Re: [Qemu-devel] [PATCH v2 18/47] hw/intc: Replace fprintf(stderr, "*\n" with error_report()

2017-10-04 Thread Cornelia Huck
On Fri, 29 Sep 2017 17:15:54 -0700 Alistair Francis wrote: > Replace a large number of the fprintf(stderr, "*\n" calls with > error_report(). The functions were renamed with these commands and then > compiler issues where manually fixed. > diff --git a/hw/intc/s390_flic_kvm.c b/hw/intc/s390_flic

Re: [Qemu-devel] [PATCH v2 47/47] target: Replace fprintf(stderr, "*\n" with error_report()

2017-10-04 Thread Cornelia Huck
On Fri, 29 Sep 2017 17:17:26 -0700 Alistair Francis wrote: > Replace a large number of the fprintf(stderr, "*\n" calls with > error_report(). The functions were renamed with these commands and then > compiler issues where manually fixed. > diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c > i

Re: [Qemu-devel] [PATCH v10 8/9] tpm: Added support for TPM emulator

2017-10-04 Thread Valluri, Amarnath
On Tue, 2017-10-03 at 17:21 -0400, Stefan Berger wrote: > On 09/29/2017 07:10 AM, Amarnath Valluri wrote: > > > > This change introduces a new TPM backend driver that can > > communicate with > > swtpm(software TPM emulator) using unix domain socket interface. > > QEMU talks to > > TPM emulator us

Re: [Qemu-devel] [PATCH v3] net: fix check for number of parameters to -netdev socket

2017-10-04 Thread Jens Freimann
ping On Wed, Sep 27, 2017 at 03:21:18PM +, Jens Freimann wrote: Since commit 0f8c289ad "net: fix -netdev socket,fd= for UDP sockets" we allow more than one parameter for -netdev socket. But now we run into an assert when no parameter at all is specified qemu-system-x86_64 -netdev socket s

Re: [Qemu-devel] [PATCH v5 18/22] s390x: implement query-hotpluggable-cpus

2017-10-04 Thread Cornelia Huck
On Mon, 02 Oct 2017 09:46:41 +0200 Markus Armbruster wrote: > David Hildenbrand writes: > > > CPU hotplug is only possible on a per core basis on s390x. So let's > > add possible_cpus and properly wire everything up. > > > > Signed-off-by: David Hildenbrand > [...] > > diff --git a/qapi-sche

Re: [Qemu-devel] [PATCH v5 00/22] s390x cleanups and CPU hotplug via device_add

2017-10-04 Thread Cornelia Huck
On Mon, 02 Oct 2017 09:47:58 +0200 Markus Armbruster wrote: > Cornelia Huck writes: > > > On Wed, 13 Sep 2017 15:23:55 +0200 > > David Hildenbrand wrote: > > > >> The first patches are a bunch of cleanups. I decided to go the > >> extra mile and implement CPU hotplug via "device_add", as wel

Re: [Qemu-devel] [PATCH v3 5/6] s390x/css: activate ChannelSubSys migration

2017-10-04 Thread Cornelia Huck
On Tue, 3 Oct 2017 13:58:29 +0200 Thomas Huth wrote: > On 11.07.2017 16:54, Halil Pasic wrote: > > Turn on migration for the channel subsystem for the next machine. For > > legacy machines we still have to do things the old way. > > > > Signed-off-by: Halil Pasic > > Reviewed-by: Cornelia Huck

Re: [Qemu-devel] [PATCH v8 0/8] Optimize VMDK I/O by allocating multiple clusters

2017-10-04 Thread Ashijeet Acharya
On Thu, Aug 10, 2017 at 11:13 PM, Stefan Hajnoczi wrote: > On Thu, Aug 10, 2017 at 9:18 AM, Ashijeet Acharya > wrote: > > On Thu, Aug 10, 2017 at 1:41 PM, Stefan Hajnoczi > wrote: > >> > >> On Thu, Jul 27, 2017 at 3:33 PM, Ashijeet Acharya > >> wrote: > >> > Previously posted series patches: >

Re: [Qemu-devel] [PATCH v8 0/8] Optimize VMDK I/O by allocating multiple clusters

2017-10-04 Thread Fam Zheng
On Wed, 10/04 13:47, Ashijeet Acharya wrote: > Fam: Ping? Hi Ashijeet, looks like this patch doesn't apply to current master, could you rebase and post another version? Fam

Re: [Qemu-devel] [PATCH v8 0/8] Optimize VMDK I/O by allocating multiple clusters

2017-10-04 Thread Ashijeet Acharya
On Wed, Oct 4, 2017 at 1:58 PM, Fam Zheng wrote: > On Wed, 10/04 13:47, Ashijeet Acharya wrote: > > Fam: Ping? > > Hi Ashijeet, looks like this patch doesn't apply to current master, could > you > rebase and post another version? > Hello Fam, I will try to do it over the weekend then and you ca

Re: [Qemu-devel] [PULL 44/50] scripts: let checkpatch.pl process an entire GIT branch

2017-10-04 Thread Daniel P. Berrange
On Tue, Oct 03, 2017 at 04:07:06PM -0600, Alex Williamson wrote: > On Tue, 19 Sep 2017 14:29:33 +0200 > Paolo Bonzini wrote: > > > From: "Daniel P. Berrange" > > > > Currently before submitting a series, devs should run checkpatch.pl > > across each patch to be submitted. This can be automated

Re: [Qemu-devel] [PATCH v2 1/2] vl: Eliminate defconfig variable

2017-10-04 Thread Daniel P. Berrange
On Tue, Oct 03, 2017 at 06:17:29PM -0300, Eduardo Habkost wrote: > Both -nodefconfig and -no-user-config options do the same thing > today, we only need one variable to keep track of them. > > Suggested-by: Markus Armbruster > Acked-by: Alistair Francis > Reviewed-by: Markus Armbruster > Signed

Re: [Qemu-devel] [PATCH v2 2/2] qemu-options: Deprecate -nodefconfig

2017-10-04 Thread Daniel P. Berrange
On Tue, Oct 03, 2017 at 06:17:30PM -0300, Eduardo Habkost wrote: > Since 2012 (commit ba6212d8 "Eliminate cpus-x86_64.conf file") we > have no default config files that would be disabled using > -nodefconfig. Update documentation and document -nodefconfig as > deprecated. > > Cc: Markus Armbruste

Re: [Qemu-devel] [PATCH v3 2/2] qemu-options: Deprecate -nodefconfig

2017-10-04 Thread Daniel P. Berrange
On Wed, Oct 04, 2017 at 12:00:25AM -0300, Eduardo Habkost wrote: > Since 2012 (commit ba6212d8 "Eliminate cpus-x86_64.conf file") we > have no default config files that would be disabled using > -nodefconfig. Update documentation and document -nodefconfig as > deprecated. > > Cc: Markus Armbruste

Re: [Qemu-devel] [PATCH v3 1/2] vl: Eliminate defconfig variable

2017-10-04 Thread Daniel P. Berrange
On Wed, Oct 04, 2017 at 12:00:24AM -0300, Eduardo Habkost wrote: > Both -nodefconfig and -no-user-config options do the same thing > today, we only need one variable to keep track of them. > > Suggested-by: Markus Armbruster > Acked-by: Alistair Francis > Reviewed-by: Markus Armbruster > Signed

Re: [Qemu-devel] [PATCH v3] linux-user/syscall.c: Handle SH4's exceptional alignment for p{read, write}64

2017-10-04 Thread John Paul Adrian Glaubitz
Hi! Any chance that this patch gets merged soon? Looks like it has been reviewed by at least Richard and Philippe. Adrian On 09/15/2017 09:33 PM, James Clarke wrote: Fixes: https://bugs.launchpad.net/qemu/+bug/1716767 Signed-off-by: James Clarke --- Changes since v2: * Fixed opening curly

[Qemu-devel] [PATCH] spapr: fix RTAS method name in comment

2017-10-04 Thread Greg Kurz
Signed-off-by: Greg Kurz --- hw/ppc/spapr.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index 352ff3d614e8..b284e0b9d43e 100644 --- a/hw/ppc/spapr.c +++ b/hw/ppc/spapr.c @@ -2334,7 +2334,8 @@ static void ppc_spapr_init(MachineState *mac

Re: [Qemu-devel] [PULL 0/3] Usb 20170929 patches

2017-10-04 Thread Gerd Hoffmann
Hi, > This fails to build from clean (x86-64 Linux host): > > make[1]: *** No rule to make target '../hw/usb/host-stub.o', needed > by > 'qemu-system-microblaze'. Stop. > > (and similar for other targets). /me looks puzzled. There is a "hw/usb/host-stub.c" file in the repo, so I fail so see

[Qemu-devel] [PATCH] s390x: sort some devices into categories

2017-10-04 Thread Cornelia Huck
Add missing categorizations for some s390x devices: - zpci device -> misc - 3270 -> display - vfio-ccw -> misc Signed-off-by: Cornelia Huck --- hw/s390x/3270-ccw.c | 1 + hw/s390x/s390-pci-bus.c | 1 + hw/vfio/ccw.c | 1 + 3 files changed, 3 insertions(+) diff --git a/hw/s390x/327

Re: [Qemu-devel] [PATCH] s390x: sort some devices into categories

2017-10-04 Thread Christian Borntraeger
On 10/04/2017 11:00 AM, Cornelia Huck wrote: > Add missing categorizations for some s390x devices: > - zpci device -> misc > - 3270 -> display > - vfio-ccw -> misc > > Signed-off-by: Cornelia Huck Acked-by: Christian Borntraeger > --- > hw/s390x/3270-ccw.c | 1 + > hw/s390x/s390-pci-bus.c

[Qemu-devel] [PATCH] spapr: sanity check size of the CAS buffer

2017-10-04 Thread Greg Kurz
The CAS buffer is provided by SLOF. A broken SLOF could pass a silly size: either smaller than the diff header, in which case the current code will try to allocate 16 Exabytes of memory and g_malloc0() will abort, or bigger than the maximum memory provisioned for SLOF (ie, 40 Megabytes), which does

[Qemu-devel] xen-pci-passthrough PCI Express support? (Re: [Xen-devel] [PATCH v2 4/5] pci: Add INTERFACE_CONVENTIONAL_PCI_DEVICE to Conventional PCI devices)

2017-10-04 Thread Jan Beulich
>>> On 03.10.17 at 02:12, wrote: > On Thu, Sep 28, 2017 at 10:12:34AM -0300, Eduardo Habkost wrote: >> On Thu, Sep 28, 2017 at 02:33:57AM -0600, Jan Beulich wrote: >> > >>> On 27.09.17 at 21:56, wrote: >> > > --- a/hw/xen/xen_pt.c >> > > +++ b/hw/xen/xen_pt.c >> > > @@ -964,6 +964,10 @@ static co

Re: [Qemu-devel] [PATCH] s390x: sort some devices into categories

2017-10-04 Thread Thomas Huth
On 04.10.2017 11:00, Cornelia Huck wrote: > Add missing categorizations for some s390x devices: > - zpci device -> misc > - 3270 -> display > - vfio-ccw -> misc > > Signed-off-by: Cornelia Huck > --- > hw/s390x/3270-ccw.c | 1 + > hw/s390x/s390-pci-bus.c | 1 + > hw/vfio/ccw.c | 1

[Qemu-devel] [qemu-web PATCH] Add a blog post about the release of version 2.10.1

2017-10-04 Thread Thomas Huth
Content is based on Michael Roth's announce e-mail for this release. Signed-off-by: Thomas Huth --- _posts/2017-10-04-qemu-2-10-1.md | 21 + 1 file changed, 21 insertions(+) create mode 100644 _posts/2017-10-04-qemu-2-10-1.md diff --git a/_posts/2017-10-04-qemu-2-10-1.md b/

Re: [Qemu-devel] [PATCH] Revert: checkpatch: check trace-events code style

2017-10-04 Thread Vladimir Sementsov-Ogievskiy
04.10.2017 01:00, Alex Williamson wrote: Commit c3e5875afc0f ("checkpatch: check trace-events code style") introduces a regression as reported: https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg05820.html Bareword found where operator expected at ./scripts/checkpatch.pl line 1350, near

Re: [Qemu-devel] [PATCH v3 5/6] s390x/css: activate ChannelSubSys migration

2017-10-04 Thread Halil Pasic
On 10/04/2017 10:16 AM, Cornelia Huck wrote: > On Tue, 3 Oct 2017 13:58:29 +0200 > Thomas Huth wrote: > >> On 11.07.2017 16:54, Halil Pasic wrote: >>> Turn on migration for the channel subsystem for the next machine. For >>> legacy machines we still have to do things the old way. >>> >>> Signe

Re: [Qemu-devel] [PULL 0/3] Usb 20170929 patches

2017-10-04 Thread Thomas Huth
On 04.10.2017 10:43, Gerd Hoffmann wrote: > Hi, > >> This fails to build from clean (x86-64 Linux host): >> >> make[1]: *** No rule to make target '../hw/usb/host-stub.o', needed >> by >> 'qemu-system-microblaze'. Stop. >> >> (and similar for other targets). > > /me looks puzzled. > > There is

Re: [Qemu-devel] [qemu-web PATCH] Add a blog post about the release of version 2.10.1

2017-10-04 Thread Paolo Bonzini
On 04/10/2017 11:31, Thomas Huth wrote: > Content is based on Michael Roth's announce e-mail for this release. > > Signed-off-by: Thomas Huth > --- > _posts/2017-10-04-qemu-2-10-1.md | 21 + > 1 file changed, 21 insertions(+) > create mode 100644 _posts/2017-10-04-qemu-2-10-

Re: [Qemu-devel] [PATCH 2/3] hmp-commands-info: Move Texinfo stanzas to conventional place

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 3:45 PM, Markus Armbruster wrote: > A command's STEXI..ETEXI stanza follows the command's initializer. > Two commands got them backwards. Correct that. > > Signed-off-by: Markus Armbruster > --- > hmp-commands-info.hx | 20 ++-- > 1 file changed, 10 inser

Re: [Qemu-devel] [PULL 0/3] Usb 20170929 patches

2017-10-04 Thread Gerd Hoffmann
Hi, > > Ah, deja vu. I think you've now run into the problem that I had with > my > patch, too - see: > >  https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg01875.html >  https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg02082.html > > I think this is a race condition when buildi

Re: [Qemu-devel] [PATCH] s390x: sort some devices into categories

2017-10-04 Thread Cornelia Huck
On Wed, 4 Oct 2017 11:20:00 +0200 Thomas Huth wrote: > On 04.10.2017 11:00, Cornelia Huck wrote: > > Add missing categorizations for some s390x devices: > > - zpci device -> misc > > - 3270 -> display > > - vfio-ccw -> misc > > > > Signed-off-by: Cornelia Huck > > --- > > hw/s390x/3270-ccw.c

Re: [Qemu-devel] [PATCH] s390x: sort some devices into categories

2017-10-04 Thread Cornelia Huck
On Wed, 4 Oct 2017 11:00:32 +0200 Cornelia Huck wrote: > Add missing categorizations for some s390x devices: > - zpci device -> misc > - 3270 -> display > - vfio-ccw -> misc > > Signed-off-by: Cornelia Huck > --- > hw/s390x/3270-ccw.c | 1 + > hw/s390x/s390-pci-bus.c | 1 + > hw/vfio/ccw.

Re: [Qemu-devel] [PATCH] spapr: sanity check size of the CAS buffer

2017-10-04 Thread David Gibson
On Wed, Oct 04, 2017 at 11:02:31AM +0200, Greg Kurz wrote: > The CAS buffer is provided by SLOF. A broken SLOF could pass a silly > size: either smaller than the diff header, in which case the current > code will try to allocate 16 Exabytes of memory and g_malloc0() will > abort, or bigger than the

Re: [Qemu-devel] [PATCH] spapr: fix RTAS method name in comment

2017-10-04 Thread David Gibson
On Wed, Oct 04, 2017 at 10:43:18AM +0200, Greg Kurz wrote: > Signed-off-by: Greg Kurz Applied, except I corrected the short description. ibm,client-architecture-support is an OF word, not actually an RTAS method. > --- > hw/ppc/spapr.c |3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-

Re: [Qemu-devel] [PULL 0/3] Usb 20170929 patches

2017-10-04 Thread Paolo Bonzini
On 04/10/2017 11:37, Thomas Huth wrote: > Ah, deja vu. I think you've now run into the problem that I had with my > patch, too - see: > > https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg01875.html > https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg02082.html > > I think this is

Re: [Qemu-devel] [PATCH] Revert: checkpatch: check trace-events code style

2017-10-04 Thread Vladimir Sementsov-Ogievskiy
04.10.2017 01:00, Alex Williamson wrote: Commit c3e5875afc0f ("checkpatch: check trace-events code style") introduces a regression as reported: https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg05820.html Bareword found where operator expected at ./scripts/checkpatch.pl line 1350, near

Re: [Qemu-devel] [PULL 0/3] Usb 20170929 patches

2017-10-04 Thread Paolo Bonzini
On 04/10/2017 11:50, Gerd Hoffmann wrote: > Hi, >> >> Ah, deja vu. I think you've now run into the problem that I had with >> my >> patch, too - see: >> >>  https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg01875.html >>  https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg02082.html

[Qemu-devel] [Bug 1721187] Re: install Centos7 or fedora27 on qemu on windows8.1

2017-10-04 Thread Thomas Huth
Which version of QEMU are you using? Did you compile QEMU on your own or are you using a pre-build binary? Anyway, to be able to use USB devices, you've got to specify the "-usb" parameter when starting QEMU. -- You received this bug notification because you are a member of qemu- devel-ml, whic

[Qemu-devel] [PATCH v2 0/3] add DEFINE_TYPES() helper to simplify batch type registration

2017-10-04 Thread Igor Mammedov
following thread '[PATCH 01/38] qom: add helper type_init_from_array()' here is reworked variant of above patch as Eduardo have suggested (+ split in several atomic parts) PS: posting for early review, I'll wait for a couple of days before reposting whole series that will use DEFINE_TYPES() to sim

[Qemu-devel] [PATCH v2 1/3] qom: update doc comment for type_register[_static]()

2017-10-04 Thread Igor Mammedov
type_register()/type_register_static() functions in current impl. can't fail returning 0, also none of the users check for error so update doc comment to reflect current behaviour. Suggested-by: Eduardo Habkost Signed-off-by: Igor Mammedov --- include/qom/object.h | 4 ++-- 1 file changed, 2 in

[Qemu-devel] [PATCH v2 2/3] qom: introduce type_register_static_array()

2017-10-04 Thread Igor Mammedov
it will help to remove code duplication of registration static types in places that have open coded loop to perform batch type registering. Signed-off-by: Igor Mammedov --- include/qom/object.h | 10 ++ qom/object.c | 9 + 2 files changed, 19 insertions(+) diff --git a/

[Qemu-devel] [PATCH v2 3/3] qom: add helper macro DEFINE_TYPES()

2017-10-04 Thread Igor Mammedov
DEFINE_TYPES() will help to simplify following routine patterns: static void foo_register_types(void) { type_register_static(&foo1_type_info); type_register_static(&foo2_type_info); ... } type_init(foo_register_types) or static void foo_register_types(void) { int i; f

Re: [Qemu-devel] [PATCH 8/8] qemu-options: Belatedly document --watchdog-action inject-nmi

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:03 PM, Markus Armbruster wrote: > Missed in commit 795dc6e46d, v2.4.0. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > qemu-options.hx | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/qemu-options.hx b/qemu-option

Re: [Qemu-devel] [PATCH 7/8] qemu-options: Polish section "Character device options"

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:03 PM, Markus Armbruster wrote: > Cc: Paolo Bonzini > CC: Marc-André Lureau > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > qemu-options.hx | 15 --- > 1 file changed, 8 insertions(+), 7 deletions(-) > > diff --git a/qemu-optio

Re: [Qemu-devel] [PATCH 01/11] qapi-schema: Fix query-vm-generation-id's doc comment markup

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > Reported-by: Daniel P. Berrange > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau (reviweing this patch makes me realize that the last expression/command in qapi-schema isn't in the generated doc..) > --- > qapi-sche

Re: [Qemu-devel] [PATCH 1/8] qemu-options: Remove stray colons from output of --help

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:03 PM, Markus Armbruster wrote: > Commit 43f187a broke --help: it put colons into blank lines. It > removed the colon from DEFHEADING(TITLE:) and added it back in the > macro expansion of DEFHEADING(TITLE), so hxtool can emit "@subsection > TITLE" more easily. Trouble is

Re: [Qemu-devel] [PATCH 6/8] qemu-options: Polish section "TPM device options"

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:03 PM, Markus Armbruster wrote: > Cc: Stefan Berger > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > qemu-options.hx | 18 +- > 1 file changed, 9 insertions(+), 9 deletions(-) > > diff --git a/qemu-options.hx b/qemu-options.

Re: [Qemu-devel] [PATCH 3/3] hmp-commands-info: Change "@findex FOO" to "@findex info FOO"

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 3:45 PM, Markus Armbruster wrote: > qemu-doc has the monitor commands in the "Function Index". The "info > FOO" are listed as "FOO" there. List them as "info FOO" instead. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > hmp-commands-info.h

Re: [Qemu-devel] [PATCH 5/8] qemu-options: Add missing -iscsi Texinfo documentation

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:03 PM, Markus Armbruster wrote: > Cc: Ronnie Sahlberg > Cc: Kevin Wolf > Cc: Max Reitz > Cc: qemu-bl...@nongnu.org > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau (that's very short doc) > --- > qemu-options.hx | 6 ++ > 1 file changed, 6 ins

Re: [Qemu-devel] [PATCH 2/8] qemu-options: Fix markup of -netdev l2tpv3

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:03 PM, Markus Armbruster wrote: > The table of option parameters lacks @table and @end table. The > parameters become items in the enclosing table of options. Screwed up > when l2tpv3 was added in commit 3fb69aa. Fix the obvious way. > > Cc: Jason Wang > Signed-off-by:

Re: [Qemu-devel] [PATCH 4/8] qemu-options: Move -iscsi under "Block device options"

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:03 PM, Markus Armbruster wrote: > -iscsi ended up under the "Device URL Syntax" heading by a sequence of > errors, as explained in the previous commit. Move it under the "Block > device options" heading. Nothing left under "Device URL Syntax"; > drop the heading. > > Cc:

Re: [Qemu-devel] [PATCH 02/11] qapi: Stop rejecting #optional

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > Commit 1d8bda1 got rid of #optional tags, and added a check to keep > them from getting added back, to make sure patches then in flight > don't add them back. It's been six months, time to drop that check. > > Signed-off-by: Markus Armbru

Re: [Qemu-devel] [PATCH 03/11] qapi: Eliminate QAPISchemaParser.__init__()'s local fname

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > scripts/qapi.py | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/scripts/qapi.py b/scripts/qapi.py > index dc92bca620..6c8001e96d 10

Re: [Qemu-devel] [PATCH 3/8] qemu-options qemu-doc: Move "Device URL Syntax" to qemu-doc

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:03 PM, Markus Armbruster wrote: > Commit 0f5314a (v1.0) added section "Device URL Syntax" to > qemu-options.hx. It's enclosed in STEXI..ETEXI, thus affects only > qemu-options.texi, not --help. It appears as a subsection under > section "Invocation". Similarly, qemu.1 h

[Qemu-devel] [Bug 1721220] [NEW] qemu crashes with assertion error `!mr->container' failed

2017-10-04 Thread R.Nageswara Sastry
Public bug reported: Re-production steps: git clone today's qemu git tree (4th Oct 2017) ./configure --target-list=ppc64-softmmu && make -j 8 Run the device-crash-test from scripts folder, seeing the following error INFO: running test case: machine=bamboo binary=ppc64-softmmu/qemu-system-ppc64

Re: [Qemu-devel] [PATCH 2/3] hmp-commands-info: Move Texinfo stanzas to conventional place

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 3:45 PM, Markus Armbruster wrote: > A command's STEXI..ETEXI stanza follows the command's initializer. > Two commands got them backwards. Correct that. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > hmp-commands-info.hx | 20 ++

Re: [Qemu-devel] [PATCH 1/3] hmp-commands-info: Fix "info rocker-FOO" misspellings

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 3:45 PM, Markus Armbruster wrote: > Screwed up in commit da76ee7. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > hmp-commands-info.hx | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/hmp-commands-info.hx b/hmp-

Re: [Qemu-devel] [PATCH 06/11] qapi2texi: Clean up texi_sections()

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > Repurposing the function parameter doc for stepping through > doc.sections.__str__() is not nice. Use new variable @text instead. > > While there, eliminate variables name and func. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc

Re: [Qemu-devel] [PATCH 04/11] qapi: Make cur_doc local to QAPISchemaParser.__init__()

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > QAPISchemaParser.cur_doc is used only by .__init__() and its helper > .reject_expr_doc(). Make it local to __init__() and pass it to > .reject_expr_doc() explicitly. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau

Re: [Qemu-devel] [PATCH 07/11] qapi: Unify representation of doc section without name

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > We have two representations of sections without a name: the main > section uses name=None, the others name=''. Standardize on name=None. > > Signed-off-by: Markus Armbruster > --- > scripts/qapi.py | 2 +- > scrip

Re: [Qemu-devel] [PATCH 05/11] tests/qapi-schema/doc-bad-section: New, factored out of doc-good

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > A negative test case crept into doc-good.json: invalid use of section > markup we currently fail to reject. Move this into its own > doc-bad-section.json. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > t

Re: [Qemu-devel] [PATCH 07/11] qapi: Unify representation of doc section without name

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > We have two representations of sections without a name: the main > section uses name=None, the others name=''. Standardize on name=None. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > scripts/qapi.py

Re: [Qemu-devel] [Qemu-block] [PATCH v3 28/44] block: Fix pending requests check in bdrv_append()

2017-10-04 Thread Alberto Garcia
On Tue 28 Feb 2017 01:54:13 PM CET, Kevin Wolf wrote: > bdrv_append() cares about isolation of the node that it modifies, but > not about activity in some subtree below it. Instead of using the > recursive bdrv_requests_pending(), directly check bs->in_flight, which > considers only the node in que

Re: [Qemu-devel] [PATCH 09/11] qapi2texi: Simplify representation of section text

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > Use a string instead of a list of strings. While there, generate > fewer superfluous blank lines. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > scripts/qapi2texi.py| 33 -

Re: [Qemu-devel] [PATCH 08/11] qapi: Simplify representation of QAPIDoc section text

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > Use a string instead of a list of strings. > > This makes qapi2texi.py generate additional blank lines. They're > harmless, and the next commit will get rid of them again. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lur

[Qemu-devel] [Bug 1721222] [NEW] qemu crashes with Assertion `fdctrl->dma' failed

2017-10-04 Thread R.Nageswara Sastry
Public bug reported: Re-production steps: git clone today's qemu git tree (4th Oct 2017) ./configure --target-list=ppc64-softmmu && make -j 8 Run the device-crash-test from scripts folder, seeing the following error INFO: running test case: machine=powernv binary=ppc64-softmmu/qemu-system-ppc64

[Qemu-devel] [Bug 1721224] [NEW] qemu crashes with Assertion `!bus->dma[0] && !bus->dma[1]' failed

2017-10-04 Thread R.Nageswara Sastry
Public bug reported: Re-production steps: git clone today's qemu git tree (4th Oct 2017) ./configure --target-list=ppc64-softmmu && make -j 8 Run the device-crash-test from scripts folder, seeing the following error INFO: running test case: machine=prep binary=ppc64-softmmu/qemu-system-ppc64 de

Re: [Qemu-devel] [PATCH 10/11] qapi: Rename QAPIDoc.parser, .section to ._parser, ._section

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > scripts/qapi.py | 52 ++-- > 1 file changed, 26 insertions(+), 26 deletions(-) > > diff --git a/scripts/qapi.py b

[Qemu-devel] [Bug 1721220] Re: qemu crashes with signal 6 with assertion error `!mr->container' failed

2017-10-04 Thread R.Nageswara Sastry
** Summary changed: - qemu crashes with signal 6 when tried with -machine bamboo,accel=kvm -device pcie-pci-bridge + qemu crashes with signal 6 with assertion error `!mr->container' failed ** Summary changed: - qemu crashes with signal 6 with assertion error `!mr->container' failed + qemu crash

Re: [Qemu-devel] [PATCH 5/8] qemu-options: Add missing -iscsi Texinfo documentation

2017-10-04 Thread ronnie sahlberg
Reviewed-by: Ronnie Sahlberg On Tue, Oct 3, 2017 at 12:03 AM, Markus Armbruster wrote: > Cc: Ronnie Sahlberg > Cc: Kevin Wolf > Cc: Max Reitz > Cc: qemu-bl...@nongnu.org > Signed-off-by: Markus Armbruster > --- > qemu-options.hx | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a

Re: [Qemu-devel] [PATCH 11/11] qapi2texi: De-duplicate code to add blank line before symbol

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster wrote: > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > scripts/qapi2texi.py | 12 ++-- > 1 file changed, 2 insertions(+), 10 deletions(-) > > diff --git a/scripts/qapi2texi.py b/scripts/qapi2texi.py > index 5

Re: [Qemu-devel] [PATCH 4/8] qemu-options: Move -iscsi under "Block device options"

2017-10-04 Thread ronnie sahlberg
Reviewed-by: Ronnie Sahlberg Would be nice if this died at some stage : [,password=password] But that is for a different patch. On Wed, Oct 4, 2017 at 8:12 PM, Marc-André Lureau wrote: > On Mon, Oct 2, 2017 at 4:03 PM, Markus Armbruster wrote: >> -iscsi ended up under the "Device URL Syntax"

Re: [Qemu-devel] [RFC PATCH 01/32] tests/qapi-schema: Improve coverage of '@'

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 5:25 PM, Markus Armbruster wrote: > Demonstrates a qapi2texi.py bug: it can fail to escape '@'. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > tests/qapi-schema/doc-good.json | 3 +++ > tests/qapi-schema/doc-good.out | 3 +++ > tests/qapi-

Re: [Qemu-devel] [PATCH 3/8] qemu-options qemu-doc: Move "Device URL Syntax" to qemu-doc

2017-10-04 Thread ronnie sahlberg
On Wed, Oct 4, 2017 at 8:12 PM, Marc-André Lureau wrote: > On Mon, Oct 2, 2017 at 4:03 PM, Markus Armbruster wrote: >> Commit 0f5314a (v1.0) added section "Device URL Syntax" to >> qemu-options.hx. It's enclosed in STEXI..ETEXI, thus affects only >> qemu-options.texi, not --help. It appears as

[Qemu-devel] [PATCH 3/6] tests: Add precopy test using deprecated commands

2017-10-04 Thread Juan Quintela
Signed-off-by: Juan Quintela --- tests/migration-test.c | 71 ++ 1 file changed, 71 insertions(+) diff --git a/tests/migration-test.c b/tests/migration-test.c index 8ef2b72459..8549e4c2c6 100644 --- a/tests/migration-test.c +++ b/tests/migration-te

[Qemu-devel] [PATCH 2/6] tests: Add basic migration precopy tcp test

2017-10-04 Thread Juan Quintela
Signed-off-by: Juan Quintela --- tests/migration-test.c | 8 1 file changed, 8 insertions(+) diff --git a/tests/migration-test.c b/tests/migration-test.c index cd954caee4..8ef2b72459 100644 --- a/tests/migration-test.c +++ b/tests/migration-test.c @@ -466,6 +466,13 @@ static void test_p

[Qemu-devel] [PATCH 4/6] tests: Add migration xbzrle test

2017-10-04 Thread Juan Quintela
Signed-off-by: Juan Quintela --- tests/migration-test.c | 80 ++ 1 file changed, 80 insertions(+) diff --git a/tests/migration-test.c b/tests/migration-test.c index 8549e4c2c6..f0734e4ba0 100644 --- a/tests/migration-test.c +++ b/tests/migration-te

[Qemu-devel] [PATCH 0/6] Add make check tests for Migration

2017-10-04 Thread Juan Quintela
His this series: - add test for precopy for unix/tcp exec and fd to came, don't know how to test rdma without hardware - add tests using deprecated interfaces - add test for xbzrle Note to myself, there is no way to set the cache size with migraton_set_parameters - Add test for compress thread

[Qemu-devel] [PATCH 5/6] tests: Add migration compress threads tests

2017-10-04 Thread Juan Quintela
For some reason, compression is not working at the moment, test is disabled until I found why. Signed-off-by: Juan Quintela --- tests/migration-test.c | 51 ++ 1 file changed, 51 insertions(+) diff --git a/tests/migration-test.c b/tests/migration-

[Qemu-devel] [PATCH 6/6] tests: Move postcopy migration test to migrate-set-parameters

2017-10-04 Thread Juan Quintela
migrate_set_speed and migrate_set_dowtime are deprecated since long ago. Signed-off-by: Juan Quintela --- tests/postcopy-test.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/postcopy-test.c b/tests/postcopy-test.c index 8142f2ab90..991b954a83 100644 --- a/test

[Qemu-devel] [PATCH 1/6] tests: Add basic migration precopy test

2017-10-04 Thread Juan Quintela
Signed-off-by: Juan Quintela --- tests/Makefile.include | 3 + tests/migration-test.c | 497 + 2 files changed, 500 insertions(+) create mode 100644 tests/migration-test.c diff --git a/tests/Makefile.include b/tests/Makefile.include index abc670

Re: [Qemu-devel] [RFC PATCH 03/32] qapi2texi: Fix for examples containing '@'

2017-10-04 Thread Marc-André Lureau
Hi On Mon, Oct 2, 2017 at 5:25 PM, Markus Armbruster wrote: > Instead of quoting example text so that the @example environment shows > it verbatim, simply use the @verbatim environment. > Not sure if this is really worth it, unless we have a real use-case of @ in examples. > Examples are no lon

[Qemu-devel] [PATCH v9 04/12] migration: Start of multiple fd work

2017-10-04 Thread Juan Quintela
We create new channels for each new thread created. We send through them a string containing multifd so we are sure that we connect the right channels in both sides. Signed-off-by: Juan Quintela -- Split SocketArgs into incoming and outgoing args Use UUID's on the initial message, so we are s

[Qemu-devel] [PATCH v9 01/12] qapi: Fix grammar in x-multifd-page-count descriptions

2017-10-04 Thread Juan Quintela
Reported-by: Eric Blake Signed-off-by: Juan Quintela --- qapi/migration.json | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/qapi/migration.json b/qapi/migration.json index f8b365e3f5..d94d6c0e46 100644 --- a/qapi/migration.json +++ b/qapi/migration.json @@ -471,7 +471,

[Qemu-devel] [PATCH v9 03/12] migration: Make migrate_fd_error() the owner of the Error

2017-10-04 Thread Juan Quintela
So far, we had to free the error after each caller, so just do it here. Once there, tls.c was leaking the error. Signed-off-by: Juan Quintela --- migration/channel.c | 1 - migration/migration.c | 10 -- migration/migration.h | 4 ++-- migration/socket.c| 1 - 4 files changed,

[Qemu-devel] [PATCH v9 00/12] Multifd

2017-10-04 Thread Juan Quintela
Hi This series is on top of my migration test series just sent, only reject should be on the test system, though. On v9 series for you: - qobject_unref() as requested by dan Yes he was right, I had a reference leak for _non_ multifd, I *thought* he mean for multifd, and that took a while to

[Qemu-devel] [PATCH v9 05/12] migration: Create ram_multifd_page

2017-10-04 Thread Juan Quintela
The function still don't use multifd, but we have simplified ram_save_page, xbzrle and RDMA stuff is gone. We have added a new counter and a new flag for this type of pages. Signed-off-by: Juan Quintela -- Add last_page parameter Add commets for done and address Remove multifd field, it is the

[Qemu-devel] [PATCH v9 06/12] migration: Send the fd number which we are going to use for this page

2017-10-04 Thread Juan Quintela
We are still sending the page through the main channel, that would change later in the series Signed-off-by: Juan Quintela Reviewed-by: Dr. David Alan Gilbert --- migration/ram.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/migration/ram.c b/migration/ram.c

[Qemu-devel] [PATCH v9 08/12] migration: Test new fd infrastructure

2017-10-04 Thread Juan Quintela
We just send the address through the alternate channels and test that it is ok. Signed-off-by: Juan Quintela -- Use qio_channel_*all functions --- migration/ram.c | 39 +++ 1 file changed, 39 insertions(+) diff --git a/migration/ram.c b/migration/ram.c inde

[Qemu-devel] [PATCH v9 02/12] migration: Improve migration thread error handling

2017-10-04 Thread Juan Quintela
We now report errors also when we finish migration, not only on info migrate. We plan to use this error from several places, and we want the first error to happen to win, so we add an mutex to order it. Signed-off-by: Juan Quintela Reviewed-by: Dr. David Alan Gilbert --- migration/migration.c

[Qemu-devel] [PATCH v9 12/12] migration: Add multifd test

2017-10-04 Thread Juan Quintela
We set the x-multifd-page-count and x-multifd-channels. Signed-off-by: Juan Quintela --- tests/Makefile.include | 3 + tests/multifd-test.c | 511 + 2 files changed, 514 insertions(+) create mode 100644 tests/multifd-test.c diff --git a/tests

[Qemu-devel] [PATCH v9 09/12] migration: Rename initial_bytes

2017-10-04 Thread Juan Quintela
Now it is called qemu_file_bytes that reflects better what it does, and we create qemu_file_bytes_now to not have to call qemu_ftell() twice. Signed-off-by: Juan Quintela Reviewed-by: Dr. David Alan Gilbert --- migration/migration.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-)

[Qemu-devel] [PATCH v9 07/12] migration: Create thread infrastructure for multifd recv side

2017-10-04 Thread Juan Quintela
We make the locking and the transfer of information specific, even if we are still receiving things through the main thread. Signed-off-by: Juan Quintela -- We split when we create the main channel and where we start the main migration thread, so we wait for the creation of the other threads.

Re: [Qemu-devel] [RFC PATCH 04/32] qapi2texi: Fix for '@' not followed by \w character

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 5:25 PM, Markus Armbruster wrote: > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > scripts/qapi2texi.py| 22 -- > tests/qapi-schema/doc-good.json | 2 +- > tests/qapi-schema/doc-good.out | 2 +- > tests/qapi-s

Re: [Qemu-devel] [RFC PATCH 08/32] qapi: Simplify check_name() parameters

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 5:25 PM, Markus Armbruster wrote: > Replace flags @enum_member, @allow_optional by string @meta: > 'member-enum' implies @enum_member, 'member-struct' implies > @allow_optional. > > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > scripts/qapi.py

[Qemu-devel] [PATCH v9 11/12] migration: Flush receive queue

2017-10-04 Thread Juan Quintela
Each time that we sync the bitmap, it is a possiblity that we receive a page that is being processed by a different thread. We fix this problem just making sure that we wait for all receiving threads to finish its work before we procedeed with the next stage. We are low on page flags, so we use a

Re: [Qemu-devel] [RFC PATCH 09/32] qapi: check_type() parameter allow_optional is now unused, drop

2017-10-04 Thread Marc-André Lureau
On Mon, Oct 2, 2017 at 5:25 PM, Markus Armbruster wrote: > Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau > --- > scripts/qapi.py | 13 + > 1 file changed, 5 insertions(+), 8 deletions(-) > > diff --git a/scripts/qapi.py b/scripts/qapi.py > index 208010c241..8f9c

[Qemu-devel] [PATCH v2 2/2] s390/kvm: make TOD setting failures fatal for migration

2017-10-04 Thread Christian Borntraeger
From: "Collin L. Walling" If we fail to set a proper TOD clock on the target system, this can already result in some problematic cases. We print several warn messages on source and target in that case. If kvm fails to set a nonzero epoch index, then we must ultimately fail the migration as this

[Qemu-devel] [PATCH v9 10/12] migration: Transfer pages over new channels

2017-10-04 Thread Juan Quintela
We switch for sending the page number to send real pages. Signed-off-by: Juan Quintela -- Remove the HACK bit, now we have the function that calculates the size of a page exported. Rename multifd_pages{_now}, to sent pages Remove multifd pages field, it is the same than normal pages --- migrat

[Qemu-devel] [PATCH v2 0/2] s390/z14: extended TOD-clock support

2017-10-04 Thread Christian Borntraeger
The last last remaining z14 item on my queue. v1->v2: - split patches - fix white spaces (also fixup kvm_s390_set_clock) - single phrase error message - use strerror Collin L. Walling (2): s390/kvm: Support for get/set of extended TOD-Clock for guest s390/kvm: make TOD setting failures fatal

  1   2   3   4   >