Re: [Qemu-devel] [PATCH V8 1/2] arm: Add an option to turn on/off vPMU support

2016-10-26 Thread Andrew Jones
On Wed, Oct 26, 2016 at 02:28:20AM -0400, Wei Huang wrote: > This patch adds a pmu=[on/off] option to enable/disable vPMU support > in guest vCPU. It allows virt tools, such as libvirt, to determine the > exsitence of vPMU and configure it. Note this option is only available > for cortex-a57/cortex

Re: [Qemu-devel] [PATCH V8 2/2] arm: virt: add PMU property to mach-virt machine type

2016-10-26 Thread Andrew Jones
On Wed, Oct 26, 2016 at 02:28:21AM -0400, Wei Huang wrote: > CPU vPMU is now turned ON by default, but this feature wasn't introduced > until virt-2.7 machine type. To solve this problem, this patch adds a > PMU option in machine state, which is used to control CPU's vPMU status. > This PMU option

Re: [Qemu-devel] [PATCH COLO-Frame (Base) v21 07/17] COLO: Add a new RunState RUN_STATE_COLO

2016-10-26 Thread Amit Shah
On (Tue) 18 Oct 2016 [20:10:03], zhanghailiang wrote: > Guest will enter this state when paused to save/restore VM state > under COLO checkpoint. > > Cc: Eric Blake > Cc: Markus Armbruster > Signed-off-by: zhanghailiang > Signed-off-by: Li Zhijian > Signed-off-by: Gonglei > Reviewed-by: Dr. D

Re: [Qemu-devel] [PATCH v5 11/17] ppc/xics: Add "native" XICS subclass

2016-10-26 Thread Cédric Le Goater
On 10/25/2016 07:08 AM, David Gibson wrote: > On Sat, Oct 22, 2016 at 11:46:44AM +0200, Cédric Le Goater wrote: >> This provides access to the MMIO based Interrupt Presentation >> Controllers (ICP) as found on a POWER8 system. >> >> A new XICSNative class is introduced to hold the MMIO region of th

[Qemu-devel] [PATCH] util/mmap-alloc: check parameter before using

2016-10-26 Thread Cao jin
Also refactor some code hunk for readability Signed-off-by: Cao jin --- util/mmap-alloc.c | 20 +--- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/util/mmap-alloc.c b/util/mmap-alloc.c index 5a85aa3..92c123a 100644 --- a/util/mmap-alloc.c +++ b/util/mmap-alloc.c

Re: [Qemu-devel] [PATCH v9 04/12] vfio iommu: Add support for mediated devices

