Re: [Qemu-devel] [PATCH] pseries: Restore support for total vcpus not a multiple of threads-per-core for old machine types

2017-05-23 Thread Laurent Vivier
On 23/05/2017 08:36, David Gibson wrote: > As of pseries-2.7 and later, we require the total number of guest vcpus to > be a multiple of the threads-per-core. pseries-2.6 and earlier machine > types, however, are supposed to allow this for the sake of migration from > old qemu versions which allow

Re: [Qemu-devel] [PATCH] pseries: Restore support for total vcpus not a multiple of threads-per-core for old machine types

2017-05-23 Thread Greg Kurz
On Tue, 23 May 2017 16:36:45 +1000 David Gibson wrote: > As of pseries-2.7 and later, we require the total number of guest vcpus to > be a multiple of the threads-per-core. pseries-2.6 and earlier machine > types, however, are supposed to allow this for the sake of migration from > old qemu vers

Re: [Qemu-devel] [PATCH v2] tests/libqtest: Print error instead of aborting when env variable is missing

2017-05-23 Thread Thomas Huth
On 22.05.2017 17:58, no-re...@patchew.org wrote: > 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. [...] >LEX convert-dtsv0-lexer.lex.c >DEP /tmp/qemu-test/s

Re: [Qemu-devel] [PATCH] acpi: change CPU container node _HID to compatible PNP0A05

2017-05-23 Thread Evgeny Yakovlev
On 22.05.2017 20:44, Michael S. Tsirkin wrote: On Mon, May 22, 2017 at 02:58:56PM +0200, Igor Mammedov wrote: On Mon, 22 May 2017 14:01:15 +0300 Evgeny Yakovlev wrote: On 22.05.2017 13:35, Igor Mammedov wrote: On Mon, 22 May 2017 12:50:30 +0300 Evgeny Yakovlev wrote: When running wind

Re: [Qemu-devel] [PATCH v2] tests/libqtest: Print error instead of aborting when env variable is missing

2017-05-23 Thread Samuel Thibault
Thomas Huth, on mar. 23 mai 2017 09:25:30 +0200, wrote: > > /tmp/qemu-test/src/slirp/tcp_input.c: In function ‘tcp_input’: > > /tmp/qemu-test/src/slirp/tcp_input.c:219: warning: ‘save_ip.ip_p’ may be > > used uninitialized in this function > > I've never seen these warnings in tcp_input.c before

Re: [Qemu-devel] [PATCH 01/31] target/s390: Use cpu_loop_exit_restore for tlb_fill

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 10 ++ > 1 file changed, 2 insertions(+), 8 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > index 0c6a0d9..e3325a4 100644 > --- a/tar

Re: [Qemu-devel] [RFC PATCH 1/8] iommu: Introduce bind_pasid_table API function

2017-05-23 Thread Liu, Yi L
On Fri, Apr 28, 2017 at 01:51:42PM +0100, Jean-Philippe Brucker wrote: > On 28/04/17 10:04, Liu, Yi L wrote: Hi Jean, Sorry for the delay response. Still have some follow-up comments on per-device or per-group. Pls refer to comments inline. > > On Wed, Apr 26, 2017 at 05:56:45PM +0100, Jean-Phili

Re: [Qemu-devel] [PATCH v1 01/13] qcow2: alloc space for COW in one chunk

2017-05-23 Thread Anton Nefedov
On 05/22/2017 10:00 PM, Eric Blake wrote: On 05/19/2017 04:34 AM, Anton Nefedov wrote: From: "Denis V. Lunev" Currently each single write operation can result in 3 write operations if guest offsets are not cluster aligned. One write is performed for the real payload and two for COW-ed areas.

Re: [Qemu-devel] [PATCH v1 03/13] qcow2: do not COW the empty areas

2017-05-23 Thread Anton Nefedov
On 05/22/2017 10:24 PM, Eric Blake wrote: On 05/19/2017 04:34 AM, Anton Nefedov wrote: If COW area of the newly allocated cluster is zeroes, there is no reason to write zero sectors in perform_cow() again now as whole clusters are zeroed out in single chunks by handle_alloc_space(). But that

Re: [Qemu-devel] [PATCH 03/31] target/s390x: Use unwind data for helper_nc

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 18 +- > target/s390x/translate.c | 1 - > 2 files changed, 9 insertions(+), 10 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper

