[Qemu-devel] [PATCH] usb: fix libusb config variable name.

Cc: Jan Kiszka Fixes: 4e5ee5b21c84fe3023a64b5cc2e12a52ab0597c1 Signed-off-by: Gerd Hoffmann --- hw/usb/Makefile.objs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/usb/Makefile.objs b/hw/usb/Makefile.objs index 9255234c63..0e6d54b21f 100644 --- a/hw/usb/Makefile.objs +++

Re: [Qemu-devel] [PATCH] usb: Fix usb-host build

On Mon, 2017-09-25 at 20:29 +0100, Peter Maydell wrote: > Gerd, ping? If you provide a reviewed-by or acked-by > I can apply this to master as a buildfix... > > thanks > -- PMM > > On 22 September 2017 at 08:42, Jan Kiszka > wrote: > > From: Jan Kiszka > > > > This reverts the incorrect commit

Re: [Qemu-devel] [PATCH] qga-win: don't hang if vss hold writes timeout

At 2017-09-22 10:46:06, "Chen Hanxiao" wrote: >From: Chen Hanxiao > >When VM is in a heavy IO, if the command "guest-fsfreeze-freeze" >is executed, VSS may timeout when trying to hold writes. > >Inside guest, Event ID 12298(VSS_ERROR_HOLD_WRITES_TIMEOUT) >is logged in the Event Viewer. > >At tha

[Qemu-devel] [PULL 00/29] Trivial patches for 2017-09-26

This is a resend of yesterday's pull request, with two patches removed: filter-mirror: segfault when specifying non existent device (causes testsuite to fail) dma/i82374: avoid double creation of i82374 device (better version is in the work) and with one patch with additional R-b line:

[Qemu-devel] [PATCH] qemu-doc.texi: remove trailing whitespace

Reported-By: Vagrant Cascadian Signed-off-by: Michael Tokarev --- qemu-doc.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qemu-doc.texi b/qemu-doc.texi index 4076226f39..de23fca9dc 100644 --- a/qemu-doc.texi +++ b/qemu-doc.texi @@ -105,7 +105,7 @@ QEMU full system emula

Re: [Qemu-devel] [PULL 00/31] Trivial patches for 2017-09-25

26.09.2017 08:57, Michael Tokarev wrote: .. > Reverting this patch makes it run again. Cc'ing Eduardo. > > I'll remove this patch, patch "hw/isa/pc87312: Mark the device with > user_creatable = false" (will be replaced with a better variant), Actually "dma/i82374: avoid double creation of i82374

Re: [Qemu-devel] [PULL 00/31] Trivial patches for 2017-09-25

26.09.2017 02:22, Peter Maydell wrote: [] >> trivial patches for 2017-09-25 > > This fails 'make check' on most of my configs: > > GTESTER check-qtest-ppc64 > qemu-system-ppc64: -object > filter-mirror,id=qtest-f0,netdev=qtest-bn0,queue=tx,outdev=mirror0: > Device 'qtest-bn0' not found > Broken

Re: [Qemu-devel] [PATCH] linux-user: remove duplicate break in syscall

Le 26/09/2017 à 06:14, Carlo Marcelo Arenas Belón a écrit : > likely introduced in 3532fa7402cda16f7b95261b0339c58630051f0b > > Signed-off-by: Carlo Marcelo Arenas Belón > --- > linux-user/syscall.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/linux-user/syscall.c b/linux-user/syscal

[Qemu-devel] A glib warning encountered with internal iothread

Hi, Generally speaking this is a question about glib, but I would like to see how the list sees this before throwing this question to glib community in case I made severe mistake. I encountered one glib warning when start to use internal iothread: (qemu-system-x86_64:19925): GLib-CRITICAL **: g_

Re: [Qemu-devel] [PATCH v2] qdev: Mark devices as non-hotpluggable by default

On Fri, Sep 22, 2017 at 11:16:34AM +0200, Thomas Huth wrote: > Historically we've marked all devices as hotpluggable by default. However, > most devices are not hotpluggable, and you also need a HotplugHandler to > support these devices. So if the user tries to "device_add" or "device_del" > such a

Re: [Qemu-devel] [RFC for-2.10 1/3] pci/pcie: Make a consistent helper for switching PCI/PCIe "hybrid" devices

On Wed, Aug 30, 2017 at 09:23:59AM -0300, Eduardo Habkost wrote: > On Wed, Aug 30, 2017 at 03:54:32PM +1000, David Gibson wrote: > > On Tue, Aug 29, 2017 at 11:12:36AM -0300, Eduardo Habkost wrote: > > > On Tue, Aug 29, 2017 at 09:42:39PM +1000, David Gibson wrote: > > > > On Wed, Apr 26, 2017 at 0

