[Qemu-devel] [PULL for-2.10 1/1] boot-serial-test: fallback to kvm accelerator

2017-08-11 Thread Cornelia Huck
Currently, at least x86_64 and s390x support building with --disable-tcg. Instead of forcing tcg (which causes the test to fail on such builds), allow to use kvm as well. Reviewed-by: Daniel P. Berrange Reviewed-by: Thomas Huth Signed-off-by: Cornelia Huck --- tests/boot-serial-test.c | 5 +++-

[Qemu-devel] [PULL for-2.10 0/1] bugfix for boot-serial-test and no tcg

2017-08-11 Thread Cornelia Huck
The following changes since commit 473a321122fd3c2c327a5a5d01a9a41f26f1734c: Merge remote-tracking branch 'remotes/gkurz/tags/for-upstream' into staging (2017-08-10 17:50:55 +0100) are available in the git repository at: git://github.com/cohuck/qemu tags/boot-serial-no-tcg-201

Re: [Qemu-devel] [PATCH 1/2] nbd: Drop connection if broken server is detected

2017-08-11 Thread Eric Blake
On 08/11/2017 02:48 AM, Vladimir Sementsov-Ogievskiy wrote: > 11.08.2017 05:37, Eric Blake wrote: >> As soon as the server is sending us garbage, we should quit >> trying to send further messages to the server, and allow all >> pending coroutines for any remaining replies to error out. >> Failure t

Re: [Qemu-devel] [PATCH RFC 2/5] target/s390x: move a couple of functions to cpu.c

2017-08-11 Thread Thomas Huth
On 11.08.2017 09:46, David Hildenbrand wrote: > Signed-off-by: David Hildenbrand > --- > target/s390x/cpu.c | 80 + > target/s390x/cpu.h | 88 > +++--- > 2 files changed, 90 insertions(+), 78 deletion

Re: [Qemu-devel] [PATCH] target/i386: fix pcmpxstrx substring search

