[PATCH v3 11/26] migration: Add Error** argument to qemu_savevm_state_setup()

2024-03-04 Thread Cédric Le Goater
This prepares ground for the changes coming next which add an Error** argument to the .save_setup() handler. Callers of qemu_savevm_state_setup() now handle the error and fail earlier setting the migration state from MIGRATION_STATUS_SETUP to MIGRATION_STATUS_FAILED. Since the previous behavior

[PATCH v3 02/26] vfio: Always report an error in vfio_save_setup()

2024-03-04 Thread Cédric Le Goater
This will prepare ground for futur changes adding an Error** argument to the save_setup() handler. We need to make sure that on failure, vfio_save_setup() always sets a new error. Signed-off-by: Cédric Le Goater --- hw/vfio/migration.c | 15 --- 1 file changed, 12 insertions(+), 3

[PATCH v3 20/26] vfio: Use new Error** argument in vfio_save_setup()

2024-03-04 Thread Cédric Le Goater
Add an Error** argument to vfio_migration_set_state() and adjust callers, including vfio_save_setup(). The error will be propagated up to qemu_savevm_state_setup() where the save_setup() handler is executed. Modify vfio_vmstate_change_prepare() and vfio_vmstate_change() to store a reported error

[PATCH v3 19/26] vfio: Add Error** argument to vfio_devices_dma_logging_stop()

2024-03-04 Thread Cédric Le Goater
This improves error reporting in the log_global_stop() VFIO handler. Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Cédric Le Goater --- Changes in v3: - Use error_setg_errno() in vfio_devices_dma_logging_stop() hw/vfio/common.c | 19 ++- 1 file changed, 14

[PATCH v3 15/26] memory: Add Error** argument to the global_dirty_log routines

2024-03-04 Thread Cédric Le Goater
Now that the log_global*() handlers take an Error** parameter and return a bool, do the same for memory_global_dirty_log_start() and memory_global_dirty_log_stop(). The error is reported in the callers for now and it will be propagated in the call stack in the next changes. To be noted a

[PATCH v3 25/26] vfio: Also trace event failures in vfio_save_complete_precopy()

2024-03-04 Thread Cédric Le Goater
vfio_save_complete_precopy() currently returns before doing the trace event. Change that. Signed-off-by: Cédric Le Goater --- hw/vfio/migration.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/hw/vfio/migration.c b/hw/vfio/migration.c index

[PATCH v3 12/26] migration: Add Error** argument to .save_setup() handler

2024-03-04 Thread Cédric Le Goater
The purpose is to record a potential error in the migration stream if qemu_savevm_state_setup() fails. Most of the current .save_setup() handlers can be modified to use the Error argument instead of managing their own and calling locally error_report(). Cc: Nicholas Piggin Cc: Harsh Prateek Bora

[PATCH v3 17/26] vfio: Add Error** argument to .set_dirty_page_tracking() handler

2024-03-04 Thread Cédric Le Goater
We will use the Error object to improve error reporting in the .log_global*() handlers of VFIO. Add documentation while at it. Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Cédric Le Goater --- Changes in v3: - Use error_setg_errno() in vfio_legacy_set_dirty_page_tracking()

[PATCH v3 03/26] migration: Always report an error in block_save_setup()

2024-03-04 Thread Cédric Le Goater
This will prepare ground for futur changes adding an Error** argument to the save_setup() handler. We need to make sure that on failure, block_save_setup() always sets a new error. Cc: Stefan Hajnoczi Signed-off-by: Cédric Le Goater --- migration/block.c | 18 +- 1 file

[PATCH v3 08/26] migration: Add documentation for SaveVMHandlers

2024-03-04 Thread Cédric Le Goater
The SaveVMHandlers structure is still in use for complex subsystems and devices. Document the handlers since we are going to modify a few later. Reviewed-by: Peter Xu Signed-off-by: Cédric Le Goater --- Changes in v3: - VMState -> VMStateDescription - Rephrased

[PATCH v3 00/26] migration: Improve error reporting

2024-03-04 Thread Cédric Le Goater
Hello, The motivation behind these changes is to improve error reporting to the upper management layer (libvirt) with a more detailed error, this to let it decide, depending on the reported error, whether to try migration again later. It would be useful in cases where migration fails due to lack

[PATCH v3 06/26] migration: Report error when shutdown fails

2024-03-04 Thread Cédric Le Goater
This will help detect issues regarding I/O channels usage. Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Peter Xu Signed-off-by: Cédric Le Goater --- migration/qemu-file.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/migration/qemu-file.c b/migration/qemu-file.c

[PATCH v3 14/26] memory: Add Error** argument to .log_global*() handlers

