Re: [Qemu-devel] [Nbd] [Qemu-block] How to online resize qemu disk with nbd protocol?

2017-11-16 Thread Eric Blake
On 11/16/2017 03:51 AM, Wouter Verhelst wrote: >> I also remember from talking with Vladimir during KVM Forum last month >> that one of the shortfalls of the NBD protocol is that you can only ever >> send a length of up to 32 bits on the command side (unless we introduce >> structured commands in

Re: [Qemu-devel] QEMU terminates during reboot after memory unplug with vhost=on

2017-11-16 Thread Greg Kurz
On Thu, 16 Nov 2017 17:28:44 +0200 "Michael S. Tsirkin" wrote: > On Thu, Nov 16, 2017 at 01:23:39PM +0100, Greg Kurz wrote: > > Hi, > > > > I'm resurrecting a thread about a QEMU crash we're still hitting on ppc64. > > It > > was reported to the list by Bharata 2 months ago: >

Re: [Qemu-devel] [PATCH v6 5/5] fw_cfg: write vmcoreinfo details

2017-11-16 Thread Michael S. Tsirkin
On Thu, Nov 16, 2017 at 08:34:23AM -0500, Marc-André Lureau wrote: > Hi > > - Original Message - > > On Mon, Nov 13, 2017 at 08:29:58PM +0100, Marc-André Lureau wrote: > > > If the "etc/vmcoreinfo" fw_cfg file is present and we are not running > > > the kdump kernel, write the addr/size

Re: [Qemu-devel] [PULL 00/10] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Michael S. Tsirkin
On Thu, Nov 16, 2017 at 04:10:22PM +, Peter Maydell wrote: > On 16 November 2017 at 15:56, Michael S. Tsirkin wrote: > > On Thu, Nov 16, 2017 at 03:45:34PM +, Daniel P. Berrange wrote: > >> On Thu, Nov 16, 2017 at 05:38:18PM +0200, Michael S. Tsirkin wrote: > >> > I think

Re: [Qemu-devel] [Question] Qemu's Heap Becomes Very Large and Never Reduce Down

2017-11-16 Thread Zhong Yang
On Wed, Nov 15, 2017 at 02:18:55PM +0100, Paolo Bonzini wrote: > On 15/11/2017 04:14, Xulei (Stone) wrote: > > Hi, guys > > > > I met a strange problem, with qemu 2.8.1: > > qemu consumes too many heap memory after several operations and can not > > release them anymore:  > > hot pulg/unplug disk

Re: [Qemu-devel] [PATCH] ipmi: check ibe status before ibe outlen at ipmi_bmc_extern_handle_command

2017-11-16 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH] ipmi: check ibe status before ibe outlen at ipmi_bmc_extern_handle_command Type: series Message-id: 20171116052511.16236-1-caoxin...@huawei.com === TEST SCRIPT BEGIN

[Qemu-devel] [RFC v4 03/27] qobject: let object_property_get_str() use new API

2017-11-16 Thread Peter Xu
We can simplify object_property_get_str() using the new qobject_get_try_str(). Reviewed-by: Fam Zheng Signed-off-by: Peter Xu --- qom/object.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/qom/object.c b/qom/object.c index

[Qemu-devel] [RFC v4 07/27] monitor: unify global init

2017-11-16 Thread Peter Xu
There are many places for monitor init its globals, at least: - monitor_init_qmp_commands() at the very beginning - single function to init monitor_lock - in the first entry of monitor_init() using "is_first_init" Unify them a bit. Reviewed-by: Fam Zheng Signed-off-by: Peter

[Qemu-devel] [RFC v4 10/27] monitor: allow to use IO thread for parsing

2017-11-16 Thread Peter Xu
For each Monitor, add one field "use_io_thr" to show whether it will be using the dedicated monitor IO thread to handle input/output. When set, monitor IO parsing work will be offloaded to dedicated monitor IO thread, rather than the original main loop thread. This only works for QMP. HMP will

[Qemu-devel] [RFC v4 11/27] qmp: introduce QMPCapability

2017-11-16 Thread Peter Xu
There was no QMP capabilities defined. Define the first "oob" as capability to allow out-of-band messages. Also, touch up qmp-test.c to test the new bits. Signed-off-by: Peter Xu --- monitor.c| 15 +-- qapi-schema.json | 13 +

[Qemu-devel] [RFC v4 19/27] qapi: introduce new cmd option "allow-oob"

2017-11-16 Thread Peter Xu
Here "oob" stands for "Out-Of-Band". When "allow-oob" is set, it means the command allows out-of-band execution. The "oob" idea is proposed by Markus Armbruster in following thread: https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg02057.html This new "allow-oob" boolean will be

Re: [Qemu-devel] [virtio-dev] [PATCH 1/1] virtio-balloon: include buffers and chached memory statistics

