[PATCH for-5.2] microvm: fix kconfig for non-default devices

2020-11-20 Thread Paolo Bonzini
PCIe and USB support can be disabled on the command line, and therefore should not be included if QEMU is configured --without-default-devices. While at it, also remove the "default y" for USB_XHCI_SYSBUS because sysbus devices are not user creatable; boards that use them will specify them

Re: [PATCH 1/2] pc-bios: s390x: Ensure Read IPL memory is clean

2020-11-20 Thread Cornelia Huck
On Thu, 19 Nov 2020 17:57:28 +0100 Eric Farman wrote: > If, for example, we boot off a virtio device and chreipl to a vfio-ccw > device, the space at lowcore will be non-zero. We build a Read IPL CCW > at address zero, but it will have leftover PSW data that will conflict > with the Format-0 CCW

Re: iotest 030 still occasionally intermittently failing

2020-11-20 Thread Vladimir Sementsov-Ogievskiy
19.11.2020 23:31, Vladimir Sementsov-Ogievskiy wrote: 19.11.2020 22:31, Vladimir Sementsov-Ogievskiy wrote: 19.11.2020 22:30, Vladimir Sementsov-Ogievskiy wrote: 19.11.2020 19:11, Vladimir Sementsov-Ogievskiy wrote: 16.11.2020 20:59, Peter Maydell wrote: On Mon, 16 Nov 2020 at 17:34, Alberto

Re: [PATCH for-5.2] microvm: fix kconfig for non-default devices

2020-11-20 Thread Philippe Mathieu-Daudé
On 11/20/20 9:55 AM, Paolo Bonzini wrote: > PCIe and USB support can be disabled on the command line, and therefore > should not be included if QEMU is configured --without-default-devices. > > While at it, also remove the "default y" for USB_XHCI_SYSBUS because > sysbus devices are not user

Re: QMP and the 'id' parameter

2020-11-20 Thread Markus Armbruster
John Snow writes: > On 11/11/20 3:27 AM, Markus Armbruster wrote: >> John Snow writes: >> >>> On 11/10/20 1:22 AM, Markus Armbruster wrote: John Snow writes: > The QMP specification states: > >> NOTE: Some errors can occur before the Server is able to read the "id"

[PATCH for-5.2] hw/display/tcx: add missing 64-bit access for framebuffer blitter

2020-11-20 Thread Mark Cave-Ayland
Commit ae5643ecc6 "hw/display/tcx: Allow 64-bit accesses to framebuffer stippler and blitter" enabled 64-bit access for the TCX framebuffer stippler and blitter but missed applying the change to one of the blitter MemoryRegions. Whilst the original change works for me on my local NetBSD test

Re: [PATCH] qga: Correct loop count in qmp_guest_get_vcpus()

2020-11-20 Thread Lin Ma
On 2020-11-19 14:46, Marc-André Lureau wrote: Hi On Thu, Nov 19, 2020 at 12:48 PM Lin Ma wrote: The guest-get-vcpus returns incorrect vcpu info in case we hotunplug vcpus(not the last one). e.g.: A VM has 4 VCPUs: cpu0 + 3 hotunpluggable online vcpus(cpu1, cpu2 and cpu3). Hotunplug cpu2,

Re: [PATCH v5 0/2] MTE support for KVM guest

2020-11-20 Thread Steven Price
On 19/11/2020 19:11, Marc Zyngier wrote: On 2020-11-19 18:42, Andrew Jones wrote: On Thu, Nov 19, 2020 at 03:45:40PM +, Peter Maydell wrote: On Thu, 19 Nov 2020 at 15:39, Steven Price wrote: > This series adds support for Arm's Memory Tagging Extension (MTE) to > KVM, allowing KVM guests

[RFC PATCH 2/2] ramlist: Resize dirty bitmap blocks after remove ramblock

2020-11-20 Thread Keqian Zhu
Use the new "dirty_bitmap_resize" interface to reduce dirty bitmap blocks after we remove a ramblock from ramlist. Signed-off-by: Keqian Zhu --- softmmu/physmem.c | 8 1 file changed, 8 insertions(+) diff --git a/softmmu/physmem.c b/softmmu/physmem.c index f6ff78378e..2a17e0a89a

Re: Peter Maydell

2020-11-20 Thread Paolo Bonzini
On 20/11/20 09:44, Liviu Ionescu wrote: On 20 Nov 2020, at 07:43, cavinnarsingh...@gmail.com wrote: This issue is about the Qemu Will the Qemu work on the new m1 macbook pro? And if yes, when will the arm version of Qemu be available for public I already distribute a binary version of the

Re: [PATCH v5 6/7] tcg: implement JIT for iOS and Apple Silicon

2020-11-20 Thread Alexander Graf
On 09.11.20 00:24, Joelle van Dyne wrote: When entitlements are available (macOS or jailbroken iOS), a hardware feature called APRR exists on newer Apple Silicon that can cheaply mark JIT pages as either RX or RW. Reverse engineered functions from libsystem_pthread.dylib are implemented to

Re: [RFC v3 9/9] i386: split cpu accelerators from cpu.c

2020-11-20 Thread Claudio Fontana
On 11/19/20 8:23 PM, Eduardo Habkost wrote: > On Thu, Nov 19, 2020 at 09:53:09AM +0100, Claudio Fontana wrote: >> Hi, >> >> On 11/18/20 7:28 PM, Eduardo Habkost wrote: >>> On Wed, Nov 18, 2020 at 11:29:36AM +0100, Claudio Fontana wrote: split cpu.c into: cpu.ccpuid and

[PATCH v2] qga: Correct loop count in qmp_guest_get_vcpus()

2020-11-20 Thread Lin Ma
The guest-get-vcpus returns incorrect vcpu info in case we hotunplug vcpus(not the last one). e.g.: A VM has 4 VCPUs: cpu0 + 3 hotunpluggable online vcpus(cpu1, cpu2 and cpu3). Hotunplug cpu2, Now only cpu0, cpu1 and cpu3 are present & online. ./qmp-shell /tmp/qmp-monitor.sock (QEMU)