2024-03-04 Thread Cédric Le Goater
Modify all log_global*() handlers to take an Error** parameter and return a bool. A new MEMORY_LISTENER_CALL_LOG_GLOBAL macro looping on the listeners is introduced to handle a possible error, which will would interrupt the loop if necessary. To be noted the change in

[PATCH v3 13/26] migration: Add Error** argument to .load_setup() handler

2024-03-04 Thread Cédric Le Goater
This will be useful to report errors at a higher level, mostly in VFIO today. Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Peter Xu Signed-off-by: Cédric Le Goater --- Changes in v3: - ERRP_GUARD() because of error_prepend use - Made sure an error is always set in case of failure in

[PATCH v3 04/26] migration: Always report an error in ram_save_setup()

2024-03-04 Thread Cédric Le Goater
This will prepare ground for futur changes adding an Error** argument to the save_setup() handler. We need to make sure that on failure, ram_save_setup() sets a new error. Signed-off-by: Cédric Le Goater --- migration/ram.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff

[PATCH v3 07/26] migration: Remove SaveStateHandler and LoadStateHandler typedefs

2024-03-04 Thread Cédric Le Goater
They are only used once. Reviewed-by: Fabiano Rosas Reviewed-by: Peter Xu Signed-off-by: Cédric Le Goater --- include/migration/register.h | 4 ++-- include/qemu/typedefs.h | 2 -- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/include/migration/register.h

[PATCH v3 10/26] migration: Move cleanup after after error reporting in qemu_savevm_state_setup()

2024-03-04 Thread Cédric Le Goater
This will help preserving the error set by .save_setup() handlers. Signed-off-by: Cédric Le Goater --- migration/savevm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/migration/savevm.c b/migration/savevm.c index

[PATCH v3 09/26] migration: Do not call PRECOPY_NOTIFY_SETUP notifiers in case of error

2024-03-04 Thread Cédric Le Goater
When commit bd2270608fa0 ("migration/ram.c: add a notifier chain for precopy") added PRECOPY_NOTIFY_SETUP notifiers at the end of qemu_savevm_state_setup(), it didn't take into account a possible error in the loop calling vmstate_save() or .save_setup() handlers. Check ret value before calling

[PATCH v3 01/26] s390/stattrib: Add Error** argument to set_migrationmode() handler

2024-03-04 Thread Cédric Le Goater
This will prepare ground for futur changes adding an Error** argument to the save_setup() handler. We need to make sure that on failure, set_migrationmode() always sets a new error. See the Rules section in qapi/error.h. Cc: Halil Pasic Cc: Christian Borntraeger Cc: Thomas Huth Signed-off-by:

Re: [PATCH v2 00/10] mirror: allow switching from background to active mode

2024-03-04 Thread Markus Armbruster
Kevin Wolf writes: > Am 28.02.2024 um 19:07 hat Vladimir Sementsov-Ogievskiy geschrieben: [...] >> About the APIs, I think, of course we should deprecate block-job-* API, >> because we already have jobs which are not block-jobs, so we can't deprecate >> job-* API. >> >> So I suggest a plan:

Re: [RFC PATCH v5 18/22] hw/intc/arm_gicv3: Implement NMI interrupt prioirty

2024-03-04 Thread Jinjie Ruan via
On 2024/3/1 7:50, Richard Henderson wrote: > On 2/29/24 03:10, Jinjie Ruan via wrote: >> If GICD_CTLR_DS bit is zero and the NMI is non-secure, the NMI prioirty >> is higher than 0x80, otherwise it is higher than 0x0. And save NMI >> super prioirty information in hppi.superprio to deliver NMI

Re: [PATCH v3 0/5] Generate x86 cpu features

2024-03-04 Thread Tim Wiederhake
ping On Tue, 2024-02-06 at 14:47 +0100, Tim Wiederhake wrote: > Synchronizing the list of cpu features and models with qemu is a > recurring > task in libvirt. For x86, this is done by reading qom-list-properties > for > max-x86_64-cpu and manually filtering out everthing that does not > look

Re: [PATCH v2 0/7] qga/commands-posix: replace code duplicating commands with a helper

2024-03-04 Thread Konstantin Kostiuk
For series Reviewed-by: Konstantin Kostiuk On Fri, Mar 1, 2024 at 7:29 PM Andrey Drobyshev < andrey.drobys...@virtuozzo.com> wrote: > v1 --> v2: > * Replace commit for guest-get-fsinfo: instead of reporting statvfs(3) >values directly, keep the old ones but add an additional optional >

Re: [PATCH] docs/conf.py: Remove usage of distutils