2017-11-16 Thread Yan Vugenfirer
Hello All, Michael asked me to comment regarding the possibility to get this statistics on Windows. By using Win32_OperatingSystem WMI class we can calculate the amount of cached memory (https://msdn.microsoft.com/en-us/library/aa394239(v=vs.85).aspx

Re: [Qemu-devel] [PATCH v3] net: Transmit zero UDP checksum as 0xFFFF

2017-11-16 Thread Ed Swierk via Qemu-devel
On Wed, Nov 15, 2017 at 7:55 PM, Eric Blake wrote: > > On 11/15/2017 09:07 PM, Ed Swierk via Qemu-devel wrote: > > This part below... > > > > > v3: > > > > Leave net_tx_pkt_update_ip_checksums() alone since it's only computing > > a partial sum of the IP pseudo-header. > > > >

[Qemu-devel] [PATCH v2 for-2.11] migration, xen: Fix block image lock issue on live migration

2017-11-16 Thread Anthony PERARD
When doing a live migration of a Xen guest with libxl, the images for block devices are locked by the original QEMU process, and this prevent the QEMU at the destination to take the lock and the migration fail. >From QEMU point of view, once the RAM of a domain is migrated, there is two QMP

Re: [Qemu-devel] [PULL 00/10] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Michael S. Tsirkin
On Thu, Nov 16, 2017 at 02:41:09PM +, Peter Maydell wrote: > On 15 November 2017 at 18:17, Michael S. Tsirkin wrote: > > The following changes since commit 1fa0f627d03cd0d0755924247cafeb42969016bf: > > > > Update version for v2.11.0-rc1 release (2017-11-14 18:37:49 +) >

Re: [Qemu-devel] QEMU terminates during reboot after memory unplug with vhost=on

2017-11-16 Thread Michael S. Tsirkin
On Thu, Nov 16, 2017 at 08:53:45AM -0700, Alex Williamson wrote: > On Thu, 16 Nov 2017 17:28:44 +0200 > "Michael S. Tsirkin" wrote: > > > On Thu, Nov 16, 2017 at 01:23:39PM +0100, Greg Kurz wrote: > > > Hi, > > > > > > I'm resurrecting a thread about a QEMU crash we're still

Re: [Qemu-devel] [RFC PATCH v2 01/21] ppc/xive: introduce a skeleton for the sPAPR XIVE interrupt controller

2017-11-16 Thread Cédric Le Goater
On 09/26/2017 05:54 AM, David Gibson wrote: > On Fri, Sep 22, 2017 at 02:42:07PM +0200, Cédric Le Goater wrote: >> On 09/22/2017 01:00 PM, David Gibson wrote: >>> On Tue, Sep 19, 2017 at 03:15:44PM +0200, Cédric Le Goater wrote: On 09/19/2017 04:27 AM, David Gibson wrote: > On Mon, Sep

Re: [Qemu-devel] [PULL 00/11] Miscellaneous patches for QEMU 2.11-rc2

2017-11-16 Thread Peter Maydell
On 16 November 2017 at 11:59, Paolo Bonzini wrote: > The following changes since commit d24aaf2a2915424962fb101142f28fa4307f4740: > > Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' > into staging (2017-11-06 11:24:14 +) > > are available in the

Re: [Qemu-devel] [PULL 00/10] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Peter Maydell
On 16 November 2017 at 15:56, Michael S. Tsirkin wrote: > On Thu, Nov 16, 2017 at 03:45:34PM +, Daniel P. Berrange wrote: >> On Thu, Nov 16, 2017 at 05:38:18PM +0200, Michael S. Tsirkin wrote: >> > I think it's test: fix detection of errors from iasl >> > I'll drop it for

Re: [Qemu-devel] QEMU terminates during reboot after memory unplug with vhost=on

2017-11-16 Thread Michael S. Tsirkin
On Thu, Nov 16, 2017 at 09:04:48AM -0700, Alex Williamson wrote: > > Range is unmapped by listener but device has not been reset yet, > > so it will get errors when attempting to access guest RAM. > > So IOMMU faults would occur if the device continues to try to DMA > into that range. Sloppy

[Qemu-devel] [PULL 4/9] hw/pcie-pci-bridge: restrict to X86 and ARM

2017-11-16 Thread Michael S. Tsirkin
From: Marcel Apfelbaum The PCIE-PCI bridge is specific to "pure" PCIe systems (on QEMU we have X86 and ARM), it does not make sense to have it in other archs. Reported-by: Thomas Huth Signed-off-by: Marcel Apfelbaum Reviewed-by: Thomas

[Qemu-devel] [PATCH v2 1/4] qcow2: multiple clusters write compressed

2017-11-16 Thread Anton Nefedov
From: Pavel Butsykin At the moment, qcow2_co_pwritev_compressed can process the requests size less than or equal to one cluster. This patch added possibility to write compressed data in the QCOW2 more than one cluster. The implementation is simple, we just split large

Re: [Qemu-devel] [PATCH] hw/arm: Silence xlnx-ep108 deprecation warning during tests

2017-11-16 Thread Wei Huang
On 11/16/2017 09:29 AM, Thomas Huth wrote: > The new deprecation warning for the xlnx-ep108 machine also pops up > during "make check" which is kind of confusing. Silence it if testing > mode is enabled. > > Signed-off-by: Thomas Huth > --- > Note: I slightly shortened the

Re: [Qemu-devel] [PATCH] ipmi: check ibe status before ibe outlen at ipmi_bmc_extern_handle_command

2017-11-16 Thread no-reply
Hi, This series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH] ipmi: check ibe status before ibe outlen at ipmi_bmc_extern_handle_command Type: series

[Qemu-devel] [PATCH] hw/arm: Silence xlnx-ep108 deprecation warning during tests

2017-11-16 Thread Thomas Huth
The new deprecation warning for the xlnx-ep108 machine also pops up during "make check" which is kind of confusing. Silence it if testing mode is enabled. Signed-off-by: Thomas Huth --- Note: I slightly shortened the message so that the code still fits into the 80-columns

