[Qemu-devel] qemu-1.7.0 vm migration with nbd usage

2014-02-12 Thread xuanmao_001
Hi, Is there a document describe vm migration with nbd server or any example about it? give me some idea, thanks!

Re: [Qemu-devel] [PATCH v3 4/9] ide: Drop redundant IDEState member bs

2014-02-12 Thread Markus Armbruster
Markus Armbruster writes: > Stefano Stabellini writes: > >> On Thu, 6 Feb 2014, Markus Armbruster wrote: >>> Stefano Stabellini writes: >>> >>> > On Wed, 5 Feb 2014, Markus Armbruster wrote: >>> >> [Note cc: Stefano] >>> >> >>> >> Kevin Wolf writes: >>> >> >>> >> > Am 30.01.2014 um 13:16 ha

[Qemu-devel] [Bug 1279257] [NEW] [hw/scsi/scsi-bus.c:910]: (style) Expression '(X & 0x4) == 0x1' is always false.

2014-02-12 Thread dcb
Public bug reported: Source code is } else if ((buf[1] & 4) == 1) { Suggest code rework. I found this bug by running static analyser cppcheck over the source code. I also checked the latest code on the web and the bug exists there also. ** Affects: qemu Importance: Undecided

Re: [Qemu-devel] [Qemu-ppc] [PATCH V9 2/5] target-ppc: Fix htab_mask calculation

2014-02-12 Thread Greg Kurz
On Wed, 12 Feb 2014 00:16:25 +0530 "Aneesh Kumar K.V" wrote: > > Hi Greg, > > can you try the below patch and see if it fix the TCG mode failure ? > > -aneesh > Hi Aneesh, The patche fixes the issue indeed. Maybe a helper could be factored out since we have 4 times the same magic formula :)

Re: [Qemu-devel] [Bug 1279257] [NEW] [hw/scsi/scsi-bus.c:910]: (style) Expression '(X & 0x4) == 0x1' is always false.