2024-03-04 Thread Thomas Huth
On 04/03/2024 12.18, Peter Maydell wrote: On Mon, 4 Mar 2024 at 11:10, Thomas Huth wrote: Using my shell script to query repology, I get: centos_stream_8: 0.3.1 centos_stream_9: 0.5.1 fedora_37: 1.0.0 fedora_38: 1.1.1 fedora_39: 1.2.2 fedora_rawhide: 2.0.0 freebsd: 1.0.0 haikuports_master:

Re: [PATCH] docs/conf.py: Remove usage of distutils

2024-03-04 Thread Peter Maydell
On Mon, 4 Mar 2024 at 11:10, Thomas Huth wrote: > Using my shell script to query repology, I get: > > centos_stream_8: 0.3.1 > centos_stream_9: 0.5.1 > fedora_37: 1.0.0 > fedora_38: 1.1.1 > fedora_39: 1.2.2 > fedora_rawhide: 2.0.0 > freebsd: 1.0.0 > haikuports_master: 1.2.1 > openbsd: 1.2.2 >

Re: [PATCH 10/19] smbios: handle errors consistently

2024-03-04 Thread Ani Sinha
On Tue, 27 Feb 2024, Igor Mammedov wrote: > Current code uses mix of error_report()+exit(1) > and error_setg() to handle errors. > Use newer error_setg() everywhere, beside consistency > it will allow to detect error condition without killing > QEMU and attempt switch-over to SMBIOS3.x

Re: [PATCH v2 00/10] mirror: allow switching from background to active mode

2024-03-04 Thread Peter Krempa
On Mon, Mar 04, 2024 at 11:48:54 +0100, Kevin Wolf wrote: > Am 28.02.2024 um 19:07 hat Vladimir Sementsov-Ogievskiy geschrieben: > > On 03.11.23 18:56, Markus Armbruster wrote: > > > Kevin Wolf writes: [...] > > > Is the job abstraction a failure? > > > > > > We have > > > > > >

Re: [PATCH] docs/conf.py: Remove usage of distutils

2024-03-04 Thread Thomas Huth
On 04/03/2024 11.56, Peter Maydell wrote: On Mon, 4 Mar 2024 at 09:36, Thomas Huth wrote: The macOS jobs in our CI recently started failing, complaining that the distutils module is not available anymore. And indeed, according to https://peps.python.org/pep-0632/ it's been deprecated since a

Re: [RISC-V][tech-server-platform] [RFC 0/2] Add RISC-V Server Platform Reference Board

2024-03-04 Thread Heinrich Schuchardt
On 04.03.24 11:25, Wu, Fei2 wrote: The RISC-V Server Platform specification[1] defines a standardized set of hardware and software capabilities, that portable system software, such as OS and hypervisors can rely on being present in a RISC-V server platform. This patchset provides a RISC-V Server

Re: [PATCH] docs/conf.py: Remove usage of distutils

2024-03-04 Thread Michael Tokarev
04.03.2024 12:36, Thomas Huth wrote: The macOS jobs in our CI recently started failing, complaining that the distutils module is not available anymore. And indeed, according to https://peps.python.org/pep-0632/ it's been deprecated since a while and now likely got removed in recent Python

Re: [PATCH] docs/conf.py: Remove usage of distutils

2024-03-04 Thread Peter Maydell
On Mon, 4 Mar 2024 at 09:36, Thomas Huth wrote: > > The macOS jobs in our CI recently started failing, complaining that > the distutils module is not available anymore. And indeed, according to > https://peps.python.org/pep-0632/ it's been deprecated since a while > and now likely got removed in

Re: [PATCH 09/19] smbios: build legacy mode code only for 'pc' machine

2024-03-04 Thread Ani Sinha
> On 27-Feb-2024, at 21:17, Igor Mammedov wrote: > > basically moving code around without functional change. > And exposing some symbols so that they could be shared > between smbbios.c and new smbios_legacy.c > > plus some meson magic to build smbios_legacy.c only > for 'pc' machine and

[PATCH v2 1/3] migration: Don't serialize devices in qemu_savevm_state_iterate()

2024-03-04 Thread Avihai Horon
Commit 90697be8896c ("live migration: Serialize vmstate saving in stage 2") introduced device serialization in qemu_savevm_state_iterate(). The rationale behind it was to first complete migration of slower changing block devices and only then migrate the RAM, to avoid sending fast changing RAM

[PATCH v2 3/3] vfio/migration: Add a note about migration rate limiting