Re: [Qemu-devel] [PULL 00/10] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Peter Maydell
On 16 November 2017 at 16:13, Daniel P. Berrange wrote: > On Thu, Nov 16, 2017 at 04:10:22PM +, Peter Maydell wrote: >> If there are known-buggy iasl versions in the wild we could >> consider a configure test and treat buggy-iasl like no-iasl. > > iasl has long been

[Qemu-devel] [PULL for-2.11 0/1] Block patches

2017-11-16 Thread Stefan Hajnoczi
The following changes since commit 8048082f7a11040a366942a2de8abb4c3d0020c9: Merge remote-tracking branch 'remotes/stefanberger/tags/pull-tpm-2017-11-15-1' into staging (2017-11-16 11:34:24 +) are available in the git repository at: git://github.com/stefanha/qemu.git

[Qemu-devel] [PULL for-2.11 1/1] throttle-groups: forget timer and schedule next TGM on detach

2017-11-16 Thread Stefan Hajnoczi
tg->any_timer_armed[] must be cleared when detaching pending timers from the AioContext. Failure to do so leads to hung I/O because it looks like there are still timers pending when in fact they have been removed. Other ThrottleGroupMembers might have requests pending too so it's necessary to

[Qemu-devel] [PULL 5/9] tests/acpi-test-data: update _CRS in DSDT

2017-11-16 Thread Michael S. Tsirkin
commit dadf988e81b15065ac1d6dbaf4b87b5b80c7b670 hw/pci-host: Fix x86 Host Bridges 64bit PCI hole Added a 64 bit hole to _CRS of PCI0. Update the expected files accordingly. Signed-off-by: Michael S. Tsirkin --- tests/acpi-test-data/pc/DSDT | Bin 5098 -> 5144 bytes

Re: [Qemu-devel] [PATCH v17 6/6] virtio-balloon: VIRTIO_BALLOON_F_FREE_PAGE_VQ

2017-11-16 Thread Wei Wang
On 11/15/2017 09:26 PM, Michael S. Tsirkin wrote: On Wed, Nov 15, 2017 at 11:47:58AM +0800, Wei Wang wrote: On 11/15/2017 05:21 AM, Michael S. Tsirkin wrote: On Tue, Nov 14, 2017 at 08:02:03PM +0800, Wei Wang wrote: On 11/14/2017 01:32 AM, Michael S. Tsirkin wrote: - guest2host_cmd: written

[Qemu-devel] [PATCH v1 for-2.11 2/3] s390x/tcg: fix DIAG 308 with > 1 VCPU (single threaded TCG)

2017-11-16 Thread David Hildenbrand
Currently, single threaded TCG with > 1 VCPU gets stuck during IPL, when the bios tries to switch to the loaded kernel via DIAG 308. pause_all_vcpus()/resume_all_vcpus() should not be called from a VCPU. And there is also no need for it. run_on_cpu() will make sure that the CPUs exit KVM/TCG,

Re: [Qemu-devel] Effect of qemu-img convert -m and -W options

2017-11-16 Thread Richard W.M. Jones
On Thu, Nov 16, 2017 at 03:51:29PM +0100, Peter Lieven wrote: > Afaik all writes to the same QCOW2 serialize because of the s->lock that > is held during the write. So its not suprising that there is no benefit from > mutliple threads as long as reading from the RAW file involves no delay. > Which

Re: [Qemu-devel] QEMU terminates during reboot after memory unplug with vhost=on

2017-11-16 Thread Michael S. Tsirkin
On Thu, Nov 16, 2017 at 01:23:39PM +0100, Greg Kurz wrote: > Hi, > > I'm resurrecting a thread about a QEMU crash we're still hitting on ppc64. It > was reported to the list by Bharata 2 months ago: > > https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg03685.html > > "Hi, > > QEMU hits

Re: [Qemu-devel] [PULL 00/10] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Daniel P. Berrange
On Thu, Nov 16, 2017 at 05:38:18PM +0200, Michael S. Tsirkin wrote: > On Thu, Nov 16, 2017 at 02:41:09PM +, Peter Maydell wrote: > > On 15 November 2017 at 18:17, Michael S. Tsirkin wrote: > > > The following changes since commit > > >

Re: [Qemu-devel] [Qemu-block] segfault in parallel blockjobs (iotest 30)

2017-11-16 Thread Alberto Garcia
On Wed 15 Nov 2017 05:31:20 PM CET, Anton Nefedov wrote: >> I have the impression that one major source of headaches is the fact >> that the reopen queue contains nodes that don't need to be reopened at >> all. Ideally this should be detected early on in bdrv_reopen_queue(), so >> there's no

Re: [Qemu-devel] [Nbd] [Qemu-block] How to online resize qemu disk with nbd protocol?

2017-11-16 Thread Wouter Verhelst
On Thu, Nov 16, 2017 at 09:30:41AM -0600, Eric Blake wrote: > On 11/16/2017 03:51 AM, Wouter Verhelst wrote: > > >> I also remember from talking with Vladimir during KVM Forum last month > >> that one of the shortfalls of the NBD protocol is that you can only ever > >> send a length of up to 32

[Qemu-devel] [PULL 3/9] hw/pci-host: Fix x86 Host Bridges 64bit PCI hole

2017-11-16 Thread Michael S. Tsirkin
From: Marcel Apfelbaum Currently there is no MMIO range over 4G reserved for PCI hotplug. Since the 32bit PCI hole depends on the number of cold-plugged PCI devices and other factors, it is very possible is too small to hotplug PCI devices with large BARs. Fix it by reserving