2016-10-26 Thread Tian, Kevin
> From: Alex Williamson [mailto:alex.william...@redhat.com] > Sent: Monday, October 24, 2016 10:32 AM > > > >> -static long vfio_unpin_pages(unsigned long pfn, long npage, > > >> - int prot, bool do_accounting) > > >> +static long __vfio_unpin_pages_remote(struct vfio_i

Re: [Qemu-devel] [PATCH v2 2/2] qapi: allow blockdev-add for NFS

2016-10-26 Thread Markus Armbruster
A few drive-by comments... Eric Blake writes: > On 10/24/2016 02:27 PM, Ashijeet Acharya wrote: >> Introduce new object 'BlockdevOptionsNFS' in qapi/block-core.json to >> support blockdev-add for NFS network protocol driver. Also make a new >> struct NFSServer to support tcp connection. >> >> S

[Qemu-devel] [PATCH 2/2] tests: Add pseries machine to the prom-env-test, too

2016-10-26 Thread Thomas Huth
Now that we also support the "-prom-env" parameter for the pseries machine, we can enable this test for this machine, too. Since booting with TCG is rather slow with the pseries machine, we also enable the "-nodefaults" parameter for this test now, so that SLOF does not have to check that much devi

[Qemu-devel] [PATCH 0/2] ppc: Allow "-prom-env" parameter for the pseries machine, too

2016-10-26 Thread Thomas Huth
Here's a slightly updated version of the patch that enables the "-prom-env" parameter for the pseries machine. The code is now only executed if nb_prom_envs > 0. That means if the user did _not_ specify a "-prom-env" parameter, there is no change in behavior (i.e. SLOF will initialize the NVRAM ins

[Qemu-devel] [PATCH 1/2] spapr_nvram: Pre-initialize the NVRAM to support the -prom-env parameter

2016-10-26 Thread Thomas Huth
In case we do not load the NVRAM contents from a file and the user specified the "-prom-env" parameter, use the new CHRP NVRAM helper functions to pre-initialize the NVRAM partitions, so that the SLOF firmware now can pick up the environment variables from the -prom-env parameter, too. Signed-off-

Re: [Qemu-devel] [PATCH v5 0/3] IOMMU: intel_iommu support map and unmap notifications

2016-10-26 Thread Chao Gao
On Fri, Oct 21, 2016 at 12:07:18AM +0300, Aviv B.D wrote: >From: "Aviv Ben-David" > >* Advertize Cache Mode capability in iommu cap register. > This capability is controlled by "cache-mode" property of intel-iommu device. > To enable this option call QEMU with "-device intel-iommu,cache-mode=tr

Re: [Qemu-devel] [PATCH 2/2] hostmem-file: allow option 'size' optional

2016-10-26 Thread Haozhong Zhang
On 10/26/16 13:56 +0800, Haozhong Zhang wrote: On 10/25/16 17:50 -0200, Eduardo Habkost wrote: On Mon, Oct 24, 2016 at 05:21:51PM +0800, Haozhong Zhang wrote: +if (memory) { +memory = memory ?: file_size; This doesn't make sense to me. You already checked if memory is zero above,

Re: [Qemu-devel] [PATCH v9 04/12] vfio iommu: Add support for mediated devices

2016-10-26 Thread Tian, Kevin
> From: Alex Williamson [mailto:alex.william...@redhat.com] > Sent: Thursday, October 20, 2016 5:03 AM > > @@ -83,6 +92,21 @@ struct vfio_group { > > }; > > > > /* > > + * Guest RAM pinning working set or DMA target > > + */ > > +struct vfio_pfn { > > + struct rb_node node; > > + uns

Re: [Qemu-devel] [PATCH v9 04/12] vfio iommu: Add support for mediated devices

2016-10-26 Thread Tian, Kevin
> From: Tian, Kevin > Sent: Wednesday, October 26, 2016 3:54 PM > > > From: Alex Williamson [mailto:alex.william...@redhat.com] > > Sent: Thursday, October 20, 2016 5:03 AM > > > @@ -83,6 +92,21 @@ struct vfio_group { > > > }; > > > > > > /* > > > + * Guest RAM pinning working set or DMA target

Re: [Qemu-devel] [PATCH v2 2/2] gdbstub: Fix vCont behaviour

2016-10-26 Thread Christian Borntraeger
On 10/14/2016 01:53 PM, Claudio Imbrenda wrote: > When GDB issues a "vCont", QEMU was not handling it correctly when > multiple VCPUs are active. > For vCont, for each thread (VCPU), it can be specified whether to > single step, continue or stop that thread. The default is to stop a > thread. > How

Re: [Qemu-devel] [PATCH v2 2/2] qapi: allow blockdev-add for NFS

2016-10-26 Thread Kevin Wolf
Am 26.10.2016 um 09:23 hat Markus Armbruster geschrieben: > A few drive-by comments... > > Eric Blake writes: > > > On 10/24/2016 02:27 PM, Ashijeet Acharya wrote: > >> Introduce new object 'BlockdevOptionsNFS' in qapi/block-core.json to > >> support blockdev-add for NFS network protocol driver.

Re: [Qemu-devel] [PATCH v5 08/13] qapi: Allow blockdev-add for NBD

2016-10-26 Thread Kevin Wolf
Am 26.10.2016 um 01:57 hat Eric Blake geschrieben: > On 10/25/2016 08:11 AM, Max Reitz wrote: > > Signed-off-by: Max Reitz > > --- > > qapi/block-core.json | 25 ++--- > > 1 file changed, 22 insertions(+), 3 deletions(-) > > > > diff --git a/qapi/block-core.json b/qapi/block-

Re: [Qemu-devel] [PATCH] util/mmap-alloc: check parameter before using

2016-10-26 Thread Thomas Huth
On 26.10.2016 09:18, Cao jin wrote: > Also refactor some code hunk for readability > > Signed-off-by: Cao jin > --- > util/mmap-alloc.c | 20 +--- > 1 file changed, 9 insertions(+), 11 deletions(-) > > diff --git a/util/mmap-alloc.c b/util/mmap-alloc.c > index 5a85aa3..92c123a 1

Re: [Qemu-devel] [PATCH v5] raw_bsd: add offset and size options

2016-10-26 Thread Kevin Wolf
Am 25.10.2016 um 23:37 hat Tomáš Golembiovský geschrieben: > I should test my code more before submitting it to ML. I have found two > bugs in the patch. Maybe one thing you could/should do is add a second patch which adds a qemu-iotests case to test that the options are working correctly. Kevin

Re: [Qemu-devel] [PATCH COLO-Frame (Base) v21 00/17] COarse-grain LOck-stepping(COLO) Virtual Machines for Non-stop Service (FT)

2016-10-26 Thread Amit Shah
On (Wed) 26 Oct 2016 [14:43:30], Hailiang Zhang wrote: > Hi Amit, > > On 2016/10/26 14:09, Amit Shah wrote: > >Hello, > > > >On (Tue) 18 Oct 2016 [20:09:56], zhanghailiang wrote: > >>This is the 21th version of COLO frame series. > >> > >>Rebase to the latest master. > > > >I've reviewed the patch

Re: [Qemu-devel] [PATCH v4 5/5] qapi: allow blockdev-add for ssh

2016-10-26 Thread Kevin Wolf
Am 25.10.2016 um 21:58 hat Eric Blake geschrieben: > On 10/25/2016 08:04 AM, Ashijeet Acharya wrote: > > Introduce new object 'BlockdevOptionsSsh' in qapi/block-core.json to > > support blockdev-add for SSH network protocol driver. Use only 'struct > > InetSocketAddress' since SSH only supports con

Re: [Qemu-devel] [PATCH v2 2/2] qapi: allow blockdev-add for NFS

2016-10-26 Thread Markus Armbruster
Kevin Wolf writes: > Am 26.10.2016 um 09:23 hat Markus Armbruster geschrieben: >> A few drive-by comments... >> >> Eric Blake writes: >> >> > On 10/24/2016 02:27 PM, Ashijeet Acharya wrote: >> >> Introduce new object 'BlockdevOptionsNFS' in qapi/block-core.json to >> >> support blockdev-add fo

[Qemu-devel] [PATCH v3] vfio: Add support for mmapping sub-page MMIO BARs

2016-10-26 Thread Yongji Xie
Now the kernel commit 05f0c03fbac1 ("vfio-pci: Allow to mmap sub-page MMIO BARs if the mmio page is exclusive") allows VFIO to mmap sub-page BARs. This is the corresponding QEMU patch. With those patches applied, we could passthrough sub-page BARs to guest, which can help to improve IO performance

Re: [Qemu-devel] [PATCH v2 2/2] qapi: allow blockdev-add for NFS

2016-10-26 Thread Kevin Wolf
Am 26.10.2016 um 10:40 hat Markus Armbruster geschrieben: > Kevin Wolf writes: > > > Am 26.10.2016 um 09:23 hat Markus Armbruster geschrieben: > >> A few drive-by comments... > >> > >> Eric Blake writes: > >> > >> > On 10/24/2016 02:27 PM, Ashijeet Acharya wrote: > >> >> Introduce new object '

Re: [Qemu-devel] [PATCH v2 2/2] qapi: allow blockdev-add for NFS

2016-10-26 Thread Kevin Wolf
Am 26.10.2016 um 09:23 hat Markus Armbruster geschrieben: > A few drive-by comments... > > Eric Blake writes: > > > On 10/24/2016 02:27 PM, Ashijeet Acharya wrote: > >> Introduce new object 'BlockdevOptionsNFS' in qapi/block-core.json to > >> support blockdev-add for NFS network protocol driver.

Re: [Qemu-devel] [PATCH 12/22] qcow2-bitmap: add IN_USE flag

2016-10-26 Thread Vladimir Sementsov-Ogievskiy
25.10.2016 13:53, Vladimir Sementsov-Ogievskiy wrote: 24.10.2016 20:18, Max Reitz wrote: On 24.10.2016 19:08, Max Reitz wrote: On 24.10.2016 13:35, Vladimir Sementsov-Ogievskiy wrote: 24.10.2016 13:32, Vladimir Sementsov-Ogievskiy пишет: 21.10.2016 22:58, Max Reitz пишет: On 21.10.2016 17:34

[Qemu-devel] [Bug 705931] Re: make ui sdl error 1 on git devel

2016-10-26 Thread Thomas Huth
** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/705931 Title: make ui sdl error 1 on git devel Status in QEMU: Incomplete Bug description: a

[Qemu-devel] [Bug 710234] Re: Qemu segfaults at start regardles if i suply an image or not

2016-10-26 Thread Thomas Huth
I assume this has been fixed with one of the later versions of QEMU Or do you still have this issue with the latest version of QEMU? If yes, can you please add the information which Linux distribution you are using, and provide a fresh backtrace with the latest version of QEMU? Thanks! ** Cha

Re: [Qemu-devel] [PATCH 2/4] block/curl: Fix return value from curl_read_cb

2016-10-26 Thread Kevin Wolf
Am 25.10.2016 um 20:37 hat Eric Blake geschrieben: > On 10/24/2016 09:54 PM, Max Reitz wrote: > > While commit 38bbc0a580f9f10570b1d1b5d3e92f0e6feb2970 is correct in that > > the callback is supposed to return the number of bytes handled; what it > > does not mention is that libcurl will throw an e

Re: [Qemu-devel] [PATCH 12/22] qcow2-bitmap: add IN_USE flag

2016-10-26 Thread Vladimir Sementsov-Ogievskiy
26.10.2016 12:04, Vladimir Sementsov-Ogievskiy wrote: 25.10.2016 13:53, Vladimir Sementsov-Ogievskiy wrote: 24.10.2016 20:18, Max Reitz wrote: On 24.10.2016 19:08, Max Reitz wrote: On 24.10.2016 13:35, Vladimir Sementsov-Ogievskiy wrote: 24.10.2016 13:32, Vladimir Sementsov-Ogievskiy пишет:

Re: [Qemu-devel] [PATCH 1/4] block/curl: Use BDRV_SECTOR_SIZE

2016-10-26 Thread Richard W.M. Jones
On Tue, Oct 25, 2016 at 04:54:28AM +0200, Max Reitz wrote: > Currently, curl defines its own constant SECTOR_SIZE. There is no > advantage over using the global BDRV_SECTOR_SIZE, so drop it. > > Cc: qemu-sta...@nongnu.org > Signed-off-by: Max Reitz > --- > block/curl.c | 11 +-- > 1 file

[Qemu-devel] [Bug 1623276] Re: qemu 2.7 / iPXE crash

2016-10-26 Thread Laszlo Ersek (Red Hat)
Can you post the host dmesg that is written at the time of the guest crash? -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1623276 Title: qemu 2.7 / iPXE crash Status in QEMU: New Bug descriptio

Re: [Qemu-devel] [PATCH v2 00/11] qapi doc generation (whole version, squashed)

2016-10-26 Thread Markus Armbruster
Marc-André Lureau writes: > Hi, > > Add a qapi2texi script to generate the documentation from the qapi > schemas. Build various new documentation targets for it: pdf, man, > txt. The 7th patch in this series is a squashed version of the > documentation move from qmp-commands.txt to the schemas. T

Re: [Qemu-devel] [PATCH 2/4] block/curl: Fix return value from curl_read_cb

2016-10-26 Thread Richard W.M. Jones
On Tue, Oct 25, 2016 at 04:54:29AM +0200, Max Reitz wrote: > While commit 38bbc0a580f9f10570b1d1b5d3e92f0e6feb2970 is correct in that > the callback is supposed to return the number of bytes handled; what it > does not mention is that libcurl will throw an error if the callback did > not "handle" a

Re: [Qemu-devel] [PATCH 0/4] block/curl: Fix FTP

2016-10-26 Thread Richard W.M. Jones
As well as testing patch 2 on its own, I also tested all 4 patches together on top of current qemu from git. In summary, it seems to work fine and doesn't break http or https as far as I can tell. Rich. $ http_proxy= LIBGUESTFS_BACKEND=direct LIBGUESTFS_HV=~/d/qemu/x86_64-softmmu/qemu-system-x8

[Qemu-devel] [Bug 905095] Re: qemu-img can't convert vmdk file: Operation not permitted

2016-10-26 Thread Thomas Huth
That "Debian 8.5 (64bit).vmdk" also works fine with the qemu-img from upstream master branch ==> I'm closing this ticket now for upstream, too. ** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscr

Re: [Qemu-devel] [PATCH v2 2/2] qapi: allow blockdev-add for NFS

2016-10-26 Thread Kevin Wolf
Am 25.10.2016 um 23:16 hat Eric Blake geschrieben: > On 10/24/2016 02:27 PM, Ashijeet Acharya wrote: > > Introduce new object 'BlockdevOptionsNFS' in qapi/block-core.json to > > support blockdev-add for NFS network protocol driver. Also make a new > > struct NFSServer to support tcp connection. > >

Re: [Qemu-devel] [PATCH v5 07/13] block/nbd: Use SocketAddress options

2016-10-26 Thread Kevin Wolf
Am 25.10.2016 um 15:11 hat Max Reitz geschrieben: > Drop the use of legacy options in favor of the SocketAddress > representation, even for internal use (i.e. for storing the result of > the filename parsing). > > Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf

Re: [Qemu-devel] [PATCH COLO-Frame (Base) v21 00/17] COarse-grain LOck-stepping(COLO) Virtual Machines for Non-stop Service (FT)

2016-10-26 Thread Li Zhijian
On 10/26/2016 04:26 PM, Amit Shah wrote: On (Wed) 26 Oct 2016 [14:43:30], Hailiang Zhang wrote: Hi Amit, On 2016/10/26 14:09, Amit Shah wrote: Hello, On (Tue) 18 Oct 2016 [20:09:56], zhanghailiang wrote: This is the 21th version of COLO frame series. Rebase to the latest master. I've re

[Qemu-devel] [Bug 1623276] Re: qemu 2.7 / iPXE crash

2016-10-26 Thread Laszlo Ersek (Red Hat)
(I should have given the pattern /sys/module/kvm_intel/parameters/*, but the result is the same.) -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1623276 Title: qemu 2.7 / iPXE crash Status in QEMU:

[Qemu-devel] [Bug 1623276] Re: qemu 2.7 / iPXE crash

2016-10-26 Thread Laszlo Ersek (Red Hat)
Please add the output of the following command too: tail /sys/module/kvm/holders/kvm_intel/parameters/* Thanks. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1623276 Title: qemu 2.7 / iPXE crash

Re: [Qemu-devel] [PATCH v5 08/13] qapi: Allow blockdev-add for NBD

2016-10-26 Thread Kevin Wolf
Am 25.10.2016 um 15:11 hat Max Reitz geschrieben: > Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf

Re: [Qemu-devel] [PATCH v5 09/13] iotests.py: Add qemu_nbd function

2016-10-26 Thread Kevin Wolf
Am 25.10.2016 um 15:11 hat Max Reitz geschrieben: > Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf

Re: [Qemu-devel] [PATCH 0/5] curses: wide character support

2016-10-26 Thread Gerd Hoffmann
On Mi, 2016-10-26 at 00:26 +0200, Samuel Thibault wrote: > Hello, > > Ping? Back online now after being sick for a while. Sorry for the delay. > Samuel Thibault, on Sat 15 Oct 2016 21:53:03 +0200, wrote: > > This patch series adds wide character support to the curses frontend of > > qemu, > > t

Re: [Qemu-devel] [RESEND PATCH v3 kernel 0/7] Extend virtio-balloon for fast (de)inflating & fast live migration

2016-10-26 Thread Li, Liang Z
> On 10/20/2016 11:24 PM, Liang Li wrote: > > Dave Hansen suggested a new scheme to encode the data structure, > > because of additional complexity, it's not implemented in v3. > > So, what do you want done with this patch set? Do you want it applied as-is > so that we can introduce a new host/gu

Re: [Qemu-devel] [PATCH] util/mmap-alloc: check parameter before using

2016-10-26 Thread Cao jin
On 10/26/2016 04:10 PM, Thomas Huth wrote: On 26.10.2016 09:18, Cao jin wrote: Also refactor some code hunk for readability Signed-off-by: Cao jin --- util/mmap-alloc.c | 20 +--- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/util/mmap-alloc.c b/util/mmap-

Re: [Qemu-devel] [RESEND PATCH v3 kernel 0/7] Extend virtio-balloon for fast (de)inflating & fast live migration

2016-10-26 Thread Li, Liang Z
> > On 10/20/2016 11:24 PM, Liang Li wrote: > > > Dave Hansen suggested a new scheme to encode the data structure, > > > because of additional complexity, it's not implemented in v3. > > > > So, what do you want done with this patch set? Do you want it applied > > as-is so that we can introduce a

Re: [Qemu-devel] [PATCH COLO-Frame (Base) v21 00/17] COarse-grain LOck-stepping(COLO) Virtual Machines for Non-stop Service (FT)

2016-10-26 Thread Li Zhijian
On 10/26/2016 04:26 PM, Amit Shah wrote: On (Wed) 26 Oct 2016 [14:43:30], Hailiang Zhang wrote: Hi Amit, On 2016/10/26 14:09, Amit Shah wrote: Hello, On (Tue) 18 Oct 2016 [20:09:56], zhanghailiang wrote: This is the 21th version of COLO frame series. Rebase to the latest master. I've re

Re: [Qemu-devel] [PATCH 3/3] Move getting XWindow ID from baum driver to graphical backend

2016-10-26 Thread Gerd Hoffmann
On So, 2016-10-23 at 21:54 +0200, Samuel Thibault wrote: > This adds two console functions, qemu_console_set_window_id and > qemu_graphic_console_get_window_id, to let graphical backend record the > window id in the QemuConsole structure, and let the baum driver read it. > > We can then move the S

Re: [Qemu-devel] [PATCH COLO-Frame (Base) v21 00/17] COarse-grain LOck-stepping(COLO) Virtual Machines for Non-stop Service (FT)

2016-10-26 Thread Li Zhijian
On 10/26/2016 05:53 PM, Li Zhijian wrote: On 10/26/2016 04:26 PM, Amit Shah wrote: On (Wed) 26 Oct 2016 [14:43:30], Hailiang Zhang wrote: Hi Amit, On 2016/10/26 14:09, Amit Shah wrote: Hello, On (Tue) 18 Oct 2016 [20:09:56], zhanghailiang wrote: This is the 21th version of COLO frame se

[Qemu-devel] [Bug 1623276] Re: qemu 2.7 / iPXE crash

2016-10-26 Thread Greg
Laszlo, I'll grab that info for you soon. In the meantime here's the bug tracker for Arch. Someone has completed a git bisect which may be helpful: https://bugs.archlinux.org/task/50778 -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

[Qemu-devel] [PATCH v12 05/19] block: Use block_job_add_bdrv() in backup_start()

2016-10-26 Thread Alberto Garcia
Use block_job_add_bdrv() instead of blocking all operations in backup_start() and unblocking them in backup_run(). Signed-off-by: Alberto Garcia --- block/backup.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/block/backup.c b/block/backup.c index 02dbe48..2da4ff0 10064

[Qemu-devel] [PATCH v12 04/19] block: Use block_job_add_bdrv() in mirror_start_job()

2016-10-26 Thread Alberto Garcia
Use block_job_add_bdrv() instead of blocking all operations in mirror_start_job() and unblocking them in mirror_exit(). Signed-off-by: Alberto Garcia --- block/mirror.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/block/mirror.c b/block/mirror.c index a433e68..a4273f6 10

[Qemu-devel] [PATCH v12 01/19] block: Add bdrv_drain_all_{begin, end}()

2016-10-26 Thread Alberto Garcia
bdrv_drain_all() doesn't allow the caller to do anything after all pending requests have been completed but before block jobs are resumed. This patch splits bdrv_drain_all() into _begin() and _end() for that purpose. It also adds aio_{disable,enable}_external() calls to disable external clients in

[Qemu-devel] [PATCH v12 07/19] block: Block all nodes involved in the block-commit operation

2016-10-26 Thread Alberto Garcia
After a successful block-commit operation all nodes between top and base are removed from the backing chain, and top's overlay needs to be updated to point to base. Because of that we should prevent other block jobs from messing with them. This patch blocks all operations in these nodes in commit_

[Qemu-devel] [PATCH v12 02/19] block: Pause all jobs during bdrv_reopen_multiple()

2016-10-26 Thread Alberto Garcia
When a BlockDriverState is about to be reopened it can trigger certain operations that need to write to disk. During this process a different block job can be woken up. If that block job completes and also needs to call bdrv_reopen() it can happen that it needs to do it on the same BlockDriverState

[Qemu-devel] [PATCH v12 09/19] block: Support streaming to an intermediate layer

2016-10-26 Thread Alberto Garcia
This makes sure that the image we are streaming into is open in read-write mode during the operation. Operation blockers are also set in all intermediate nodes, since they will be removed from the chain afterwards. Finally, this also unblocks the stream operation in backing files. Signed-off-by:

[Qemu-devel] [PATCH v12 08/19] block: Block all intermediate nodes in commit_active_start()

2016-10-26 Thread Alberto Garcia
When block-commit is launched without the top parameter, it uses internally a mirror block job. In that case all intermediate nodes between the active and base nodes must be blocked as well. Signed-off-by: Alberto Garcia Reviewed-by: Kevin Wolf --- block/mirror.c | 8 1 file changed, 8

[Qemu-devel] [PATCH v12 03/19] block: Add block_job_add_bdrv()

2016-10-26 Thread Alberto Garcia
When a block job is created on a certain BlockDriverState, operations are blocked there while the job exists. However, some block jobs may involve additional BDSs, which must be blocked separately when the job is created and unblocked manually afterwards. This patch adds block_job_add_bdrv(), that

[Qemu-devel] [PATCH v12 17/19] qemu-iotests: Test streaming to a Quorum child

2016-10-26 Thread Alberto Garcia
Quorum children are special in the sense that they're not directly attached to a block backend but they're not used as backing images either. However the intermediate block streaming code supports streaming to them. This is a test case for that scenario. Signed-off-by: Alberto Garcia --- tests/q

[Qemu-devel] [PATCH v12 15/19] qemu-iotests: Test block-stream and block-commit in parallel

2016-10-26 Thread Alberto Garcia
As with test_stream_parallel(), we allow mixing block-stream and block-commit operations in the same backing chain as long as there's no overlap among the involved nodes. Signed-off-by: Alberto Garcia --- tests/qemu-iotests/030 | 30 ++ tests/qemu-iotests/030.out

[Qemu-devel] [PATCH v12 16/19] qemu-iotests: Add iotests.supports_quorum()

2016-10-26 Thread Alberto Garcia
There's many tests that need Quorum support in order to run. At the moment each test implements its own check to see if Quorum is enabled. This patch centralizes all those checks in a new function called iotests.supports_quorum(). Signed-off-by: Alberto Garcia --- tests/qemu-iotests/041|

[Qemu-devel] [PATCH v12 00/19] Support streaming to an intermediate layer

2016-10-26 Thread Alberto Garcia
Hi all, here's version 12 of the series. The problem with version 11 and the bdrv_drain_all() split is that it won't work if new block jobs are created or new BDSs are added between the _begin() and _end() calls. We were discussing this on the mailing list and on IRC. Adding a global quiesce cou

[Qemu-devel] [PATCH v12 06/19] block: Check blockers in all nodes involved in a block-commit job

2016-10-26 Thread Alberto Garcia
qmp_block_commit() checks for op blockers in the active and destination (base) images. However all nodes between top_bs and base are also involved, and they are removed from the chain afterwards. In addition to that, if top_bs is not the active layer then top_bs's overlay also needs to be checked

[Qemu-devel] [PATCH v12 12/19] qemu-iotests: Test streaming to an intermediate layer

2016-10-26 Thread Alberto Garcia
This adds test_stream_intermediate(), similar to test_stream() but streams to the intermediate image instead. It also removes the usage of blkdebug, which is unnecessary for this test. Signed-off-by: Alberto Garcia --- tests/qemu-iotests/030 | 21 - tests/qemu-iotests/03

[Qemu-devel] [PATCH v12 11/19] docs: Document how to stream to an intermediate layer

2016-10-26 Thread Alberto Garcia
Signed-off-by: Alberto Garcia --- docs/live-block-ops.txt | 36 +--- 1 file changed, 25 insertions(+), 11 deletions(-) diff --git a/docs/live-block-ops.txt b/docs/live-block-ops.txt index a257087..2211d14 100644 --- a/docs/live-block-ops.txt +++ b/docs/live-block-

[Qemu-devel] [PATCH v12 10/19] block: Add QMP support for streaming to an intermediate layer

2016-10-26 Thread Alberto Garcia
This patch makes the 'device' parameter of the 'block-stream' command accept a node name that is not a root node. In addition to that, operation blockers will be checked in all intermediate nodes between the top and the base node. Signed-off-by: Alberto Garcia --- blockdev.c | 15

[Qemu-devel] [PATCH v12 19/19] qemu-iotests: Test the 'base-node' parameter of 'block-stream'

2016-10-26 Thread Alberto Garcia
The block-stream command has traditionally used the 'base' parameter to indicate the image to copy the data from. This test checks that the 'base-node' parameter can also be used for the same purpose. Signed-off-by: Alberto Garcia --- tests/qemu-iotests/030 | 33 +

[Qemu-devel] [Bug 1636770] [NEW] mouse wheel works only with -usbdevice tablet

2016-10-26 Thread Bug Report
Public bug reported: 2.7.0 tested with windows 10 ** Affects: qemu Importance: Undecided Status: New -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1636770 Title: mouse wheel works

[Qemu-devel] [PATCH v12 13/19] qemu-iotests: Test block-stream operations in parallel

2016-10-26 Thread Alberto Garcia
This test case checks that it's possible to launch several stream operations in parallel in the same snapshot chain, each one involving a different set of nodes. Signed-off-by: Alberto Garcia --- tests/qemu-iotests/030 | 80 ++ tests/qemu-iotests/0

[Qemu-devel] [PATCH v12 18/19] block: Add 'base-node' parameter to the 'block-stream' command

2016-10-26 Thread Alberto Garcia
The way to specify the node from which to copy data in the block-stream operation is by using the 'base' parameter. This parameter however takes a file name, not a node name. Since we want to be able to perform this operation using only node names, this patch adds a new 'base-node' parameter. Sig

Re: [Qemu-devel] [PATCH RFC] acpi: don't build acpi tables for xen hvm guests

2016-10-26 Thread Wei Liu
On Tue, Oct 25, 2016 at 04:33:03PM -0700, Stefano Stabellini wrote: > CC'ing maintainers > > On Tue, 25 Oct 2016, Wei Liu wrote: > > Xen's toolstack is in charge of building ACPI tables. Skip acpi table > > building if running on Xen. > > > > This issue is discovered due to direct kernel boot on

Re: [Qemu-devel] [PATCH] exec.c: workaround regression caused by alignment change in d2f39ad

2016-10-26 Thread Christian Borntraeger
On 10/24/2016 02:58 PM, Paolo Bonzini wrote: > > > On 24/10/2016 14:49, Haozhong Zhang wrote: >> Commit d2f39ad "exec.c: Ensure right alignment also for file backed ram" >> added an additional alignment requirement on the size of backend file >> besides the previous page size. On x86, the alignme

[Qemu-devel] [Bug 1619438] Re: GTK+ UI, delete key deletes to the left in the monitor

2016-10-26 Thread Bug Report
2.7.0 this issue has not been fixed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1619438 Title: GTK+ UI, delete key deletes to the left in the monitor Status in QEMU: New Bug description: i

[Qemu-devel] [PATCH v12 14/19] qemu-iotests: Test overlapping stream and commit operations

2016-10-26 Thread Alberto Garcia
These test cases check that it's not possible to perform two block-stream or block-commit operations if there are nodes involved in both. Signed-off-by: Alberto Garcia --- tests/qemu-iotests/030 | 89 ++ tests/qemu-iotests/030.out | 4 +-- 2 files

Re: [Qemu-devel] [PATCH v5 12/13] iotests: Add assert_json_filename_equal() method

2016-10-26 Thread Kevin Wolf
Am 25.10.2016 um 15:11 hat Max Reitz geschrieben: > Since the order of keys in JSON filenames is not necessarily fixed, they > should not be compared to fixed strings. This method takes a Python dict > as a reference, parses a given JSON filename and compares both. > > Signed-off-by: Max Reitz >

Re: [Qemu-devel] [PATCH v8 00/36] block: Image locking series

2016-10-26 Thread Fam Zheng
On Tue, 10/25 16:57, Kevin Wolf wrote: > Am 25.10.2016 um 15:30 hat Max Reitz geschrieben: > > On 25.10.2016 10:24, Kevin Wolf wrote: > > > Am 24.10.2016 um 20:03 hat Max Reitz geschrieben: > > >> On 24.10.2016 12:11, Kevin Wolf wrote: > > >> > > >> [...] > > >> > > >>> Now, the big question is how

Re: [Qemu-devel] [PATCH 2/2] tests: send error_report to test log

2016-10-26 Thread Halil Pasic
On 10/25/2016 01:34 PM, Paolo Bonzini wrote: > > On 25/10/2016 13:06, Halil Pasic wrote: >> > >> > >> > On 10/24/2016 06:31 PM, Paolo Bonzini wrote: >>> >> Implement error_vprintf to send the output of error_report to >>> >> the test log. This silences test-vmstate. >>> >> >>> >> Signed-off-b

[Qemu-devel] [PATCH v2] util/mmap-alloc: check parameter before using

2016-10-26 Thread Cao jin
Also refactor a little for readability Signed-off-by: Cao jin --- util/mmap-alloc.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/util/mmap-alloc.c b/util/mmap-alloc.c index 5a85aa3..a32ee53 100644 --- a/util/mmap-alloc.c +++ b/util/mmap-alloc.c @@ -61,7 +61,7

Re: [Qemu-devel] [PATCH 3/3] Move getting XWindow ID from baum driver to graphical backend

2016-10-26 Thread Samuel Thibault
Hello, Gerd Hoffmann, on Wed 26 Oct 2016 12:17:44 +0200, wrote: > On So, 2016-10-23 at 21:54 +0200, Samuel Thibault wrote: > > This adds two console functions, qemu_console_set_window_id and > > qemu_graphic_console_get_window_id, to let graphical backend record the > > window id in the QemuConsol

Re: [Qemu-devel] [PATCH v3 00/13] Refactor common part of xen backend and frontend

2016-10-26 Thread Anthony PERARD
On Tue, Oct 25, 2016 at 08:50:06AM +0300, Emil Condrea wrote: > This patch series was splitted from QEMU:Xen stubdom vTPM for HVM virtual > machine > http://markmail.org/message/fkix7g3a5zdj7lvr > > It contains a reorganization of xen backend and frontend functions together > with code style fixe

[Qemu-devel] [PULL 03/49] tests: fix memory leak in virtio-scsi-test

2016-10-26 Thread David Gibson
From: Laurent Vivier vs is allocated in qvirtio_scsi_pci_init() and never freed. Signed-off-by: Laurent Vivier Reviewed-by: Greg Kurz Reviewed-by: Thomas Huth Signed-off-by: David Gibson --- tests/virtio-scsi-test.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/virtio-scsi-test.

[Qemu-devel] [PULL 02/49] ppc/xics: Add xics to the monitor "info pic" command

2016-10-26 Thread David Gibson
From: Benjamin Herrenschmidt Useful to debug interrupt problems. Signed-off-by: Benjamin Herrenschmidt [clg: - updated for qemu-2.7 - added a test on ->irqs as it is not necessarily allocated (PHB3_MSI) - removed static variable g_xics and replace with a loop on all

[Qemu-devel] [PULL 10/49] nvram: Introduce helper functions for CHRP "system" and "free space" partitions

2016-10-26 Thread David Gibson
From: Thomas Huth The "system partition" and "free space" partition layouts are defined by the CHRP and LoPAPR specification, and used by OpenBIOS and SLOF. We can re-use this code for other machines that use OpenBIOS and SLOF, too. So let's make this code independent from the MAC NVRAM environme

[Qemu-devel] [PULL 21/49] ppc: Fix single step with gdb stub

2016-10-26 Thread David Gibson
From: Benjamin Herrenschmidt Signed-off-by: Benjamin Herrenschmidt Signed-off-by: David Gibson --- target-ppc/translate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target-ppc/translate.c b/target-ppc/translate.c index 94989b2..43505a9 100644 --- a/target-ppc/translate

[Qemu-devel] [PULL 04/49] tests: don't check if qtest_spapr_boot() returns NULL

2016-10-26 Thread David Gibson
From: Laurent Vivier qtest_spapr_boot()/qtest_pc_boot()/qtest_boot() call qtest_vboot() and qtest_vboot() calls g_malloc(), and g_malloc() never fails: if memory allocation fails, the application is terminated. Signed-off-by: Laurent Vivier Reviewed-by: Thomas Huth Reviewed-by: Greg Kurz Sign

[Qemu-devel] [PULL 09/49] spapr_pci: advertise explicit numa IDs even when there's 1 node

2016-10-26 Thread David Gibson
From: Michael Roth With the addition of "numa_node" properties for PHBs we began advertising NUMA affinity in cases where nb_numa_nodes > 1. Since the default on the guest side is to make no assumptions about PHB NUMA affinity (defaulting to -1), there is still a valid use-case for explicitly de

Re: [Qemu-devel] [PATCH 0/5] curses: wide character support

2016-10-26 Thread Samuel Thibault
Hello, Gerd Hoffmann, on Wed 26 Oct 2016 12:05:55 +0200, wrote: > Back online now after being sick for a while. Oh, sorry about that, hoping you're fine. > > Samuel Thibault, on Sat 15 Oct 2016 21:53:03 +0200, wrote: > > > This patch series adds wide character support to the curses frontend of

[Qemu-devel] [PULL 05/49] tests: move QVirtioBus pointer into QVirtioDevice

2016-10-26 Thread David Gibson
From: Laurent Vivier This allows to not have to pass bus and device for every virtio functions. Signed-off-by: Laurent Vivier Reviewed-by: Greg Kurz Reviewed-by: Thomas Huth [dwg: Fix style nit] Signed-off-by: David Gibson --- tests/libqos/virtio-mmio.c | 1 + tests/libqos/virtio-pci.c |

[Qemu-devel] [PULL 19/49] ppc: fix MSR_ME handling for system reset interrupt

2016-10-26 Thread David Gibson
From: Nicholas Piggin Power ISA specifies ME bit handling for system reset interrupt: if the interrupt occurred while the thread was in power-saving mode, set to 1; otherwise not altered Power ISA 3.0, section 6.5 "Interrupt Definitions", Figure 64. Signed-off-by: Nicholas Piggin Revi

[Qemu-devel] [PULL 00/49] ppc-for-2.8 queue 20161026

2016-10-26 Thread David Gibson
Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2016-10-25' into staging (2016-10-25 17:03:11 +0100) are available in the git repository at: git://github.com/dgibson/qemu.git tags/ppc-for-2.8-20161026 for you to fetch changes up to acb8eed32d455851773be209a1d9cd

[Qemu-devel] [PULL 07/49] tests: use qtest_pc_boot()/qtest_shutdown() in virtio tests

2016-10-26 Thread David Gibson
From: Laurent Vivier This patch replaces calls to qtest_start() and qtest_end() by calls to qtest_pc_boot() and qtest_shutdown(). This allows to initialize memory allocator and PCI interface functions. This will ease to enable virtio tests on other architectures by only adding a specific qtest_X

[Qemu-devel] [PULL 20/49] pseries: Remove unused callbacks from sPAPR VIO bus state

2016-10-26 Thread David Gibson
The original QOMification of the spapr VIO devices in 3954d33 "spapr: convert to QEMU Object Model (v2)" moved some callbacks from the VIOsPAPRBus structure to the VIOsPAPRDeviceClass. Except, that it forgot to actually remove them from the VIOsPAPRBus structure (which still exists, though it does

[Qemu-devel] [PULL 08/49] tests: enable virtio tests on SPAPR

2016-10-26 Thread David Gibson
From: Laurent Vivier but disable MSI-X tests on SPAPR as we can't check the result (the memory region used on PC is not readable on SPAPR). Signed-off-by: Laurent Vivier Signed-off-by: David Gibson --- tests/Makefile.include| 3 ++- tests/libqos/virtio-pci.c | 24 ++--

[Qemu-devel] [PULL 12/49] nvram: Move the remaining CHRP NVRAM related code to chrp_nvram.[ch]

2016-10-26 Thread David Gibson
From: Thomas Huth Everything that is related to CHRP NVRAM should rather reside in chrp_nvram.c / chrp_nvram.h instead of openbios_firmware_abi.h. Signed-off-by: Thomas Huth Tested-by: Mark Cave-Ayland Signed-off-by: David Gibson --- hw/nvram/chrp_nvram.c| 31

[Qemu-devel] [PULL 06/49] tests: rename target_big_endian() as qvirtio_is_big_endian()

2016-10-26 Thread David Gibson
From: Laurent Vivier Move the definition to libqos/virtio.h as it must be used only with virtio functions. Add a QVirtioDevice parameter as it will be needed to know if the virtio device is using virtio 1.0 specification and thus is always little-endian (to do) Signed-off-by: Laurent Vivier Re

[Qemu-devel] [PULL 31/49] ppc/pnv: add a ISA bus

2016-10-26 Thread David Gibson
From: Cédric Le Goater As Qemu only supports a single instance of the ISA bus, we use the LPC controller of chip 0 to create one and plug in a couple of useful devices, like an UART and RTC. An IPMI BT device, which is also an ISA device, can be defined on the command line to connect an external

[Qemu-devel] [PULL 14/49] target-ppc: implement vnegw/d instructions

2016-10-26 Thread David Gibson
From: Nikunj A Dadhania Vector Integer Negate Instructions: vnegw: Vector Negate Word vnegd: Vector Negate Doubleword Signed-off-by: Nikunj A Dadhania Reviewed-by: Thomas Huth Reviewed-by: Richard Henderson Signed-off-by: David Gibson --- target-ppc/helper.h | 2 ++ target

[Qemu-devel] [PULL 33/49] pseries: Split device tree construction from device tree load

2016-10-26 Thread David Gibson
spapr_finalize_fdt() both finishes building the device tree for the guest and loads it into guest memory. For future cleanups, it's going to be more convenient to do these two things separately. The loading portion is pretty trivial, so we move it inline into the caller, ppc_spapr_reset(). We al

[Qemu-devel] [PULL 37/49] pseries: Consolidate RTAS loading

2016-10-26 Thread David Gibson
At each system reset, the pseries machine needs to load RTAS (the runtime portion of the guest firmware) into the VM. This means copying the actual RTAS code into guest memory, and also updating the device tree so that the guest OS and boot firmware can locate it. For historical reasons the copy

  1   2   3   4   5   6   >