2017-08-11 Thread Paolo Bonzini
On 10/08/2017 23:40, Joseph Myers wrote: > One of the cases of the SSE4.2 pcmpestri / pcmpestrm / pcmpistri / > pcmpistrm instructions does a substring search. The implementation of > this case in the pcmpxstrx helper is incorrect. The operation in this > case is a search for a string (argument d

[Qemu-devel] [RFC PATCH] memory: fix access_with_adjusted_size() on big-endian

2017-08-11 Thread Philippe Mathieu-Daudé
remove unnecessary memory_region_big_endian() Signed-off-by: Philippe Mathieu-Daudé --- This is part of a branch with many cross-endianness tests for 2.11 Frederic does it helps your issues on armeb? memory.c | 28 ++-- 1 file changed, 10 insertions(+), 18 deletions(-)

[Qemu-devel] [PULL 6/7] file-posix: Do runtime check for ofd lock API

2017-08-11 Thread Kevin Wolf
From: Fam Zheng It is reported that on Windows Subsystem for Linux, ofd operations fail with -EINVAL. In other words, QEMU binary built with system headers that exports F_OFD_SETLK doesn't necessarily run in an environment that actually supports it: $ qemu-system-aarch64 ... -drive file=test.vhd

[Qemu-devel] [PULL 7/7] qemu-iotests: fix 185

2017-08-11 Thread Kevin Wolf
From: Vladimir Sementsov-Ogievskiy 185 can sometimes produce wrong output like this: 185 2s ... - output mismatch (see 185.out.bad) --- /work/src/qemu/master/tests/qemu-iotests/185.out2017-07-14 \ 15:14:29.520343805 +0300 +++ 185.out.bad 2017-08-07 16:51:02.231922900 +030

[Qemu-devel] [PULL 5/7] osdep: Add runtime OFD lock detection

2017-08-11 Thread Kevin Wolf
From: Fam Zheng Build time check of OFD lock is not sufficient and can cause image open errors when the runtime environment doesn't support it. Add a helper function to probe it at runtime, additionally. Also provide a qemu_has_ofd_lock() for callers to check the status. Signed-off-by: Fam Zhen

[Qemu-devel] [PULL 4/7] qcow2: Check failure of bdrv_getlength()

2017-08-11 Thread Kevin Wolf
From: Eric Blake qcow2_co_pwritev_compressed() should not call bdrv_truncate() if determining the size failed. Reported-by: Markus Armbruster Signed-off-by: Eric Blake Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Jeff Cody Reviewed-by: Kevin Wolf Reviewed-by: John Snow Signed-off-by: K

[Qemu-devel] [PULL 2/7] vpc: Check failure of bdrv_getlength()

2017-08-11 Thread Kevin Wolf
From: Eric Blake vpc_open() was checking for bdrv_getlength() failure in one, but not the other, location. Reported-by: Markus Armbruster Signed-off-by: Eric Blake Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Jeff Cody Reviewed-by: John Snow Signed-off-by: Kevin Wolf --- block/vpc.c |

[Qemu-devel] [PULL 3/7] qcow2: Drop debugging dump_refcounts()

2017-08-11 Thread Kevin Wolf
From: Eric Blake It's been #if 0'd since its introduction in 2006, commit 585f8587. We can revive dead code if we need it, but in the meantime, it has bit-rotted (for example, not checking for failure in bdrv_getlength()). Signed-off-by: Eric Blake Reviewed-by: Philippe Mathieu-Daudé Reviewed-

[Qemu-devel] [PULL 0/7] Block/Multiboot patches for 2.10.0-rc3

2017-08-11 Thread Kevin Wolf
The following changes since commit 95766c2cd04395e5712b4d5967b3251f35d537df: Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into staging (2017-08-10 18:53:39 +0100) are available in the git repository at: git://repo.or.cz/qemu/kevin.git tags/for-upstream for you to

[Qemu-devel] [PULL 1/7] tests/multiboot: Fix whitespace failure

2017-08-11 Thread Kevin Wolf
From: Eric Blake Commit b43671f8 accidentally broke run_test.sh within tests/multiboot; due to a subtle change in whitespace. These two commands produce theh same output (at least, for sane $IFS of space-tab-newline): echo -e "...$@..." echo -e "...$*..." But that's only because echo inserts s

Re: [Qemu-devel] [PATCH RFC 1/5] target/s390x: introduce internal.h

2017-08-11 Thread Thomas Huth
On 11.08.2017 09:46, David Hildenbrand wrote: > cpu.h should only contain what really has to be accessed outside of > target/s390x/. Add internal.h which can only be used inside target/s390x/. > > Move everything that isn't fast enough to run away and restructure it > right away. > > Minor style

Re: [Qemu-devel] big endian arm.

2017-08-11 Thread KONRAD Frederic
On 08/11/2017 01:29 PM, Philippe Mathieu-Daudé wrote: Hi Frederic, I'm slowly working on Hercules MCU peripherals and use a R4F core. It seems that when I try to load a big endian image on a Cortex-R5 it gets confused: * the instructions are fine it executes some code. So far no problem

Re: [Qemu-devel] [PATCH 2/2] loader: Ignore zero-sized ELF segments

2017-08-11 Thread Richard Henderson
On 08/07/2017 07:39 AM, Peter Maydell wrote: > Some ELF files have program headers that specify segments that > are of zero size. Ignore them, rather than trying to create > zero-length ROM blobs for them, because the zero-length blob > can falsely trigger the overlapping-ROM-blobs check. > > Sign

Re: [Qemu-devel] [PATCH 1/2] loader: Handle ELF files with overlapping zero-initialized data

2017-08-11 Thread Richard Henderson
On 08/07/2017 07:39 AM, Peter Maydell wrote: > For embedded systems, notably ARM, one common use of ELF > file segments is that the 'physical addresses' represent load addresses > and the 'virtual addresses' execution addresses, such that > the load addresses are packed into ROM or flash, and the >

Re: [Qemu-devel] [PATCH v2] boot-serial-test: fallback to kvm accelerator

2017-08-11 Thread Paolo Bonzini
On 11/08/2017 15:34, Cornelia Huck wrote: > On Fri, 11 Aug 2017 14:29:53 +0100 > "Daniel P. Berrange" wrote: > >> On Fri, Aug 11, 2017 at 03:18:01PM +0200, Cornelia Huck wrote: >>> Currently, at least x86_64 and s390x support building with --disable-tcg. >>> Instead of forcing tcg (which causes t

Re: [Qemu-devel] [PATCH v2] boot-serial-test: fallback to kvm accelerator

2017-08-11 Thread Cornelia Huck
On Fri, 11 Aug 2017 14:29:53 +0100 "Daniel P. Berrange" wrote: > On Fri, Aug 11, 2017 at 03:18:01PM +0200, Cornelia Huck wrote: > > Currently, at least x86_64 and s390x support building with --disable-tcg. > > Instead of forcing tcg (which causes the test to fail on such builds), > > allow to use

Re: [Qemu-devel] [PATCH v2] boot-serial-test: fallback to kvm accelerator

2017-08-11 Thread Thomas Huth
On 11.08.2017 15:18, Cornelia Huck wrote: > Currently, at least x86_64 and s390x support building with --disable-tcg. > Instead of forcing tcg (which causes the test to fail on such builds), > allow to use kvm as well. > > Signed-off-by: Cornelia Huck > --- > > v1->v2: switch around kvm and tcg

Re: [Qemu-devel] [PATCH v2] boot-serial-test: fallback to kvm accelerator

2017-08-11 Thread Daniel P. Berrange
On Fri, Aug 11, 2017 at 03:18:01PM +0200, Cornelia Huck wrote: > Currently, at least x86_64 and s390x support building with --disable-tcg. > Instead of forcing tcg (which causes the test to fail on such builds), > allow to use kvm as well. > > Signed-off-by: Cornelia Huck > --- > > v1->v2: switc

Re: [Qemu-devel] [PATCH RFC 1/5] target/s390x: introduce internal.h

2017-08-11 Thread David Hildenbrand
On 11.08.2017 15:15, Richard Henderson wrote: > On 08/11/2017 12:46 AM, David Hildenbrand wrote: >> + >> +static const char *cc_names[] = { >> +[CC_OP_CONST0]= "CC_OP_CONST0", > > This should get moved out of the header to the c file that uses it. > (Why it's ever there in the first place

Re: [Qemu-devel] [PATCH RFC 5/5] target/s390x: cleanup cpu.h

2017-08-11 Thread Richard Henderson
On 08/11/2017 12:46 AM, David Hildenbrand wrote: > Let's reshuffle the function prototypes so we get a cleaner outline > of the files. > > Signed-off-by: David Hildenbrand > --- > target/s390x/cpu.h | 140 > ++--- > 1 file changed, 70 insertions(+

Re: [Qemu-devel] [PATCH RFC 3/5] s390x: avoid calling kvm_ functions outside of target/s390x/

2017-08-11 Thread Richard Henderson
On 08/11/2017 12:46 AM, David Hildenbrand wrote: > } > +void s390_enable_css_support(S390CPU *cpu) Spacing. Otherwise, Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH RFC 4/5] target/s390x: remove all CONFIG_KVM from cpu.h

2017-08-11 Thread Richard Henderson
On 08/11/2017 12:46 AM, David Hildenbrand wrote: > Let's move everything into internal.h > > Signed-off-by: David Hildenbrand > --- > target/s390x/cpu.h | 62 > - > target/s390x/internal.h | 52 + > 2 f

[Qemu-devel] [PATCH v2] boot-serial-test: fallback to kvm accelerator

2017-08-11 Thread Cornelia Huck
Currently, at least x86_64 and s390x support building with --disable-tcg. Instead of forcing tcg (which causes the test to fail on such builds), allow to use kvm as well. Signed-off-by: Cornelia Huck --- v1->v2: switch around kvm and tcg Added Paolo on cc:; I can still take this through the s39

Re: [Qemu-devel] [PATCH RFC 2/5] target/s390x: move a couple of functions to cpu.c

2017-08-11 Thread Richard Henderson
On 08/11/2017 12:46 AM, David Hildenbrand wrote: > Signed-off-by: David Hildenbrand > --- > target/s390x/cpu.c | 80 + > target/s390x/cpu.h | 88 > +++--- > 2 files changed, 90 insertions(+), 78 delet

Re: [Qemu-devel] [PATCH RFC 1/5] target/s390x: introduce internal.h

2017-08-11 Thread Richard Henderson
On 08/11/2017 12:46 AM, David Hildenbrand wrote: > + > +static const char *cc_names[] = { > +[CC_OP_CONST0]= "CC_OP_CONST0", This should get moved out of the header to the c file that uses it. (Why it's ever there in the first place is a mystery...) Otherwise, Reviewed-by: Richard Hender

[Qemu-devel] [PATCH 2/2] backup: QEMU Backup Tool

2017-08-11 Thread Ishani Chugh
qemu-backup will be a command-line tool for performing full and incremental disk backups on running VMs. It is intended as a reference implementation for management stack and backup developers to see QEMU's backup features in action. The tool writes details of guest in a configuration file and the

[Qemu-devel] [PATCH 1/2] Add manpage for QEMU Backup Tool

2017-08-11 Thread Ishani Chugh
qemu-backup will be a command-line tool for performing full and incremental disk backups on running VMs. It is intended as a reference implementation for management stack and backup developers to see QEMU's backup features in action. The following commit is an initial implementation of manpage list

[Qemu-devel] [PATCH 0/2] QEMU Backup Tool

2017-08-11 Thread Ishani Chugh
This patch series is intended to introduce QEMU Backup tool. qemu-backup will be a command-line tool for performing full and incremental disk backups on running VMs. It is intended as a reference implementation for management stack and backup developers to see QEMU's backup features in action. This

Re: [Qemu-devel] [PATCH] boot-serial-test: fallback to kvm accelerator

2017-08-11 Thread Cornelia Huck
On Fri, 11 Aug 2017 13:56:37 +0200 Thomas Huth wrote: > On 11.08.2017 13:47, Cornelia Huck wrote: > > diff --git a/tests/boot-serial-test.c b/tests/boot-serial-test.c > > index 11f48b049c..38664a0594 100644 > > --- a/tests/boot-serial-test.c > > +++ b/tests/boot-serial-test.c > > @@ -78,7 +78,8

Re: [Qemu-devel] [Update PATCH V2] x86: Increase max vcpu number to 8192

2017-08-11 Thread Eduardo Habkost
On Thu, Aug 10, 2017 at 10:26:06PM -0400, Lan Tianyu wrote: > Intel Xeon phi chip will support 352 logical threads. For HPC > usage case, it will create a huge VM with vcpus number as same as host > cpus. This patch is to increase max vcpu number from 288 to 8192 which > is current default maximum

Re: [Qemu-devel] [PATCH 2/8] io: introduce a network socket listener API

2017-08-11 Thread Daniel P. Berrange
On Fri, Aug 11, 2017 at 01:39:43PM +0100, Dr. David Alan Gilbert wrote: > * Daniel P. Berrange (berra...@redhat.com) wrote: > > On Fri, Aug 11, 2017 at 01:26:00PM +0100, Dr. David Alan Gilbert wrote: > > > * Daniel P. Berrange (berra...@redhat.com) wrote: > > > > The existing QIOChannelSocket class

Re: [Qemu-devel] [PATCH v2 for 2.10] iotests: fix 185

2017-08-11 Thread Kevin Wolf
Am 09.08.2017 um 17:17 hat Vladimir Sementsov-Ogievskiy geschrieben: > 185 can sometimes produce wrong output like this: > > = > 185 2s ... - output mismatch (see 185.out.bad) > --- /work/src/qemu/master/tests/qemu-iotests/185.out2017-07-14 \ > 15:14

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-11 Thread Fam Zheng
On Fri, 08/11 14:06, Paolo Bonzini wrote: > On 11/08/2017 09:54, Fam Zheng wrote: > > On Thu, 08/10 12:25, Cornelia Huck wrote: > >>> I think the UI (giving no consideration to how we might implement > >>> this!) would ideally be something like: > >>> * if anybody mails a patch which touches an "u

Re: [Qemu-devel] [PATCH 2/8] io: introduce a network socket listener API

2017-08-11 Thread Dr. David Alan Gilbert
* Daniel P. Berrange (berra...@redhat.com) wrote: > On Fri, Aug 11, 2017 at 01:26:00PM +0100, Dr. David Alan Gilbert wrote: > > * Daniel P. Berrange (berra...@redhat.com) wrote: > > > The existing QIOChannelSocket class provides the ability to > > > listen on a single socket at a time. This patch i

Re: [Qemu-devel] migration issue with qemu 2.10-rc2: QEMU command 'nbd-server-add': Block node is read-only

2017-08-11 Thread Kevin Wolf
Am 11.08.2017 um 14:04 hat Fam Zheng geschrieben: > On Fri, 08/11 13:07, Christian Ehrhardt wrote: > > Simplifying that to a smaller test: > > > > $ qemu-img create -f qcow2 /tmp/test.qcow2 100M > > $ qemu-system-x86_64 -S -m 512 -smp 1 -nodefaults --nographic -monitor > > stdio -drive > > file=/t

Re: [Qemu-devel] [PATCH 2/8] io: introduce a network socket listener API

2017-08-11 Thread Daniel P. Berrange
On Fri, Aug 11, 2017 at 01:26:00PM +0100, Dr. David Alan Gilbert wrote: > * Daniel P. Berrange (berra...@redhat.com) wrote: > > The existing QIOChannelSocket class provides the ability to > > listen on a single socket at a time. This patch introduces > > a QIONetListener class that provides a highe

Re: [Qemu-devel] [PATCH 2/8] io: introduce a network socket listener API

2017-08-11 Thread Dr. David Alan Gilbert
* Daniel P. Berrange (berra...@redhat.com) wrote: > The existing QIOChannelSocket class provides the ability to > listen on a single socket at a time. This patch introduces > a QIONetListener class that provides a higher level API > concept around listening for network services, allowing > for list

Re: [Qemu-devel] [PATCH for-2.10? v3 0/2] block: Do OFD lock check at runtime

2017-08-11 Thread Kevin Wolf
Am 11.08.2017 um 13:44 hat Fam Zheng geschrieben: > v3: Fix mingw build. [patchew] > > v2: Probe /dev/null to save LOC. [Eric] > Mention "new glibc + old kernel" in commit message. [Kevin, Daniel, Eric, > Christian] > > This fixes the image opening failure reported by Andrew Baumann: > >

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-11 Thread Paolo Bonzini
On 11/08/2017 09:54, Fam Zheng wrote: > On Thu, 08/10 12:25, Cornelia Huck wrote: >>> I think the UI (giving no consideration to how we might implement >>> this!) would ideally be something like: >>> * if anybody mails a patch which touches an "unmaintained" file, >>>a robot should send a repl

Re: [Qemu-devel] migration issue with qemu 2.10-rc2: QEMU command 'nbd-server-add': Block node is read-only

2017-08-11 Thread Fam Zheng
On Fri, 08/11 13:07, Christian Ehrhardt wrote: > Hi, > testing on 2.10-rc2 I ran into an issue around: > unable to execute QEMU command 'nbd-server-add': Block node is read-only > > ### TL;DR ### > - triggered by livbirt driven live migration with --copy-storage-all > - buils down to nbd_server_

Re: [Qemu-devel] [PATCH] boot-serial-test: fallback to kvm accelerator

2017-08-11 Thread Thomas Huth
On 11.08.2017 13:47, Cornelia Huck wrote: > Currently, at least x86_64 and s390x support building with --disable-tcg. > Instead of forcing tcg (which causes the test to fail on such builds), > allow to use kvm as well. > > Signed-off-by: Cornelia Huck > --- > > Not sure who should merge this. I

Re: [Qemu-devel] [PATCH] x86: Increase max vcpu number to 352

2017-08-11 Thread Eduardo Habkost
On Thu, Aug 10, 2017 at 02:08:38PM -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Aug 09, 2017 at 11:58:20PM -0400, Lan Tianyu wrote: > > Intel Xeon phi chip will support 352 logical threads. For HPC > > usage case, it will create a huge VM with vcpus number as same as host > > cpus. This patch is t

[Qemu-devel] [PATCH] boot-serial-test: fallback to kvm accelerator

2017-08-11 Thread Cornelia Huck
Currently, at least x86_64 and s390x support building with --disable-tcg. Instead of forcing tcg (which causes the test to fail on such builds), allow to use kvm as well. Signed-off-by: Cornelia Huck --- Not sure who should merge this. I can add it to the s390 tree if nobody else takes it. ---

[Qemu-devel] [PATCH for-2.10? v3 2/2] file-posix: Do runtime check for ofd lock API

2017-08-11 Thread Fam Zheng
It is reported that on Windows Subsystem for Linux, ofd operations fail with -EINVAL. In other words, QEMU binary built with system headers that exports F_OFD_SETLK doesn't necessarily run in an environment that actually supports it: $ qemu-system-aarch64 ... -drive file=test.vhdx,if=none,id=hd0 \

[Qemu-devel] [PATCH for-2.10? v3 0/2] block: Do OFD lock check at runtime

2017-08-11 Thread Fam Zheng
v3: Fix mingw build. [patchew] v2: Probe /dev/null to save LOC. [Eric] Mention "new glibc + old kernel" in commit message. [Kevin, Daniel, Eric, Christian] This fixes the image opening failure reported by Andrew Baumann: > I'm running a recent Linux build of qemu on Windows Subsystem for

[Qemu-devel] [PATCH for-2.10? v3 1/2] osdep: Add runtime OFD lock detection

2017-08-11 Thread Fam Zheng
Build time check of OFD lock is not sufficient and can cause image open errors when the runtime environment doesn't support it. Add a helper function to probe it at runtime, additionally. Also provide a qemu_has_ofd_lock() for callers to check the status. Signed-off-by: Fam Zheng --- include/qe

Re: [Qemu-devel] [PULL 0/6] ppc patch queue 2017-08-09

2017-08-11 Thread Philippe Mathieu-Daudé
Hi David, Peter, On 08/11/2017 05:59 AM, Peter Maydell wrote: On 11 August 2017 at 09:51, David Gibson wrote: On Thu, Aug 10, 2017 at 01:47:03PM +0100, Peter Maydell wrote: On 9 August 2017 at 08:03, David Gibson wrote: I haven't completed a Travis build for this, which is part of my usual

Re: [Qemu-devel] [PATCH v2 for-2.11 0/2] Improvements for the pxe tester

2017-08-11 Thread Cornelia Huck
On Fri, 11 Aug 2017 12:52:54 +0200 Thomas Huth wrote: > On 11.08.2017 12:19, Cornelia Huck wrote: > > A quick test on x86_64 with --disable-tcg showed no further problems > > than boot-serial-test, so at least it's only the one for now (and it > > might make sense to simply use accel=tcg:kvm for

Re: [Qemu-devel] [PATCH for-2.11] s390x: introduce 2.11 compat machine

2017-08-11 Thread Halil Pasic
; On 08/10/2017 02:37 PM, Cornelia Huck wrote: > Signed-off-by: Cornelia Huck > --- > > It's that time again. I plan to put this into my first 2.11 pullreq. > Yes, and I assume that the next version is indeed 2.11 :) > > --- > hw/s390x/s390-virtio-ccw.c | 17 - > include/hw/com

Re: [Qemu-devel] big endian arm.

2017-08-11 Thread Philippe Mathieu-Daudé
Hi Frederic, I'm slowly working on Hercules MCU peripherals and use a R4F core. It seems that when I try to load a big endian image on a Cortex-R5 it gets confused: * the instructions are fine it executes some code. So far no problem here * GDB address / insns are wrong endianness. Ma

Re: [Qemu-devel] [PATCH for-2.10 v2 0/5] More bdrv_getlength() fixes

2017-08-11 Thread Kevin Wolf
Am 10.08.2017 um 17:08 hat Eric Blake geschrieben: > On 08/10/2017 08:02 AM, Kevin Wolf wrote: > > Am 09.08.2017 um 22:38 hat Eric Blake geschrieben: > >> We already have a lot of bdrv_getlength() fixes in -rc2; so I think > >> this is still okay for -rc3. > >> > >> v1 was here (with a typo'd subje

Re: [Qemu-devel] big endian arm.

2017-08-11 Thread KONRAD Frederic
On 08/11/2017 01:05 PM, Peter Maydell wrote: On 11 August 2017 at 12:03, KONRAD Frederic wrote: On 08/11/2017 12:18 PM, Peter Maydell wrote: That said, if you specify a BE elf file then we do set the SCTLR.EE and CPSR.E bits on reset in do_cpu_reset() (a change added in the commit you quote)

Re: [Qemu-devel] [PATCH] error: Improve documentation of error_append_hint()

2017-08-11 Thread Halil Pasic
On 08/10/2017 12:06 PM, Markus Armbruster wrote: > Suggested-by: Halil Pasic > Signed-off-by: Markus Armbruster Nice clarification! Reviewed-by: Halil Pasic

[Qemu-devel] migration issue with qemu 2.10-rc2: QEMU command 'nbd-server-add': Block node is read-only

2017-08-11 Thread Christian Ehrhardt
Hi, testing on 2.10-rc2 I ran into an issue around: unable to execute QEMU command 'nbd-server-add': Block node is read-only ### TL;DR ### - triggered by livbirt driven live migration with --copy-storage-all - buils down to nbd_server_add failing - can be reproduced on a single system without li

Re: [Qemu-devel] big endian arm.

2017-08-11 Thread Peter Maydell
On 11 August 2017 at 12:03, KONRAD Frederic wrote: > On 08/11/2017 12:18 PM, Peter Maydell wrote: >> That said, if you specify a BE elf file then we do >> set the SCTLR.EE and CPSR.E bits on reset in do_cpu_reset() >> (a change added in the commit you quote), which is probably >> why we haven't no

Re: [Qemu-devel] big endian arm.

2017-08-11 Thread KONRAD Frederic
On 08/11/2017 12:18 PM, Peter Maydell wrote: On 11 August 2017 at 10:59, KONRAD Frederic wrote: Hi Peters, I got some strange results since this commit: commit 9776f636455b6f0d9c14dce112242ed653f954b4 Author: Peter Crosthwaite Date: Fri Mar 4 11:30:21 2016 + arm: boot: Support

Re: [Qemu-devel] [PATCH v2 for-2.11 0/2] Improvements for the pxe tester

2017-08-11 Thread Thomas Huth
On 11.08.2017 12:19, Cornelia Huck wrote: > On Fri, 11 Aug 2017 11:49:22 +0200 > Cornelia Huck wrote: > >> On Fri, 11 Aug 2017 07:57:54 +0200 >> Thomas Huth wrote: >> >>> The first patch improves the buffer handling in the pxe tester a >>> little bit by allocating a separate buffer on the heap f

Re: [Qemu-devel] [PATCH v4 5/7] block: add throttle block filter driver

2017-08-11 Thread Manos Pitsidianakis
On Wed, Aug 09, 2017 at 01:07:32PM +0300, Manos Pitsidianakis wrote: +static int coroutine_fn throttle_co_preadv(BlockDriverState *bs, + uint64_t offset, uint64_t bytes, + QEMUIOVector *qiov, int flags) +{ + +

Re: [Qemu-devel] [Qemu-block] [PATCH] nbd: Fix trace message for disconnect

2017-08-11 Thread Stefan Hajnoczi
On Thu, Aug 10, 2017 at 08:57:48PM -0500, Eric Blake wrote: > NBD_CMD_DISC is a disconnect request, not a data discard request. > > Signed-off-by: Eric Blake > --- > > Although this is not 2.10 material in isolation (it is only a > bad trace message), I don't mind including it in a larger pull >

Re: [Qemu-devel] [PATCH] virtio-scsi: Add virtqueue_size parameter allowing virtqueue size to be set.

2017-08-11 Thread Stefan Hajnoczi
On Thu, Aug 10, 2017 at 05:52:55PM +0100, Richard W.M. Jones wrote: > Since Linux switched to blk-mq as the default in Linux commit > 5c279bd9e406 ("scsi: default to scsi-mq"), virtio-scsi LUNs consume > about 10x as much guest kernel memory. > > This commit allows you to choose the virtqueue size

Re: [Qemu-devel] [PATCH v2 for-2.11 0/2] Improvements for the pxe tester

2017-08-11 Thread Cornelia Huck
On Fri, 11 Aug 2017 11:49:22 +0200 Cornelia Huck wrote: > On Fri, 11 Aug 2017 07:57:54 +0200 > Thomas Huth wrote: > > > The first patch improves the buffer handling in the pxe tester a > > little bit by allocating a separate buffer on the heap for each > > architecture. This also gets rid of th

Re: [Qemu-devel] big endian arm.

2017-08-11 Thread Peter Maydell
On 11 August 2017 at 10:59, KONRAD Frederic wrote: > Hi Peters, > > I got some strange results since this commit: > > commit 9776f636455b6f0d9c14dce112242ed653f954b4 > Author: Peter Crosthwaite > Date: Fri Mar 4 11:30:21 2016 + > > arm: boot: Support big-endian elfs > > Support ARM

Re: [Qemu-devel] [PATCH 2/8] io: introduce a network socket listener API

2017-08-11 Thread Daniel P. Berrange
On Fri, Aug 11, 2017 at 10:15:04AM +0100, Daniel P. Berrange wrote: > On Thu, Aug 10, 2017 at 01:12:25PM -0500, Eric Blake wrote: > > On 08/10/2017 11:04 AM, Daniel P. Berrange wrote: > > > The existing QIOChannelSocket class provides the ability to > > > listen on a single socket at a time. This p

Re: [Qemu-devel] [PATCH v5 4/4] docs: update documentation considering PCIE-PCI bridge

2017-08-11 Thread Laszlo Ersek
On 08/11/17 01:31, Aleksandr Bezzubikov wrote: > +PCIE-PCI bridge hot-plug > +=== > +Guest OSes require extra efforts to enable PCIE-PCI bridge hot-plug. > +Motivation - now on init any PCI Express root port which doesn't have > +any device plugged in, has no free buses reserve

Re: [Qemu-devel] [PATCH v3 1/6] seccomp: changing from whitelist to blacklist

2017-08-11 Thread Daniel P. Berrange
On Fri, Aug 11, 2017 at 11:51:12AM +0200, Eduardo Otubo wrote: > On Thu, Aug 03, 2017 at 06:54:15PM +0200, Thomas Huth wrote: > > On 28.07.2017 14:10, Eduardo Otubo wrote: > > > This patch changes the default behavior of the seccomp filter from > > > whitelist to blacklist. By default now all syste

Re: [Qemu-devel] [PATCH 0/2] QEMU Backup Tool

2017-08-11 Thread Stefan Hajnoczi
On Thu, Aug 10, 2017 at 11:14:33PM +0530, Ishani Chugh wrote: > This patch series is intended to introduce QEMU Backup tool. > qemu-backup will be a command-line tool for performing full and > incremental disk backups on running VMs. It is intended as a > reference implementation for management sta

[Qemu-devel] big endian arm.

2017-08-11 Thread KONRAD Frederic
Hi Peters, I got some strange results since this commit: commit 9776f636455b6f0d9c14dce112242ed653f954b4 Author: Peter Crosthwaite Date: Fri Mar 4 11:30:21 2016 + arm: boot: Support big-endian elfs Support ARM big-endian ELF files in system-mode emulation. When loading an e

[Qemu-devel] [PATCH for-2.10? v2 2/2] file-posix: Do runtime check for ofd lock API

2017-08-11 Thread Fam Zheng
It is reported that on Windows Subsystem for Linux, ofd operations fail with -EINVAL. In other words, QEMU binary built with system headers that exports F_OFD_SETLK doesn't necessarily run in an environment that actually supports it: $ qemu-system-aarch64 ... -drive file=test.vhdx,if=none,id=hd0 \

[Qemu-devel] [PATCH for-2.10? v2 0/2] block: Do OFD lock check at runtime

2017-08-11 Thread Fam Zheng
v2: Probe /dev/null to save LOC. [Eric] Mention "new glibc + old kernel" in commit message. [Kevin, Daniel, Eric, Christian] This fixes the image opening failure reported by Andrew Baumann: > I'm running a recent Linux build of qemu on Windows Subsystem for Linux (WSL) > which doesn't app

[Qemu-devel] [PATCH for-2.10? v2 1/2] osdep: Add runtime OFD lock detection

2017-08-11 Thread Fam Zheng
Build time check of OFD lock is not sufficient and can cause image open errors when the runtime environment doesn't support it. Add a helper function to probe it at runtime, additionally. Also provide a qemu_has_ofd_lock() for callers to check the status. Signed-off-by: Fam Zheng --- include/qe

Re: [Qemu-devel] [PATCH v3 1/6] seccomp: changing from whitelist to blacklist

2017-08-11 Thread Eduardo Otubo
On Thu, Aug 03, 2017 at 06:54:15PM +0200, Thomas Huth wrote: > On 28.07.2017 14:10, Eduardo Otubo wrote: > > This patch changes the default behavior of the seccomp filter from > > whitelist to blacklist. By default now all system calls are allowed and > > a small black list of definitely forbidden

Re: [Qemu-devel] [PATCH v2 for-2.11 0/2] Improvements for the pxe tester

2017-08-11 Thread Cornelia Huck
On Fri, 11 Aug 2017 07:57:54 +0200 Thomas Huth wrote: > The first patch improves the buffer handling in the pxe tester a > little bit by allocating a separate buffer on the heap for each > architecture. This also gets rid of the huge pre-initialized > array in the tester, shrinking the size of th

Re: [Qemu-devel] [PATCH v3 2/6] seccomp: add obsolete argument to command line

2017-08-11 Thread Eduardo Otubo
On Fri, Aug 11, 2017 at 11:12:48AM +0200, Eduardo Otubo wrote: > On Wed, Aug 02, 2017 at 01:33:56PM +0100, Daniel P. Berrange wrote: > > On Fri, Jul 28, 2017 at 02:10:36PM +0200, Eduardo Otubo wrote: > > > This patch introduces the argument [,obsolete=allow] to the `-sandbox on' > > > option. It al

Re: [Qemu-devel] [PATCH v3 2/6] seccomp: add obsolete argument to command line

2017-08-11 Thread Daniel P. Berrange
On Fri, Aug 11, 2017 at 11:12:48AM +0200, Eduardo Otubo wrote: > On Wed, Aug 02, 2017 at 01:33:56PM +0100, Daniel P. Berrange wrote: > > On Fri, Jul 28, 2017 at 02:10:36PM +0200, Eduardo Otubo wrote: > > > This patch introduces the argument [,obsolete=allow] to the `-sandbox on' > > > option. It al

Re: [Qemu-devel] [PATCH 8/8] sockets: fix parsing of ipv4/ipv6 opts in parse_socket_addr

2017-08-11 Thread Daniel P. Berrange
On Thu, Aug 10, 2017 at 01:35:15PM -0500, Eric Blake wrote: > On 08/10/2017 11:04 AM, Daniel P. Berrange wrote: > > The inet_parse() function looks for 'ipv4' and 'ipv6' > > flags, but only treats them as bare bool flags. The normal > > QemuOpts parsing would allow on/off values to be set too. > >

Re: [Qemu-devel] [PATCH 2/8] io: introduce a network socket listener API

2017-08-11 Thread Daniel P. Berrange
On Thu, Aug 10, 2017 at 01:12:25PM -0500, Eric Blake wrote: > On 08/10/2017 11:04 AM, Daniel P. Berrange wrote: > > The existing QIOChannelSocket class provides the ability to > > listen on a single socket at a time. This patch introduces > > a QIONetListener class that provides a higher level API

Re: [Qemu-devel] [PATCH for-2.11 0/4] ppc64: add e6500

2017-08-11 Thread Alexander Graf
On 09.08.17 08:13, David Gibson wrote: On Mon, Aug 07, 2017 at 05:50:44PM +0200, KONRAD Frederic wrote: Hi, Those are some patches to add basic e6500 support for the moment e5500 with a correct MMU configuration and supported instructions. Some (maybe a lot of) things are missing (ie: the thr

Re: [Qemu-devel] [PATCH 05/15] ui: use QKeyCode exclusively in InputKeyEvent

2017-08-11 Thread Daniel P. Berrange
On Thu, Aug 10, 2017 at 02:02:32PM -0500, Eric Blake wrote: > On 08/10/2017 10:55 AM, Daniel P. Berrange wrote: > > Now that keycode numbers are converted to QKeyCodes immediately > > when creating input events, the InputKeyEvent struct can be > > changed to only accept a QKeyCode, instead of a Key

Re: [Qemu-devel] [PATCH 02/15] ui: convert common input code to keycodemapdb

2017-08-11 Thread Daniel P. Berrange
On Thu, Aug 10, 2017 at 01:59:09PM -0500, Eric Blake wrote: > On 08/10/2017 10:55 AM, Daniel P. Berrange wrote: > > Replace the number_to_qcode, qcode_to_number and linux_to_qcode > > tables with automatically generated tables. > > > > Missing entries in linux_to_qcode now fixed: > > > In additio

Re: [Qemu-devel] [PATCH v3 2/6] seccomp: add obsolete argument to command line

2017-08-11 Thread Eduardo Otubo
On Wed, Aug 02, 2017 at 01:33:56PM +0100, Daniel P. Berrange wrote: > On Fri, Jul 28, 2017 at 02:10:36PM +0200, Eduardo Otubo wrote: > > This patch introduces the argument [,obsolete=allow] to the `-sandbox on' > > option. It allows Qemu to run safely on old system that still relies on > > old syst

Re: [Qemu-devel] [PATCH] tests/qmp-test: Add generic, basic test of query commands

2017-08-11 Thread Markus Armbruster
Eric Blake writes: > On 08/10/2017 01:30 PM, Markus Armbruster wrote: >> A command is a query if it has no side effect and yields a result. >> Such commands are typically named query-FOO, but there are exceptions. >> >> The basic idea is to find candidates with query-qmp-schema, filter out >> th

Re: [Qemu-devel] [PATCH 01/15] ui: add keycodemapdb repository as a GIT submodule

2017-08-11 Thread Daniel P. Berrange
On Thu, Aug 10, 2017 at 01:23:08PM -0500, Eric Blake wrote: > On 08/10/2017 10:55 AM, Daniel P. Berrange wrote: > > The https://gitlab.com/keycodemap/keycodemapdb/ repo contains a > > data file mapping between all the different scancode/keycode/keysym > > sets that are known, and a tool to auto-gen

Re: [Qemu-devel] [PULL 0/6] ppc patch queue 2017-08-09

2017-08-11 Thread Peter Maydell
On 11 August 2017 at 09:51, David Gibson wrote: > On Thu, Aug 10, 2017 at 01:47:03PM +0100, Peter Maydell wrote: >> On 9 August 2017 at 08:03, David Gibson wrote: >> > I haven't completed a Travis build for this, which is part of my usual >> > test regime, since the first dozen or so Travis build

Re: [Qemu-devel] [PATCH RFC 0/5] target/s390x: introduce internal.h and cleanup cpu.h

2017-08-11 Thread David Hildenbrand
> /var/tmp/patchew-tester-tmp-05mor12r/src/hw/intc/s390_flic_kvm.c:125:5: > error: no previous prototype for ‘kvm_s390_inject_flic’ > [-Werror=missing-prototypes] > int kvm_s390_inject_flic(struct kvm_s390_irq *irq) > ^~~~ > cc1: all warnings being treated as errors Easily

Re: [Qemu-devel] [PULL 0/6] ppc patch queue 2017-08-09

2017-08-11 Thread David Gibson
On Thu, Aug 10, 2017 at 01:47:03PM +0100, Peter Maydell wrote: > On 9 August 2017 at 08:03, David Gibson wrote: > > The following changes since commit 54affb3a3623b1d36c95e34faa722a5831323a74: > > > > Update version for v2.10.0-rc2 release (2017-08-08 19:07:46 +0100) > > > > are available in the

[Qemu-devel] [Update PATCH V2] x86: Increase max vcpu number to 8192

2017-08-11 Thread Lan Tianyu
Intel Xeon phi chip will support 352 logical threads. For HPC usage case, it will create a huge VM with vcpus number as same as host cpus. This patch is to increase max vcpu number from 288 to 8192 which is current default maximum cpu number for Linux kernel. Signed-off-by: Lan Tianyu --- Change

Re: [Qemu-devel] [PULL for-2.10 0/3] Block patches

2017-08-11 Thread Peter Maydell
On 10 August 2017 at 18:48, Stefan Hajnoczi wrote: > The following changes since commit b38df311c174c98ef8cce7dec9f46603b083018e: > > Merge remote-tracking branch 'remotes/dgibson/tags/ppc-for-2.10-20170809' > into staging (2017-08-10 11:12:36 +0100) > > are available in the git repository at:

[Qemu-devel] [PATCH v3 2/2] tests/pxe-test: add testcase using vhost-user-bridge

2017-08-11 Thread Jens Freimann
From: Jens Freimann Add a PXE testcase tunneling traffic through vhost-user-bridge process. Create a vhost-user-bridge process and connect it to qemu via a socket. Signed-off-by: Jens Freimann --- tests/Makefile.include | 4 +- tests/pxe-test.c | 114 ++

Re: [Qemu-devel] [PATCH v2] x86: Increase max vcpu number to 8192

2017-08-11 Thread no-reply
Hi, This series failed build test on FreeBSD host. Please find the details below. Type: series Message-id: 1502417472-15511-1-git-send-email-tianyu@intel.com Subject: [Qemu-devel] [PATCH v2] x86: Increase max vcpu number to 8192 === TEST SCRIPT BEGIN === #!/bin/sh # Testing script will be in

Re: [Qemu-devel] [PATCH v2] x86: Increase max vcpu number to 8192

2017-08-11 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. Message-id: 1502417472-15511-1-git-send-email-tianyu@intel.com Subject: [Qemu-devel] [PATCH v2] x86: Increase max vcpu num

[Qemu-devel] [PATCH v2] x86: Increase max vcpu number to 8192

2017-08-11 Thread Lan Tianyu
Intel Xeon phi chip will support 352 logical threads. For HPC usage case, it will create a huge VM with vcpus number as same as host cpus. This patch is to increase max vcpu number from 288 to 8192 which is current default maximum cpu number for Linux kernel. Signed-off-by: Lan Tianyu --- Change

[Qemu-devel] [PATCH for-2.11 v2] file-posix: Clear out first sector in hdev_create

2017-08-11 Thread Fam Zheng
People get surprised when, after "qemu-img create -f raw /dev/sdX", they still see qcow2 with "qemu-img info", if previously the bdev had a qcow2 header. While this is natural because raw doesn't need to write any magic bytes during creation, hdev_create is free to clear out the first sector to mak

Re: [Qemu-devel] [PATCH] file-posix: Clear out first sector in hdev_create

2017-08-11 Thread Fam Zheng
On Fri, 08/11 15:44, Xie Changlong wrote: > 在 8/10/2017 4:01 PM, Fam Zheng 写道: > > People get surprised when, after "qemu-imc create -f raw /dev/sdX", they > > s/qemu-imc/qemu-img/ Thanks, will fix in v2. Fam

[Qemu-devel] [PATCH V2 for-2.10] xlnx-qspi: add a property for mmio-execution

2017-08-11 Thread KONRAD Frederic
This adds mmio-exec property to workaround the migration bug. When enabled the migration is blocked and will return an error. Signed-off-by: KONRAD Frederic V1 -> V2: * changed mmio-exec to x-mmio-exec --- hw/ssi/xilinx_spips.c | 36 +--- 1 file changed, 33 ins

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-11 Thread Fam Zheng
On Thu, 08/10 12:25, Cornelia Huck wrote: > > I think the UI (giving no consideration to how we might implement > > this!) would ideally be something like: > > * if anybody mails a patch which touches an "unmaintained" file, > >a robot should send a reply along the lines of "thanks for the > >

[Qemu-devel] [PATCH RFC 4/5] target/s390x: remove all CONFIG_KVM from cpu.h

2017-08-11 Thread David Hildenbrand
Let's move everything into internal.h Signed-off-by: David Hildenbrand --- target/s390x/cpu.h | 62 - target/s390x/internal.h | 52 + 2 files changed, 52 insertions(+), 62 deletions(-) diff --git a/targ

<    1   2   3   >