[Qemu-devel] [PULL 1/9] fix: unrealize virtio device if we fail to hotplug it

2017-11-16 Thread Michael S. Tsirkin
From: linzhecheng If we fail to hotplug virtio-blk device and then suspend or shutdown VM, qemu is likely to crash. Re-production steps: 1. Run VM named vm001 2. Create a virtio-blk.xml which contains wrong configurations: 3. Run command : virsh attach-device

Re: [Qemu-devel] [PATCH v8 10/14] migration: add postcopy migration of dirty bitmaps

2017-11-16 Thread Dr. David Alan Gilbert
* John Snow (js...@redhat.com) wrote: > > > On 10/30/2017 12:33 PM, Vladimir Sementsov-Ogievskiy wrote: > > Postcopy migration of dirty bitmaps. Only named dirty bitmaps, > > associated with root nodes and non-root named nodes are migrated. > > > > If destination qemu is already containing a

Re: [Qemu-devel] [PATCH] hw/arm: Silence xlnx-ep108 deprecation warning during tests

2017-11-16 Thread Alistair Francis
On Thu, Nov 16, 2017 at 7:29 AM, Thomas Huth wrote: > The new deprecation warning for the xlnx-ep108 machine also pops up > during "make check" which is kind of confusing. Silence it if testing > mode is enabled. > > Signed-off-by: Thomas Huth > --- > Note: I

Re: [Qemu-devel] [PULL 00/10] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Thomas Huth
On 16.11.2017 15:41, Peter Maydell wrote: > On 15 November 2017 at 18:17, Michael S. Tsirkin wrote: >> The following changes since commit 1fa0f627d03cd0d0755924247cafeb42969016bf: >> >> Update version for v2.11.0-rc1 release (2017-11-14 18:37:49 +) >> >> are available in

Re: [Qemu-devel] [PATCH] tests: report errors when iasl exits with non-zero status

2017-11-16 Thread Michael S. Tsirkin
On Wed, Oct 18, 2017 at 02:15:28PM +0100, Daniel P. Berrange wrote: > If iasl exits with non-zero status, the test unhelpfully just reports > that the AML did not match, because the data files it thought iasl > generated do not exist. This adds an explicit check for the exit status > of iasl and

Re: [Qemu-devel] [PATCH] test: fix detection of errors from iasl

2017-11-16 Thread Michael S. Tsirkin
On Wed, Oct 18, 2017 at 02:14:56PM +0100, Daniel P. Berrange wrote: > The conditional looking for errors while loading asl files would ignore > errors from loading the expected data, if the actual data succeeded. > > Signed-off-by: Daniel P. Berrange Problem is, some

Re: [Qemu-devel] [PULL 00/10] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Michael S. Tsirkin
On Thu, Nov 16, 2017 at 03:45:34PM +, Daniel P. Berrange wrote: > On Thu, Nov 16, 2017 at 05:38:18PM +0200, Michael S. Tsirkin wrote: > > On Thu, Nov 16, 2017 at 02:41:09PM +, Peter Maydell wrote: > > > On 15 November 2017 at 18:17, Michael S. Tsirkin wrote: > > > > The

[Qemu-devel] 2 bytes of memory leaking due to MigrationParameters unshadowing