Re: [Qemu-devel] [PATCH v1 02/13] qcow2: is_zero_sectors(): return true if area is outside of backing file

2017-05-23 Thread Anton Nefedov
On 05/22/2017 10:14 PM, Eric Blake wrote: On 05/22/2017 02:12 PM, Eric Blake wrote: +++ b/block/qcow2.c @@ -2482,7 +2482,7 @@ static bool is_zero_sectors(BlockDriverState *bs, int64_t start, int64_t res; if (start + count > bs->total_sectors) { -count = bs->total_sectors -

Re: [Qemu-devel] [PATCH] qapi: Fix some blockdev-add documentation regressions

2017-05-23 Thread Markus Armbruster
Eric Blake writes: > In the process of getting rid of docs/qmp-commands.txt, we > managed to regress on any text that changed after the point > where the move was first branched and when the move actually > occurred. For example, commit 3282eca for blockdev-snapshot > re-added the extra "options

Re: [Qemu-devel] [Qemu-ppc] [RFC PATCH v2 4/4] spapr: Fix migration of Radix guests

2017-05-23 Thread Suraj Jitindar Singh
On Tue, 2017-05-23 at 10:18 +0530, Bharata B Rao wrote: > On Mon, May 22, 2017 at 04:30:50PM +1000, Suraj Jitindar Singh wrote: > > On Fri, 2017-05-19 at 11:10 +0530, Bharata B Rao wrote: > > > Fix migration of radix guests by ensuring that we issue > > > KVM_PPC_CONFIGURE_V3_MMU for radix case pos

Re: [Qemu-devel] [PATCH 4/4] qapi: Reject alternates that can't work with keyval_parse()

2017-05-23 Thread Markus Armbruster
Eric Blake writes: > On 05/22/2017 11:42 AM, Markus Armbruster wrote: >> Alternates are sum types like unions, but use the JSON type on the >> wire / QType in QObject instead of an explicit tag. That's why we >> require alternate members to have distinct QTypes. >> >> The recently introduced ke

[Qemu-devel] [PATCH] usb: don't wakeup during coldplug

2017-05-23 Thread Gerd Hoffmann
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1452512 Signed-off-by: Gerd Hoffmann --- hw/usb/core.c | 8 1 file changed, 8 insertions(+) diff --git a/hw/usb/core.c b/hw/usb/core.c index 45fa00c517..241ae66b15 100644 --- a/hw/usb/core.c +++ b/hw/usb/core.c @@ -98,6 +98,14 @@ void u

Re: [Qemu-devel] [PATCH 04/31] target/s390x: Use unwind data for helper_xc

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 25 - > target/s390x/translate.c | 1 - > 2 files changed, 12 insertions(+), 14 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/me

Re: [Qemu-devel] [PATCH 05/31] target/s390x: Use unwind data for helper_oc

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 18 +- > target/s390x/translate.c | 1 - > 2 files changed, 9 insertions(+), 10 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper

Re: [Qemu-devel] [Qemu-ppc] [RFC PATCH v2 4/4] spapr: Fix migration of Radix guests

2017-05-23 Thread Bharata B Rao
On Tue, May 23, 2017 at 06:42:28PM +1000, Suraj Jitindar Singh wrote: > On Tue, 2017-05-23 at 10:18 +0530, Bharata B Rao wrote: > > On Mon, May 22, 2017 at 04:30:50PM +1000, Suraj Jitindar Singh wrote: > > > On Fri, 2017-05-19 at 11:10 +0530, Bharata B Rao wrote: > > > > Fix migration of radix gues

Re: [Qemu-devel] [PATCH v1 01/13] qcow2: alloc space for COW in one chunk

2017-05-23 Thread Denis V. Lunev
On 05/22/2017 10:00 PM, Eric Blake wrote: > On 05/19/2017 04:34 AM, Anton Nefedov wrote: >> From: "Denis V. Lunev" >> >> Currently each single write operation can result in 3 write operations >> if guest offsets are not cluster aligned. One write is performed for the >> real payload and two for CO

Re: [Qemu-devel] [PATCH v1 03/13] qcow2: do not COW the empty areas

2017-05-23 Thread Denis V. Lunev
On 05/22/2017 10:24 PM, Eric Blake wrote: > On 05/19/2017 04:34 AM, Anton Nefedov wrote: >> If COW area of the newly allocated cluster is zeroes, there is no reason >> to write zero sectors in perform_cow() again now as whole clusters are >> zeroed out in single chunks by handle_alloc_space(). > Bu

Re: [Qemu-devel] [PATCH v5 4/4] net/rocker: Fix the unusual macro name

2017-05-23 Thread Markus Armbruster
Mao Zhongyi writes: > Suggested-by: Markus Armbruster > Signed-off-by: Mao Zhongyi Reviewed-by: Markus Armbruster

Re: [Qemu-devel] [PATCH 06/31] target/s390x: Use unwind data for helper_mvc

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 19 ++- > target/s390x/translate.c | 1 - > 2 files changed, 10 insertions(+), 10 deletions(-) Reviewed-by: Thomas Huth

Re: [Qemu-devel] [PATCH 11/31] target/s390x: Use unwind data for helper_mvpg

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/helper.h | 2 +- > target/s390x/mem_helper.c | 9 + > target/s390x/translate.c | 3 +-- > 3 files changed, 7 insertions(+), 7 deletions(-) > > diff --git a/target/s390x/helper.h b/t

Re: [Qemu-devel] [PATCH 26/31] target/s390x: Use unwind data for helper_testblock

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 3 +++ > target/s390x/translate.c | 1 - > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > index 1e31bd3.

Re: [Qemu-devel] [PATCH 07/31] target/s390x: Use unwind data for helper_clc

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 18 +- > target/s390x/translate.c | 1 - > 2 files changed, 9 insertions(+), 10 deletions(-) Reviewed-by: Thomas Huth

Re: [Qemu-devel] [PATCH v5 1/4] net/rocker: Remove the dead error handling

2017-05-23 Thread Markus Armbruster
Mao Zhongyi writes: > Memory allocation functions like world_alloc, desc_ring_alloc etc, > they are all wrappers around g_malloc, g_new etc. But g_malloc and > similar functions doesn't return null. Because they ignore the fact don't > that g_malloc() of 0 bytes returns null. So error checks

Re: [Qemu-devel] [PATCH 08/31] target/s390x: Use unwind data for helper_clm

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 11 ++- > target/s390x/translate.c | 1 - > 2 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > ind

Re: [Qemu-devel] [PATCH] vhost_net: do not expose MTU feature bit to kernel backend

2017-05-23 Thread Maxime Coquelin
On 05/22/2017 07:24 PM, Michael S. Tsirkin wrote: On Sat, May 20, 2017 at 10:06:58AM +0200, Maxime Coquelin wrote: This patch removes MTU from the list of features supported by the kernel backend, so that vhost kernel backend does not have to advertise it to be negotiated. Cc: Aaron Conole S

Re: [Qemu-devel] [PATCH 09/31] target/s390x: Use unwind data for helper_srst

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 3 ++- > target/s390x/translate.c | 1 - > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > index 5f38ac3

Re: [Qemu-devel] [PATCH 10/31] target/s390x: Use unwind data for helper_clst

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 5 +++-- > target/s390x/translate.c | 1 - > 2 files changed, 3 insertions(+), 3 deletions(-) Reviewed-by: Thomas Huth

Re: [Qemu-devel] [PATCH 12/31] target/s390x: Use unwind data for helper_mvst

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 5 +++-- > target/s390x/translate.c | 1 - > 2 files changed, 3 insertions(+), 3 deletions(-) Reviewed-by: Thomas Huth

Re: [Qemu-devel] [PATCH 13/31] target/s390x: Use unwind data for helper_lam

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 3 ++- > target/s390x/translate.c | 1 - > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > index d1a7bcd

Re: [Qemu-devel] [PATCH 14/31] target/s390x: Use unwind data for helper_stam

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 3 ++- > target/s390x/translate.c | 1 - > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > index 4ecec74

Re: [Qemu-devel] [PATCH 15/31] target/s390x: Use unwind data for helper_mvcl

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 7 --- > target/s390x/translate.c | 1 - > 2 files changed, 4 insertions(+), 4 deletions(-) Reviewed-by: Thomas Huth

Re: [Qemu-devel] [PATCH 16/31] target/s390x: Use unwind data for helper_mvcle

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 7 --- > target/s390x/translate.c | 1 - > 2 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > index b76

Re: [Qemu-devel] [PATCH 17/31] target/s390x: Use unwind data for helper_clcle

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 6 +++--- > target/s390x/translate.c | 1 - > 2 files changed, 3 insertions(+), 4 deletions(-) Reviewed-by: Thomas Huth

Re: [Qemu-devel] [PATCH 18/31] target/s390x: Use unwind data for helper_cksm

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 11 ++- > target/s390x/translate.c | 1 - > 2 files changed, 6 insertions(+), 6 deletions(-) Reviewed-by: Thomas Huth

Re: [Qemu-devel] [PATCH 19/31] target/s390x: Use unwind data for helper_unpk

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 9 + > target/s390x/translate.c | 1 - > 2 files changed, 5 insertions(+), 5 deletions(-) Reviewed-by: Thomas Huth

Re: [Qemu-devel] [PATCH 20/31] target/s390x: Use unwind data for helper_tr

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 10 +- > target/s390x/translate.c | 1 - > 2 files changed, 5 insertions(+), 6 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > inde

Re: [Qemu-devel] [PATCH 1/1] Improve Cocoa modifier key handling

2017-05-23 Thread Gerd Hoffmann
Hi, > looked at the keyCode to determine what modifier key changed. This > usually works fine but sometimes the keyCode is 0 and the app should > instead be looking at the modifierFlags bitmask. Key code 0 is the > 'a' > key. > > I added code that handles keyCode == 0 differently. It checks the

Re: [Qemu-devel] [PATCH v5 1/4] net/rocker: Remove the dead error handling

2017-05-23 Thread Mao Zhongyi
Hi, Markus On 05/23/2017 05:27 PM, Markus Armbruster wrote: Mao Zhongyi writes: Memory allocation functions like world_alloc, desc_ring_alloc etc, they are all wrappers around g_malloc, g_new etc. But g_malloc and similar functions doesn't return null. Because they ignore the fact don't

Re: [Qemu-devel] [PATCH 25/31] target/s390x: Use unwind data for helper_stctl

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 10 ++ > target/s390x/translate.c | 2 -- > 2 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > ind

Re: [Qemu-devel] [PATCH 27/31] target/s390x: Use unwind data for helper_tprot

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 1 - > target/s390x/translate.c | 1 - > 2 files changed, 2 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > index a874f8a..7f22bf0 100644 >

Re: [Qemu-devel] [PATCH 31/31] target/s390x: Use unwind data for helper_mvcs/mvcp

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 8 ++-- > target/s390x/translate.c | 2 -- > 2 files changed, 6 insertions(+), 4 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > index 4

Re: [Qemu-devel] [PATCH 21/31] target/s390x: Use unwind data for helper_tre

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 15 --- > target/s390x/translate.c | 1 - > 2 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c >

Re: [Qemu-devel] [virtio-dev] Re: [PATCH RFC] virtio-net: enable configurable tx queue size

2017-05-23 Thread Wei Wang
On 05/23/2017 02:24 PM, Jason Wang wrote: On 2017年05月23日 13:15, Wei Wang wrote: On 05/23/2017 10:04 AM, Jason Wang wrote: On 2017年05月22日 19:52, Wei Wang wrote: On 05/20/2017 04:42 AM, Michael S. Tsirkin wrote: On Fri, May 19, 2017 at 10:32:19AM +0800, Wei Wang wrote: This patch enables t

Re: [Qemu-devel] [PATCH 22/31] target/s390x: Use unwind data for helper_trt

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 9 + > target/s390x/translate.c | 1 - > 2 files changed, 5 insertions(+), 5 deletions(-) Reviewed-by: Thomas Huth

Re: [Qemu-devel] [PATCH RFC 1/2] Increased the size of requester_id field from MemTxAttrs

2017-05-23 Thread Diana Madalina Craciun
On 04/04/2017 03:27 PM, Peter Maydell wrote: > On 23 March 2017 at 13:36, Diana Craciun wrote: >> The PCI requester ID field is 16 bits. The requester_id field >> from MemTxAttrs is used for MSIs to specify the device ID for >> the platforms where this device ID is needed (e.g virt machine + GICv3

[Qemu-devel] [PULL 0/1] Silence "make check" warnings on NUMA test

2017-05-23 Thread Eduardo Habkost
The following changes since commit 56821559f0ba682fe6b367815572e6f974d329ab: Merge remote-tracking branch 'dgilbert/tags/pull-hmp-20170517' into staging (2017-05-18 13:36:15 +0100) are available in the git repository at: git://github.com/ehabkost/qemu.git tags/numa-pull-request for you to

Re: [Qemu-devel] [virtio-dev] Re: [virtio-dev] Re: [PATCH v2 00/16] Vhost-pci for inter-VM communication

2017-05-23 Thread Wei Wang
On 05/23/2017 02:32 PM, Jason Wang wrote: On 2017年05月23日 13:47, Wei Wang wrote: On 05/23/2017 10:08 AM, Jason Wang wrote: On 2017年05月22日 19:46, Wang, Wei W wrote: On Monday, May 22, 2017 10:28 AM, Jason Wang wrote: On 2017年05月19日 23:33, Stefan Hajnoczi wrote: On Fri, May 19, 2017 at 11:1

[Qemu-devel] [PULL 1/1] numa: Silence incomplete mapping warning under qtest

2017-05-23 Thread Eduardo Habkost
From: Igor Mammedov Silence "make check" warnings triggered by the numa/mon/cpus/partial test case. Suggested-by: Markus Armbruster Signed-off-by: Igor Mammedov Message-Id: <1495094971-177754-4-git-send-email-imamm...@redhat.com> Signed-off-by: Eduardo Habkost --- hw/core/machine.c | 3 ++-

Re: [Qemu-devel] [PATCH 01/31] target/s390: Use cpu_loop_exit_restore for tlb_fill

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 10 ++ > 1 file changed, 2 insertions(+), 8 deletions(-) > Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4096R/1DDD8C9B aurel...@aurel32.

Re: [Qemu-devel] [PATCH 23/31] target/s390x: Use unwind data for helper_lctlg

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 8 > target/s390x/translate.c | 1 - > 2 files changed, 4 insertions(+), 5 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > index 90

Re: [Qemu-devel] [PATCH 03/31] target/s390x: Use unwind data for helper_nc

2017-05-23 Thread Aurelien Jarno
On 2017-05-23 10:33, Thomas Huth wrote: > On 23.05.2017 05:02, Richard Henderson wrote: > > Signed-off-by: Richard Henderson > > --- > > target/s390x/mem_helper.c | 18 +- > > target/s390x/translate.c | 1 - > > 2 files changed, 9 insertions(+), 10 deletions(-) Reviewed-by: Aur

Re: [Qemu-devel] [PATCH 02/31] target/s390x: Implement EXECUTE via new TranslationBlock

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Previously, helper_ex would construct the insn and then implement > the insn via direct calls other helpers. This was sufficient to > boot Linux but that is all. > > It is easy enough to go the whole nine yards by stashing state for > EXECUTE within

Re: [Qemu-devel] [PATCH 06/31] target/s390x: Use unwind data for helper_mvc

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 19 ++- > target/s390x/translate.c | 1 - > 2 files changed, 10 insertions(+), 10 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno

Re: [Qemu-devel] [PATCH 24/31] target/s390x: Use unwind data for helper_lctl

2017-05-23 Thread Thomas Huth
On 23.05.2017 05:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 9 + > target/s390x/translate.c | 1 - > 2 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c > index 2

Re: [Qemu-devel] [PATCH 04/31] target/s390x: Use unwind data for helper_xc

2017-05-23 Thread Aurelien Jarno
On 2017-05-23 10:56, Thomas Huth wrote: > On 23.05.2017 05:02, Richard Henderson wrote: > > Signed-off-by: Richard Henderson > > --- > > target/s390x/mem_helper.c | 25 - > > target/s390x/translate.c | 1 - > > 2 files changed, 12 insertions(+), 14 deletions(-) Reviewed

Re: [Qemu-devel] [PATCH v5 1/4] net/rocker: Remove the dead error handling

2017-05-23 Thread Markus Armbruster
Mao Zhongyi writes: > Hi, Markus > > > On 05/23/2017 05:27 PM, Markus Armbruster wrote: >> Mao Zhongyi writes: >> >>> Memory allocation functions like world_alloc, desc_ring_alloc etc, >>> they are all wrappers around g_malloc, g_new etc. But g_malloc and >>> similar functions doesn't return nul

Re: [Qemu-devel] [PATCH 05/31] target/s390x: Use unwind data for helper_oc

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 18 +- > target/s390x/translate.c | 1 - > 2 files changed, 9 insertions(+), 10 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno

Re: [Qemu-devel] [PATCH 05/31] target/s390x: Use unwind data for helper_oc

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 18 +- > target/s390x/translate.c | 1 - > 2 files changed, 9 insertions(+), 10 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno

Re: [Qemu-devel] [PATCH 08/31] target/s390x: Use unwind data for helper_clm

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 11 ++- > target/s390x/translate.c | 1 - > 2 files changed, 6 insertions(+), 6 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GP

Re: [Qemu-devel] [PATCH 09/31] target/s390x: Use unwind data for helper_srst

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 3 ++- > target/s390x/translate.c | 1 - > 2 files changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4096R/1

Re: [Qemu-devel] [PATCH 07/31] target/s390x: Use unwind data for helper_clc

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 18 +- > target/s390x/translate.c | 1 - > 2 files changed, 9 insertions(+), 10 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno

Re: [Qemu-devel] [PATCH 14/31] target/s390x: Use unwind data for helper_stam

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 3 ++- > target/s390x/translate.c | 1 - > 2 files changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4096R/1

Re: [Qemu-devel] [PATCH 15/31] target/s390x: Use unwind data for helper_mvcl

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 7 --- > target/s390x/translate.c | 1 - > 2 files changed, 4 insertions(+), 4 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 409

Re: [Qemu-devel] [PATCH 10/31] target/s390x: Use unwind data for helper_clst

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 5 +++-- > target/s390x/translate.c | 1 - > 2 files changed, 3 insertions(+), 3 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4096R

Re: [Qemu-devel] [PATCH 16/31] target/s390x: Use unwind data for helper_mvcle

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 7 --- > target/s390x/translate.c | 1 - > 2 files changed, 4 insertions(+), 4 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 409

Re: [Qemu-devel] [PATCH 11/31] target/s390x: Use unwind data for helper_mvpg

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/helper.h | 2 +- > target/s390x/mem_helper.c | 9 + > target/s390x/translate.c | 3 +-- > 3 files changed, 7 insertions(+), 7 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien

Re: [Qemu-devel] [PATCH 17/31] target/s390x: Use unwind data for helper_clcle

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 6 +++--- > target/s390x/translate.c | 1 - > 2 files changed, 3 insertions(+), 4 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4096

Re: [Qemu-devel] [PATCH 21/31] target/s390x: Use unwind data for helper_tre

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 15 --- > target/s390x/translate.c | 1 - > 2 files changed, 8 insertions(+), 8 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno

Re: [Qemu-devel] [PATCH 19/31] target/s390x: Use unwind data for helper_unpk

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 9 + > target/s390x/translate.c | 1 - > 2 files changed, 5 insertions(+), 5 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4

Re: [Qemu-devel] [PATCH 12/31] target/s390x: Use unwind data for helper_mvst

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 5 +++-- > target/s390x/translate.c | 1 - > 2 files changed, 3 insertions(+), 3 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4096R

Re: [Qemu-devel] [PATCH 13/31] target/s390x: Use unwind data for helper_lam

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 3 ++- > target/s390x/translate.c | 1 - > 2 files changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4096R/1

Re: [Qemu-devel] [PATCH 23/31] target/s390x: Use unwind data for helper_lctlg

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 8 > target/s390x/translate.c | 1 - > 2 files changed, 4 insertions(+), 5 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 40

Re: [Qemu-devel] [PATCH 18/31] target/s390x: Use unwind data for helper_cksm

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:02, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 11 ++- > target/s390x/translate.c | 1 - > 2 files changed, 6 insertions(+), 6 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GP

Re: [Qemu-devel] [PATCH 24/31] target/s390x: Use unwind data for helper_lctl

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 9 + > target/s390x/translate.c | 1 - > 2 files changed, 5 insertions(+), 5 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4

Re: [Qemu-devel] [PATCH 22/31] target/s390x: Use unwind data for helper_trt

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 9 + > target/s390x/translate.c | 1 - > 2 files changed, 5 insertions(+), 5 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4

Re: [Qemu-devel] [PATCH 25/31] target/s390x: Use unwind data for helper_stctl

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 10 ++ > target/s390x/translate.c | 2 -- > 2 files changed, 6 insertions(+), 6 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GP

Re: [Qemu-devel] [PATCH 20/31] target/s390x: Use unwind data for helper_tr

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 10 +- > target/s390x/translate.c | 1 - > 2 files changed, 5 insertions(+), 6 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG

Re: [Qemu-devel] [PATCH 26/31] target/s390x: Use unwind data for helper_testblock

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 3 +++ > target/s390x/translate.c | 1 - > 2 files changed, 3 insertions(+), 1 deletion(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4096R/1D

Re: [Qemu-devel] [PATCH 27/31] target/s390x: Use unwind data for helper_tprot

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 1 - > target/s390x/translate.c | 1 - > 2 files changed, 2 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4096R/1DDD8C9B aurel...@au

[Qemu-devel] [PULL 03/21] pc-bios/s390-ccw: Break up virtio-scsi read into multiples

2017-05-23 Thread Cornelia Huck
From: Eric Farman A virtio-scsi request that goes through the host sd driver and exceeds the maximum transfer size is automatically broken up for us. But the equivalent request going to the sg driver presumes that any length requirements have already been honored. Let's use the max_sectors fiel

[Qemu-devel] [PULL 01/21] pc-bios/s390-ccw: Remove duplicate blk_factor adjustment

2017-05-23 Thread Cornelia Huck
From: Eric Farman When using virtio-scsi, we multiply the READ(10) data_size by a block factor twice when building the I/O. This is fine, since it's only 1 for SCSI disks, but let's clean it up. Signed-off-by: Eric Farman Reviewed-by: Christian Borntraeger Message-Id: <20170510155359.32727-2-

Re: [Qemu-devel] [virtio-dev] Re: [virtio-dev] Re: [PATCH v2 00/16] Vhost-pci for inter-VM communication

2017-05-23 Thread Wei Wang
On 05/20/2017 04:44 AM, Michael S. Tsirkin wrote: On Fri, May 19, 2017 at 05:00:37PM +0800, Wei Wang wrote: That being said, we compared to vhost-user, instead of vhost_net, because vhost-user is the one that is used in NFV, which we think is a major use case for vhost-pci. If this is true, why

[Qemu-devel] [PULL 09/21] linux-headers: update

2017-05-23 Thread Cornelia Huck
Update against Linux v4.12-rc1. Also include the new vfio_ccw.h header. Signed-off-by: Cornelia Huck --- include/standard-headers/asm-x86/hyperv.h | 7 +- include/standard-headers/linux/input-event-codes.h | 1 + include/standard-headers/linux/input.h | 11 +--- i

[Qemu-devel] [PATCH v2 2/2] Add a unique ID in the virt machine to be used as device ID

2017-05-23 Thread Diana Craciun
Device IDs are required by the ARM GICv3 ITS for IRQ remapping. Currently, for PCI devices, the requester ID was used as device ID in the virt machine. If the system has multiple masters that use MSIs a unique ID accross the platform is needed. A static scheme is used and each master is allocated a

[Qemu-devel] [PATCH v2 0/2] Add global device ID in virt machine

2017-05-23 Thread Diana Craciun
The NXP DPAA2 is a hardware architecture designed for high-speeed network packet processing. The DPAA2 hardware components are managed by a hardware component called the Management Complex (or MC) which provides an object-base abstraction for software drivers to use the DPAA2 hardware. For more det

[Qemu-devel] [PULL 14/21] vfio/ccw: vfio based subchannel passthrough driver

2017-05-23 Thread Cornelia Huck
From: Xiao Feng Ren We use the IOMMU_TYPE1 of VFIO to realize the subchannels passthrough, implement a vfio based subchannels passthrough driver called "vfio-ccw". Support qemu parameters in the style of: "-device vfio-ccw,sysfsdev=$mdev_file_path,devno=xx.x.' Reviewed-by: Eric Auger Acked

[Qemu-devel] [PULL 17/21] s390x/css: introduce and realize ccw-request callback

2017-05-23 Thread Cornelia Huck
From: Xiao Feng Ren Introduce a new callback on subchannel to handle ccw-request. Realize the callback in vfio-ccw device. Besides, resort to the event notifier handler to handling the ccw-request results. 1. Pread the I/O results via MMIO region. 2. Update the scsw info to guest. 3. Inject an I/

Re: [Qemu-devel] [PATCH 30/31] target/s390x: Implement CSPG

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/insn-data.def | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/target/s390x/insn-data.def b/target/s390x/insn-data.def > index 4c91f30..8604847 100644 > --- a/target/s390x/insn-data.def

[Qemu-devel] [PULL 04/21] pc-bios/s390-ccw: Refactor scsi_inquiry function

2017-05-23 Thread Cornelia Huck
From: Eric Farman If we want to issue any of the SCSI Inquiry EVPD pages, which we do, we could use this function to issue both types of commands with a little bit of refactoring. Signed-off-by: Eric Farman Message-Id: <20170510155359.32727-5-far...@linux.vnet.ibm.com> Signed-off-by: Cornelia H

[Qemu-devel] [PATCH 0/4] spapr: disable hotplugging without OS

2017-05-23 Thread Laurent Vivier
If the OS is not started, QEMU sends an event to the OS that is lost and cannot be recovered. An unplug is not able to restore QEMU in a coherent state. So, while the OS is not started, disable CPU and memory hotplug. We use option vector 6 to know if the OS is started This series moves error chec

[Qemu-devel] [PULL 18/21] s390x/css: ccw translation infrastructure

2017-05-23 Thread Cornelia Huck
From: Xiao Feng Ren Implement a basic infrastructure of handling channel I/O instruction interception for passed through subchannels: 1. Branch the code path of instruction interception handling by SubChannel type. 2. For a passed-through subchannel, issue the ORB to kernel to do ccw transl

[Qemu-devel] [PULL 07/21] pc-bios/s390-ccw: Build a reasonable max_sectors limit

2017-05-23 Thread Cornelia Huck
From: Eric Farman Now that we've read all the possible limits that have been defined for a virtio-scsi controller and the disk we're booting from, it's possible that we are STILL going to exceed the limits of the host device. For example, a "-device scsi-generic" device does not support the Block

[Qemu-devel] [PULL 15/21] vfio/ccw: get io region info

2017-05-23 Thread Cornelia Huck
From: Dong Jia Shi vfio-ccw provides an MMIO region for I/O operations. We fetch its information via ioctls here, then we can use it performing I/O instructions and retrieving I/O results later on. Reviewed-by: Eric Auger Acked-by: Alex Williamson Signed-off-by: Dong Jia Shi Message-Id: <2017

[Qemu-devel] [PULL 08/21] pc-bios/s390-ccw.img: rebuild image

2017-05-23 Thread Cornelia Huck
From: Eric Farman Contains the following commits: - pc-bios/s390-ccw: Remove duplicate blk_factor adjustment - pc-bios/s390-ccw: Move SCSI block factor to outer read - pc-bios/s390-ccw: Break up virtio-scsi read into multiples - pc-bios/s390-ccw: Refactor scsi_inquiry function - pc-bios/s390-ccw:

Re: [Qemu-devel] [PATCH 31/31] target/s390x: Use unwind data for helper_mvcs/mvcp

2017-05-23 Thread Aurelien Jarno
On 2017-05-22 20:03, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/s390x/mem_helper.c | 8 ++-- > target/s390x/translate.c | 2 -- > 2 files changed, 6 insertions(+), 4 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4

[Qemu-devel] [PATCH 2/7] qcow2: Use unsigned int for both members of Qcow2COWRegion

2017-05-23 Thread Alberto Garcia
Qcow2COWRegion has two attributes: - The offset of the COW region from the start of the first cluster touched by the I/O request. Since it's always going to be positive and the maximum request size is at most INT_MAX, we can use a regular unsigned int to store this offset. - The size of the

[Qemu-devel] [PULL 19/21] vfio/ccw: update sense data if a unit check is pending

2017-05-23 Thread Cornelia Huck
From: Dong Jia Shi Concurrent-sense data is currently not delivered. This patch stores the concurrent-sense data to the subchannel if a unit check is pending and the concurrent-sense bit is enabled. Then a TSCH can retreive the right IRB data back to the guest. Acked-by: Alex Williamson Signed-

  1   2   3   4   >