Re: [Qemu-devel] [PATCH v3 1/4] qom: provide root container for internal objs

On Tue, Sep 26, 2017 at 12:58:24PM +0800, Fam Zheng wrote: > On Tue, 09/26 12:52, Peter Xu wrote: > > We have object_get_objects_root() to keep user created objects, however > > no place for objects that will be used internally. Create such a > > container for internal objects. > > > > CC: Andrea

Re: [Qemu-devel] [PATCH v3 2/4] iothread: provide helpers for internal use

On Tue, 09/26 12:52, Peter Xu wrote: > IOThread is a general framework that contains IO loop environment and a > real thread behind. It's also good to be used internally inside qemu. > Provide some helpers for it to create iothreads to be used internally. > > Put all the internal used iothreads i

Re: [Qemu-devel] [PATCH v3 1/4] qom: provide root container for internal objs

On Tue, 09/26 12:52, Peter Xu wrote: > We have object_get_objects_root() to keep user created objects, however > no place for objects that will be used internally. Create such a > container for internal objects. > > CC: Andreas Färber > CC: Markus Armbruster > CC: Paolo Bonzini > Suggested-by:

[Qemu-devel] [PATCH v3 4/4] iothread: delay the context release to finalize

When gcontext is used with iothread, the context will be destroyed during iothread_stop(). That's not good since sometimes we would like to keep the resources until iothread is destroyed, but we may want to stop the thread before that point. Delay the destruction of gcontext to iothread finalize.

[Qemu-devel] [PATCH v3 3/4] iothread: export iothread_stop()

So that internal iothread users can explicitly stop one iothread without destroying it. Since at it, fix iothread_stop() to allow it to be called multiple times. Before this patch we may call iothread_stop() more than once on single iothread, while that may not be correct since qemu_thread_join()

[Qemu-devel] [PATCH v3 2/4] iothread: provide helpers for internal use

IOThread is a general framework that contains IO loop environment and a real thread behind. It's also good to be used internally inside qemu. Provide some helpers for it to create iothreads to be used internally. Put all the internal used iothreads into the internal object container. Signed-off-

[Qemu-devel] [PATCH v3 1/4] qom: provide root container for internal objs

We have object_get_objects_root() to keep user created objects, however no place for objects that will be used internally. Create such a container for internal objects. CC: Andreas Färber CC: Markus Armbruster CC: Paolo Bonzini Suggested-by: Daniel P. Berrange Signed-off-by: Peter Xu --- in

[Qemu-devel] [PATCH v3 0/4] iothread: allow to create internal iothreads