2024-03-04 Thread Avihai Horon
VFIO migration buffer size is currently limited to 1MB. Therefore, there is no need to check if migration rate exceeded, as in the worst case it will exceed by only 1MB. However, if the buffer size is later changed to a bigger value, vfio_save_iterate() should enforce migration rate (similar to

[PATCH v2 0/3] migration: Don't serialize devices in qemu_savevm_state_iterate()

2024-03-04 Thread Avihai Horon
Hi, This small series is v2 of the single patch I previously sent [1]. It removes device serialization in qemu_savevm_state_iterate() and does some VFIO migration touch ups. More info provided in the commit messages. Thanks. Changes from V1 -> V2: * Remove device serialization in

[PATCH v2 2/3] vfio/migration: Refactor vfio_save_state() return value

2024-03-04 Thread Avihai Horon
Currently, vfio_save_state() returns 1 regardless of whether there is more data to send or not. This was done to prevent a fast changing VFIO device from potentially blocking other devices from sending their data, as qemu_savevm_state_iterate() serialized devices. Now that

Re: [PATCH v2 00/10] mirror: allow switching from background to active mode

2024-03-04 Thread Kevin Wolf
Am 28.02.2024 um 19:07 hat Vladimir Sementsov-Ogievskiy geschrieben: > On 03.11.23 18:56, Markus Armbruster wrote: > > Kevin Wolf writes: > > > > > Am 03.11.2023 um 10:36 hat Markus Armbruster geschrieben: > > > > Vladimir Sementsov-Ogievskiy writes: > > > > > > > > > On 11.10.23 13:18, Fiona

[PATCH 1/3] hw/cxl/cxl-cdat: Fix type of buf in ct3_load_cdat()

2024-03-04 Thread Thomas Huth
When setting GLIB_VERSION_MAX_ALLOWED to GLIB_VERSION_2_58 or higher (which we'll certainly do in the not too distant future), glib adds type safety checks to the g_steal_pointer() macro. This trigger an error in the ct3_load_cdat() function: The local char *buf variable is assigned to uint8_t

[PATCH 0/3] cxl: Fix issues with g_steal_pointer()

2024-03-04 Thread Thomas Huth
When setting GLIB_VERSION_MAX_ALLOWED to GLIB_VERSION_2_58 or higher (which we'll certainly do in the not too distant future), glib adds type safety checks to the g_steal_pointer() macro. This triggers errors in the cxl code since the pointer types do not always match here. Let's fix those errors

[PATCH 3/3] hw/mem/cxl_type3: Fix problem with g_steal_pointer()

2024-03-04 Thread Thomas Huth
When setting GLIB_VERSION_MAX_ALLOWED to GLIB_VERSION_2_58 or higher, glib adds type safety checks to the g_steal_pointer() macro. This triggers errors in the ct3_build_cdat_entries_for_mr() function which uses the g_steal_pointer() for type-casting from one pointer type to the other (which also

[PATCH 2/3] hw/pci-bridge/cxl_upstream: Fix problem with g_steal_pointer()

2024-03-04 Thread Thomas Huth
When setting GLIB_VERSION_MAX_ALLOWED to GLIB_VERSION_2_58 or higher, glib adds type safety checks to the g_steal_pointer() macro. This triggers errors in the build_cdat_table() function which uses the g_steal_pointer() for type-casting from one pointer type to the other (which also looks quite

Re: [PATCH] meson: Remove --warn-common ldflag

2024-03-04 Thread Peter Maydell
On Sun, 3 Mar 2024 at 16:26, Akihiko Odaki wrote: > > --warn-common ldflag causes warnings for multiple definitions of > ___asan_globals_registered when enabling AddressSanitizer with clang. > The warning is somewhat obsolete so just remove it. > > The common block is used to allow duplicate

Re: [PATCH v3 01/12] util/log: convert debug_regions to GList

2024-03-04 Thread Alex Bennée
Sven Schnelle writes: > In preparation of making the parsing part of qemu_set_dfilter_ranges() > available to other users, convert it to return a GList, so the result > can be used with other functions taking a GList of struct Range. Why do we need to convert it to a Glist? When I originally

Re: [PATCH v2 03/21] migration: Add documentation for SaveVMHandlers

2024-03-04 Thread Cédric Le Goater
On 3/4/24 10:05, Avihai Horon wrote: Hi Cedric, A few nits below. Just in time for v3 I was about to send ! I will include these suggestions. Thanks, C. On 27/02/2024 20:03, Cédric Le Goater wrote: External email: Use caution opening links or attachments The SaveVMHandlers

[RFC 2/2] target/riscv: Add server platform reference cpu

2024-03-04 Thread Fei Wu
The harts requirements of RISC-V server platform [1] require RVA23 ISA profile support, plus Sv48, Svadu, H, Sscofmpf etc. This patch provides a virt CPU type (rvsp-ref) as compliant as possible. [1]

[RFC 1/2] hw/riscv: Add server platform reference machine

2024-03-04 Thread Fei Wu
The RISC-V Server Platform specification[1] defines a standardized set of hardware and software capabilities, that portable system software, such as OS and hypervisors can rely on being present in a RISC-V server platform. A corresponding Qemu RISC-V server platform reference (rvsp-ref for short)

[RFC 0/2] Add RISC-V Server Platform Reference Board

2024-03-04 Thread Fei Wu
The RISC-V Server Platform specification[1] defines a standardized set of hardware and software capabilities, that portable system software, such as OS and hypervisors can rely on being present in a RISC-V server platform. This patchset provides a RISC-V Server Platform (RVSP) reference

Re: [PATCH v8 0/5] Support message-based DMA in vfio-user server

2024-03-04 Thread Peter Xu
On Mon, Mar 04, 2024 at 02:05:49AM -0800, Mattias Nissler wrote: > This series adds basic support for message-based DMA in qemu's vfio-user > server. This is useful for cases where the client does not provide file > descriptors for accessing system memory via memory mappings. My motivating use >

RE: [PATCH v1 6/6] intel_iommu: Block migration if cap is updated

2024-03-04 Thread Duan, Zhenzhong
>-Original Message- >From: Prasad Pandit >Subject: Re: [PATCH v1 6/6] intel_iommu: Block migration if cap is updated > >On Mon, 4 Mar 2024 at 13:41, Duan, Zhenzhong > wrote: >> This is to deal with a special case when cold plugged vfio device is >unplugged >> at runtime, then migration

[PATCH v8 1/5] softmmu: Per-AddressSpace bounce buffering

2024-03-04 Thread Mattias Nissler
Instead of using a single global bounce buffer, give each AddressSpace its own bounce buffer. The MapClient callback mechanism moves to AddressSpace accordingly. This is in preparation for generalizing bounce buffer handling further to allow multiple bounce buffers, with a total allocation limit

[PATCH v8 2/5] softmmu: Support concurrent bounce buffers

2024-03-04 Thread Mattias Nissler
When DMA memory can't be directly accessed, as is the case when running the device model in a separate process without shareable DMA file descriptors, bounce buffering is used. It is not uncommon for device models to request mapping of several DMA regions at the same time. Examples include: *

[PATCH v8 4/5] vfio-user: Message-based DMA support

2024-03-04 Thread Mattias Nissler
Wire up support for DMA for the case where the vfio-user client does not provide mmap()-able file descriptors, but DMA requests must be performed via the VFIO-user protocol. This installs an indirect memory region, which already works for pci_dma_{read,write}, and pci_dma_map works thanks to the

[PATCH v8 3/5] Update subprojects/libvfio-user

2024-03-04 Thread Mattias Nissler
Brings in assorted bug fixes. The following are of particular interest with respect to message-based DMA support: * bb308a2 "Fix address calculation for message-based DMA" Corrects a bug in DMA address calculation. * 1569a37 "Pass server->client command over a separate socket pair" Adds

[PATCH v8 5/5] vfio-user: Fix config space access byte order

2024-03-04 Thread Mattias Nissler
PCI config space is little-endian, so on a big-endian host we need to perform byte swaps for values as they are passed to and received from the generic PCI config space access machinery. Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Stefan Hajnoczi Reviewed-by: Jagannathan Raman

[PATCH v8 0/5] Support message-based DMA in vfio-user server

2024-03-04 Thread Mattias Nissler
This series adds basic support for message-based DMA in qemu's vfio-user server. This is useful for cases where the client does not provide file descriptors for accessing system memory via memory mappings. My motivating use case is to hook up device models as PCIe endpoints to a hardware design.

Re: [PATCH] docs/conf.py: Remove usage of distutils

2024-03-04 Thread Daniel P . Berrangé
On Mon, Mar 04, 2024 at 10:36:18AM +0100, Thomas Huth wrote: > The macOS jobs in our CI recently started failing, complaining that > the distutils module is not available anymore. And indeed, according to > https://peps.python.org/pep-0632/ it's been deprecated since a while > and now likely got

Re: [PULL 5/5] roms/hppa: Add build rules for hppa-firmware

2024-03-04 Thread Michael Tokarev
03.03.2024 08:46, del...@kernel.org : From: Helge Deller Signed-off-by: Helge Deller Suggested-by: Michael Tokarev Reviewed-by: Richard Henderson Reviewed-by: Michael Tokarev Thank you! /mjt roms/Makefile | 7 +++ 1 file changed, 7 insertions(+) diff --git a/roms/Makefile

RE: [PATCH v1 8/8] aspeed: Add an AST2700 eval board

2024-03-04 Thread Jamin Lin
> -Original Message- > From: Cédric Le Goater > Sent: Thursday, February 29, 2024 4:32 PM > To: Jamin Lin ; peter.mayd...@linaro.org; > and...@codeconstruct.com.au; j...@jms.id.au; qemu-...@nongnu.org; > qemu-devel@nongnu.org > Cc: Troy Lee ; Yunlin Tang > > Subject: Re: [PATCH v1 8/8]

Re: [PATCH 07/19] smbios: avoid mangling user provided tables

2024-03-04 Thread Ani Sinha
On Tue, 27 Feb 2024, Igor Mammedov wrote: > currently smbios_entry_add() preserves internally '-smbios type=' > options but tables provided with '-smbios file=' are stored directly > into blob that eventually will be exposed to VM. And then later > QEMU adds default/'-smbios type' entries on

Re: [PATCH v1 6/6] intel_iommu: Block migration if cap is updated

2024-03-04 Thread Prasad Pandit
On Mon, 4 Mar 2024 at 13:41, Duan, Zhenzhong wrote: > This is to deal with a special case when cold plugged vfio device is unplugged > at runtime, then migration triggers. > > When qemu on source side starts with cold plugged vfio device, vIOMMU qemu -> QEMU > capability/extended capability

Re: QNX VM hang on Qemu

2024-03-04 Thread Clément Chigot
On Fri, Mar 1, 2024 at 6:48 PM Faiq Ali Sayed wrote: > > Hi Clément, > > So the output of the command > | $ qemu-system-aarch64 -M xlnx-zcu102 -m 4G -no-reboot -nographic > -kernel qnx.img > is > > $ pulseaudio: set_sink_input_volume() failed > $ pulseaudio: Reason: Invalid argument > $

[PATCH] docs/conf.py: Remove usage of distutils

2024-03-04 Thread Thomas Huth
The macOS jobs in our CI recently started failing, complaining that the distutils module is not available anymore. And indeed, according to https://peps.python.org/pep-0632/ it's been deprecated since a while and now likely got removed in recent Python versions. Fortunately, we only use it for a

[PATCH v2 6/9] aspeed/intc: Add AST2700 support

2024-03-04 Thread Jamin Lin via
AST2700 interrupt controller(INTC) provides hardware interrupt interfaces to interrupt of processors PSP, SSP and TSP. In INTC, each interrupt of INT 128 to INT136 combines 32 interrupts. Introduce a new aspeed_intc class with instance_init and realize handlers. QEMU supports ARM Generic

[PATCH v2 4/9] aspeed/smc: Add AST2700 support

2024-03-04 Thread Jamin Lin via
AST2700 fmc/spi controller's address decoding unit is 64KB and only bits [31:16] are used for decoding. Introduce seg_to_reg and reg_to_seg handlers for ast2700 fmc/spi controller. In addition, adds ast2700 fmc, spi0, spi1, and spi2 class init handler. AST2700 support the maximum dram size is

[PATCH v2 0/9] Add AST2700 support

2024-03-04 Thread Jamin Lin via
Changes from v1: The patch series supports WDT, SDMC, SMC, SCU, SLI and INTC for AST2700 SoC. Changes from v2: - replace is_aarch64 with is_bus64bit for sdmc patch review. - fix incorrect dram size for AST2700 Test steps: 1. Download openbmc image for AST2700 from

[PATCH v2 3/9] aspeed/sdmc: Add AST2700 support

2024-03-04 Thread Jamin Lin via
The SDRAM memory controller(DRAMC) controls the access to external DDR4 and DDR5 SDRAM and power up to DDR4 and DDR5 PHY. The DRAM memory controller of AST2700 is not backward compatible to previous chips such AST2600, AST2500 and AST2400. Max memory is now 8GiB on the AST2700. Introduce new

[PATCH v2 5/9] aspeed/scu: Add AST2700 support

2024-03-04 Thread Jamin Lin via
AST2700 have two SCU controllers which are SCU and SCUIO. Both SCU and SCUIO registers are not compatible previous SOCs , introduces new registers and adds ast2700 scu, sucio class init handler. The pclk divider selection of SCUIO is defined in SCUIO280[20:18] and the pclk divider selection of

[PATCH v1 6/9] aspeed/intc: Add AST2700 support

2024-03-04 Thread Jamin Lin via
AST2700 interrupt controller(INTC) provides hardware interrupt interfaces to interrupt of processors PSP, SSP and TSP. In INTC, each interrupt of INT 128 to INT136 combines 32 interrupts. Introduce a new aspeed_intc class with instance_init and realize handlers. QEMU supports ARM Generic

[PATCH v1 9/9] aspeed/soc: fix incorrect dram size for AST2700

2024-03-04 Thread Jamin Lin via
AST2700 dram size calculation is not back compatible AST2600. According to the DDR capacity hardware behavior, if users write the data to address which is beyond the ram size, it would write the data to address 0. For example: a. sdram base address "0x4 " b. sdram size is 1 GiB The

[PATCH v1 0/9] Add AST2700 support

2024-03-04 Thread Jamin Lin via
Changes from v1: The patch series supports WDT, SDMC, SMC, SCU, SLI and INTC for AST2700 SoC. Test steps: 1. Download openbmc image for AST2700 from https://github.com/AspeedTech-BMC/openbmc/releases/tag/v09.00 https://github.com/AspeedTech-BMC/openbmc/releases/download/v09.00/

[PATCH v1 2/9] aspeed/sli: Add AST2700 support

2024-03-04 Thread Jamin Lin via
AST2700 SLI engine is designed to accelerate the throughput between cross-die connections. It have CPU_SLI at CPU die and IO_SLI at IO die. Introduce new ast2700_sli and ast2700_sliio class with instance_init and realize handlers. Signed-off-by: Troy Lee Signed-off-by: Jamin Lin ---

[PATCH v1 1/9] aspeed/wdt: Add AST2700 support

2024-03-04 Thread Jamin Lin via
AST2700 wdt controller is similiar to AST2600's wdt, but the AST2700 has 8 watchdogs, and they each have a 0x80 of registers. Introduce ast2700 object class and increse the number of regs(offset) of ast2700 model. Signed-off-by: Troy Lee Signed-off-by: Jamin Lin --- hw/watchdog/wdt_aspeed.c

[PATCH v1 3/9] aspeed/sdmc: Add AST2700 support

2024-03-04 Thread Jamin Lin via
The SDRAM memory controller(DRAMC) controls the access to external DDR4 and DDR5 SDRAM and power up to DDR4 and DDR5 PHY. The DRAM memory controller of AST2700 is not backward compatible to previous chips such AST2600, AST2500 and AST2400. Max memory is now 8GiB on the AST2700. Introduce new

Re: [PATCH v2 03/21] migration: Add documentation for SaveVMHandlers

2024-03-04 Thread Avihai Horon
Hi Cedric, A few nits below. On 27/02/2024 20:03, Cédric Le Goater wrote: External email: Use caution opening links or attachments The SaveVMHandlers structure is still in use for complex subsystems and devices. Document the handlers since we are going to modify a few later. Signed-off-by:

Re: [RFC 0/4] mirror: implement incremental and bitmap modes

2024-03-04 Thread Fabian Grünbichler
On February 29, 2024 11:41 am, Fiona Ebner wrote: > Am 28.02.24 um 17:24 schrieb Vladimir Sementsov-Ogievskiy: >> On 16.02.24 13:55, Fiona Ebner wrote: >>> Previous discussion from when this was sent upstream [0] (it's been a >>> while). I rebased the patches and re-ordered and squashed like >>>

Re: [PATCH v2 3/5] hw/loongarch: Add compat machine for 9.0

2024-03-04 Thread gaosong
在 2024/2/27 10:30, Bibo Mao 写道: Since migration test case requires compat machine type support, compat machine is added for qemu 9.0 here. Signed-off-by: Bibo Mao --- hw/loongarch/virt.c | 60 +++-- 1 file changed, 47 insertions(+), 13 deletions(-)

Re: [PATCH v2 4/5] hw/loongarch: Set minimium memory size as 256M

2024-03-04 Thread gaosong
在 2024/2/27 10:30, Bibo Mao 写道: The minimum memory size for LoongArch UEFI bios is 256M, also some test cases such as migration and qos use 256M memory by default. Here set minimum memory size for Loongarch VirtMachine with 256M rather than 1G, so that test cases with 256M memory can pass to

Re: [PATCH 3/5] hw/loongarch: Add compat machine for 9.0

2024-03-04 Thread gaosong
在 2024/2/20 20:41, Bibo Mao 写道: Since migration test case requires compat machine type support, compat machine is added for qemu 9.0 here. Signed-off-by: Bibo Mao --- hw/loongarch/virt.c | 60 +++-- 1 file changed, 47 insertions(+), 13 deletions(-)

Re: [PATCH v2] target/loongarch: Add TCG macro in structure CPUArchState

2024-03-04 Thread gaosong
在 2024/3/4 10:18, Bibo Mao 写道: In structure CPUArchState some struct elements are only used in TCG mode, and it is not used in KVM mode. Macro CONFIG_TCG is added to make it simpiler in KVM mode, also there is the same modification in c code when these struct elements are used. When VM runs

Re: [PATCH] Fixed tlb huge page loading issue

2024-03-04 Thread gaosong
Hi, Title 'target/loongarch: ' ... Thanks. Song Gao 在 2024/2/28 14:55, Xianglai Li 写道: The lddir and ldpte instruction emulation has a problem with the use of large page processing above level 2. The page size is not correctly calculated, resulting in the wrong page size of the table entry

Re: [PATCH 09/13] qga/qapi-schema: Delete useless "Returns" sections

2024-03-04 Thread Konstantin Kostiuk
Reviewed-by: Konstantin Kostiuk On Tue, Feb 27, 2024 at 1:39 PM Markus Armbruster wrote: > Signed-off-by: Markus Armbruster > --- > qga/qapi-schema.json | 12 > 1 file changed, 12 deletions(-) > > diff --git a/qga/qapi-schema.json b/qga/qapi-schema.json > index

Re: [PATCH 10/13] qga/qapi-schema: Clean up "Returns" sections

2024-03-04 Thread Konstantin Kostiuk
Reviewed-by: Konstantin Kostiuk On Tue, Feb 27, 2024 at 1:39 PM Markus Armbruster wrote: > Drop "on success" where it is redundant with "Returns:". > > Signed-off-by: Markus Armbruster > --- > qga/qapi-schema.json | 16 > 1 file changed, 8 insertions(+), 8 deletions(-) > >

Re: [PATCH 08/13] qga/qapi-schema: Move error documentation to new "Errors" sections

2024-03-04 Thread Konstantin Kostiuk
Reviewed-by: Konstantin Kostiuk On Tue, Feb 27, 2024 at 1:39 PM Markus Armbruster wrote: > Signed-off-by: Markus Armbruster > --- > qga/qapi-schema.json | 22 ++ > 1 file changed, 10 insertions(+), 12 deletions(-) > > diff --git a/qga/qapi-schema.json

Re: [PATCH] hw/core/machine-smp: Remove deprecated "parameter=0" SMP configurations

2024-03-04 Thread Prasad Pandit
Hello Zhao, On Mon, 4 Mar 2024 at 12:19, Zhao Liu wrote: > > unsigned maxcpus = config->has_maxcpus ? config->maxcpus : 0; > > This indicates the default maxcpus is initialized as 0 if user doesn't > specifies it. * 'has_maxcpus' should be set only if maxcpus > 0. If maxcpus == 0, then setting

Re: [PATCH 11/13] qga/qapi-schema: Tweak documentation of fsfreeze commands

2024-03-04 Thread Konstantin Kostiuk
Reviewed-by: Konstantin Kostiuk On Tue, Feb 27, 2024 at 1:39 PM Markus Armbruster wrote: > "Returns:" sections of guest-fsfreeze-freeze and > guest-fsfreeze-freeze-list describe both command behavior and success > response. Move behavior out, so "Returns:" is only about success > response. >

Re: [PATCH 12/13] qga/qapi-schema: Fix guest-set-memory-blocks documentation

2024-03-04 Thread Konstantin Kostiuk
Reviewed-by: Konstantin Kostiuk On Tue, Feb 27, 2024 at 1:39 PM Markus Armbruster wrote: > Documentation claims the command can "return NULL". "NULL" doesn't > exist in JSON. "null" does, but the command returns lists, and null > isn't. Correct documentation to "return an empty list". > >

RE: [PATCH v1 6/6] intel_iommu: Block migration if cap is updated

2024-03-04 Thread Duan, Zhenzhong
>-Original Message- >From: Prasad Pandit >Subject: Re: [PATCH v1 6/6] intel_iommu: Block migration if cap is updated > >On Wed, 28 Feb 2024 at 15:17, Zhenzhong Duan > wrote: >> When there is VFIO device and vIOMMU cap/ecap is updated based on >host > >* cap/ecap -> capability/extended

Re: Does "-object" support structured options now?

2024-03-04 Thread Daniel P . Berrangé
On Mon, Mar 04, 2024 at 06:43:19AM +, Chun Feng Wu wrote: > Hi, > > I noticed that throttle-group can be created with “-object”, however, per > qemu doc(https://github.com/qemu/qemu/blob/master/docs/throttle.txt), > “-object” doesn’t support structured options at that moment: > > “ > A

<    1   2   3   4