Re: [PATCH v5 7/7] block: check availablity for preadv/pwritev on mac

2020-11-20 Thread Philippe Mathieu-Daudé
On 11/9/20 12:24 AM, Joelle van Dyne wrote: > macOS 11/iOS 14 added preadv/pwritev APIs. Due to weak linking, configure > will succeed with CONFIG_PREADV even when targeting a lower OS version. We > therefore need to check at run time if we can actually use these APIs. > > Signed-off-by: Joelle

Re: [PATCH v5 0/2] MTE support for KVM guest

2020-11-20 Thread Marc Zyngier
On 2020-11-20 09:50, Steven Price wrote: On 19/11/2020 19:11, Marc Zyngier wrote: Does this sound reasonable? I'll clean up the set_pte_at() change and post a v6 later today. Please hold on. I still haven't reviewed your v5, nor have I had time to read your reply to my comments on v4.

[RFC PATCH 1/2] ramlist: Make dirty bitmap blocks of ramlist resizable

2020-11-20 Thread Keqian Zhu
When we remove a ramblock, we should decrease the dirty bitmap blocks of ramlist to avoid memory leakage. This patch rebuilds dirty_memory_ extend to support both "extend" and "decrease". Signed-off-by: Keqian Zhu --- softmmu/physmem.c | 28 1 file changed, 20

[RFC PATCH 0/2] bugfix: Decrease dirty bitmap blocks after we remove ramblock

2020-11-20 Thread Keqian Zhu
Hi all, I failed to find where we free dirty bitmap blocks of ramlist. If this is a memory leakage problem, I hope this patch series can fix it properly :-). Thanks, Keqian. Keqian Zhu (2): ramlist: Make dirty bitmap blocks of ramlist resizable ramlist: Resize dirty bitmap blocks after

Re: [PATCH v2 3/8] qnum: QNumValue type for QNum value literals

2020-11-20 Thread Markus Armbruster
Eduardo Habkost writes: > On Thu, Nov 19, 2020 at 01:21:58PM -0500, Eduardo Habkost wrote: >> On Thu, Nov 19, 2020 at 11:24:52AM +0100, Markus Armbruster wrote: > [...] >> > >> > > +return qnum_from_value((QNumValue) QNUM_VAL_INT(value)); >> > >> > No space between between (type) and its

Re: Peter Maydell

2020-11-20 Thread Stefan Weil
Am 20.11.20 um 09:54 schrieb Paolo Bonzini: On 20/11/20 09:44, Liviu Ionescu wrote: On 20 Nov 2020, at 07:43, cavinnarsingh...@gmail.com wrote: This issue is about the Qemu Will the Qemu work on the new m1 macbook pro? And if yes, when will the arm version of Qemu be available for public I

Re: [PATCH for-5.2] microvm: fix kconfig for non-default devices

2020-11-20 Thread Gerd Hoffmann
On Fri, Nov 20, 2020 at 09:55:24AM +0100, Paolo Bonzini wrote: > PCIe and USB support can be disabled on the command line, and therefore > should not be included if QEMU is configured --without-default-devices. I don't think this will work. I expect linking will fail due to xhci_sysbus_build_aml

Re: [PATCH v5 0/2] MTE support for KVM guest

2020-11-20 Thread Steven Price
On 20/11/2020 09:56, Marc Zyngier wrote: On 2020-11-20 09:50, Steven Price wrote: On 19/11/2020 19:11, Marc Zyngier wrote: Does this sound reasonable? I'll clean up the set_pte_at() change and post a v6 later today. Please hold on. I still haven't reviewed your v5, nor have I had time to

Re: [PATCH for-5.2] hw/display/tcx: add missing 64-bit access for framebuffer blitter

2020-11-20 Thread Philippe Mathieu-Daudé
On 11/20/20 9:17 AM, Mark Cave-Ayland wrote: > Commit ae5643ecc6 "hw/display/tcx: Allow 64-bit accesses to framebuffer > stippler > and blitter" enabled 64-bit access for the TCX framebuffer stippler and > blitter > but missed applying the change to one of the blitter MemoryRegions. > > Whilst

Re: [PATCH] hmp: Changed hmp_netdev_add() using qmp_marshal_netdev_add()

2020-11-20 Thread Andrew Melnichenko
Ping On Thu, Jul 16, 2020 at 6:26 AM wrote: > From: Andrew Melnychenko > > There is an issue, that netdev can't be removed if it was added using hmp. > The bug appears after 08712fcb851034228b61f75bd922863a984a4f60 commit. > It happens because of unclear QemuOpts that was created during >

Re: [PATCH 1/2] pc-bios: s390x: Ensure Read IPL memory is clean

2020-11-20 Thread Janosch Frank
On 11/19/20 5:57 PM, Eric Farman wrote: > If, for example, we boot off a virtio device and chreipl to a vfio-ccw > device, the space at lowcore will be non-zero. We build a Read IPL CCW > at address zero, but it will have leftover PSW data that will conflict > with the Format-0 CCW being

Re: [PATCH 00/10] vhost/qemu: thread per IO SCSI vq

2020-11-20 Thread Stefan Hajnoczi
On Thu, Nov 19, 2020 at 5:08 PM Stefan Hajnoczi wrote: > > On Thu, Nov 19, 2020 at 4:43 PM Mike Christie > wrote: > > > > On 11/19/20 10:24 AM, Stefan Hajnoczi wrote: > > > On Thu, Nov 19, 2020 at 4:13 PM Mike Christie > > > wrote: > > >> > > >> On 11/19/20 8:46 AM, Michael S. Tsirkin wrote: >

Re: Peter Maydell