v3: - pick up r-bs (one missing for Fam's on last patch) - fix patch 1 to create isolated internal container v2: - add one patch to provide object_get_internal_root() [Daniel] - patch 2: use the new object_get_internal_root() - patch 3: fix commit message, "reentrant" is wrongly used by me. it s

[Qemu-devel] [Bug 1354727] Re: build error with GCC 4.9

[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1354727 Title: build error

Re: [Qemu-devel] [PATCH] linux-user: remove duplicate break in syscall

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. Type: series Message-id: 20170926041420.32673-1-care...@gmail.com Subject: [Qemu-devel] [PATCH] linux-user: remove duplicate b

Re: [Qemu-devel] [RFC 2/6] plugin: add initial plugin support

On 06.09.2017 22:28, Emilio G. Cota wrote: [...] > diff --git a/plugin.c b/plugin.c > new file mode 100644 > index 000..3cd19df > --- /dev/null > +++ b/plugin.c > @@ -0,0 +1,519 @@ > +/* plugin.c - QEMU Plugin interface > + * > + * Copyright (C) 2017, Emilio G. Cota > + * > + * License: GNU GP

Re: [Qemu-devel] [PATCH 2/2] thunk.h uses TARGET_ABI_BITS without including abitypes.h

> On Sep 25, 2017, at 9:10 AM, Eric Blake wrote: > > On 09/24/2017 08:02 PM, Daniel Loffgren wrote: >> >> Signed-off-by: Daniel Loffgren > > >> --- >> include/exec/user/thunk.h | 1 + >> 1 file changed, 1 insertion(+) > > meta-comment: your patch is titled 2/2, but w

Re: [Qemu-devel] [PATCH 2/2] thunk.h uses TARGET_ABI_BITS without including abitypes.h

I am attempting to get ppc-darwin-user in a good working state again, and this broke one of the .c files that included it. I figured this change should be made regardless of my branch, and would help reduce the size of my branch for future rebasing. > On Sep 25, 2017, at 9:37 AM, Thomas Huth w

Re: [Qemu-devel] [RFC 0/6] initial plugin support

On 06.09.2017 22:28, Emilio G. Cota wrote: > Related threads: > [PATCH 00/13] instrument: Add basic event instrumentation > Date: Mon, 24 Jul 2017 20:02:24 +0300 > https://lists.gnu.org/archive/html/qemu-devel/2017-07/msg07419.html > and > [PATCH v4 00/20] instrument: Add basic event instru

Re: [Qemu-devel] [PATCH 1/2] fprintf-fn.h uses GCC_FMT, but doesn't include the definition of it

I am working on getting ppc-darwin-user working again, and this was one of the many things preventing it from compiling. Your explanation sounds correct. I believe it was the lack of osdep.h in the right .c files, since I added osdep.h to the tops of all of the necessary files well after this ch

[Qemu-devel] [PATCH] linux-user: remove duplicate break in syscall

likely introduced in 3532fa7402cda16f7b95261b0339c58630051f0b Signed-off-by: Carlo Marcelo Arenas Belón --- linux-user/syscall.c | 1 - 1 file changed, 1 deletion(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c index 9b6364a266..3ef4d1c568 100644 --- a/linux-user/syscall.c +++ b/lin

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

On 26.09.2017 02:09, 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. [...] > @@ -9281,18 +9281,19 @@ static void init_ppc_proc(PowerPCCPU *cpu)

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

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

Re: [Qemu-devel] [PATCH 6/7] mac_dbdma: change DBDMA_register_channel to a MAC_DBDMA type method

On Sun, Sep 24, 2017 at 03:47:45PM +0100, Mark Cave-Ayland wrote: > Using this we can change the MACIO_IDE instance to register the channel > itself via a type method instead of requiring a separate > DBDMA_register_channel() function. > > As a consequence of this it is now possible to remove the

Re: [Qemu-devel] [PATCH v1 3/8] hw: Replace fprintf(stderr, "*\n" with error_report()

On 26.09.2017 02:08, 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. > > find ./* -type f -exec sed -i \ > 'N;N;N;N;N;N;N;N;N;N;N;N; {s|fpr

Re: [Qemu-devel] [PATCH v2] qdev: Mark devices as non-hotpluggable by default

On 26.09.2017 04:59, Eduardo Habkost wrote: > On Mon, Sep 25, 2017 at 07:05:26PM +0100, Peter Maydell wrote: >> On 25 September 2017 at 18:59, Eduardo Habkost wrote: >>> Finding the full list of devices that can be instantiated >>> internally at hotplug-time sounds tricky. >> >> If we just diff "l

Re: [Qemu-devel] [PATCH v2 1/4] qom: provide root container for internal objs

On Mon, Sep 25, 2017 at 11:00:33AM +0200, Andreas Färber wrote: > Hi, > > Am 25.09.2017 um 10:14 schrieb Peter Xu: > > On Mon, Sep 25, 2017 at 09:14:21AM +0200, Andreas Färber wrote: > >> Am 25.09.2017 um 08:37 schrieb Peter Xu: > >>> We have object_get_objects_root() to keep user created objects,

Re: [Qemu-devel] [PATCH] spapr: move registration of "host" CPU core type to machine code

On Mon, Sep 25, 2017 at 11:47:33AM +0200, Greg Kurz wrote: > The CPU core abstraction belongs to the machine code. This also gets > rid of some code duplication. > > Signed-off-by: Greg Kurz > --- > > hw/ppc/spapr_cpu_core.h is also included elsewhere in target/ppc/kvm.c > but this is already ha

Re: [Qemu-devel] [PATCH v2 6/6] migration: Route more error paths

On Mon, Sep 25, 2017 at 12:29:17PM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > vmstate_save is called in a few places, and vmstate_save_state is > called in lots of places. vmstate_save() should have been processed with previous patch? > > Route error return

Re: [Qemu-devel] [PATCH v2] qdev: Mark devices as non-hotpluggable by default

On Mon, Sep 25, 2017 at 07:05:26PM +0100, Peter Maydell wrote: > On 25 September 2017 at 18:59, Eduardo Habkost wrote: > > Finding the full list of devices that can be instantiated > > internally at hotplug-time sounds tricky. > > If we just diff "list of devices marked hotplug before this patch"

Re: [Qemu-devel] [PATCH v2 5/6] migration: Route errors up through vmstate_save

On Mon, Sep 25, 2017 at 12:29:16PM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Route the errors from vsmtate_save_state back up through > vmstate_save and out to the normal device state path. > That's the normal error path done. > > Signed-off-by: Dr. David Al

Re: [Qemu-devel] [PATCH v2 4/6] migration: wire vmstate_save_state errors up to vmstate_subsection_save

On Mon, Sep 25, 2017 at 12:29:15PM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Route the errors from vmstate_subsection_save up through > vmstate_subsection_save (and back down, all rather recursive). I guess here one of the "vmstate_subsection_save" should be

Re: [Qemu-devel] [PATCH v2 3/6] migration: Check field save returns

On Mon, Sep 25, 2017 at 12:29:14PM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Check the return values from vmstate_save_state for fields and also the > return values from 'put' for fields that use that. > > Signed-off-by: Dr. David Alan Gilbert > --- > migr

Re: [Qemu-devel] [PATCH v2 2/6] migration: check pre_save return in vmstate_save_state

On Mon, Sep 25, 2017 at 12:29:13PM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Check the return value of pre_save state and fail vmstate_save_state > if the pre_save failed. > > Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Peter Xu -- Peter Xu

Re: [Qemu-devel] [PATCH v2 1/6] migration: pre_save return int

On Mon, Sep 25, 2017 at 12:29:12PM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Modify the pre_save method on VMStateDescription to return an int > rather than void so that it potentially can fail. > > Changed zillions of devices to make them return 0; the only

Re: [Qemu-devel] [PATCH v10 18/20] qcow2: Switch store_bitmap_data() to byte-based iteration

On Mon, 09/25 09:55, Eric Blake wrote: > Now that we have adjusted the majority of the calls this function > makes to be byte-based, it is easier to read the code if it makes > passes over the image using bytes rather than sectors. > > iotests 165 was rather weak - on a default 64k-cluster image,

Re: [Qemu-devel] [PATCH v10 05/20] dirty-bitmap: Avoid size query failure during truncate

On Mon, 09/25 09:55, Eric Blake wrote: > We've previously fixed several places where we failed to account > for possible errors from bdrv_nb_sectors(). Fix another one by > making bdrv_dirty_bitmap_truncate() take the new size from the > caller instead of querying itself; then adjust the sole call

Re: [Qemu-devel] [PATCH v1 2/8] tests: Replace fprintf(stderr, "*\n" with error_report()

On Mon, Sep 25, 2017 at 17:08:48 -0700, Alistair Francis wrote: > diff --git a/tests/atomic_add-bench.c b/tests/atomic_add-bench.c > index caa1e8e689..41ba1600c0 100644 > --- a/tests/atomic_add-bench.c > +++ b/tests/atomic_add-bench.c > @@ -29,7 +29,7 @@ static const char commands_string[] = > sta

Re: [Qemu-devel] [PATCH v1 0/8] Remove some of the fprintf(stderr, "*

Hi, This series failed build test on s390x host. Please find the details below. Type: series Subject: [Qemu-devel] [PATCH v1 0/8] Remove some of the fprintf(stderr, "* Message-id: cover.1506384414.git.alistair.fran...@xilinx.com === TEST SCRIPT BEGIN === #!/bin/bash # Testing script will be inv

[Qemu-devel] [PATCH v1 1/8] Replace all occurances of __FUNCTION__ with __func__

Replace all occurs of __FUNCTION__ except for the check in checkpatch with the non GCC specific __func__. One line in hcd-musb.c was manually tweaked to pass checkpatch. Signed-off-by: Alistair Francis Cc: Gerd Hoffmann Cc: Andrzej Zaborowski Cc: Stefano Stabellini Cc: Anthony Perard Cc: Joh

[Qemu-devel] [PATCH v1 6/8] ui: Replace fprintf(stderr, "*\n" with error_report()

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. find ./* -type f -exec sed -i \ 'N;N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr, "\(.*\)\\n"\(.*\));|error_report("\1"\2);|I

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

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. find ./* -type f -exec sed -i \ 'N;N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr, "\(.*\)\\n"\(.*\));|error_report("\1"\2);|I

[Qemu-devel] [PATCH v1 5/8] util: Replace fprintf(stderr, "*\n" with error_report()

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. find ./* -type f -exec sed -i \ 'N;N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr, "\(.*\)\\n"\(.*\));|error_report("\1"\2);|I

[Qemu-devel] [PATCH v1 7/8] tcg: Replace fprintf(stderr, "*\n" with error_report()

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. find ./* -type f -exec sed -i \ 'N;N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr, "\(.*\)\\n"\(.*\));|error_report("\1"\2);|I

[Qemu-devel] [PATCH v1 4/8] block: Replace fprintf(stderr, "*\n" with error_report()

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. find ./* -type f -exec sed -i \ 'N;N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr, "\(.*\)\\n"\(.*\));|error_report("\1"\2);|I

[Qemu-devel] [PATCH v1 0/8] Remove some of the fprintf(stderr, "*

Continue on improving QEMUs logging/error messages by removing more fprintf()'s. Unfortunatley my Coccinelle skills aren't that great so it's all done in some nasty regex and a little bit of manual work. Alistair Francis (8): Replace all occurances of __FUNCTION__ with __func__ tests: Repl

Re: [Qemu-devel] [PATCH v2 6/8] arm: Support Capstone in disas_set_info

Richard Henderson writes: > Cc: qemu-...@nongnu.org > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée Tested-by: Alex Bennée And BTW better than libvixl at least w.r.t wfi. > --- > disas.c | 3 +++ > target/arm/cpu.c | 21 ++--- > 2 files changed, 21 in

Re: [Qemu-devel] [PATCH] docker: Fix test-mingw

On Mon, Sep 25, 2017 at 1:29 AM, Fam Zheng wrote: > Feature "dtc" is explicitly required by test-mingw, but is not detected > by the run script since we switched to archive-source.sh in b7f404201e4. > Since it isn't available in the Fedora image which runs this test on > patchew, the way we get dt

Re: [Qemu-devel] [PATCH 0/7] mac_dbdma: tidy-up and QOMify

On Sun, Sep 24, 2017 at 03:47:39PM +0100, Mark Cave-Ayland wrote: > Whilst looking at implementing another DBDMA device for the Mac machines > I noticed a couple of things: firstly there were some unused fields still > in DBDMAState, and secondly the existing code still used global functions > to r

Re: [Qemu-devel] [PATCH v1 16/27] s390x/kvm: factor out SIGP code into sigp.c

On 09/18/2017 09:00 AM, David Hildenbrand wrote: > We want to use the same code base for TCG, so let's cleanly factor it > out. > > The sigp mutex is currently not really needed, as everything is > protected by the iothread mutex. But this could change later, so leave > it in place and initialize

Re: [Qemu-devel] [PATCH v1 15/27] s390x/kvm: drop two debug prints

On 09/18/2017 09:00 AM, David Hildenbrand wrote: > Preparation for moving it out of kvm.c. > > Signed-off-by: David Hildenbrand > --- > target/s390x/kvm.c | 2 -- > 1 file changed, 2 deletions(-) Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH v1 14/27] s390x/kvm: factor out storing of adtl CPU status

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > Called from SIGP code to be factored out, so let's move it. Add a > FIXME for TCG code in the future. > > Signed-off-by: David Hildenbrand > --- > target/s390x/helper.c | 29 + > target/s390x/internal.h | 1 + > ta

Re: [Qemu-devel] [PATCH v3] spapr: fix the value of SDR1 in kvmppc_put_books_sregs()

On Mon, Sep 25, 2017 at 01:00:02PM +0200, Greg Kurz wrote: > When running with KVM PR, if a new HPT is allocated we need to inform > KVM about the HPT address and size. This is currently done by hacking > the value of SDR1 and pushing it to KVM in several places. > > Also, migration breaks the gue

Re: [Qemu-devel] [PATCH v2] ppc/pnv: check for OPAL firmware file presence

On Mon, Sep 25, 2017 at 10:58:25AM +0200, Cédric Le Goater wrote: > and exit before uselessly trying to load it if the file does not > exists. > > Issue discovered by Coverity Scan. > > Signed-off-by: Cédric Le Goater Applied to ppc-for-2.11, thanks. > --- > > Changes since v1: > > - fixed

Re: [Qemu-devel] [PATCH v1 14/27] s390x/kvm: factor out storing of adtl CPU status

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > Called from SIGP code to be factored out, so let's move it. Add a > FIXME for TCG code in the future. > > Signed-off-by: David Hildenbrand > --- > target/s390x/helper.c | 29 + > target/s390x/internal.h | 1 + > ta

Re: [Qemu-devel] [PULL 00/31] Trivial patches for 2017-09-25

On 24 September 2017 at 22:22, Michael Tokarev wrote: > This is a collection of trivial stuff collected for quite some time. > It includes various stuff, and just one series from > Philippe Mathieu-Daudé (MAINTAINERS update), - other his series are > in the works. > > Thanks, > > /mjt > > The foll

Re: [Qemu-devel] [PATCH v1 11/27] s390x/kvm: generalize SIGP stop and restart interrupt injection

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > Preparation for factoring it out into !kvm code. > > Signed-off-by: David Hildenbrand > --- > target/s390x/internal.h | 2 ++ > target/s390x/interrupt.c | 20 > target/s390x/kvm-stub.c | 8 > target/s390x/kvm.c

Re: [Qemu-devel] [PATCH v1 09/27] target/s390x: factor out handling of WAIT PSW into handle_wait()

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > This will now also detect crashes under TCG. We can directly use > cpu->env.psw.addr instead of kvm_run, as we do a > cpu_synchronize_state(). > > Signed-off-by: David Hildenbrand > --- > target/s390x/helper.c | 28 ++--

Re: [Qemu-devel] [PATCH v1 08/27] s390x/tcg: a CPU cannot switch state due to an interrupt

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > Going to OPERATING here looks wrong. A CPU should even never be > !OPERATING at this point. Unhalting will already be done in > cpu_handle_halt() if there is work, so we can drop this statement > completely. > > Signed-off-by: David Hildenbrand >

Re: [Qemu-devel] [PATCH v1 10/27] s390x/kvm: pass ipb directly into handle_sigp()

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > No need to pass kvm_run. > > Signed-off-by: David Hildenbrand > --- > target/s390x/kvm.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH v1 07/27] s390x/tcg: STOPPED cpus can never wake up

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > Interrupts can't wake such CPUs up. SIGP from other CPUs has to be used > to toggle the state. > > Signed-off-by: David Hildenbrand > --- > target/s390x/cpu.c | 6 ++ > 1 file changed, 6 insertions(+) Reviewed-by: Richard Henderson r~

[Qemu-devel] [PATCH v2 6/6] linux_user: consolidate sock_type

remove unnecessary sock_type enum and other unused surrounding code to allow for per arch sockbits to mirror better linux headers for maintenance Signed-off-by: Carlo Marcelo Arenas Belón --- linux-user/alpha/sockbits.h | 36 linux-user/hppa/sockbits.h | 30

[Qemu-devel] [PATCH v2 4/6] linux-user: refactor socket.h for sparc

fixes SOL_SOCKET and SO_LINGER and all other values that didn't match the default (SO_PASSSEC being the exception as it only changed base) TARGET_SOCK_{NONBLOCK,CLOEXEC} updated to match the values for the header: arch/sparc/include/uapi/asm/fcntl.h Signed-off-by: Carlo Marcelo Arenas Belón Revi

[Qemu-devel] [PATCH v2 5/6] linux-user: update default socket.h

enable SO_REUSEPORT as a sideeffect and add SO_GET_FILTER alias Signed-off-by: Carlo Marcelo Arenas Belón --- linux-user/socket.h | 59 +++-- 1 file changed, 48 insertions(+), 11 deletions(-) diff --git a/linux-user/socket.h b/linux-user/socket.h

[Qemu-devel] [PATCH v2 2/6] linux-user: refactor socket.h for alpha

based on fresh bits from linux 4.14 and therefore enabling SO_REUSEPORT as a side effect to easy on maintenance SO_NONBLOCK updated to match the value from linux headers (in hexadecimal) as seen in arch/alpha/include/asm/socket.h Signed-off-by: Carlo Marcelo Arenas Belón --- linux-user/alpha/so

[Qemu-devel] [PATCH v2 3/6] linux-user: refactor socket.h for mips

fresh bits from linux 4.14, redefine SO_STYLE and {SO,SCM}_TIMESTAMP to the right target values and enables SO_REUSEPORT to easy on maintenance SOCK_NONBLOCK has been updated to use the same value (in hexadecimal) as defined for O_NONBLOCK from the linux header arch/mips/linux/uapi/asm/fcntl.h Si

[Qemu-devel] [PATCH v2 1/6] linux-user: update hppa sockbits

updated to match arch/parisc/include/uapi/asm/socket.h from linux 4.14 include in socket.h changed to prefer a local path like all other qemu headers and for consistency and clarity when adding all other arch Signed-off-by: Carlo Marcelo Arenas Belón --- linux-user/hppa/sockbits.h | 148

[Qemu-devel] linux-user: refactor socket.h into architecture specific sockbits

the definitions in socket.h are meant to reflect the ones in linux for each respective target, but are sometimes difficult to maintain. hppa (AKA parisc) was initially merged with an independent file that mirrors more closely the corresponding one in linux but hasn't been updated since. this seri

Re: [Qemu-devel] [PATCH v1 05/27] s390/tcg: turn INTERRUPT_EXT into a mask

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > +} else if (env->pending_int | INTERRUPT_EXT_FLOATING) { Surely & here. Otherwise, Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH v1 04/27] s390x: introduce and use S390_MAX_CPUS

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > Will be handy in the next patches. > > Signed-off-by: David Hildenbrand > --- > hw/s390x/s390-virtio-ccw.c | 2 +- > target/s390x/cpu.h | 2 ++ > 2 files changed, 3 insertions(+), 1 deletion(-) Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH v1 03/27] target/s390x: get rid of next_core_id

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > /* sync cs->cpu_index and env->core_id. The latter is needed for TCG. */ > -cs->cpu_index = env->core_id; > +cs->cpu_index = cpu->env.core_id; > +#endif Any reason not to drop core_id entirely in favour of cpu_index? (Since cpu_index

Re: [Qemu-devel] [PATCH v1 01/27] s390x: raise CPU hotplug irq after really hotplugged

On 09/18/2017 08:59 AM, David Hildenbrand wrote: > Let's move it into the machine, so we trigger the IRQ after setting > ms->possible_cpus (which SCLP uses to construct the list of > online CPUs). > > Signed-off-by: David Hildenbrand > --- > hw/s390x/s390-virtio-ccw.c | 4 > target/s390x/cp

Re: [Qemu-devel] [PATCH v2] hw/sd: fix out-of-bounds check for multi block reads

On 25 September 2017 at 23:53, Alistair Francis wrote: > On Mon, Sep 25, 2017 at 3:38 PM, Peter Maydell > wrote: >> On 25 September 2017 at 22:16, Alistair Francis wrote: >>> On Mon, Sep 25, 2017 at 12:27 PM, Peter Maydell >>> wrote: Alistair, were you planning to provide a reviewed-by: f

Re: [Qemu-devel] [PATCH v2] hw/sd: fix out-of-bounds check for multi block reads

On Mon, Sep 25, 2017 at 3:38 PM, Peter Maydell wrote: > On 25 September 2017 at 22:16, Alistair Francis wrote: >> On Mon, Sep 25, 2017 at 12:27 PM, Peter Maydell >> wrote: >>> Alistair, were you planning to provide a reviewed-by: for this >>> patch (or did you have more review comments on it)? >

Re: [Qemu-devel] xen/disk: don't leak stack data via response ring

On Sun, 24 Sep 2017, Michael Tokarev wrote: > 23.09.2017 19:05, Michael Tokarev wrote: > > 28.06.2017 01:04, Stefano Stabellini wrote: > >> Rather than constructing a local structure instance on the stack, fill > >> the fields directly on the shared ring, just like other (Linux) > >> backends do. B

Re: [Qemu-devel] [PATCH v4 01/23] block: Allow NULL file for bdrv_get_block_status()

On 09/13/2017 12:03 PM, Eric Blake wrote: > Not all callers care about which BDS owns the mapping for a given > range of the file. This patch merely simplifies the callers by > consolidating the logic in the common call point, while guaranteeing > a non-NULL file to all the driver callbacks, for

Re: [Qemu-devel] [PATCH v2] hw/sd: fix out-of-bounds check for multi block reads

On 25 September 2017 at 22:16, Alistair Francis wrote: > On Mon, Sep 25, 2017 at 12:27 PM, Peter Maydell > wrote: >> Alistair, were you planning to provide a reviewed-by: for this >> patch (or did you have more review comments on it)? > > Ah woops, this slipped through. Looks fine to me then. > >

Re: [Qemu-devel] [PULL 0/3] slirp updates

On 24 September 2017 at 19:08, Samuel Thibault wrote: > warning: redirection vers https://people.debian.org/~sthibault/qemu.git/ > The following changes since commit 460b6c8e581aa06b86f59eebd9e52edfe7adf417: > > Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into > staging (20

Re: [Qemu-devel] [PATCH 8/8] nbd: Minimal structured read for client

On 09/25/2017 08:58 AM, Vladimir Sementsov-Ogievskiy wrote: > Minimal implementation: drop most of additional error information. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/nbd-client.h | 2 + > include/block/nbd.h | 15 - > block/nbd-client.c | 97 +++

Re: [Qemu-devel] [PATCH v10 00/20] make dirty-bitmap byte-based

On 09/25/2017 10:55 AM, Eric Blake wrote: > There are patches floating around to add NBD_CMD_BLOCK_STATUS, > but NBD wants to report status on byte granularity (even if the > reporting will probably be naturally aligned to sectors or even > much higher levels). I've therefore started the task of

[Qemu-devel] [PULL 0/1] tcg patch queue

at: git://github.com/rth7680/qemu.git tags/pull-tcg-20170925 for you to fetch changes up to 8b81253332b5a3f3c67b6462f39caef47a00dd29: accel/tcg/cputlb: avoid recursive BQL (fixes #1706296) (2017-09-25 11:23:30 -0700) B

[Qemu-devel] [PULL 1/1] accel/tcg/cputlb: avoid recursive BQL (fixes #1706296)

From: Alex Bennée The mmio path (see exec.c:prepare_mmio_access) already protects itself against recursive locking and it makes sense to do the same for io_readx/writex. Otherwise any helper running in the BQL context will assert when it attempts to write to device memory as in the case of the bu

Re: [Qemu-devel] [PATCH 2/8] block/nbd-client: refactor nbd_co_receive_reply

On 09/25/2017 08:57 AM, Vladimir Sementsov-Ogievskiy wrote: > Pass handle parameter directly, as the whole request isn't needed. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/nbd-client.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > Reviewed-by: Eric Blake

Re: [Qemu-devel] [PATCH 1/8] block/nbd-client: assert qiov len once in nbd_co_request

On 09/25/2017 08:57 AM, Vladimir Sementsov-Ogievskiy wrote: > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/nbd-client.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > @@ -233,6 +231,7 @@ static int nbd_co_request(BlockDriverState *bs, > > assert(!qiov || reque

Re: [Qemu-devel] [Qemu-ppc] [PATCH] spapr: move registration of "host" CPU core type to machine code

On Mon, 25 Sep 2017 17:48:57 +0200 Greg Kurz wrote: > On Mon, 25 Sep 2017 15:41:34 +0200 > Igor Mammedov wrote: > > > On Mon, 25 Sep 2017 11:47:33 +0200 > > Greg Kurz wrote: > > > > > The CPU core abstraction belongs to the machine code. This also gets > > > rid of some code duplication. >

Re: [Qemu-devel] [PATCH v2 2/2] qcow2: truncate the tail of the image file after shrinking the image

On 09/22/2017 05:39 AM, Pavel Butsykin wrote: > Now after shrinking the image, at the end of the image file, there might be a > tail that probably will never be used. So we can find the last used cluster > and > cut the tail. > > Signed-off-by: Pavel Butsykin Reviewed-by: John Snow

Re: [Qemu-devel] [PATCH v2] hw/sd: fix out-of-bounds check for multi block reads

On Mon, Sep 25, 2017 at 12:27 PM, Peter Maydell wrote: > On 20 September 2017 at 07:19, Michael Olbrich > wrote: >> On Tue, Sep 19, 2017 at 05:09:51PM -0700, Alistair Francis wrote: >>> On Tue, Sep 19, 2017 at 1:23 AM, Michael Olbrich >>> wrote: >>> > On Mon, Sep 18, 2017 at 02:28:26PM -0700, A

Re: [Qemu-devel] [PATCH v10 18/20] qcow2: Switch store_bitmap_data() to byte-based iteration

On 09/25/2017 10:55 AM, Eric Blake wrote: > Now that we have adjusted the majority of the calls this function > makes to be byte-based, it is easier to read the code if it makes > passes over the image using bytes rather than sectors. > > iotests 165 was rather weak - on a default 64k-cluster im

Re: [Qemu-devel] [PATCH 00/12] Patch Round-up for stable 2.10.1, freeze on 2017-09-27

Quoting Michael Roth (2017-09-19 19:45:09) > Hi everyone, > > The following new patches are queued for QEMU stable v2.10.1: > > https://github.com/mdroth/qemu/commits/stable-2.10-staging > > The release is planned for 2017-10-02: > > https://wiki.qemu.org/Planning/2.10 Thank you for the su

Re: [Qemu-devel] [PATCH] accel/tcg/cputlb: avoid recursive BQL (fixes #1706296)

Quoting Alex Bennée (2017-09-21 06:06:25) > The mmio path (see exec.c:prepare_mmio_access) already protects itself > against recursive locking and it makes sense to do the same for > io_readx/writex. Otherwise any helper running in the BQL context will > assert when it attempts to write to device m

Re: [Qemu-devel] [Qemu-stable] [PATCH] migration: disable auto-converge during bulk block migration

Quoting Peter Lieven (2017-09-21 07:32:32) > auto-converge and block migration currently do not play well together. > During block migration the auto-converge logic detects that ram > migration makes no progress and thus throttles down the vm until > it nearly stalls completely. Avoid this by disab

Re: [Qemu-devel] [PATCH] throttle-groups: update tg->any_timer_armed[] on detach

Quoting Stefan Hajnoczi (2017-09-19 10:50:25) > Clear tg->any_timer_armed[] when throttling timers are destroy during > AioContext attach/detach. Failure to do so causes throttling to hang > because we believe the timer is already scheduled! > > The following was broken at least since QEMU 2.10.0

Re: [Qemu-devel] [PATCH] block/qcow2-bitmap: fix use of uninitialized pointer

Quoting Vladimir Sementsov-Ogievskiy (2017-09-22 09:43:53) > Without initialization to zero dirty_bitmap field may be not zero > for a bitmap which should not be stored and > qcow2_store_persistent_dirty_bitmaps will erroneously call > store_bitmap for it which leads to SYGSEGV on bdrv_dirty_bitmap

Re: [Qemu-devel] [PATCH] virtio/vhost: reset dev->log after syncing

Quoting Felipe Franciosi (2017-09-20 13:53:06) > vhost_log_put() is called to decomission the dirty log between qemu and > a vhost device when stopping the device. Such a call can happen from > migration_completion(). > > Present code sets dev->log_size to zero too early in vhost_log_put(), > caus

  1   2   3   4   >