2014-02-12 Thread Markus Armbruster
dcb <1279...@bugs.launchpad.net> writes: > Public bug reported: > > Source code is > >} else if ((buf[1] & 4) == 1) { > > Suggest code rework. I found this bug by running > static analyser cppcheck over the source code. > > I also checked the latest code on the web and the > bug exists the

Re: [Qemu-devel] [PATCH 0/4] spapr-pci: prepare for vfio

2014-02-12 Thread Alexey Kardashevskiy
On 01/15/2014 06:10 PM, Alexey Kardashevskiy wrote: > On 12/20/2013 01:47 PM, Alexey Kardashevskiy wrote: >> On 12/05/2013 08:39 PM, Alexey Kardashevskiy wrote: >>> On 11/21/2013 03:08 PM, Alexey Kardashevskiy wrote: Here are few reworks for spapr-pci PHB which I'd like to have to support >>>

Re: [Qemu-devel] PC value translation from guest VA to host VA.

2014-02-12 Thread Gaurav Sharma
To elaborate a bit more ... If the target is arm, in disas_arm_insn function we fetch the opcode as per the pc value : insn = arm_ldl_code(env, s->pc, s->bswap_code) This PC value i could trace back and see actually come from the cpu register set for arm regs[15]. which is the pc value. Is this PC

Re: [Qemu-devel] Ping for patch "linux-user: add binfmt wrapper for argv[0] handling"

2014-02-12 Thread Andreas Färber
Am 12.02.2014 08:35, schrieb Alexander Graf: > > On 12.02.2014, at 03:30, Dennis Lan (dlan) wrote: > >> Hi Folks: >> Any chance to get this patch accepted into master branch? >> I found it useful when chroot into another system. Origianl post can >> be found here[1], would be quite easy to for

Re: [Qemu-devel] [PATCH v3 4/9] ide: Drop redundant IDEState member bs

2014-02-12 Thread Stefano Stabellini
On Wed, 12 Feb 2014, Markus Armbruster wrote: > Markus Armbruster writes: > > > Stefano Stabellini writes: > > > >> On Thu, 6 Feb 2014, Markus Armbruster wrote: > >>> Stefano Stabellini writes: > >>> > >>> > On Wed, 5 Feb 2014, Markus Armbruster wrote: > >>> >> [Note cc: Stefano] > >>> >> > >

Re: [Qemu-devel] [Qemu-trivial] [PATCH] sparc/leon3: Initialize stack pointer

2014-02-12 Thread Sebastian Huber
On 2014-02-06 09:57, Peter Maydell wrote: On 6 February 2014 08:40, Michael Tokarev wrote: >03.02.2014 13:18, Sebastian Huber wrote: >>A lot of real world LEON3 systems are shipped with the GRMON boot >>loader. This boot loader initializes the stack pointer with the end of >>RAM address. The

Re: [Qemu-devel] [Qemu-trivial] [PATCH] sparc/leon3: Initialize stack pointer

2014-02-12 Thread Peter Maydell
On 12 February 2014 13:54, Sebastian Huber wrote: > On 2014-02-06 09:57, Peter Maydell wrote: >> On 6 February 2014 08:40, Michael Tokarev wrote: >>> >I don't really know much about sparc or leon3 stuff. But having a >>> >reviewed-by from Fabien I can apply it to trivial-patches. I'm not >>> >r

Re: [Qemu-devel] QEMU & KVM at devconf.cz 7-9 February in Brno, Czech Republic

2014-02-12 Thread Karen Noel
On 02/11/2014 12:33 PM, Paolo Bonzini wrote: Il 11/02/2014 18:11, Eduardo Habkost ha scritto: On Tue, Feb 11, 2014 at 06:07:39PM +0100, Stefan Hajnoczi wrote: On Fri, Jan 31, 2014 at 5:10 PM, Stefan Hajnoczi wrote: Several QEMU and KVM community members will be at devconf.cz 7-9 February in

[Qemu-devel] Regarding QEMU API's to interact with hardware peripherals like Memory

2014-02-12 Thread Brajesh Sharma
Hi All , I am new to Qemu Emulator i need some Info regarding this . I am working on something which needs Qemu to interact with hardware peripherals . I want to know what are those API to communicate with hardware and how can i find and see them . Any help would be appreciated . Thanks in Adva

Re: [Qemu-devel] *BSD kernel crash at boot in qemu with seabios 1.7.4 (bisected)

2014-02-12 Thread Xin Li
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 2/6/14, 4:44 AM, Michael Tokarev wrote: > [Changing subject to reflect reality] > > I don't really understand what it is all about, as I initially > said. But I've been told on freebsd IRC channels to post the issue > to freebsd-emulation list, w

[Qemu-devel] [PATCH 0/2] PC: improve emulation correctness

2014-02-12 Thread Hervé Poussineau
With those two patches, I'm able to run a real BIOS in QEMU (Iwill P55XB2) However, QEMU crashes when rebooting the computer after leaving BIOS settings. Hervé Poussineau (2): pam: partly fix write-only mode pckbd: return 'keyboard enabled' on read input port command hw/input/pckbd.c |2

[Qemu-devel] [PATCH 2/2] pckbd: return 'keyboard enabled' on read input port command

2014-02-12 Thread Hervé Poussineau
Bit 7 of Input Port is the keyboard inhibit switch. 0 means keyboard inhibited, while 1 means keyboard enabled. Incidentaly, this also fixes an error encountered while booting an Award BIOS: "Keyboard is locked out - Unlock the key". Signed-off-by: Hervé Poussineau --- hw/input/pckbd.c |2 +

[Qemu-devel] [PATCH 1/2] pam: partly fix write-only mode

2014-02-12 Thread Hervé Poussineau
In write-only mode, writes are forwarded to RAM, while reads should not be handled (ie should return 0xff). Assume that in this mode, no read access is ever done, as they shouldn't give any sensible result. So, in write-only mode, alias PAM region to RAM, instead of PCI memory (which can even be m

Re: [Qemu-devel] [PATCH v10 13/13] dump: add 'query-dump-guest-memory-capability' command

2014-02-12 Thread Luiz Capitulino
On Wed, 12 Feb 2014 14:34:19 +0800 Qiao Nuohan wrote: > 'query-dump-guest-memory-capability' is used to query the available formats > for > 'dump-guest-memory'. The output of the command will be like: > > -> { "execute": "query-dump-guest-memory-capability" } > <- { "return": { "formats": >

Re: [Qemu-devel] [PATCH v2 09/10] hw/mips: malta: Add KVM support

2014-02-12 Thread James Hogan
On 21/12/13 20:42, Peter Maydell wrote: > On 16 December 2013 14:12, James Hogan wrote: >> +#include "linux/kvm.h" > > You can't include linux/kvm.h like this -- on Linux this will pull in > whatever the host's system kvm.h happens to be, and on non-Linux > it won't compile: > > /Users/pm215/src

[Qemu-devel] [PATCH] target-ppc: fix 32 bit build break in the page table lookup code

2014-02-12 Thread Greg Kurz
The 396bb9874 commit reworked page table lookup to support kvm. Unfortunately this breaks 32 bit build: target-ppc/mmu-hash64.h: In function ‘ppc_hash64_load_hpte0’: target-ppc/mmu-hash64.h:90:23: error: cast to pointer from integer of different size target-ppc/mmu-hash64.h: In function ‘ppc_hash

Re: [Qemu-devel] [RFC] translate-all.c: add hack for coverage testing

2014-02-12 Thread Andreas Färber
Am 11.02.2014 16:31, schrieb Peter Maydell: > Actually this barely even rises to the level of an RFC, but > since I'm not entirely sure how or if it might be possible to > turn this into upstreamable code I thought I might as well > send it out as-is in case anybody else finds it useful. > > The b

[Qemu-devel] [Bug 685096] Re: USB Passthrough not working for Windows 7 guest

2014-02-12 Thread Manuel Baesler
RIght, with '-usb' qemu creates then 'piix3-usb-uhci' device: 00:01.2 USB controller: Intel Corporation 82371SB PIIX3 USB [Natoma/Triton II] (rev 01) -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/685

[Qemu-devel] [PATCH] target-ppc: fix warn_unused_result build break with in-kernel HTAB support

2014-02-12 Thread Greg Kurz
The 7029677e4 commit introduced the following build break: target-ppc/kvm.c: In function ‘kvmppc_hash64_write_pte’: target-ppc/kvm.c:2017:10: error: ignoring return value of ‘write’, declared with attribute warn_unused_result [-Werror=unused-result] write(htab_fd, &hpte_buf, sizeof(hpte_buf))

[Qemu-devel] [PATCH 1/1] Fix vmstate_info_int32_le comparison/assign

2014-02-12 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" Fix comparison of vmstate_info_int32_le so that it succeeds if loaded value is (l)ess than or (e)qual When the comparison succeeds, assign the value loaded This is a change in behaviour but I think the original intent, since the idea is to check if the version/

[Qemu-devel] [PATCH 2/2] block: Open by reference will try device then node_name.

2014-02-12 Thread Benoît Canet
Since we introduced node_name for named bs of the graph modify the opening by reference to use it as a fallback. This patch also enforce the separation of the device id and graph node namespaces. Signed-off-by: Benoit Canet --- block.c| 10 -- blockdev.c | 6 ++ 2 files changed

[Qemu-devel] [PATCH 1/2] block: Relax bdrv_lookup_bs constraints.

2014-02-12 Thread Benoît Canet
The following patch will reuse bdrv_lookup_bs in order to open images by references so the rules of usage of bdrv_lookup_bs must be relaxed a bit. Signed-off-by: Benoit Canet --- block.c | 26 +++--- 1 file changed, 11 insertions(+), 15 deletions(-) diff --git a/block.c b/bl

[Qemu-devel] [PATCH 0/2] Make open by reference use id then node-name

2014-02-12 Thread Benoît Canet
This series was asked by Kevin. It sit on top of the snapshot fix patch and max openv2 series. I have done the following tests. check qemu-iotest ok snapshot blockbackend by id ok open quorum by node-name reference and id reference ok Error messages seems ok Benoît Canet (2): block: Relax bdrv

Re: [Qemu-devel] [Qemu-trivial] [PATCH] sparc/leon3: Initialize stack pointer

2014-02-12 Thread Michael Tokarev
12.02.2014 18:00, Peter Maydell wrote: [] > It's sparc-specific, it's pretty small, it's been reviewed. > MJT: I suggest we take the pragmatic approach and apply it > via -trivial. I actually applied it back at Feb-06. Should be in the next pull request at Friday. Thanks, /mjt

[Qemu-devel] [PATCH] gitignore: anchor all ignored names

2014-02-12 Thread Michael Tokarev
by default, patterns/names in .gitignore are applied recursively to all subdirectories. So any name mentioned in .gitignore is ignored in all subdirectores. This is good for, say. object files (*.o), but not good for particular names which should be ignored only in one directory. For example, qe

Re: [Qemu-devel] [Qemu-trivial] [PATCH] sparc/leon3: Initialize stack pointer

2014-02-12 Thread Andreas Färber
Am 12.02.2014 14:54, schrieb Sebastian Huber: > On 2014-02-06 09:57, Peter Maydell wrote: >> On 6 February 2014 08:40, Michael Tokarev wrote: >>> >03.02.2014 13:18, Sebastian Huber wrote: >>A lot of real world LEON3 systems are shipped with the GRMON boot >>loader. This boot loader init

Re: [Qemu-devel] [PATCH 1/1] Fix vmstate_info_int32_le comparison/assign

2014-02-12 Thread Juan Quintela
"Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > Fix comparison of vmstate_info_int32_le so that it succeeds if loaded > value is (l)ess than or (e)qual > > When the comparison succeeds, assign the value loaded > This is a change in behaviour but I think the original i

[Qemu-devel] [PATCH 1/1] Trivial typo: ctz->cto

2014-02-12 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" Signed-off-by: Dr. David Alan Gilbert --- include/qemu/host-utils.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/qemu/host-utils.h b/include/qemu/host-utils.h index de85d28..285c5fb 100644 --- a/include/qemu/host-utils.h +++ b/includ

[Qemu-devel] Open by references and close on exit

2014-02-12 Thread Benoît Canet
Hi Max, I have a crash when quorum open it's files by references and close them on exit: It seems that bdrv_close_all tries to close them a second time because they are still registered in bdrv_states. Do you think you have the same problem with blkverify ? Should bdrv_open remove the bs from b

Re: [Qemu-devel] [PATCH v13 00/14] Drop in_use from BlockDriverState and enable point-in-time snapshot exporting over NBD

2014-02-12 Thread Ian Main
On Wed, Jan 29, 2014 at 01:07:27PM +0800, Fam Zheng wrote: > This series adds for point-in-time snapshot NBD exporting based on > blockdev-backup (variant of drive-backup with existing device as target). > > We get a thin point-in-time snapshot by COW mechanism of drive-backup, and > export it thr

[Qemu-devel] [PATCH v2] RFC: Add blockdev-del QMP command

2014-02-12 Thread Ian Main
This is the sister command to blockdev-add. In Fam's example he uses the drive_del HMP command to clean up but it would be much nicer to have a way to do this via QMP. Signed-off-by: Ian Main --- v2: - s/blockdev-delete/blockdev-del - Fixed example. blockdev.c | 52 +

[Qemu-devel] [PATCH V2 1/1] Fix vmstate_info_int32_le comparison/assign

2014-02-12 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" Fix comparison of vmstate_info_int32_le so that it succeeds if loaded value is (l)ess than or (e)qual When the comparison succeeds, assign the value loaded This is a change in behaviour but I think the original intent, since the idea is to check if the version/

Re: [Qemu-devel] [PULL 00/18] Block patches

2014-02-12 Thread Peter Maydell
On 9 February 2014 08:46, Kevin Wolf wrote: > The following changes since commit 1f6b12f75f2c22f861d0202374033a7594c91707: > > Merge remote-tracking branch 'remotes/mwalle/tags/lm32-fixes/20140204' into > staging (2014-02-08 15:57:51 +) > > are available in the git repository at: > > > gi

Re: [Qemu-devel] net: Adding netmap network backend

2014-02-12 Thread Michael Tokarev
After this patch, qemu fails to build on freebsd with the following error message: net/netmap.c: In function 'netmap_receive_iov': net/netmap.c:307: error: 'NS_MOREFRAG' undeclared (first use in this function) net/netmap.c:307: error: (Each undeclared identifier is reported only once net/netmap.c:

Re: [Qemu-devel] [PATCH v2] RFC: Add blockdev-del QMP command

2014-02-12 Thread Eric Blake
On 02/12/2014 10:36 AM, Ian Main wrote: > This is the sister command to blockdev-add. In Fam's example he uses > the drive_del HMP command to clean up but it would be much nicer to > have a way to do this via QMP. > > Signed-off-by: Ian Main > --- > Reviewed-by: Eric Blake Nice to finally ge

Re: [Qemu-devel] [PULL 0/1 v2] libcacard glusterfs fix

2014-02-12 Thread Peter Maydell
On 9 February 2014 11:13, Alon Levy wrote: > changes from v1: Added Signed-of by me. > > The following changes since commit 1f6b12f75f2c22f861d0202374033a7594c91707: > > Merge remote-tracking branch 'remotes/mwalle/tags/lm32-fixes/20140204' into > staging (2014-02-08 15:57:51 +) > > are ava

[Qemu-devel] Guest IOMMU and Cisco usnic

2014-02-12 Thread Benoît Canet
Hi Alex, After the IRC conversation we had a few days ago I understood that guest IOMMU was not implemented. I have a real use case for it: Cisco usnic allow to write MPI applications while driving the network card in userspace in order to optimize the latency. It's made for compute clusters.

Re: [Qemu-devel] [PATCH v3] QMP: Allow dot separated dict path arguments in qmp-shell

2014-02-12 Thread Eric Blake
On 02/11/2014 08:05 PM, Fam Zheng wrote: > As another convinience to allow using commands that expect a dict as You missed this, even though I pointed it out in v2: s/convinience/convenience/ > argument, this patch adds support for foo.bar=value syntax, similar to > command line argument style:

Re: [Qemu-devel] [PATCH v13 00/14] Drop in_use from BlockDriverState and enable point-in-time snapshot exporting over NBD

2014-02-12 Thread Richard W.M. Jones
On Wed, Feb 12, 2014 at 09:22:29AM -0800, Ian Main wrote: > On Wed, Jan 29, 2014 at 01:07:27PM +0800, Fam Zheng wrote: > > This series adds for point-in-time snapshot NBD exporting based on > > blockdev-backup (variant of drive-backup with existing device as target). > > > > We get a thin point-in

Re: [Qemu-devel] [PATCH target-arm v1 6/7] dma/pl330: Fix buffer depth

2014-02-12 Thread Peter Maydell
On 11 February 2014 06:31, Peter Crosthwaite wrote: > This is the product of the data-width and the depth arguments, I.e the > depth of the FIFO is in terms of data entries and not bytes (which is > what the original implementation was suggesting). Fix. > > Signed-off-by: Peter Crosthwaite > ---

[Qemu-devel] [PATCH] [RFC] Convert Qemu Timer List and Active Timers to RCU

2014-02-12 Thread Mike Day
Allow readers to use RCU when reading Qemu timer lists. Applies to Paolo Bonzini's RCU branch, https://github.com/bonzini/qemu/tree/rcu. This patch is for comment and review only. The rcu branch needs to be rebased on upstream. Signed-off-by: Mike Day --- include/qemu/timer.h | 9 +- qem

Re: [Qemu-devel] [PATCH target-arm v1 7/7] dma/pl330: implement dmaadnh instruction

2014-02-12 Thread Peter Maydell
On 11 February 2014 06:32, Peter Crosthwaite wrote: > Implement the missing DMAADNH instruction. This is a minor variant > of the DMAADDH instruction, so factor out to a common implementation > for both (dmaadxh). > > Signed-off-by: Peter Crosthwaite > --- > > hw/dma/pl330.c | 19 +++

Re: [Qemu-devel] [PATCH target-arm v1 0/7] PL330 Changes

2014-02-12 Thread Peter Maydell
On 11 February 2014 06:28, Peter Crosthwaite wrote: > Hi Peter, > > Some minor fixups and tweaks to PL330. Thanks. For patches 1-5, and also 6 if you fix the double-space: Reviewed-by: Peter Maydell -- PMM

Re: [Qemu-devel] [PATCH target-arm v1 1/2] char/cadence_uart: Handle qemu_chr_fe_write errors

2014-02-12 Thread Peter Maydell
On 12 February 2014 03:36, Peter Crosthwaite wrote: > By just ignoring them and trying again later. This handles the > EGAIN case properly (the previous implementation was only dealing > with short returns and not errors). > > Signed-off-by: Peter Crosthwaite > --- > > hw/char/cadence_uart.c | 7

Re: [Qemu-devel] [PATCH target-arm v1 2/2] char/cadence_uart: Add NULL guards against chr

2014-02-12 Thread Peter Maydell
On 12 February 2014 03:37, Peter Crosthwaite wrote: > It's possible and valid for users of this device model to instantiate > it without a backing chr device. To avoid crashes, guard all uses of > the backing chr device against NULL. > > Signed-off-by: Peter Crosthwaite Reviewed-by: Peter Maydel

Re: [Qemu-devel] [PATCH target-arm v5 1/1] target-arm: Implements the ARM PMCCNTR register

2014-02-12 Thread Peter Maydell
On 31 January 2014 04:44, Alistair Francis wrote: > This patch implements the ARM PMCCNTR register including > the disable and reset components of the PMCR register. > > Signed-off-by: Alistair Francis I'm afraid you'll need to respin this against my target-arm.next branch, because I've landed t

Re: [Qemu-devel] Guest IOMMU and Cisco usnic

2014-02-12 Thread Alex Williamson
On Wed, 2014-02-12 at 19:10 +0100, Benoît Canet wrote: > Hi Alex, > > After the IRC conversation we had a few days ago I understood that guest IOMMU > was not implemented. > > I have a real use case for it: > > Cisco usnic allow to write MPI applications while driving the network card in > users

Re: [Qemu-devel] [PATCH] linux-user: simplify/refactor socketcall implementation

2014-02-12 Thread Peter Maydell
On 6 February 2014 06:56, Michael Tokarev wrote: > socketcall is just a dispatcher, it accepts an array of ulongs and should > call the right socket function. We tried to handle arguments for every > function case, and did that differently, which in the past caused errors > due to wrong types or

[Qemu-devel] [PATCH] block: Don't throw away errno via error_setg

2014-02-12 Thread Jeff Cody
There are a handful of places in the block layer where a failure path has a valid -errno value, yet error_setg() is used. Those instances should instead use error_setg_errno(), to preserve as much error information as possible. This patch replaces those instances with error_setg_errno(), so that

[Qemu-devel] [PATCH] block: mirror - insure that errp is not NULL

2014-02-12 Thread Jeff Cody
When starting a block job, commit_active_start() relies on whether *errp is set by mirror_start_job. This allows it to determine if the mirror job start failed, so that it can clean up any changes to open flags from the bdrv_reopen(). If errp is NULL, then it will not be able to determine if mirr

Re: [Qemu-devel] [FIX V2] block: Fix device snapshots broken by the block filter snapshots patchset.

2014-02-12 Thread Benoît Canet
The Tuesday 11 Feb 2014 à 16:12:17 (+0800), Fam Zheng wrote : > On Mon, 02/10 22:49, Benoît Canet wrote: > > Take into account the fact that a block filter like quorum will be in > > bs->file > > while a regular block driver device is really on the top level. > > > > Signed-off-by: Benoit Canet

Re: [Qemu-devel] [PATCH] block: Don't throw away errno via error_setg

2014-02-12 Thread Eric Blake
On 02/12/2014 12:46 PM, Jeff Cody wrote: > There are a handful of places in the block layer where a failure path > has a valid -errno value, yet error_setg() is used. Those instances > should instead use error_setg_errno(), to preserve as much error > information as possible. > > This patch repla

Re: [Qemu-devel] [PATCH] block: mirror - insure that errp is not NULL

2014-02-12 Thread Eric Blake
On 02/12/2014 12:46 PM, Jeff Cody wrote: > When starting a block job, commit_active_start() relies on whether *errp > is set by mirror_start_job. This allows it to determine if the mirror > job start failed, so that it can clean up any changes to open flags from > the bdrv_reopen(). If errp is NU

[Qemu-devel] [Bug 1279500] [NEW] system_powerdown causes SMP OpenBSD guest to freeze

2014-02-12 Thread Robert Urban
version) qemu-1.7.50 (latest development snapshot, "git cloned" today, 20140212) all of the above hosts are running x86_64 linux. The first OpenBSD version that I ran as a VM, v5.1, experienced the problem. All subsequent versions experience the problem. The above tests were perfo

Re: [Qemu-devel] [PATCH 1/5] qmp: rename query_option_descs() to get_param_infolist()

2014-02-12 Thread Eric Blake
On 01/27/2014 08:53 PM, Amos Kong wrote: > Signed-off-by: Amos Kong > --- > util/qemu-config.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) Reviewed-by: Eric Blake -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization library http://libvirt.org sign

Re: [Qemu-devel] [PATCH 2/5] introduce two marcos to dump the options info

2014-02-12 Thread Eric Blake
On 01/27/2014 08:53 PM, Amos Kong wrote: s/marcos/macros/ in the subject line > We will use the marocs to generate two tables, which contain s/marocs/macros/ (wow, two different ways to mis-spell the word) > the option name and argument information. > > Signed-off-by: Amos Kong > --- > qemu

Re: [Qemu-devel] [PATCH 3/5] query-command-line-options: query all the options in qemu-options.hx

2014-02-12 Thread Eric Blake
On 01/27/2014 08:53 PM, Amos Kong wrote: > vm_config_groups[] contain the options which have parameter, but some > legcy options haven't been added to vm_config_groups[]. s/legcy/legacy/ > > All the options can be found in qemu-options.hx, this patch used two > new marcos to generate two tables,

Re: [Qemu-devel] [PATCH 4/5] introduce QEMU_OPTIONS_GENERATE_HELPMSG

2014-02-12 Thread Eric Blake
On 01/27/2014 08:53 PM, Amos Kong wrote: > This patch introduced a new maroc, it will be used to dump the help > messages of all the options. > > Signed-off-by: Amos Kong > --- > qemu-options-wrapper.h | 9 + > 1 file changed, 9 insertions(+) Again, this patch is useless on its own, I'd

Re: [Qemu-devel] [PATCH 5/5] query-command-line-options: return help message for legacy options

2014-02-12 Thread Eric Blake
On 02/11/2014 05:19 AM, Markus Armbruster wrote: > [Note cc: Eric] > > Amos Kong writes: > >> Some legacy options that have arguments weren't added to >> vm_config_groups[], so query-command-line-options returns a >> NULL parameters infolist. This patch try to return help message >> for this kin

[Qemu-devel] [Bug 1279500] Re: system_powerdown causes SMP OpenBSD guest to freeze

2014-02-12 Thread Robert Urban
(qemu-kvm-0.12.5) - Fedora 20: qemu-system-x86-1.6.1 (from Fedora repository) qemu-1.7.0 (latest release version) qemu-1.7.50 (latest development snapshot, "git cloned" today, 20140212) all of the above hosts are running x86_64 linux. The first OpenBSD vers

Re: [Qemu-devel] [PATCH 5/5] query-command-line-options: return help message for legacy options

2014-02-12 Thread Eric Blake
On 01/27/2014 08:53 PM, Amos Kong wrote: > Some legacy options that have arguments weren't added to > vm_config_groups[], so query-command-line-options returns a > NULL parameters infolist. This patch try to return help message > for this kind of legacy options. > > Example: > { > "helpmsg":

[Qemu-devel] [PATCH 00/28] target-ppc: Altivec 2.07

2014-02-12 Thread Tom Musta
This patch series implements the changes to Altivec introduced by Power ISA Version 2.07. Tom Musta (28): target-ppc: Altivec 2.07: Add Instruction Flag target-ppc: Altivec 2.07: Update AVR Structure target-ppc: Altivec 2.07: Add GEN_VXFORM3 target-ppc: Altivec 2.07: Add Support for Dual A

[Qemu-devel] [PATCH 01/28] target-ppc: Altivec 2.07: Add Instruction Flag

2014-02-12 Thread Tom Musta
This patch adds a flag that will be used to tag the Altivec instructions introduced in Power ISA Version 2.07. The flag is added to Power8 model since P8 supports these instructions. Signed-off-by: Tom Musta --- target-ppc/cpu.h|5 - target-ppc/translate_init.c |2 +- 2

[Qemu-devel] [PATCH 02/28] target-ppc: Altivec 2.07: Update AVR Structure

2014-02-12 Thread Tom Musta
This patch updates the ppc_avr_t data structure to include elements for signed 64-bit integers and (conditionally) unsigned 128 bit integers. These elements will be in instructions models later on in this patch series. Signed-off-by: Tom Musta --- target-ppc/cpu.h |4 1 files changed, 4

[Qemu-devel] [PATCH 05/28] target-ppc: Altivec 2.07: Add Opcode Macro for VX Form Instructions

2014-02-12 Thread Tom Musta
This patch adds a macro to insert an entry into the opcode table for Altivec Power ISA Version 2.07 instructions. The macro is similar to the GEN_VXFORM macro except that it tags the entry with the PPC2_ALTIVEC_207 flag rather than PPC_ALTIVEC. Signed-off-by: Tom Musta --- target-ppc/translate

[Qemu-devel] [PATCH 04/28] target-ppc: Altivec 2.07: Add Support for Dual Altivec Instructions

2014-02-12 Thread Tom Musta
Some Alitvec instructions introduced in Power ISA Version 2.07 use bit 31 (aka the "Rc" bit) as an opcode bit. However, QEMU for PowerPC uses bits 0-5 and 21-30 for opcodes and not bit 31. This patch introduces macros that will handle this situation by injecting an auxiliary handler which decodes

[Qemu-devel] [PATCH 07/28] target-ppc: Altivec 2.07: Vector Logical Instructions

2014-02-12 Thread Tom Musta
This patch adds the Vector Logical Instructions that are introduced in Power ISA Version 2.07: veqv, vnand and vorc. Signed-off-by: Tom Musta --- target-ppc/translate.c | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/target-ppc/translate.c b/target-ppc/transla

[Qemu-devel] [PATCH 11/28] target-ppc: Altivec 2.07: vmuluw Instruction

2014-02-12 Thread Tom Musta
This patch adds the Vector Multiply Unsigned Word Modulo (vmuluwm) instruction. The existing VARITH_DO macro is re-used to (trivially) instantiate the helper code. Since bits 21-31 of any vmuluwm instruction is 137, the instruction is coded as a dual to vmulouw (bits 21-31 = 136). Signed-off-by:

[Qemu-devel] [PATCH 17/28] target-ppc: Altivec 2.07: Vector Merge Instructions

2014-02-12 Thread Tom Musta
This patch adds the Vector Merge Even Word (vmrgew) and Vector Merge Odd Word (vmrgow) instructions introduced in Power ISA Version 2.07. Signed-off-by: Tom Musta --- target-ppc/translate.c | 37 + 1 files changed, 37 insertions(+), 0 deletions(-) diff --gi

[Qemu-devel] [PATCH 14/28] target-ppc: Altivec 2.07: Vector Min/Max Doubleword Instructions

2014-02-12 Thread Tom Musta
This patch adds the Vector Minimum and Maximum Doubleword instructions that are introduced in Power ISA Version 2.07. Signed-off-by: Tom Musta --- target-ppc/helper.h |4 target-ppc/int_helper.c |2 ++ target-ppc/translate.c |8 3 files changed, 14 insertions(+), 0

[Qemu-devel] [PATCH 18/28] target-ppc: Altivec 2.07: Change Bit Masks to Support 64-bit Rotates and Shifts

2014-02-12 Thread Tom Musta
Existing code in the VROTATE, VSL and VSR macros for the Altivec rotate and shift helpers uses a formula to compute a bit mask used to extract the rotate/shift amount from the VRB register. What is desired is: mask = (1 << (3 + log2(sizeof(element - 1 but what is implemented is: ma

[Qemu-devel] [PATCH 27/28] target-ppc: Altivec 2.07: Vector SHA Sigma Instructions

2014-02-12 Thread Tom Musta
This patch adds the Vector SHA Sigma instructions introduced in Power ISA Version 2.07: - Vector SHA-512 Sigma Doubleword (vshasigmad) - Vector SHA-256 Sigma Word (vshasigmaw) Signed-off-by: Tom Musta --- target-ppc/helper.h |2 + target-ppc/int_helper.c | 82 +

[Qemu-devel] [PATCH 06/28] target-ppc: Altivec 2.07: Add Support for R-Form Dual Instructions

2014-02-12 Thread Tom Musta
Some Alitvec instructions introduced in Power ISA Version 2.07 use bit 31 (aka the "Rc" bit) as an opcode but also use bit 21 as an actual Rc bit. QEMU for PowerPC typically uses bits 0-5 and 21-30 for opcodes. This patch introduces a generator macro that injects an auxiliary handler which decode

[Qemu-devel] [PATCH 15/28] target-ppc: Altivec 2.07: Pack Doubleword Instructions

2014-02-12 Thread Tom Musta
This patch adds the Vector Pack Doubleword instructions introduced in Power ISA Version 2.07: - Vector Pack Signed Doubleword Signed Saturate (vpksdss) - Vector Pack Signed Doubleword Unsigned Saturate (vpksdus) - Vector Pack Unsigned Doubleword Unsigned Modulo (vpkudum) - Vector Pack Unsigned

[Qemu-devel] [PATCH 22/28] target-ppc: Altivec 2.07: Doubleword Compares

2014-02-12 Thread Tom Musta
This patch adds the Vector Compare Doubleword instructions introduced by Power ISA Version 2.07: - Vector Compare Equal to Unsigned Doubleword (vcmpequd) - Vector Compare Greater Than Signed Doubleword (vcmpgtsd) - Vector Compare Greater Than Unsigned Doubleword (vcmpgtud) These instruction

[Qemu-devel] [PATCH 0/2] qemu-iotest fixes and updates for 070

2014-02-12 Thread Jeff Cody
This fixes a bug with test 070 (incorrect output expectation), and adds additional test for log replay. Jeff Cody (2): block: qemu-iotests - fix test 070 (vhdx) block: qemu-iotests - add vhdx log replay tests for qemu-img tests/qemu-iotests/070 | 13 - tests/qemu-iotests/070

[Qemu-devel] [PATCH 24/28] target-ppc: Altivec 2.07: Vector Polynomial Multiply Sum

2014-02-12 Thread Tom Musta
This patch adds the Vectory Polynomial Multiply Sum instructions introduced in Power ISA Version 2.07: - Vectory Polynomial Multiply Sum Byte (vpmsumb) - Vectory Polynomial Multiply Sum Halfword (vpmsumh) - Vectory Polynomial Multiply Sum Word (vpmsumw) - Vectory Polynomial Multiply Sum Do

[Qemu-devel] [PATCH 1/2] block: qemu-iotests - fix test 070 (vhdx)

2014-02-12 Thread Jeff Cody
VHDX test 070 failed, due to different output from qemu-io / qemu when opening an image read-only that contains a log file. Filter the output, and update the expected results to match the correct output. Signed-off-by: Jeff Cody --- tests/qemu-iotests/070 | 3 ++- tests/qemu-iotests/070.out

[Qemu-devel] [PATCH 25/28] target-ppc: Altivec 2.07: Binary Coded Decimal Instructions

2014-02-12 Thread Tom Musta
This patch add the Binary Coded Decimal instructions bcdadd. and bcdsub. Signed-off-by: Tom Musta --- target-ppc/helper.h |3 + target-ppc/int_helper.c | 201 +++ target-ppc/translate.c | 45 ++- 3 files changed, 245 insertions(+),

[Qemu-devel] [PATCH 26/28] target-ppc: Altivec 2.07: AES Instructions

2014-02-12 Thread Tom Musta
This patch adds the Vector AES instructions introduced in Power ISA Version 2.07: - Vector AES Cipher (vcipher) - Vector AES Cipher Last (vcipherlast) - Vector AES Inverse Cipher (vncipher) - Vector AES Inverse Cipher Last (vncipherlast) - Vector AES SubBytes (vsbox) Note that the

[Qemu-devel] [PATCH 2/2] block: qemu-iotests - add vhdx log replay tests for qemu-img

2014-02-12 Thread Jeff Cody
VHDX logs can now be replayed via 'qemu-img check -r all'. Add tests to verify that the log replay is successful when using qemu-img. Signed-off-by: Jeff Cody --- tests/qemu-iotests/070 | 10 ++ tests/qemu-iotests/070.out | 11 +++ 2 files changed, 21 insertions(+) diff --g

Re: [Qemu-devel] [FIX V2] block: Fix device snapshots broken by the block filter snapshots patchset.

2014-02-12 Thread Benoît Canet
The Wednesday 12 Feb 2014 à 20:49:18 (+0100), Benoît Canet wrote : > The Tuesday 11 Feb 2014 à 16:12:17 (+0800), Fam Zheng wrote : > > On Mon, 02/10 22:49, Benoît Canet wrote: > > > Take into account the fact that a block filter like quorum will be in > > > bs->file > > > while a regular block dri

[Qemu-devel] [PATCH] virtio-net: Do not filter VLANs without F_CTRL_VLAN

2014-02-12 Thread Stefan Fritsch
If VIRTIO_NET_F_CTRL_VLAN is not negotiated, do not filter out all VLAN-tagged packets but send them to the guest. Signed-off-by: Stefan Fritsch --- This time CCing the maintainers. This fixes VLANs with OpenBSD guests (and probably NetBSD, too, because the OpenBSD driver started as a port from

[Qemu-devel] [PATCH 12/28] target-ppc: Altivec 2.07: Add Vector Count Leading Zeroes

2014-02-12 Thread Tom Musta
This patch adds the Vector Count Leading Zeroes instructions introduced in Power ISA Version 2.07 - vclzb, vclzh, vclzw and vclzd. Signed-off-by: Tom Musta --- target-ppc/helper.h |5 + target-ppc/int_helper.c | 29 + target-ppc/translate.c |9 +

[Qemu-devel] [PATCH 28/28] target-ppc: Altivec 2.07: Vector Permute and Exclusive OR

2014-02-12 Thread Tom Musta
This patch adds the Vector Permuate and Exclusive OR (vpermxor) instruction introduced in Power ISA Version 2.07. Signed-off-by: Tom Musta --- target-ppc/helper.h |1 + target-ppc/int_helper.c | 14 ++ target-ppc/translate.c |7 ++- 3 files changed, 21 insertions(+

[Qemu-devel] [PATCH 16/28] target-ppc: Altivec 2.07: Unpack Signed Word Instructions

2014-02-12 Thread Tom Musta
This patch adds the Unpack Signed Word instructions introduced in Power ISA Version 2.07: - Vector Unpack High Signed Word (vupkusw) - Vector Unpack Low Signed Word (vupklsw) Signed-off-by: Tom Musta --- target-ppc/helper.h |2 ++ target-ppc/int_helper.c |2 ++ target-ppc/transl

[Qemu-devel] [PATCH V17 02/12] quorum: Create BDRVQuorumState and BlkDriver and do init.

2014-02-12 Thread Benoît Canet
From: Benoît Canet Create the structure holding the quorum settings and write the minimal block driver instanciation boilerplate. Signed-off-by: Benoit Canet Reviewed-by: Max Reitz --- block/quorum.c | 31 +++ 1 file changed, 31 insertions(+) diff --git a/block/qu

[Qemu-devel] [PATCH V17 07/12] quorum: Add quorum_getlength().

2014-02-12 Thread Benoît Canet
From: Benoît Canet Check that every bs file returns the same length. Otherwise, return -EIO to disable the quorum and avoid length discrepancy. Signed-off-by: Benoit Canet Reviewed-by: Max Reitz --- block/quorum.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a

[Qemu-devel] [PATCH 09/28] target-ppc: Altivec 2.07: Change VMUL_DO to Support 64-bit Integers

2014-02-12 Thread Tom Musta
This VMUL_DO macro provides support for the various vmule* and vmulo* instructions. These instructions multiply vector elements, producing products that are one size larger; e.g. vmuleub multiplies unsigned 8-bit elements and produces a 16 bit unsigned element. The existing macro works correctly

[Qemu-devel] [PATCH V17 06/12] quorum: Add quorum mechanism.

2014-02-12 Thread Benoît Canet
From: Benoît Canet This patchset enable the core of the quorum mechanism. The num_children reads are compared to get the majority version and if this version exist more than threshold time the guest won't see the error at all. QMP events are used to report individual driver errors or whole quorum

[Qemu-devel] [PATCH V17 03/12] quorum: Add quorum_aio_writev and its dependencies.

2014-02-12 Thread Benoît Canet
From: Benoît Canet Writes are mirrored num_children times on num_children devices. Signed-off-by: Benoit Canet --- block/quorum.c | 103 + 1 file changed, 103 insertions(+) diff --git a/block/quorum.c b/block/quorum.c index 36c5bb8..197c

[Qemu-devel] [PATCH V17 08/12] quorum: Add quorum_invalidate_cache().

2014-02-12 Thread Benoît Canet
From: Benoît Canet We really want that live migration works with quorum so implement quorum_invalidate_cache(). Signed-off-by: Benoit Canet Reviewed-by: Max Reitz --- block/quorum.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/block/quorum.c b/block/quorum.c index 2e54921.

[Qemu-devel] [PATCH V17 01/12] quorum: Create quorum.c, add QuorumChildRequest and QuorumAIOCB.

2014-02-12 Thread Benoît Canet
From: Benoît Canet Quorum is a block filter mirroring writes to num_children children. For reads quorum reads each children and does a vote. If more than vote_threshold versions are identicals the quorum is reached and this winning version is returned to the guest. So quorum prevents bit corrupt

[Qemu-devel] [PATCH V17 04/12] blkverify: Extract qemu_iovec_clone() and qemu_iovec_compare() from blkverify.

2014-02-12 Thread Benoît Canet
From: Benoît Canet qemu_iovec_compare() will be used to compare IOs vectors in quorum blkverify mode. The patch extract these functions in order to factorize the code. Signed-off-by: Benoit Canet Reviewed-by: Max Reitz --- block/blkverify.c | 108 +-

[Qemu-devel] [PATCH V17 05/12] quorum: Add quorum_aio_readv.

2014-02-12 Thread Benoît Canet
From: Benoît Canet Add code to do num_children reads in parallel and cleanup the structures afterward. Signed-off-by: Benoit Canet Reviewed-by: Max Reitz --- block/quorum.c | 39 ++- 1 file changed, 38 insertions(+), 1 deletion(-) diff --git a/block/quorum

[Qemu-devel] [PATCH 08/28] target-ppc: Altivec 2.07: Add/Subtract Unsigned Doubleword Modulo

2014-02-12 Thread Tom Musta
This patch adds two Altivec unsigned doublword modulo instructions that are introduced in Power ISA Version V2.07: - vaddudm : Vector Add Unsigned Doubleword Modulo - vsubudm : Vector Subtrace Unsigned Doubleword Modulo Signed-off-by: Tom Musta --- target-ppc/helper.h |2 ++ target-

  1   2   >