2020-11-20 Thread Liviu Ionescu
> On 20 Nov 2020, at 07:43, cavinnarsingh...@gmail.com wrote: > > This issue is about the Qemu > Will the Qemu work on the new m1 macbook pro? > And if yes, when will the arm version of Qemu be available for public I already distribute a binary version of the xPack QEMU Arm fork which runs

Re: Peter Maydell

2020-11-20 Thread Liviu Ionescu
> On 20 Nov 2020, at 10:54, Paolo Bonzini wrote: > > new m1 ... Richard Henderson and Joelle van Dyne are working on it Ok, it's good to know that someone takes care of this. Personally I think that the new Apple hardware is great, but to match developer needs it might need a few more

Re: [PATCH v4 2/2] arm64: kvm: Introduce MTE VCPU feature

2020-11-20 Thread Steven Price
On 19/11/2020 16:24, Catalin Marinas wrote: On Thu, Nov 19, 2020 at 12:45:52PM +, Steven Price wrote: On 18/11/2020 17:05, Andrew Jones wrote: On Wed, Nov 18, 2020 at 04:50:01PM +, Catalin Marinas wrote: On Wed, Nov 18, 2020 at 04:01:20PM +, Steven Price wrote: On 17/11/2020

Re: [PATCH] qga: Correct loop count in qmp_guest_get_vcpus()

2020-11-20 Thread Marc-André Lureau
Hi On Fri, Nov 20, 2020 at 1:28 PM Lin Ma wrote: > On 2020-11-19 14:46, Marc-André Lureau wrote: > > Hi > > > > On Thu, Nov 19, 2020 at 12:48 PM Lin Ma wrote: > > > >> The guest-get-vcpus returns incorrect vcpu info in case we hotunplug > >> vcpus(not > >> the last one). > >> e.g.: > >> A VM

Re: [PATCH v5 2/7] configure: cross-compiling with empty cross_prefix

2020-11-20 Thread Philippe Mathieu-Daudé
On 11/9/20 12:24 AM, Joelle van Dyne wrote: > The iOS toolchain does not use the host prefix naming convention. So we need > to enable cross-compile options while allowing the PREFIX to be blank. > > Signed-off-by: Joelle van Dyne > --- > configure | 6 -- > 1 file changed, 4 insertions(+),

Re: [PATCH 4/4] [broken] meson: try link tracepoints to module

2020-11-20 Thread Gerd Hoffmann
Hi, > > Nope. Tried s/module_ss/kraxel_ss/, still not working. > > > > I get tons of "undefined reference to `_TRACE_something'" > > errors (*not* qxl). Seems trace_ss is not updated as intended. > > Okay. There is a workaround available: > [PATCH v2] trace: use STAP_SDT_V2 to work around

Re: iotest 030 still occasionally intermittently failing

2020-11-20 Thread Philippe Mathieu-Daudé
Cc'ing Stefan/Paolo On 11/20/20 10:51 AM, Vladimir Sementsov-Ogievskiy wrote: > 19.11.2020 23:31, Vladimir Sementsov-Ogievskiy wrote: >> 19.11.2020 22:31, Vladimir Sementsov-Ogievskiy wrote: >>> 19.11.2020 22:30, Vladimir Sementsov-Ogievskiy wrote: 19.11.2020 19:11, Vladimir

[PATCH v3] qga: Correct loop count in qmp_guest_get_vcpus()

2020-11-20 Thread Lin Ma
The guest-get-vcpus returns incorrect vcpu info in case we hotunplug vcpus(not the last one). e.g.: A VM has 4 VCPUs: cpu0 + 3 hotunpluggable online vcpus(cpu1, cpu2 and cpu3). Hotunplug cpu2, Now only cpu0, cpu1 and cpu3 are present & online. ./qmp-shell /tmp/qmp-monitor.sock (QEMU)

Re: iotest 030 still occasionally intermittently failing

2020-11-20 Thread Vladimir Sementsov-Ogievskiy
20.11.2020 13:34, Philippe Mathieu-Daudé wrote: Cc'ing Stefan/Paolo On 11/20/20 10:51 AM, Vladimir Sementsov-Ogievskiy wrote: 19.11.2020 23:31, Vladimir Sementsov-Ogievskiy wrote: 19.11.2020 22:31, Vladimir Sementsov-Ogievskiy wrote: 19.11.2020 22:30, Vladimir Sementsov-Ogievskiy wrote:

Re: [PATCH for-5.2] microvm: fix kconfig for non-default devices

2020-11-20 Thread Gerd Hoffmann
On Fri, Nov 20, 2020 at 01:08:40PM +0100, Paolo Bonzini wrote: > On 20/11/20 10:38, Gerd Hoffmann wrote: > > On Fri, Nov 20, 2020 at 09:55:24AM +0100, Paolo Bonzini wrote: > > > PCIe and USB support can be disabled on the command line, and therefore > > > should not be included if QEMU is

Re: [PATCH v3 3/7] support UFFD write fault processing in ram_save_iterate()