2017-11-16 Thread Christian Borntraeger
Markus, valgrind complains about the following two 1 byte memory leaks during migration: ==40773== 1 bytes in 1 blocks are definitely lost in loss record 3 of 2,127 ==40773==at 0x4834E60: malloc (vg_replace_malloc.c:299) ==40773==by 0x18B1A177: g_malloc (in

Re: [Qemu-devel] [Qemu-block] segfault in parallel blockjobs (iotest 30)

2017-11-16 Thread Anton Nefedov
On 16/11/2017 6:54 PM, Alberto Garcia wrote: On Wed 15 Nov 2017 05:31:20 PM CET, Anton Nefedov wrote: I have the impression that one major source of headaches is the fact that the reopen queue contains nodes that don't need to be reopened at all. Ideally this should be detected early on in

[Qemu-devel] [PATCH for-2.11] Fix build of console and GUI executables for Windows

2017-11-16 Thread Stefan Weil
It was broken by commit 8ecc89f6e792152496eccb684d6c8c48aba8027d which moved the SDL linker flags from macro libs_softmmu to macro SDL_LIBS. Signed-off-by: Stefan Weil --- Peter, can you apply this fix directly, or do you need a pull request? Regards Stefan Makefile.target

[Qemu-devel] [PULL 2/9] pci: Initialize pci_dev->name before use

2017-11-16 Thread Michael S. Tsirkin
From: Alexey Kardashevskiy This moves pci_dev->name initialization earlier so pci_dev->bus_master_as could get a name instead of an empty string. Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Peter Xu Signed-off-by: Alexey Kardashevskiy

Re: [Qemu-devel] [PULL 6/9] NUMA: Enable adding NUMA node implicitly

2017-11-16 Thread Thadeu Lima de Souza Cascardo
Acked-by: Thadeu Lima de Souza Cascardo

[Qemu-devel] [PATCH v2 2/4] block: support compressed write for copy-on-read

2017-11-16 Thread Anton Nefedov
Signed-off-by: Anton Nefedov Reviewed-by: Max Reitz --- block/io.c | 23 +-- block/trace-events | 2 +- 2 files changed, 18 insertions(+), 7 deletions(-) diff --git a/block/io.c b/block/io.c index 3d5ef2c..2ba62a3

[Qemu-devel] [PATCH v2 3/4] block-stream: add compress option

2017-11-16 Thread Anton Nefedov
It might be useful to compress images during block-stream; this way the user can merge compressed images of a backing chain and the result will remain compressed. Signed-off-by: Anton Nefedov --- qapi/block-core.json | 4 include/block/block_int.h | 4

Re: [Qemu-devel] [PATCH for-2.11] target/arm: Report GICv3 sysregs present in ID registers if needed

2017-11-16 Thread Alistair Francis
On Wed, Nov 15, 2017 at 3:57 AM, Peter Maydell wrote: > On 15 November 2017 at 01:14, Alistair Francis wrote: >> On Tue, Nov 7, 2017 at 7:01 AM, Peter Maydell >> wrote: >>> The CPU ID registers ID_AA64PFR0_EL1,

[Qemu-devel] [PATCH v1 for-2.11 3/3] s390x/tcg: fix DIAG 308 with > 1 VCPU (MTTCG)

2017-11-16 Thread David Hildenbrand
Currently, multi threaded TCG with > 1 VCPU gets stuck during IPL, when the bios tries to switch to the loaded kernel via DIAG 308. As run_on_cpu() is used, we run into a deadlock after handling the reset. We need the iolock (just like KVM). Signed-off-by: David Hildenbrand

Re: [Qemu-devel] [PATCH v1 for-2.11 0/3] s390x: fixes for SIGP and DIAG 308

2017-11-16 Thread Cornelia Huck
On Thu, 16 Nov 2017 18:05:23 +0100 David Hildenbrand wrote: > First one also applies to KVM SIGP handling. The other two only > apply to TCG (allowing to IPL from disk with more than 1 VCPU- I > never tested that before). > > David Hildenbrand (3): > s390x: fix storing CPU

Re: [Qemu-devel] [PATCH v6 3/5] fw_cfg: do DMA read operation

2017-11-16 Thread Michael S. Tsirkin
On Thu, Nov 16, 2017 at 08:17:12AM -0500, Marc-André Lureau wrote: > Hi > > - Original Message - > > On Mon, Nov 13, 2017 at 08:29:56PM +0100, Marc-André Lureau wrote: > > > Modify fw_cfg_read_blob() to use DMA if the device supports it. > > > Return errors, because the operation may

Re: [Qemu-devel] [PULL 00/10] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Thomas Huth
On 16.11.2017 17:15, Peter Maydell wrote: > On 16 November 2017 at 16:13, Daniel P. Berrange wrote: >> On Thu, Nov 16, 2017 at 04:10:22PM +, Peter Maydell wrote: >>> If there are known-buggy iasl versions in the wild we could >>> consider a configure test and treat

[Qemu-devel] [PULL 6/9] NUMA: Enable adding NUMA node implicitly

2017-11-16 Thread Michael S. Tsirkin
From: Dou Liyang Linux and Windows need ACPI SRAT table to make memory hotplug work properly, however currently QEMU doesn't create SRAT table if numa options aren't present on CLI. Which breaks both linux and windows guests in certain conditions: * Windows: won't

[Qemu-devel] [PATCH v2 4/4] iotests: 030: add compressed block-stream test

2017-11-16 Thread Anton Nefedov
Signed-off-by: Anton Nefedov --- tests/qemu-iotests/030 | 66 +- tests/qemu-iotests/030.out | 4 +-- 2 files changed, 67 insertions(+), 3 deletions(-) diff --git a/tests/qemu-iotests/030 b/tests/qemu-iotests/030 index

[Qemu-devel] [PATCH v2 0/4] compressed block-stream

2017-11-16 Thread Anton Nefedov
v2: - 1st patch omitted (applied to 2.11) - remarks applied v1: http://lists.nongnu.org/archive/html/qemu-devel/2017-11/msg02515.html It might be useful to compress images during block-stream; this way the user can merge compressed images of a backing chain and the result will remain

Re: [Qemu-devel] [PATCH v2 1/2] Fix eepro100 simple transmission mode

2017-11-16 Thread Thomas Huth
On 06.11.2017 21:35, Mike Nawrocki wrote: > The simple transmission mode was treating the area immediately after the > transmit command block (TCB) as if it were a transmit buffer descriptor, > when in reality it is simply the packet data. This change simply copies > the data following the TCB

[Qemu-devel] [PATCH v1 for-2.11 1/3] s390x: fix storing CPU status (again)

2017-11-16 Thread David Hildenbrand
Looks like the last fix + cleanup introduced another bug. (for now Linux guests don't seem to care) - we store the crs into ars. Fixes: 947a38bd6f13 ("s390x/kvm: fix and cleanup storing CPU status") Signed-off-by: David Hildenbrand --- target/s390x/helper.c | 2 +- 1 file

[Qemu-devel] [PATCH v1 for-2.11 0/3] s390x: fixes for SIGP and DIAG 308

2017-11-16 Thread David Hildenbrand
First one also applies to KVM SIGP handling. The other two only apply to TCG (allowing to IPL from disk with more than 1 VCPU- I never tested that before). David Hildenbrand (3): s390x: fix storing CPU status (again) s390x/tcg: fix DIAG 308 with > 1 VCPU (single threaded TCG) s390x/tcg: fix

Re: [Qemu-devel] Effect of qemu-img convert -m and -W options

2017-11-16 Thread Richard W.M. Jones
On Thu, Nov 16, 2017 at 02:47:46PM +, Stefan Hajnoczi wrote: > The threads you observed are the thread pool that performs > preadv(2)/pwritev(2) syscalls. The Linux AIO API could be used instead > and does not use threads for read and write operations. I guess if I used AIO then I wouldn't

Re: [Qemu-devel] QEMU terminates during reboot after memory unplug with vhost=on

2017-11-16 Thread Alex Williamson
On Thu, 16 Nov 2017 17:28:44 +0200 "Michael S. Tsirkin" wrote: > On Thu, Nov 16, 2017 at 01:23:39PM +0100, Greg Kurz wrote: > > Hi, > > > > I'm resurrecting a thread about a QEMU crash we're still hitting on ppc64. > > It > > was reported to the list by Bharata 2 months ago: >

Re: [Qemu-devel] QEMU terminates during reboot after memory unplug with vhost=on

2017-11-16 Thread Alex Williamson
On Thu, 16 Nov 2017 17:59:17 +0200 "Michael S. Tsirkin" wrote: > On Thu, Nov 16, 2017 at 08:53:45AM -0700, Alex Williamson wrote: > > On Thu, 16 Nov 2017 17:28:44 +0200 > > "Michael S. Tsirkin" wrote: > > > > > On Thu, Nov 16, 2017 at 01:23:39PM +0100, Greg

Re: [Qemu-devel] [PULL 00/10] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Daniel P. Berrange
On Thu, Nov 16, 2017 at 04:10:22PM +, Peter Maydell wrote: > On 16 November 2017 at 15:56, Michael S. Tsirkin wrote: > > On Thu, Nov 16, 2017 at 03:45:34PM +, Daniel P. Berrange wrote: > >> On Thu, Nov 16, 2017 at 05:38:18PM +0200, Michael S. Tsirkin wrote: > >> > I think

Re: [Qemu-devel] [RESEND PATCH 2/6] memory: introduce AddressSpaceOps and IOMMUObject

2017-11-16 Thread Liu, Yi L
Hi David, On Tue, Nov 14, 2017 at 11:59:34AM +1100, David Gibson wrote: > On Mon, Nov 13, 2017 at 04:28:45PM +0800, Peter Xu wrote: > > On Mon, Nov 13, 2017 at 04:56:01PM +1100, David Gibson wrote: > > > On Fri, Nov 03, 2017 at 08:01:52PM +0800, Liu, Yi L wrote: > > > > From: Peter Xu

[Qemu-devel] [PULL 0/9] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Michael S. Tsirkin
The following changes since commit 6a7cb8c3d674815cab08d884740d203fded12249: Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-20171115' into staging (2017-11-16 12:45:14 +) are available in the git repository at: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git

Re: [Qemu-devel] [RFC PATCH v2 01/21] ppc/xive: introduce a skeleton for the sPAPR XIVE interrupt controller

2017-11-16 Thread Cédric Le Goater
On 09/26/2017 11:45 AM, Benjamin Herrenschmidt wrote: > On Tue, 2017-09-26 at 13:54 +1000, David Gibson wrote: Which ones? My impression was that there needed to be at least #cpus * #priority-levels EQs, but there could be more than that, >>> >>> euh no, not in spapr mode at

[Qemu-devel] [PULL 8/9] build-sys: restrict vmcoreinfo to fw_cfg+dma capable targets

2017-11-16 Thread Michael S. Tsirkin
From: Marc-André Lureau vmcoreinfo is built for all targets. However, it requires fw_cfg with DMA operations support (write operation). Restrict vmcoreinfo exposure to architectures that are supporting FW_CFG_DMA, that is arm-virt and x86 only atm. Signed-off-by:

[Qemu-devel] [PULL 7/9] vmcoreinfo: put it in the 'misc' device category

2017-11-16 Thread Michael S. Tsirkin
From: Marc-André Lureau Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/misc/vmcoreinfo.c | 1 + 1 file changed, 1 insertion(+) diff --git

[Qemu-devel] [PULL 9/9] tests/bios-tables-test: Fix endianess problems when passing data to iasl

2017-11-16 Thread Michael S. Tsirkin
From: Thomas Huth The bios-tables-test was writing out files that we pass to iasl in with the wrong endianness in the header when running on a big endian host. So instead of storing mixed endian information in our structures, let's keep everything in little endian and byte-swap

Re: [Qemu-devel] Effect of qemu-img convert -m and -W options

2017-11-16 Thread Stefan Hajnoczi
On Thu, Nov 16, 2017 at 3:12 PM, Richard W.M. Jones wrote: > On Thu, Nov 16, 2017 at 03:51:29PM +0100, Peter Lieven wrote: >> Afaik all writes to the same QCOW2 serialize because of the s->lock that >> is held during the write. So its not suprising that there is no benefit from

Re: [Qemu-devel] [Question] why need to start all queues in vhost_net_start

2017-11-16 Thread Jason Wang
On 2017年11月16日 17:32, Longpeng (Mike) wrote: Hi Jason, On 2017/11/16 17:13, Jason Wang wrote: On 2017年11月16日 17:01, Gonglei (Arei) wrote: No, Windows guest + vhost-user/DPDK. BTW pls see virtio spec in : "If VIRTIO_NET_F_MQ is negotiated, each of receiveq1. . .receiveqN that will be

[Qemu-devel] [PULL 07/11] cpu-exec: don't overwrite exception_index

2017-11-16 Thread Paolo Bonzini
From: Pavel Dovgalyuk This patch adds a condition before overwriting exception_index fiels. It is needed when exception_index is already set to some meaningful value. Signed-off-by: Pavel Dovgalyuk Message-Id:

[Qemu-devel] [PULL 08/11] cpu-exec: avoid cpu_exec_nocache infinite loop with record/replay

2017-11-16 Thread Paolo Bonzini
From: Pavel Dovgalyuk This patch ensures that icount_decr.u32.high is clear before calling cpu_exec_nocache when exception is pending. Because the exception is caused by the first instruction in the block and it cannot be executed without resetting the flag. There are

Re: [Qemu-devel] [PATCH] block: Fix error path in bdrv_backing_update_filename()

2017-11-16 Thread Kevin Wolf
Am 15.11.2017 um 20:41 hat Peter Maydell geschrieben: > On 6 November 2017 at 16:55, Kevin Wolf wrote: > > error_setg_errno() takes a positive errno code. > > > > Signed-off-by: Kevin Wolf > > --- > > block.c | 2 +- > > 1 file changed, 1 insertion(+), 1

Re: [Qemu-devel] [PATCH] Revert "Add new PCI ID for i82559a"

2017-11-16 Thread Stefan Weil
Am 16.11.2017 um 13:20 schrieb Jason Wang: > This reverts commit 5e89dc01133f8f5e621f6b66b356c6f37d31dafb since: > > - we should use ID in the spec instead the one used by OEM > - in the future, we should allow changing id through either property > or EEPROM file. > > Cc: Stefan Weil

[Qemu-devel] [Bug 1732671] Re: vnc websocket compatibility issue

2017-11-16 Thread Daniel Berrange
No problem, it is a valid bug report, since we've not actually released the fix yet, so changing status. ** Changed in: qemu Status: Invalid => Fix Committed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

Re: [Qemu-devel] [PULL 0/4] Merge tpm 2017/11/15 v1

2017-11-16 Thread Peter Maydell
On 15 November 2017 at 12:31, Stefan Berger wrote: > This pull request is for 2.11 and extends documentation as well as fixes > bugs related to concurrency and failure mode. > > The following changes since commit 4ffa88c99c54d2a30f79e3dbecec50b023eff1c8: > > Merge

[Qemu-devel] [RFC v4 01/27] qobject: introduce qstring_get_try_str()

2017-11-16 Thread Peter Xu
The only difference from qstring_get_str() is that it allows the qstring to be NULL. If so, NULL is returned. CC: Eric Blake CC: Markus Armbruster Reviewed-by: Fam Zheng Signed-off-by: Peter Xu ---

[Qemu-devel] [RFC v4 09/27] monitor: create monitor dedicate iothread

2017-11-16 Thread Peter Xu
Create one IOThread for the monitors, prepared to handle all the input/output IOs using existing iothread framework. Signed-off-by: Peter Xu --- monitor.c | 29 + 1 file changed, 29 insertions(+) diff --git a/monitor.c b/monitor.c index

[Qemu-devel] [RFC v4 05/27] qjson: add "opaque" field to JSONMessageParser

2017-11-16 Thread Peter Xu
It'll be passed to emit() as well when it happens. Since at it, add a typedef for the emitter function. Reviewed-by: Fam Zheng Signed-off-by: Peter Xu --- include/qapi/qmp/json-streamer.h | 10 -- monitor.c| 7 ---

[Qemu-devel] [RFC v4 15/27] monitor: let monitor_{suspend|resume} thread safe

2017-11-16 Thread Peter Xu
Monitor code now can be run in more than one thread. Let the suspend and resume code be thread safe. Reviewed-by: Fam Zheng Signed-off-by: Peter Xu --- monitor.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/monitor.c b/monitor.c

Re: [Qemu-devel] [PATCH 5/5] iotest 030: add compressed block-stream test

2017-11-16 Thread Anton Nefedov
On 15/11/2017 10:51 PM, Max Reitz wrote: On 2017-11-14 11:16, Anton Nefedov wrote: Signed-off-by: Anton Nefedov --- tests/qemu-iotests/030 | 69 +- tests/qemu-iotests/030.out | 4 +-- 2 files changed, 70

[Qemu-devel] [RFC v4 22/27] qmp: isolate responses into io thread

2017-11-16 Thread Peter Xu
For those monitors who has enabled IO thread, we'll offload the responding procedure into IO thread. The main reason is that chardev is not thread safe, and we need to do all the read/write IOs in the same thread. For use_io_thr=true monitors, that thread is the IO thread. We do this isolation

Re: [Qemu-devel] [PULL 00/10] pc, pci, virtio: fixes for rc1

2017-11-16 Thread Peter Maydell
On 15 November 2017 at 18:17, Michael S. Tsirkin wrote: > The following changes since commit 1fa0f627d03cd0d0755924247cafeb42969016bf: > > Update version for v2.11.0-rc1 release (2017-11-14 18:37:49 +) > > are available in the git repository at: > >

Re: [Qemu-devel] Effect of qemu-img convert -m and -W options

2017-11-16 Thread Peter Lieven
Am 16.11.2017 um 15:47 schrieb Stefan Hajnoczi: On Wed, Nov 15, 2017 at 11:52:46AM +, Richard W.M. Jones wrote: [CC to qemu-devel since I'm obviously doing something wrong here, I'm just not sure what.] I was getting ready to add multiple threads to ‘qemu-img convert’ (the longest part of

[Qemu-devel] [Bug 1732679] [NEW] Cisco NX-OSv 9k crashes during boot with qemu 2.10.1(Debian 1:2.10.0+dfsg-2)

2017-11-16 Thread jean-christophe manciot
Public bug reported: Ubuntu 17.04 qemu 2.10.1(Debian 1:2.10.0+dfsg-2) gns3 2.0.3 NX-OSv 9k 7.0.3.I6.1 - No such issue with previous qemu 2.8.x - the issue does not seem to come from the debian packaging - the issue does not seem to come from GNS3 either, as confirmed by Jeremy Grossmann at

[Qemu-devel] [RFC v4 00/27] QMP: out-of-band (OOB) execution support

2017-11-16 Thread Peter Xu
v4: - drop first patch to fix IOWatchPool [Stefan, Dan] - add s-o-b where missing, and newly got r-bs - fix English error in commit msg [Fam] - some tunes on request-dropped event: [Stefan] - firstly let 'id' be any type, meanwhile make sure "id" is there as long as OOB is enabled for the

[Qemu-devel] [RFC v4 02/27] qobject: introduce qobject_get_try_str()

2017-11-16 Thread Peter Xu
A quick way to fetch string from qobject when it's a QString. Reviewed-by: Fam Zheng Signed-off-by: Peter Xu --- include/qapi/qmp/qstring.h | 1 + qobject/qstring.c | 11 +++ 2 files changed, 12 insertions(+) diff --git

[Qemu-devel] [RFC v4 12/27] qmp: negociate QMP capabilities

2017-11-16 Thread Peter Xu
After this patch, we will allow QMP clients to enable QMP capabilities when sending the first "qmp_capabilities" command. Originally we are starting QMP session with no arguments like: { "execute": "qmp_capabilities" } Now we can enable some QMP capabilities using (take OOB as example, which

[Qemu-devel] [RFC v4 14/27] monitor: introduce monitor_qmp_respond()

2017-11-16 Thread Peter Xu
A tiny refactoring, preparing to split the QMP dispatcher away. Reviewed-by: Fam Zheng Signed-off-by: Peter Xu --- monitor.c | 48 +++- 1 file changed, 31 insertions(+), 17 deletions(-) diff --git a/monitor.c

[Qemu-devel] [RFC v4 08/27] monitor: let mon_list be tail queue

2017-11-16 Thread Peter Xu
It was QLIST. I want to use this list to do monitor priority job later, which need tail insertion ability. So switching to a tail queue. Reviewed-by: Fam Zheng Signed-off-by: Peter Xu --- monitor.c | 10 +- 1 file changed, 5 insertions(+), 5

Re: [Qemu-devel] [Libguestfs] [qemu-img] support for XVA

2017-11-16 Thread Tomáš Golembiovský
On Thu, 16 Nov 2017 13:56:16 +0100 Tomáš Golembiovský wrote: > On Wed, 15 Nov 2017 21:41:20 +0100 > Gandalf Corvotempesta wrote: > > > 2017-11-15 21:29 GMT+01:00 Richard W.M. Jones : > > > Gandalf, is there an XVA file

[Qemu-devel] [PATCH] ipmi: check ibe status before ibe outlen at ipmi_bmc_extern_handle_command

2017-11-16 Thread xinhua.Cao
When we always kill vm's ipmi_sim program. qemu will do handling chr_event to reconnect ipmi_sim. handling chain is chr_event -> continue_send -> qemu_chr_fe_write. if ipmi_sim program was killed again. then qemu_chr_fe_write will failed. then ibe's outlen and outbuf will not cleared. so if vcpu

[Qemu-devel] [PULL 06/11] vhost-user-scsi: add missing virtqueue_size param

2017-11-16 Thread Paolo Bonzini
From: Dariusz Stojaczyk Commit 5c0919d0 [1] introduced virtqueue_size parameter for common virtio-scsi path, without updaing the vhost-user-scsi code. vhost-user-scsi devices right now report size 0 for each vq. This patch introduces virtqueue_size param to

[Qemu-devel] [PULL 01/11] Enable 8-byte wide MMIO for 16550 serial devices

2017-11-16 Thread Paolo Bonzini
From: Mike Nawrocki Some drivers for the PPMC7400 PowerPC evaluation board accesses the serial registers through the floating point unit (stfd/ldfd), which is an 8-byte wide access. This patch enables that behavior. Signed-off-by: Mike Nawrocki

[Qemu-devel] [PULL 05/11] target-i386: adds PV_TLB_FLUSH CPUID feature bit

2017-11-16 Thread Paolo Bonzini
From: Wanpeng Li Adds PV_TLB_FLUSH CPUID feature bit. Cc: Paolo Bonzini Cc: Radim KrÄmář Cc: Richard Henderson Cc: Eduardo Habkost Signed-off-by: Wanpeng Li

[Qemu-devel] [PULL 09/11] util/stats64: Fix min/max comparisons

2017-11-16 Thread Paolo Bonzini
From: Max Reitz stat64_min_slow() and stat64_max_slow() compare the wrong way. This makes iotest 136 fail with clang and -m32. Signed-off-by: Max Reitz Message-Id: <20171114232223.25207-1-mre...@redhat.com> Signed-off-by: Paolo Bonzini

  1   2   3   >