2020-11-20 Thread Andrey Gruzdev
On 19.11.2020 21:25, Peter Xu wrote: On Thu, Nov 19, 2020 at 03:59:36PM +0300, Andrey Gruzdev via wrote: [...] +/** + * ram_find_block_by_host_address: find RAM block containing host page + * + * Returns true if RAM block is found and pss->block/page are + * pointing to the given host page,

Re: [PATCH V13 2/9] meson.build: Re-enable KVM support for MIPS

2020-11-20 Thread Philippe Mathieu-Daudé
On 11/20/20 5:28 AM, Huacai Chen wrote: > On Wed, Nov 18, 2020 at 1:17 AM Philippe Mathieu-Daudé > wrote: >> On 10/7/20 10:39 AM, Huacai Chen wrote: >>> After converting from configure to meson, KVM support is lost for MIPS, >>> so re-enable it in meson.build. >>> >>> Fixes:

Re: [PATCH-for-5.2 v2] hw/core/qdev-properties-system: Rewrite set_pci_host_devaddr using GLib

2020-11-20 Thread Philippe Mathieu-Daudé
On 11/17/20 12:16 PM, Philippe Mathieu-Daudé wrote: > ping??? > > On 11/9/20 3:16 PM, Philippe Mathieu-Daudé wrote: >> Cc'ing PCI developers (rc2 is scheduled for tomorrow). >> >> On 11/7/20 9:59 AM, Philippe Mathieu-Daudé wrote: >>> Ping for 5.2 as this is a bugfix. >>> >>> On 10/13/20 12:22 PM,

Re: [PATCH 4/4] [broken] meson: try link tracepoints to module

2020-11-20 Thread Gerd Hoffmann
Hi, > Guess I'll go dig into the meson documentation, this looks more like a > build system than a tracing problem to me. Looking at https://mesonbuild.com/Syntax.html ... "all objects are immutable". So "var2 = var1" creates a copy not a reference I guess? Which implies that ...

Re: [PATCH v3 1/7] introduce 'track-writes-ram' migration capability

2020-11-20 Thread Andrey Gruzdev
On 19.11.2020 21:51, Peter Xu wrote: On Thu, Nov 19, 2020 at 03:59:34PM +0300, Andrey Gruzdev via wrote: Signed-off-by: Andrey Gruzdev --- migration/migration.c | 96 +++ migration/migration.h | 1 + qapi/migration.json | 7 +++- 3 files

Re: [PATCH v3 7/7] introduce simple linear scan rate limiting mechanism

2020-11-20 Thread Andrey Gruzdev
On 19.11.2020 23:02, Peter Xu wrote: On Thu, Nov 19, 2020 at 03:59:40PM +0300, Andrey Gruzdev wrote: Since reading UFFD events and saving paged data are performed from the same thread, write fault latencies are sensitive to migration stream stalls. Limiting total page saving rate is a method to

Re: [PATCH 4/4] [broken] meson: try link tracepoints to module

2020-11-20 Thread Philippe Mathieu-Daudé
On 11/20/20 12:25 PM, Gerd Hoffmann wrote: > Hi, > >> Guess I'll go dig into the meson documentation, this looks more like a >> build system than a tracing problem to me. > > Looking at https://mesonbuild.com/Syntax.html ... > > "all objects are immutable". > > So "var2 = var1" creates a

Re: [PATCH for-5.2] Revert "hw/core/qdev-properties: Use qemu_strtoul() in set_pci_host_devaddr()"

2020-11-20 Thread Paolo Bonzini
On 20/11/20 14:04, Michael S. Tsirkin wrote: This reverts commit bccb20c49df1bd683248a366021973901c11982f as it introduced a regression blocking bus addresses > 0x1f or higher. Legal bus numbers go up to 0xff. Cc: "Philippe Mathieu-Daudé" Fixes: bccb20c49df ("Use qemu_strtoul() in

Re: [PATCH] meson.build: convert --with-default-devices to meson

2020-11-20 Thread Richard Henderson
On 11/19/20 11:38 PM, Paolo Bonzini wrote: > Pass the boolean option directly instead of writing > CONFIG_MINIKCONF_MODE to config-host.mak. > > Signed-off-by: Paolo Bonzini > --- > configure | 12 > meson.build | 5 +++-- > meson_options.txt | 2 ++ > 3 files

Re: [PATCH 1/2] pc-bios: s390x: Ensure Read IPL memory is clean

2020-11-20 Thread Eric Farman
On 11/20/20 3:26 AM, Janosch Frank wrote: On 11/19/20 5:57 PM, Eric Farman wrote: If, for example, we boot off a virtio device and chreipl to a vfio-ccw device, the space at lowcore will be non-zero. We build a Read IPL CCW at address zero, but it will have leftover PSW data that will

Availability of physical devices and migration (was: Re: [RFC PATCH 2/2] vfio-ccw: Connect the device request notifier)

2020-11-20 Thread Cornelia Huck
On Fri, 20 Nov 2020 03:51:07 +0100 Halil Pasic wrote: > On Tue, 17 Nov 2020 04:26:05 +0100 > Eric Farman wrote: > > > Now that the vfio-ccw code has a notifier interface to request that > > a device be unplugged, let's wire that together. > > I'm aware of the fact that performing an unplug

Re: [PATCH] hmp: Changed hmp_netdev_add() using qmp_marshal_netdev_add()

2020-11-20 Thread Markus Armbruster
Andrew Melnichenko writes: > Ping > > On Thu, Jul 16, 2020 at 6:26 AM wrote: > >> From: Andrew Melnychenko >> >> There is an issue, that netdev can't be removed if it was added using hmp. >> The bug appears after 08712fcb851034228b61f75bd922863a984a4f60 commit. >> It happens because of unclear

Re: [PATCH for-5.2] Revert "hw/core/qdev-properties: Use qemu_strtoul() in set_pci_host_devaddr()"

2020-11-20 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20201120130409.956956-1-...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20201120130409.956956-1-...@redhat.com Subject: [PATCH for-5.2] Revert

Re: [PATCH 4/4] [broken] meson: try link tracepoints to module

2020-11-20 Thread Gerd Hoffmann
> > So "var2 = var1" creates a copy not a reference I guess? > > > > Which implies that ... > > > > foo_ss.add(something) > > > > ... is different from ... > > > > bar_ss = foo_ss > > bar_ss.add(something) > > > > ... which in turn explains why the patch doesn't work at all. > >

[Bug 1905037] [NEW] Qemu SPARC64 Panics on Sun Solaris 5.8 - BOP_ALLOC failed

2020-11-20 Thread Morteza NourelahiAlamdari
Public bug reported: Hi, Running Sun Solaris 5.8 by SPARC64, will panic by "BOP_ALLOC failed": $ qemu-system-sparc64 \ -drive file=sparc.qcow2,if=ide,bus=0,unit=0 \ -drive file=sun5.8.no1.iso,format=raw,if=ide,bus=1,unit=0,media=cdrom,readonly=on \ -boot d $ qemu-system-sparc64 -M sun4u

Re: [PATCH v5 6/7] tcg: implement JIT for iOS and Apple Silicon

2020-11-20 Thread Alexander Graf
On 20.11.20 10:08, Alexander Graf wrote: On 09.11.20 00:24, Joelle van Dyne wrote: When entitlements are available (macOS or jailbroken iOS), a hardware feature called APRR exists on newer Apple Silicon that can cheaply mark JIT pages as either RX or RW. Reverse engineered functions from

Re: [PATCH v1 6/6] gitlab-ci: Move trace backend tests across to gitlab

2020-11-20 Thread Thomas Huth
On 18/11/2020 10.54, Thomas Huth wrote: > On 17/11/2020 18.36, Alex Bennée wrote: >> From: Philippe Mathieu-Daudé >> >> Similarly to commit 8cdb2cef3f1, move the trace backend >> tests to GitLab. >> >> Note the User-Space Tracer backend is still tested on >> Ubuntu by the s390x jobs on Travis-CI.

Re: [PATCH v3 2/7] introduce UFFD-WP low-level interface helpers

2020-11-20 Thread Andrey Gruzdev
On 19.11.2020 21:39, Peter Xu wrote: On Thu, Nov 19, 2020 at 03:59:35PM +0300, Andrey Gruzdev via wrote: +/** + * uffd_register_memory: register memory range with UFFD + * + * Returns 0 in case of success, negative value on error + * + * @uffd: UFFD file descriptor + * @start: starting virtual

Re: [PATCH v3 1/7] introduce 'track-writes-ram' migration capability

2020-11-20 Thread Andrey Gruzdev
On 19.11.2020 22:07, Peter Xu wrote: On Thu, Nov 19, 2020 at 01:51:50PM -0500, Peter Xu wrote: On Thu, Nov 19, 2020 at 03:59:34PM +0300, Andrey Gruzdev via wrote: Signed-off-by: Andrey Gruzdev --- migration/migration.c | 96 +++ migration/migration.h

Re: [PATCH for-5.2] microvm: fix kconfig for non-default devices

2020-11-20 Thread Paolo Bonzini
On 20/11/20 10:38, Gerd Hoffmann wrote: On Fri, Nov 20, 2020 at 09:55:24AM +0100, Paolo Bonzini wrote: PCIe and USB support can be disabled on the command line, and therefore should not be included if QEMU is configured --without-default-devices. I don't think this will work. I expect

Re: [RFC v3 8/9] module: introduce MODULE_INIT_ACCEL_CPU

2020-11-20 Thread Claudio Fontana
On 11/18/20 11:07 PM, Eduardo Habkost wrote: > On Wed, Nov 18, 2020 at 08:13:18PM +0100, Paolo Bonzini wrote: >> On 18/11/20 18:30, Eduardo Habkost wrote: Adding a layer of indirect calls is not very different from monkey patching though. >>> >>> I'm a little bothered by monkey patching,

Re: Availability of physical devices and migration (was: Re: [RFC PATCH 2/2] vfio-ccw: Connect the device request notifier)

2020-11-20 Thread Halil Pasic
On Fri, 20 Nov 2020 12:38:37 +0100 Cornelia Huck wrote: > On Fri, 20 Nov 2020 03:51:07 +0100 > Halil Pasic wrote: > > > On Tue, 17 Nov 2020 04:26:05 +0100 > > Eric Farman wrote: > > > > > Now that the vfio-ccw code has a notifier interface to request that > > > a device be unplugged, let's

Re: [PATCH] qtest: do not return freed argument vector from qtest_rsp

2020-11-20 Thread Thomas Huth
On 20/11/2020 08.31, Paolo Bonzini wrote: > If expected_args is 0, qtest frees the argument vector and then returns it > nevertheless. Coverity complains; in practice this is not an issue because > expected_args == 0 means that the caller is not interested in the argument > vector, but it would

Re: [RFC PATCH v3 01/13] hw/arm/virt: Spell out smp.cpus and smp.max_cpus

2020-11-20 Thread Andrew Jones
On Tue, Nov 17, 2020 at 06:27:54PM +0800, Ying Fang wrote: > > > On 11/9/2020 6:45 PM, Salil Mehta wrote: > > Hi Fangying, > > A trivial thing. This patch looks bit of a noise in this patch-set. Better > > to send it as a separate patch-set and get it accepted. > > > Hmm, this patch looks like

[PATCH for-5.2] Revert "hw/core/qdev-properties: Use qemu_strtoul() in set_pci_host_devaddr()"

2020-11-20 Thread Michael S. Tsirkin
This reverts commit bccb20c49df1bd683248a366021973901c11982f as it introduced a regression blocking bus addresses > 0x1f or higher. Legal bus numbers go up to 0xff. Cc: "Philippe Mathieu-Daudé" Fixes: bccb20c49df ("Use qemu_strtoul() in set_pci_host_devaddr()") Reported-by: Klaus Herman

Re: [PATCH 4/4] [broken] meson: try link tracepoints to module

2020-11-20 Thread Paolo Bonzini
On 20/11/20 12:25, Gerd Hoffmann wrote: Hi, Guess I'll go dig into the meson documentation, this looks more like a build system than a tracing problem to me. Looking at https://mesonbuild.com/Syntax.html ... "all objects are immutable". So "var2 = var1" creates a copy not a reference I

Re: [PATCH 21/29] vl: separate qemu_resolve_machine_memdev

2020-11-20 Thread Igor Mammedov
On Tue, 27 Oct 2020 14:21:36 -0400 Paolo Bonzini wrote: > This is a bit nasty: the machine is storing a string and later > resolving it. We probably want to remove the memdev property > and instead make this a memory-set command. "-M memdev" can be > handled as a legacy option that is special

Re: [PATCH-for-5.2 v2] hw/intc: fix heap-buffer-overflow in rxicu_realize()

2020-11-20 Thread Peter Maydell
On Wed, 11 Nov 2020 at 14:18, Chen Qun wrote: > > When 'j = icu->nr_sense – 1', the 'j < icu->nr_sense' condition is true, > then 'j = icu->nr_sense', the'icu->init_sense[j]' has out-of-bounds access. > > The asan showed stack: > ERROR: AddressSanitizer: heap-buffer-overflow on address

Re: [PATCH v3 4/7] implementation of write-tracking migration thread

2020-11-20 Thread Andrey Gruzdev
On 19.11.2020 21:47, Peter Xu wrote: On Thu, Nov 19, 2020 at 03:59:37PM +0300, Andrey Gruzdev via wrote: Signed-off-by: Andrey Gruzdev Some commit message would always be appreciated... Thanks, Yep, missed it.. -- Andrey Gruzdev, Principal Engineer Virtuozzo GmbH +7-903-247-6397

Re: [PATCH 4/4] [broken] meson: try link tracepoints to module

2020-11-20 Thread Paolo Bonzini
On 20/11/20 13:30, Philippe Mathieu-Daudé wrote: Maybe managing it all in the main meson.build, like the e28ab096bf8..da33fc09873 cleanup? ("Move the creation of the library to the main meson.build") That was a different issue due to variables being defined in many different meson.build

Re: [PATCH-for-5.2 v2] hw/core/qdev-properties-system: Rewrite set_pci_host_devaddr using GLib

2020-11-20 Thread Philippe Mathieu-Daudé
On Fri, Nov 20, 2020 at 1:55 PM Michael S. Tsirkin wrote: > On Fri, Nov 20, 2020 at 12:00:56PM +0100, Philippe Mathieu-Daudé wrote: > > On 11/17/20 12:16 PM, Philippe Mathieu-Daudé wrote: > > > ping??? > > > > > > On 11/9/20 3:16 PM, Philippe Mathieu-Daudé wrote: > > >> Cc'ing PCI developers

Re: [PATCH for-5.2] Revert "hw/core/qdev-properties: Use qemu_strtoul() in set_pci_host_devaddr()"

2020-11-20 Thread Philippe Mathieu-Daudé
On 11/20/20 2:04 PM, Michael S. Tsirkin wrote: > This reverts commit bccb20c49df1bd683248a366021973901c11982f as it > introduced a regression blocking bus addresses > 0x1f or higher. > Legal bus numbers go up to 0xff. > > Cc: "Philippe Mathieu-Daudé" > Fixes: bccb20c49df ("Use qemu_strtoul() in

Re: [PATCH v5 6/7] tcg: implement JIT for iOS and Apple Silicon

2020-11-20 Thread Richard Henderson
On 11/20/20 6:15 AM, Alexander Graf wrote: > MAP_JIT is definitely missing to make it work on macos. As per the cover, Based-on: 20201106032921.600200-1-richard.hender...@linaro.org ([PATCH v3 00/41] Mirror map JIT memory for TCG) which contains

Re: [PATCH v3 5/7] implementation of vm_start() BH

2020-11-20 Thread Andrey Gruzdev
On 19.11.2020 21:46, Peter Xu wrote: On Thu, Nov 19, 2020 at 03:59:38PM +0300, Andrey Gruzdev wrote: To avoid saving updated versions of memory pages we need to start tracking RAM writes before we resume operation of vCPUs. This sequence is especially critical for virtio device backends whos

Re: [PATCH v3] qga: Correct loop count in qmp_guest_get_vcpus()

2020-11-20 Thread Marc-André Lureau
Hi On Fri, Nov 20, 2020 at 4:27 PM Lin Ma wrote: > The guest-get-vcpus returns incorrect vcpu info in case we hotunplug > vcpus(not > the last one). > e.g.: > A VM has 4 VCPUs: cpu0 + 3 hotunpluggable online vcpus(cpu1, cpu2 and > cpu3). > Hotunplug cpu2, Now only cpu0, cpu1 and cpu3 are

Re: [PATCH 00/10] vhost/qemu: thread per IO SCSI vq

2020-11-20 Thread Michael S. Tsirkin
On Fri, Nov 20, 2020 at 08:45:49AM +, Stefan Hajnoczi wrote: > On Thu, Nov 19, 2020 at 5:08 PM Stefan Hajnoczi wrote: > > > > On Thu, Nov 19, 2020 at 4:43 PM Mike Christie > > wrote: > > > > > > On 11/19/20 10:24 AM, Stefan Hajnoczi wrote: > > > > On Thu, Nov 19, 2020 at 4:13 PM Mike

Re: [PATCH-for-5.2 v2] hw/core/qdev-properties-system: Rewrite set_pci_host_devaddr using GLib

2020-11-20 Thread Michael S. Tsirkin
On Fri, Nov 20, 2020 at 12:00:56PM +0100, Philippe Mathieu-Daudé wrote: > On 11/17/20 12:16 PM, Philippe Mathieu-Daudé wrote: > > ping??? > > > > On 11/9/20 3:16 PM, Philippe Mathieu-Daudé wrote: > >> Cc'ing PCI developers (rc2 is scheduled for tomorrow). > >> > >> On 11/7/20 9:59 AM, Philippe

Re: [PATCH] qtest: do not return freed argument vector from qtest_rsp

2020-11-20 Thread Markus Armbruster
Paolo Bonzini writes: > If expected_args is 0, qtest frees the argument vector and then returns it > nevertheless. Coverity complains; in practice this is not an issue because > expected_args == 0 means that the caller is not interested in the argument > vector, but it would be a potential

Re: [PATCH RFC] vfio: Move the saving of the config space to the right place in VFIO migration

2020-11-20 Thread Shenming Lu
On 2020/11/20 1:41, Alex Williamson wrote: > On Thu, 19 Nov 2020 14:13:24 +0530 > Kirti Wankhede wrote: > >> On 11/14/2020 2:47 PM, Shenming Lu wrote: >>> When running VFIO migration, I found that the restoring of VFIO PCI device’s >>> config space is before VGIC on ARM64 target. But generally,

Re: [PATCH 2/2] pc-bios: s390x: Give precedence to reset PSW

2020-11-20 Thread Eric Farman
On 11/20/20 1:02 AM, Thomas Huth wrote: On 19/11/2020 22.11, Eric Farman wrote: On 11/19/20 3:20 PM, Thomas Huth wrote: On 19/11/2020 17.57, Eric Farman wrote: Let's look at the Reset PSW first instead of the contents of memory. It might be leftover from an earlier system boot when

[RFC v4 3/9] i386: move hax accel files into hax/

2020-11-20 Thread Claudio Fontana
Signed-off-by: Claudio Fontana --- MAINTAINERS | 2 +- target/i386/{ => hax}/hax-all.c | 0 target/i386/{ => hax}/hax-cpus.c | 0 target/i386/{ => hax}/hax-cpus.h | 0 target/i386/{ => hax}/hax-i386.h | 6 +++--- target/i386/{ =>

[RFC v4 6/9] i386: move cpu dump out of helper.c into cpu-dump.c

2020-11-20 Thread Claudio Fontana
Signed-off-by: Claudio Fontana --- target/i386/cpu-dump.c | 538 target/i386/cpu.h | 1 + target/i386/helper.c| 514 -- target/i386/meson.build | 1 + 4 files changed, 540 insertions(+), 514 deletions(-)

Re: [PATCH 23/29] vl: move -global check earlier

2020-11-20 Thread Igor Mammedov
On Tue, 27 Oct 2020 14:21:38 -0400 Paolo Bonzini wrote: > The check has no effect after both builtin and user-specified devices > have been created. > > Signed-off-by: Paolo Bonzini Reviewed-by: Igor Mammedov > --- > softmmu/vl.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) >

Re: [PATCH-for-5.2 2/4] docs/system/arm: Document the various raspi boards

2020-11-20 Thread Philippe Mathieu-Daudé
On 11/20/20 4:21 PM, Philippe Mathieu-Daudé wrote: > Document the following Raspberry Pi models: > > - raspi0 Raspberry Pi Zero (revision 1.2) > - raspi1ap Raspberry Pi A+ (revision 1.1) > - raspi2b Raspberry Pi 2B (revision 1.1) > - raspi3ap

[RFC PATCH-for-5.2 4/4] docs/system/arm: Document the Sharp Zaurus SL-6000

2020-11-20 Thread Philippe Mathieu-Daudé
List the 'tosa' machine with the XScale-based PDAs models. Signed-off-by: Philippe Mathieu-Daudé --- docs/system/arm/xscale.rst | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/docs/system/arm/xscale.rst b/docs/system/arm/xscale.rst index

Re: [PATCH v3 3/7] support UFFD write fault processing in ram_save_iterate()

2020-11-20 Thread Andrey Gruzdev
On 20.11.2020 18:07, Peter Xu wrote: On Fri, Nov 20, 2020 at 01:44:53PM +0300, Andrey Gruzdev wrote: On 19.11.2020 21:25, Peter Xu wrote: On Thu, Nov 19, 2020 at 03:59:36PM +0300, Andrey Gruzdev via wrote: [...] +/** + * ram_find_block_by_host_address: find RAM block containing host page +

[PATCH for-6.0 2/2] aspeed/smc: Add extra controls to request DMA

2020-11-20 Thread Cédric Le Goater
The controller has a set of hidden bits to request/grant DMA access. Cc: Chin-Ting Kuo Signed-off-by: Cédric Le Goater --- hw/ssi/aspeed_smc.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/hw/ssi/aspeed_smc.c b/hw/ssi/aspeed_smc.c index e3d5e26058c0..c1557ef5d848 100644 ---

[PATCH 1/5] abort-on-set-to-true

2020-11-20 Thread Vladimir Sementsov-Ogievskiy
Signed-off-by: Vladimir Sementsov-Ogievskiy --- block.c | 34 +- 1 file changed, 25 insertions(+), 9 deletions(-) diff --git a/block.c b/block.c index f1cedac362..5e8dd98cec 100644 --- a/block.c +++ b/block.c @@ -84,6 +84,8 @@ static BlockDriverState

[PATCH 2/5] iotest-30-shorten: concentrate on failing test case

2020-11-20 Thread Vladimir Sementsov-Ogievskiy
Signed-off-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/030 | 52 +++--- tests/qemu-iotests/030.out | 4 +-- 2 files changed, 28 insertions(+), 28 deletions(-) diff --git a/tests/qemu-iotests/030 b/tests/qemu-iotests/030 index

[PATCH 5/5] block: protect some graph-modifyng things by mutex

2020-11-20 Thread Vladimir Sementsov-Ogievskiy
Iotest 30 accidentally fails due to interleaving of mirror and stream graph-modifying procedures. Protect these things by global co-mutex. Signed-off-by: Vladimir Sementsov-Ogievskiy --- include/block/block.h | 2 ++ block.c | 2 ++ block/mirror.c| 4 block/stream.c

Re: [PATCH v2] usb: fix kconfig for usb-xhci-sysbus

2020-11-20 Thread Bruce Rogers
On Fri, 2020-11-20 at 10:45 -0500, Paolo Bonzini wrote: > Remove the "default y" for USB_XHCI_SYSBUS because > sysbus devices are not user creatable; boards that use them will > specify them manually with "imply" or "select" clauses. > > It would be nice to keep the ability to remove PCIe and USB

Re: [PATCH 28/29] vl: remove separate preconfig main_loop

2020-11-20 Thread Igor Mammedov
On Tue, 27 Oct 2020 14:21:43 -0400 Paolo Bonzini wrote: > Move post-preconfig initialization to the x-exit-preconfig. If preconfig > is not requested, just exit preconfig mode immediately with the QMP > command. > > As a result, the preconfig loop will run with accel_setup_post > and

Re: [PATCH v3 3/7] support UFFD write fault processing in ram_save_iterate()

2020-11-20 Thread Peter Xu
On Fri, Nov 20, 2020 at 07:15:07PM +0300, Andrey Gruzdev wrote: > Yeah, I think we can re-use the postcopy queue code for faulting pages. I'm > worring a little about some additional overhead dealing with urgent request > semaphore. Also, the code won't change a lot, something like: > > [...] >

Re: [RFC PATCH-for-5.2 4/4] docs/system/arm: Document the Sharp Zaurus SL-6000

2020-11-20 Thread Peter Maydell
On Fri, 20 Nov 2020 at 15:22, Philippe Mathieu-Daudé wrote: > > List the 'tosa' machine with the XScale-based PDAs models. > > Signed-off-by: Philippe Mathieu-Daudé > --- > docs/system/arm/xscale.rst | 13 - > 1 file changed, 8 insertions(+), 5 deletions(-) > > diff --git

Re: [PATCH-for-5.2 2/4] docs/system/arm: Document the various raspi boards

2020-11-20 Thread Philippe Mathieu-Daudé
On 11/20/20 5:48 PM, Peter Maydell wrote: > On Fri, 20 Nov 2020 at 15:21, Philippe Mathieu-Daudé wrote: >> >> Document the following Raspberry Pi models: >> >> - raspi0 Raspberry Pi Zero (revision 1.2) >> - raspi1ap Raspberry Pi A+ (revision 1.1) >> - raspi2b

Re: [PATCH 5/6] qapi: Add support for aliases

2020-11-20 Thread Peter Krempa
On Fri, Nov 13, 2020 at 10:46:02 +0100, Kevin Wolf wrote: > Am 12.11.2020 um 19:34 hat Eric Blake geschrieben: > > On 11/12/20 11:28 AM, Kevin Wolf wrote: > > > Introduce alias definitions for object types (structs and unions). This > > > allows using the same QAPI type and visitor for many syntax

[RFC v4 7/9] i386: move TCG cpu class initialization out of helper.c

2020-11-20 Thread Claudio Fontana
Signed-off-by: Claudio Fontana --- target/i386/cpu.c | 33 -- target/i386/cpu.h | 97 ++--- target/i386/helper-tcg.h | 112 ++ target/i386/helper.c | 23 --- target/i386/meson.build

[RFC v4 0/9] i386 cleanup

2020-11-20 Thread Claudio Fontana
Hi all, this is v4 of the i386 cleanup, with the most interesting patches at the end. v3 -> v4: QOMification of X86CPUAccelClass In this version I basically QOMified X86CPUAccel, taking the suggestions from Eduardo as the starting point, but stopping just short of making it an actual QOM

[RFC v4 5/9] i386: move TCG accel files into tcg/

2020-11-20 Thread Claudio Fontana
Signed-off-by: Claudio Fontana --- target/i386/meson.build | 14 +- target/i386/{ => tcg}/bpt_helper.c | 0 target/i386/{ => tcg}/cc_helper.c | 0 target/i386/{ => tcg}/excp_helper.c | 0 target/i386/{ => tcg}/fpu_helper.c | 0 target/i386/{ => tcg}/int_helper.c

[PATCH] sbsa-ref: allow to use Cortex-A53/57/72 cpus

2020-11-20 Thread Marcin Juszkiewicz
Trusted Firmware now supports A72 on sbsa-ref by default [1] so enable it for QEMU as well. A53 was already enabled there. 1. https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/7117 Signed-off-by: Marcin Juszkiewicz diff --git hw/arm/sbsa-ref.c hw/arm/sbsa-ref.c index

[PATCH-for-5.2 2/4] docs/system/arm: Document the various raspi boards

2020-11-20 Thread Philippe Mathieu-Daudé
Document the following Raspberry Pi models: - raspi0 Raspberry Pi Zero (revision 1.2) - raspi1ap Raspberry Pi A+ (revision 1.1) - raspi2b Raspberry Pi 2B (revision 1.1) - raspi3ap Raspberry Pi 3A+ (revision 1.0) - raspi3b

[PATCH v2] python, tests: do not use short-form boolean options

2020-11-20 Thread Paolo Bonzini
They are going to be deprecated, avoid warnings on stdout while the tests run. Signed-off-by: Paolo Bonzini --- python/qemu/machine.py | 2 +- tests/qtest/pflash-cfi02-test.c | 4 ++-- tests/qtest/test-filter-redirector.c | 8 tests/qtest/vhost-user-test.c| 8

[PATCH-for-5.2 1/6] MAINTAINERS: Cover system/arm/cpu-features.rst with ARM TCG CPUs

2020-11-20 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé --- Cc: Alex Bennée Cc: Richard Henderson --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 1ae9fc9649b..8d7a6026b91 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -158,6 +158,7 @@ F: disas/arm.c F:

[PATCH-for-5.2 4/6] MAINTAINERS: Fix system/arm/orangepi.rst path

2020-11-20 Thread Philippe Mathieu-Daudé
Fixes: 0553ef42571 ("docs: add Orange Pi PC document") Signed-off-by: Philippe Mathieu-Daudé --- Cc: Niek Linnenbank --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 2623cf85f66..311a55317ce 100644 --- a/MAINTAINERS +++

Re: [PATCH 26/29] hmp: introduce cmd_available

2020-11-20 Thread Igor Mammedov
On Tue, 27 Oct 2020 14:21:41 -0400 Paolo Bonzini wrote: > Combine the RUN_STATE_PRECONFIG and cmd_can_preconfig checks > into a single function, to avoid repeating the same expression > (or its negation after applying DeMorgan's rule) over and > over again. > > Signed-off-by: Paolo Bonzini

  1   2   3   >