[PATCH v5 00/29] Support AST2700 A1

2025-03-07 Thread Jamin Lin via
v1: 1. Refactor INTC model to support both INTC0 and INTC1. 2. Support AST2700 A1. 3. Create ast2700a0-evb machine. v2: To streamline the review process, split the following patch series into three parts. https://patchwork.kernel.org/project/qemu-devel/cover/20250121070424.2465942-1-jam

Re: [PATCH v5 26/29] tests/functional/aspeed: Update temperature hwmon path

2025-03-07 Thread Cédric Le Goater
On 3/6/25 11:38, Jamin Lin wrote: Modified the temperature hwmon path to use a wildcard to handle different SDK versions: "cat /sys/bus/i2c/devices/1-004d/hwmon/hwmon*/temp1_input". Signed-off-by: Jamin Lin Reviewed-by: Cédric Le Goater Thanks, C. --- tests/functional/test_aarch64_as

Re: [PATCH 56/57] docs/qapidoc: add intermediate output debugger

2025-03-07 Thread Markus Armbruster
John Snow writes: > Add debugging output for the qapidoc transmogrifier - setting DEBUG=1 > will produce .ir files (one for each qapidoc directive) that write the > generated rst file to disk to allow for easy debugging and verification > of the generated document. > > Signed-off-by: John Snow

Re: [PATCH] doc: add missing 'Asset' type in function test doc

2025-03-07 Thread Thomas Huth
On 06/03/2025 07.44, Philippe Mathieu-Daudé wrote: On 6/3/25 07:07, Aditya Gupta wrote: Seems 'Asset' got missed in the documentation by mistake. Also fix the one spellcheck issue pointed by spellcheck Signed-off-by: Aditya Gupta ---   docs/devel/testing/functional.rst | 6 --   1 file cha

Re: New git repository for Go bindings

2025-03-07 Thread Daniel P . Berrangé
On Fri, Mar 07, 2025 at 01:30:38PM +0100, Victor Toso wrote: > Hi, > > I've been working on and off in having Go bindings for QEMU's > QAPI specification. The last version [0] seems to be accepted so > far but we would like not to have that in qemu.git. > > In the past [1], Daniel suggested creat

Re: [PATCH v2 0/5] ppc/amigaone patches

2025-03-07 Thread BALATON Zoltan
On Thu, 27 Feb 2025, BALATON Zoltan wrote: Hello, v2: - change unused read function to g_assert_not_reached() - new patch to add defines to constants - added R-b tags Ping? Will there be a pull request for soft freeze with this series included or do I need to do anything with this? Regards,

[PATCH v3 2/3] bsd-user: Propagate alignment argument to mmap_find_vma()

2025-03-07 Thread Philippe Mathieu-Daudé
Propagate the alignment to mmap_find_vma(), effectively embedding mmap_find_vma_aligned() within mmap_find_vma(). Since we ignore the alignment in do_bsd_shmat(), leave a FIXME comment. Signed-off-by: Philippe Mathieu-Daudé --- bsd-user/bsd-mem.h | 2 +- bsd-user/qemu.h| 2 +- bsd-user/mm

Re: [PATCH v3 2/3] bsd-user: Propagate alignment argument to mmap_find_vma()

2025-03-07 Thread Richard Henderson
On 3/7/25 05:09, Philippe Mathieu-Daudé wrote: Propagate the alignment to mmap_find_vma(), effectively embedding mmap_find_vma_aligned() within mmap_find_vma(). Since we ignore the alignment in do_bsd_shmat(), leave a FIXME comment. Signed-off-by: Philippe Mathieu-Daudé --- bsd-user/bsd-mem.

Re: [PATCH 3/3] hw/riscv/riscv_iommu: Remove the "bus" property

2025-03-07 Thread Daniel Henrique Barboza
On 3/2/25 6:12 AM, Jason Chien wrote: This property was originally intended to set the bus number for non-root endpoints. However, since the PCIe bus number is assigned and modified at runtime, setting this property before software execution is incorrect. Additionally, the property incorrectly

Re: [PATCH v3 2/3] bsd-user: Propagate alignment argument to mmap_find_vma()

2025-03-07 Thread Philippe Mathieu-Daudé
On 7/3/25 17:08, Richard Henderson wrote: On 3/7/25 05:09, Philippe Mathieu-Daudé wrote: Propagate the alignment to mmap_find_vma(), effectively embedding mmap_find_vma_aligned() within mmap_find_vma(). Since we ignore the alignment in do_bsd_shmat(), leave a FIXME comment. Signed-off-by: Phil

[PULL 07/10] qdev: Change values of PropertyInfo member @type to be QAPI types

2025-03-07 Thread Markus Armbruster
PropertyInfo member @type is externally visible via QMP device-list-properties and qom-list-properies. Its meaning is not documented at its definition. It gets passed as @type argument to object_property_add() and object_class_property_add(). This argument's documentation isn't of much help, eit

[PATCH v3 01/10] vfio/igd: Remove GTT write quirk in IO BAR 4

2025-03-07 Thread Tomita Moeko
The IO BAR4 of IGD devices contains a pair of 32-bit address/data registers, MMIO_Index (0x0) and MMIO_Data (0x4), which provide access to the MMIO BAR0 (GTTMMADR) from IO space. These registers are probably only used by the VBIOS, and are not documented by intel. The observed layout of MMIO_Index

Re: [PATCH v1 1/1] xen: No need to flush the mapcache for grants

2025-03-07 Thread Anthony PERARD
On Thu, Feb 06, 2025 at 08:49:15PM +0100, Edgar E. Iglesias wrote: > From: Stefano Stabellini > > On IOREQ_TYPE_INVALIDATE we need to invalidate the mapcache for regular > mappings. Since recently we started reusing the mapcache also to keep > track of grants mappings. However, there is no need t

RE: [PATCH 08/38] target/hexagon: Add guest, system reg number defs

2025-03-07 Thread Sid Manning
> -Original Message- > From: ltaylorsimp...@gmail.com > Sent: Thursday, March 6, 2025 3:30 PM > To: 'Brian Cain' ; qemu-devel@nongnu.org > Cc: richard.hender...@linaro.org; phi...@linaro.org; Matheus Bernardino > (QUIC) ; a...@rev.ng; a...@rev.ng; Marco > Liebel (QUIC) ; alex.ben...@lina

Re: [RFC PATCH 0/5] hw/arm/virt: Add support for user-creatable nested SMMUv3

2025-03-07 Thread Eric Auger
Hi Shammeer, On 2/18/25 7:52 AM, Shameerali Kolothum Thodi wrote: > Hi Zhenzhong, > >> -Original Message- >> From: Duan, Zhenzhong >> Sent: Monday, February 17, 2025 9:17 AM >> To: Shameerali Kolothum Thodi >> ; Nicolin Chen >> ; Donald Dutile >> Cc: eric.au...@redhat.com; Peter Maydell

Re: [PATCH v7 0/5] Allow to enable multifd and postcopy migration together

2025-03-07 Thread Prasad Pandit
Hello Fabiano, On Wed, 5 Mar 2025 at 19:26, Fabiano Rosas wrote: > Note that none of this is out of the ordinary, you'll find such > discussions in any thread on this community. It may feel arbitrary to > you because that's tacit knowledge we gathered along the years. * I understand. I don't fin

Re: [PATCH 19/57] docs/qapi-domain: add qapi:event directive

2025-03-07 Thread Markus Armbruster
John Snow writes: > On Fri, Mar 7, 2025 at 5:26 AM Markus Armbruster wrote: > >> John Snow writes: >> >> > Adds the .. qapi:event:: directive, object, and :qapi:event:`name` >> > cross-referencing role. >> > >> > Adds the :memb type name: field list syntax for documenting event data >> > member

[PATCH v2 00/18] accel/tcg: Compile more files once

2025-03-07 Thread Philippe Mathieu-Daudé
Since v1: - Patches 1-13,16 unmodified - Restrict GETPC_ADJ() - Rename accel/tcg/getpc.h - Guard GETPC() within CONFIG_TCG Take care of some easy cases in accel/tcg/. >From here it starts getting harder. 🙂 Philippe Mathieu-Daudé (7): exec: Declare tlb_reset_dirty*() in 'exec/cputlb.h' exec:

[PATCH v2 17/18] accel/tcg: Build tcg-runtime.c once

2025-03-07 Thread Philippe Mathieu-Daudé
From: Richard Henderson Signed-off-by: Richard Henderson Reviewed-by: Pierrick Bouvier Signed-off-by: Philippe Mathieu-Daudé --- accel/tcg/tcg-runtime.c | 7 +-- accel/tcg/meson.build | 2 +- 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/accel/tcg/tcg-runtime.c b/accel/

[PATCH v2 16/18] accel/tcg: Split out getpc.h

2025-03-07 Thread Philippe Mathieu-Daudé
From: Richard Henderson Split out GETPC to a target-independent header. Signed-off-by: Richard Henderson Reviewed-by: Pierrick Bouvier Signed-off-by: Philippe Mathieu-Daudé --- include/accel/tcg/getpc.h | 24 include/exec/exec-all.h | 11 ++- accel/tcg/tcg-

[PATCH v2 15/18] accel/tcg: Restrict GETPC_ADJ() to 'tb-internal.h'

2025-03-07 Thread Philippe Mathieu-Daudé
GETPC_ADJ() is only used within accel/tcg/, no need to expose it to all the code base. Signed-off-by: Philippe Mathieu-Daudé --- accel/tcg/tb-internal.h | 11 +++ include/exec/exec-all.h | 9 - 2 files changed, 11 insertions(+), 9 deletions(-) diff --git a/accel/tcg/tb-internal

[PATCH v2 1/7] hw/hyperv/hv-balloon-stub: common compilation unit

2025-03-07 Thread Pierrick Bouvier
Signed-off-by: Pierrick Bouvier --- hw/hyperv/meson.build | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/hyperv/meson.build b/hw/hyperv/meson.build index d3d2668c71a..f4aa0a5ada9 100644 --- a/hw/hyperv/meson.build +++ b/hw/hyperv/meson.build @@ -2,4 +2,5 @@ specific_ss.a

Re: [PATCH 09/16] system: Build watchpoint.c once

2025-03-07 Thread Pierrick Bouvier
On 3/7/25 10:56, Richard Henderson wrote: Now that watchpoint.c uses cputlb.h instead of exec-all.h, it can be built once. Signed-off-by: Richard Henderson --- system/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/system/meson.build b/system/meson.build index

[PATCH v2 05/14] target/i386/hvf: use emul_ops->read_mem in x86_emu.c

2025-03-07 Thread Wei Liu
No functional change. Signed-off-by: Wei Liu --- target/i386/hvf/x86_emu.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/target/i386/hvf/x86_emu.c b/target/i386/hvf/x86_emu.c index e59a73e00d5c..7b816b5a1dab 100644 --- a/target/i386/hvf/x86_emu.c +++ b/target/i38

[PATCH v2 00/14] Factor out HVF's instruction emulator

2025-03-07 Thread Wei Liu
Hi, Microsoft's Linux Systems Group developed a Linux driver for the Microsoft Hypervisor (MSHV for short). The driver is being upstreamed. The first supported VMM is Cloud Hypervisor. We want to add QEMU as the second supported VMM. The plan is to write an mshv accelerator in QEMU. The accelerat

[PATCH v2 04/14] target/i386: rename hvf_mmio_buf to emu_mmio_buf

2025-03-07 Thread Wei Liu
We want to refactor HVF's instruction emulator to a common component. Renaming hvf_mmio_buf removes the association between HVF and the instruction emulator. The definition of the field is still guarded by CONFIG_HVF for now, since it is the only user. No functional change. Signed-off-by: Wei Li

[PATCH v2 06/14] taret/i386/hvf: provide and use write_mem in emul_ops

2025-03-07 Thread Wei Liu
Signed-off-by: Wei Liu --- target/i386/hvf/hvf.c | 6 ++ target/i386/hvf/x86_emu.c | 8 target/i386/hvf/x86_emu.h | 1 + 3 files changed, 11 insertions(+), 4 deletions(-) diff --git a/target/i386/hvf/hvf.c b/target/i386/hvf/hvf.c index 1cecb765952b..e4f48a79fb7c 100644 --- a/tar

[PATCH v2 11/14] target/i386: add a directory for x86 instruction emulator

2025-03-07 Thread Wei Liu
Signed-off-by: Wei Liu --- v2: name the directory emulate --- target/i386/emulate/meson.build | 0 target/i386/meson.build | 1 + 2 files changed, 1 insertion(+) create mode 100644 target/i386/emulate/meson.build diff --git a/target/i386/emulate/meson.build b/target/i386/emulate/meson.b

RE: [PATCH 15/38] target/hexagon: Add handlers for guest/sysreg r/w

2025-03-07 Thread ltaylorsimpson
> -Original Message- > From: Brian Cain > Sent: Friday, February 28, 2025 11:26 PM > To: qemu-devel@nongnu.org > Cc: brian.c...@oss.qualcomm.com; richard.hender...@linaro.org; > phi...@linaro.org; quic_mathb...@quicinc.com; a...@rev.ng; a...@rev.ng; > quic_mlie...@quicinc.com; ltaylorsi

[PATCH v2 07/14] target/i386/hvf: provide and use simulate_{wrmsr, rdmsr} in emul_ops

2025-03-07 Thread Wei Liu
Change the first argument's type to be CPUState to match other hooks. Signed-off-by: Wei Liu --- target/i386/hvf/hvf-i386.h | 4 ++-- target/i386/hvf/hvf.c | 18 ++ target/i386/hvf/x86_emu.c | 4 ++-- target/i386/hvf/x86_emu.h | 2 ++ 4 files changed, 16 insertions(+),

[PATCH v2 10/14] target/i386/hvf: rename some include guards

2025-03-07 Thread Wei Liu
These headers will be moved out to its own component. Signed-off-by: Wei Liu --- target/i386/hvf/x86.h| 4 ++-- target/i386/hvf/x86_decode.h | 4 ++-- target/i386/hvf/x86_flags.h | 6 +++--- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/target/i386/hvf/x86.h b/target/i3

[PATCH v2 09/14] target/i386/hvf: drop unused headers

2025-03-07 Thread Wei Liu
Signed-off-by: Wei Liu --- target/i386/hvf/x86_decode.c | 3 --- target/i386/hvf/x86_emu.c| 4 2 files changed, 7 deletions(-) diff --git a/target/i386/hvf/x86_decode.c b/target/i386/hvf/x86_decode.c index 728e15963817..ddd7b60bcfe1 100644 --- a/target/i386/hvf/x86_decode.c +++ b/target

[PATCH v2 08/14] target/i386: rename lazy flags field and its type

2025-03-07 Thread Wei Liu
The same structure and code can be used by other accelerators. Drop the hvf prefix in the type and field name. No functional change. Signed-off-by: Wei Liu --- target/i386/cpu.h | 6 ++-- target/i386/hvf/x86_flags.c | 56 ++--- 2 files changed, 31 inse

[PATCH v2 0/7] hw/hyperv: remove duplication compilation units

2025-03-07 Thread Pierrick Bouvier
Work towards having a single binary, by removing duplicated object files. hw/hyperv/hyperv.c was excluded at this time, because it depends on target dependent symbols: - from system/kvm.h - kvm_check_extension - kvm_vm_ioctl - from exec/cpu-all.h | memory_ldst_phys.h.inc - ldq_phys v2

Re: [PATCH 04/57] docs/sphinx: add compat.py module and nested_parse helper

2025-03-07 Thread John Snow
On Fri, Mar 7, 2025 at 12:46 AM Markus Armbruster wrote: > John Snow writes: > > > Create a compat module that handles sphinx cross-version compatibility > > issues. For the inaugural function, add a nested_parse() helper that > > handles differences in line number tracking for nested directive

Re: [PATCH 08/14] system/kvm: Expose kvm_irqchip_[add,remove]_change_notifier()

2025-03-07 Thread Richard Henderson
On 3/7/25 10:03, Philippe Mathieu-Daudé wrote: Currently kvm_irqchip_add_irqfd_notifier() and kvm_irqchip_remove_irqfd_notifier() are only declared on target specific code. There is not particular reason to, as their prototypes don't use anything target related. Move their declaration with commo

Re: [PATCH 09/14] hw/vfio/pci: Convert CONFIG_KVM check to runtime one

2025-03-07 Thread Richard Henderson
On 3/7/25 10:03, Philippe Mathieu-Daudé wrote: Use the runtime kvm_enabled() helper to check whether KVM is available or not. Signed-off-by: Philippe Mathieu-Daudé --- hw/vfio/pci.c | 19 +-- 1 file changed, 9 insertions(+), 10 deletions(-) Reviewed-by: Richard Henderson r

[PATCH 01/16] include/exec: Split out exec/cpu-interrupt.h

2025-03-07 Thread Richard Henderson
Some of these bits are actually common to all cpus; while the reset have common reservations for target-specific usage. While generic code cannot know what the target-specific usage is, common code can know what to do with the bits, e.g. single-step. Signed-off-by: Richard Henderson --- include/

[PATCH 07/16] exec: Declare tlb_hit*() in 'exec/cputlb.h'

2025-03-07 Thread Richard Henderson
From: Philippe Mathieu-Daudé Move CPU TLB related methods to "exec/cputlb.h". Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Pierrick Bouvier Signed-off-by: Richard Henderson Message-ID: <20241114011310.3615-20-phi...@linaro.org> --- include/exec/cpu-all.h | 23 --- a

[PATCH 05/16] exec: Declare tlb_set_page_with_attrs() in 'exec/cputlb.h'

2025-03-07 Thread Richard Henderson
From: Philippe Mathieu-Daudé Move CPU TLB related methods to "exec/cputlb.h". Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Pierrick Bouvier Signed-off-by: Richard Henderson Message-ID: <20241114011310.3615-17-phi...@linaro.org> --- include/exec/cputlb.h| 28

[PATCH 06/16] exec: Declare tlb_set_page() in 'exec/cputlb.h'

2025-03-07 Thread Richard Henderson
From: Philippe Mathieu-Daudé Move CPU TLB related methods to "exec/cputlb.h". Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Pierrick Bouvier Signed-off-by: Richard Henderson Message-ID: <20241114011310.3615-18-phi...@linaro.org> --- include/exec/cputlb.h | 11 +++

[PATCH 15/16] accel/tcg: Build tcg-runtime.c once

2025-03-07 Thread Richard Henderson
Signed-off-by: Richard Henderson --- accel/tcg/tcg-runtime.c | 8 ++-- accel/tcg/meson.build | 2 +- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/accel/tcg/tcg-runtime.c b/accel/tcg/tcg-runtime.c index 9fa539ad3d..72c4c6bd31 100644 --- a/accel/tcg/tcg-runtime.c +++ b/accel

[PATCH 00/16] accel/tcg: Compile more files once

2025-03-07 Thread Richard Henderson
Take care of some easy cases in accel/tcg/. >From here it starts getting harder. :-) r~ Philippe Mathieu-Daudé (6): exec: Declare tlb_reset_dirty*() in 'exec/cputlb.h' exec: Declare tlb_set_page_full() in 'exec/cputlb.h' exec: Declare tlb_set_page_with_attrs() in 'exec/cputlb.h' exec:

[PATCH 02/16] accel/tcg: Compile watchpoint.c once

2025-03-07 Thread Richard Henderson
Move tb_check_watchpoint declaration from tb-internal.h, which is still target-specific, to internal-common.h, which isn't. Otherwise, all that is required to build watchpoint.c once is to include the new exec/cpu-interrupt.h instead of exec/exec-all.h. Signed-off-by: Richard Henderson --- accel

[PATCH 03/16] exec: Declare tlb_reset_dirty*() in 'exec/cputlb.h'

2025-03-07 Thread Richard Henderson
From: Philippe Mathieu-Daudé Move CPU TLB related methods to "exec/cputlb.h". Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Pierrick Bouvier Message-ID: <20241114011310.3615-14-phi...@linaro.org> Signed-off-by: Richard Henderson --- include/exec/cputlb.h | 7 +++ include/exec/exec

[PATCH 14/16] include/exec: Split out helper-getpc.h

2025-03-07 Thread Richard Henderson
Split out GETPC and GETPC_ADJ to a target-independent header. Signed-off-by: Richard Henderson --- include/exec/exec-all.h | 19 +-- include/exec/helper-getpc.h | 31 +++ 2 files changed, 32 insertions(+), 18 deletions(-) create mode 100644 includ

[PATCH 11/16] accel/tcg: Build tcg-accel-ops-icount.c once

2025-03-07 Thread Richard Henderson
All that is required is to avoid including exec-all.h. Signed-off-by: Richard Henderson --- accel/tcg/tcg-accel-ops-icount.c | 2 +- accel/tcg/meson.build| 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/accel/tcg/tcg-accel-ops-icount.c b/accel/tcg/tcg-accel-ops-

[PATCH 13/16] accel/tcg: Build tcg-accel-ops-mttcg.c once

2025-03-07 Thread Richard Henderson
All that is required is to avoid including exec-all.h. Signed-off-by: Richard Henderson --- accel/tcg/tcg-accel-ops-mttcg.c | 1 - accel/tcg/meson.build | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/accel/tcg/tcg-accel-ops-mttcg.c b/accel/tcg/tcg-accel-ops-mttcg

[PATCH 16/16] accel/tcg: Build tcg-runtime-gvec.c once

2025-03-07 Thread Richard Henderson
Signed-off-by: Richard Henderson --- accel/tcg/tcg-runtime-gvec.c | 1 - accel/tcg/meson.build| 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/accel/tcg/tcg-runtime-gvec.c b/accel/tcg/tcg-runtime-gvec.c index afca89baa1..ff927c5dd8 100644 --- a/accel/tcg/tcg-runtime-g

[PATCH 04/16] exec: Declare tlb_set_page_full() in 'exec/cputlb.h'

2025-03-07 Thread Richard Henderson
From: Philippe Mathieu-Daudé Move CPU TLB related methods to "exec/cputlb.h". Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Pierrick Bouvier Signed-off-by: Richard Henderson Message-ID: <20241114011310.3615-16-phi...@linaro.org> --- include/exec/cputlb.h | 23 +++

RE: [PATCH 12/38] target/hexagon: Add imported macro, attr defs for sysemu

2025-03-07 Thread ltaylorsimpson
> -Original Message- > From: Brian Cain > Sent: Friday, February 28, 2025 11:26 PM > To: qemu-devel@nongnu.org > Cc: brian.c...@oss.qualcomm.com; richard.hender...@linaro.org; > phi...@linaro.org; quic_mathb...@quicinc.com; a...@rev.ng; a...@rev.ng; > quic_mlie...@quicinc.com; ltaylorsi

[PATCH] rust: pl011: Allow NULL chardev argument to pl011_create()

2025-03-07 Thread Peter Maydell
It's valid for the caller to pass a NULL chardev to pl011_create(); this means "don't set the chardev property on the device", which in turn means "act like there's no chardev". All the chardev frontend APIs (in C, at least) accept a NULL pointer to mean "do nothing". This fixes some failures in '

[PATCH v2 14/14] MAINTAINERS: add an entry for the x86 instruction emulator

2025-03-07 Thread Wei Liu
Add myself as a reviewer. Signed-off-by: Wei Liu --- v2: new --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 5df6020ed545..02c763e9d14f 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -533,6 +533,14 @@ S: Supported F: target/i386/whpx/

[PATCH v8 10/20] acpi/ghes: add a notifier to notify when error data is ready

2025-03-07 Thread Mauro Carvalho Chehab
Some error injection notify methods are async, like GPIO notify. Add a notifier to be used when the error record is ready to be sent to the guest OS. Signed-off-by: Mauro Carvalho Chehab Reviewed-by: Jonathan Cameron Acked-by: Igor Mammedov --- hw/acpi/ghes.c | 5 - include/hw/acpi

Re: [PATCH 00/16] accel/tcg: Compile more files once

2025-03-07 Thread Pierrick Bouvier
On 3/7/25 10:56, Richard Henderson wrote: Take care of some easy cases in accel/tcg/. From here it starts getting harder. :-) It would be convenient to work on top of it. Could you rebase it on top of master? Thanks, Pierrick r~ Philippe Mathieu-Daudé (6): exec: Declare tlb_reset_di

[PATCH v3 6/7] hw/hyperv/balloon: common balloon compilation units

2025-03-07 Thread Pierrick Bouvier
Reviewed-by: Richard Henderson Signed-off-by: Pierrick Bouvier --- hw/hyperv/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/hyperv/meson.build b/hw/hyperv/meson.build index a9f2045a9af..5acd709bdd5 100644 --- a/hw/hyperv/meson.build +++ b/hw/hyperv/meson.build

Re: [PATCH 10/14] system/iommufd: Introduce iommufd_builtin() helper

2025-03-07 Thread Pierrick Bouvier
On 3/7/25 10:03, Philippe Mathieu-Daudé wrote: iommufd_builtin() can be used to check at runtime whether the IOMMUFD feature is built in a qemu-system binary. Signed-off-by: Philippe Mathieu-Daudé --- docs/devel/vfio-iommufd.rst | 2 +- include/system/iommufd.h| 8 2 files chan

[PULL 02/21] hw/gpio: npcm7xx: fixup out-of-bounds access

2025-03-07 Thread Peter Maydell
From: Patrick Venture The reg isn't validated to be a possible register before it's dereferenced for one case. The mmio space registered for the gpio device is 4KiB but there aren't that many registers in the struct. Cc: qemu-sta...@nongnu.org Fixes: 526dbbe0874 ("hw/gpio: Add GPIO model for Nu

[PULL 29/42] vfio/migration: Add vfio_add_bytes_transferred()

2025-03-07 Thread Cédric Le Goater
From: "Maciej S. Szmigiero" This way bytes_transferred can also be incremented in other translation units than migration.c. Signed-off-by: Maciej S. Szmigiero Reviewed-by: Cédric Le Goater Link: https://lore.kernel.org/qemu-devel/d1fbc27ac2417b49892f354ba20f6c6b3f7209f8.1741124640.git.maciej.

[PULL 04/10] qdev: Delete unused qdev_prop_enum

2025-03-07 Thread Markus Armbruster
Signed-off-by: Markus Armbruster Message-ID: <20250227085601.4140852-2-arm...@redhat.com> Reviewed-by: Daniel P. Berrangé --- include/hw/qdev-properties.h | 1 - hw/core/qdev-properties.c| 7 --- 2 files changed, 8 deletions(-) diff --git a/include/hw/qdev-properties.h b/include/hw/qdev

[PULL 18/42] migration: Add thread pool of optional load threads

2025-03-07 Thread Cédric Le Goater
From: "Maciej S. Szmigiero" Some drivers might want to make use of auxiliary helper threads during VM state loading, for example to make sure that their blocking (sync) I/O operations don't block the rest of the migration process. Add a migration core managed thread pool to facilitate this use c

Re: [PATCH qemu v2 1/3] hw/s390x: add CPI identifiers to QOM

2025-03-07 Thread shalini
On 2025-03-05 16:56, Thomas Huth wrote: On 24/02/2025 13.04, Shalini Chellathurai Saroja wrote: Add Control-Program Identification (CPI) to the QEMU Object Model (QOM). The CPI identifiers provide information about the guest operating system. The CPI identifiers are: system type, system name, sy

Re: [PATCH v2 5/7] hw/virtio/virtio-mem: Convert VIRTIO_MEM_USABLE_EXTENT to runtime

2025-03-07 Thread Philippe Mathieu-Daudé
On 7/3/25 23:39, Philippe Mathieu-Daudé wrote: Use qemu_arch_available() to check at runtime if a target architecture is built in. Consider the maximum extent size of any architecture built in. Signed-off-by: Philippe Mathieu-Daudé --- hw/virtio/virtio-mem.c | 28

Re: [PULL 8/8] migration: Add qtest for migration over RDMA

2025-03-07 Thread Philippe Mathieu-Daudé
Hi, On 7/3/25 19:15, Fabiano Rosas wrote: From: Li Zhijian This qtest requires there is a RDMA(RoCE) link in the host. In order to make the test work smoothly, introduce a scripts/rdma-migration-helper.sh to - setup a new Soft-RoCE(aka RXE) if it's root - detect existing RoCE link Test will b

Re: [PATCH] usb-storage: Allow manually adding SCSI device

2025-03-07 Thread Akihiko Odaki
On 2025/03/03 21:17, Paolo Bonzini wrote: On 3/3/25 11:28, Akihiko Odaki wrote: usb-storage automatically adds a SCSI device, but it limits configurability of the added SCSI device and causes usability problems as observed in: https://gitlab.com/libvirt/libvirt/-/issues/368 Allow manually addin

Re: [PATCH 15/57] docs/qapi-domain: add "Errors:" field lists

2025-03-07 Thread Markus Armbruster
John Snow writes: > On Fri, Mar 7, 2025 at 2:48 AM Markus Armbruster wrote: > >> John Snow writes: >> >> > ``:error: descr`` can now be used to document error conditions. The >> > format of the description is not defined here; so the ability to name >> > specific types is left to the document w

Re: [PATCH 16/57] docs/qapi-domain: add "Returns:" field lists

2025-03-07 Thread Markus Armbruster
John Snow writes: > On Fri, Mar 7, 2025 at 2:58 AM Markus Armbruster wrote: > >> John Snow writes: >> >> > Add "Returns:" field list syntax to QAPI Commands. >> > >> > Like "Arguments:" and "Errors:", the type name isn't currently processed >> > for cross-referencing, but this will be addressed

Re: [PATCH 18/57] docs/qapi-domain: add qapi:alternate directive

2025-03-07 Thread Markus Armbruster
John Snow writes: > On Fri, Mar 7, 2025 at 5:18 AM Markus Armbruster wrote: > >> John Snow writes: >> >> > Add the .. qapi:alternate:: directive, object, and qapi:alt:`name` >> > cross-reference role. >> > >> > Add the "Choices:" field list for describing alternate choices. Like >> > other fiel

[PATCH v6 00/29] Support AST2700 A1

2025-03-07 Thread Jamin Lin via
v1: 1. Refactor INTC model to support both INTC0 and INTC1. 2. Support AST2700 A1. 3. Create ast2700a0-evb machine. v2: To streamline the review process, split the following patch series into three parts. https://patchwork.kernel.org/project/qemu-devel/cover/20250121070424.2465942-1-jam

Re: [PATCH v3 07/10] vfio/igd: Decouple common quirks from legacy mode

2025-03-07 Thread Tomita Moeko
On 2025/3/7 6:49, Alex Williamson wrote: > On Fri, 7 Mar 2025 02:01:27 +0800 > Tomita Moeko wrote: > >> So far, IGD-specific quirks all require enabling legacy mode, which is >> toggled by assigning IGD to 00:02.0. However, some quirks, like the BDSM >> and GGC register quirks, should be applied

Re: [PATCH v2 2/7] hw/hyperv/hyperv.h: header cleanup

2025-03-07 Thread Richard Henderson
On 3/7/25 11:09, Pierrick Bouvier wrote: Signed-off-by: Pierrick Bouvier --- include/hw/hyperv/hyperv.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/include/hw/hyperv/hyperv.h b/include/hw/hyperv/hyperv.h index d717b4e13d4..63a8b65278f 100644 --- a/include/hw/hyperv/h

RE: [PATCH 14/38] target/hexagon: Add new macro definitions for sysemu

2025-03-07 Thread ltaylorsimpson
> -Original Message- > From: Brian Cain > Sent: Friday, February 28, 2025 11:26 PM > To: qemu-devel@nongnu.org > Cc: brian.c...@oss.qualcomm.com; richard.hender...@linaro.org; > phi...@linaro.org; quic_mathb...@quicinc.com; a...@rev.ng; a...@rev.ng; > quic_mlie...@quicinc.com; ltaylorsi

Re: [PATCH 01/14] hw/vfio/common: Include missing 'system/tcg.h' header

2025-03-07 Thread Richard Henderson
On 3/7/25 10:03, Philippe Mathieu-Daudé wrote: Always include necessary headers explicitly, to avoid when refactoring unrelated ones: hw/vfio/common.c:1176:45: error: implicit declaration of function ‘tcg_enabled’; 1176 | tcg_enabled() ? DIRTY

[PATCH 10/16] accel/tcg: Build tcg-accel-ops.c once

2025-03-07 Thread Richard Henderson
Now that tcg-accel-ops.c uses cputlb.h instead of exec-all.h, it can be built once. Signed-off-by: Richard Henderson --- accel/tcg/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/accel/tcg/meson.build b/accel/tcg/meson.build index 979ce90eb0..70ada21f42 100644 ---

[PATCH v8 15/20] arm/virt: Wire up a GED error device for ACPI / GHES

2025-03-07 Thread Mauro Carvalho Chehab
Adds support to ARM virtualization to allow handling generic error ACPI Event via GED & error source device. It is aligned with Linux Kernel patch: https://lore.kernel.org/lkml/1272350481-27951-8-git-send-email-ying.hu...@intel.com/ Co-authored-by: Mauro Carvalho Chehab Co-authored-by: Jonathan

Re: [PATCH v2 3/7] hw/hyperv/vmbus: common compilation unit

2025-03-07 Thread Richard Henderson
On 3/7/25 11:09, Pierrick Bouvier wrote: Replace TARGET_PAGE.* by runtime calls. Signed-off-by: Pierrick Bouvier --- hw/hyperv/vmbus.c | 50 +-- hw/hyperv/meson.build | 2 +- 2 files changed, 26 insertions(+), 26 deletions(-) Reviewed-by: Richar

Re: [PATCH v2 18/18] hw/rtc: Add Ricoh RS5C372 RTC emulation

2025-03-07 Thread Bernhard Beschow
Am 4. März 2025 18:53:10 UTC schrieb Bernhard Beschow : > > >Am 23. Februar 2025 11:47:08 UTC schrieb Bernhard Beschow : >>The implementation just allows Linux to determine date and time. >> >>Signed-off-by: Bernhard Beschow >>--- >> MAINTAINERS| 2 + >> hw/rtc/rs5c372.c

[PATCH v8 11/20] acpi/generic_event_device: Update GHES migration to cover hest addr

2025-03-07 Thread Mauro Carvalho Chehab
The GHES migration logic should now support HEST table location too. Signed-off-by: Mauro Carvalho Chehab Reviewed-by: Jonathan Cameron Reviewed-by: Igor Mammedov --- hw/acpi/generic_event_device.c | 29 + 1 file changed, 29 insertions(+) diff --git a/hw/acpi/gener

Re: [PATCH 10/16] accel/tcg: Build tcg-accel-ops.c once

2025-03-07 Thread Pierrick Bouvier
On 3/7/25 10:56, Richard Henderson wrote: Now that tcg-accel-ops.c uses cputlb.h instead of exec-all.h, it can be built once. Signed-off-by: Richard Henderson --- accel/tcg/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/accel/tcg/meson.build b/accel/tcg/meson

Re: [PATCH v2 4/7] hw/hyperv/hyperv-proto: move SYNDBG definition from target/i386

2025-03-07 Thread Richard Henderson
On 3/7/25 11:10, Pierrick Bouvier wrote: Allows them to be available for common compilation units. Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Pierrick Bouvier --- include/hw/hyperv/hyperv-proto.h | 12 target/i386/kvm/hyperv-proto.h | 12 2 files changed

Re: [PATCH v2 6/7] hw/hyperv/balloon: common balloon compilation units

2025-03-07 Thread Richard Henderson
On 3/7/25 11:10, Pierrick Bouvier wrote: Signed-off-by: Pierrick Bouvier --- hw/hyperv/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Richard Henderson r~

Re: [PATCH v2 7/7] hw/hyperv/hyperv_testdev: common compilation unit

2025-03-07 Thread Richard Henderson
On 3/7/25 11:10, Pierrick Bouvier wrote: Signed-off-by: Pierrick Bouvier --- hw/hyperv/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/hyperv/meson.build b/hw/hyperv/meson.build index 5acd709bdd5..ef5a596c8ab 100644 --- a/hw/hyperv/meson.build +++ b/hw/hyper

[PATCH v3 1/7] hw/hyperv/hv-balloon-stub: common compilation unit

2025-03-07 Thread Pierrick Bouvier
Reviewed-by: Richard Henderson Signed-off-by: Pierrick Bouvier --- hw/hyperv/meson.build | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/hyperv/meson.build b/hw/hyperv/meson.build index d3d2668c71a..f4aa0a5ada9 100644 --- a/hw/hyperv/meson.build +++ b/hw/hyperv/meson.bui

Re: [PATCH v2 5/7] hw/hyperv/syndbg: common compilation unit

2025-03-07 Thread Pierrick Bouvier
On 3/7/25 11:10, Pierrick Bouvier wrote: Replace TARGET_PAGE.* by runtime calls Signed-off-by: Pierrick Bouvier --- hw/hyperv/syndbg.c| 10 +++--- hw/hyperv/meson.build | 2 +- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/hw/hyperv/syndbg.c b/hw/hyperv/syndbg.c in

[PATCH v2 03/14] target/i386/hvf: provide and use handle_io in emul_ops

2025-03-07 Thread Wei Liu
This drops the calls to hvf_handle_io from x86_emu.c. Signed-off-by: Wei Liu --- target/i386/hvf/hvf.c | 1 + target/i386/hvf/x86_emu.c | 29 +++-- target/i386/hvf/x86_emu.h | 2 ++ 3 files changed, 18 insertions(+), 14 deletions(-) diff --git a/target/i386/hvf/hvf

Re: [PATCH 14/16] include/exec: Split out helper-getpc.h

2025-03-07 Thread Philippe Mathieu-Daudé
On 7/3/25 23:00, Philippe Mathieu-Daudé wrote: On 7/3/25 19:56, Richard Henderson wrote: Split out GETPC and GETPC_ADJ to a target-independent header. Signed-off-by: Richard Henderson ---   include/exec/exec-all.h | 19 +--   include/exec/helper-getpc.h | 31

Re: [PATCH v4 0/7] hw/hyperv: remove duplication compilation units

2025-03-07 Thread Philippe Mathieu-Daudé
Hi Maciej, On 7/3/25 22:56, Pierrick Bouvier wrote: Work towards having a single binary, by removing duplicated object files. Pierrick Bouvier (7): hw/hyperv/hv-balloon-stub: common compilation unit hw/hyperv/hyperv.h: header cleanup hw/hyperv/vmbus: common compilation unit hw/hyp

Re: [PATCH v4 0/7] hw/hyperv: remove duplication compilation units

2025-03-07 Thread Maciej S. Szmigiero
Hi Philippe, On 7.03.2025 23:25, Philippe Mathieu-Daudé wrote: Hi Maciej, On 7/3/25 22:56, Pierrick Bouvier wrote: Work towards having a single binary, by removing duplicated object files. Pierrick Bouvier (7):    hw/hyperv/hv-balloon-stub: common compilation unit    hw/hyperv/hyperv.h: hea

[PATCH 5/5] aio-posix: Separate AioPolledEvent per AioHandler

2025-03-07 Thread Kevin Wolf
Adaptive polling has a big problem: It doesn't consider that an event loop can wait for many different events that may have very different typical latencies. For example, think of a guest that tends to send a new I/O request soon after the previous I/O request completes, but the storage on the hos

[PATCH v4 7/7] hw/hyperv/hyperv_testdev: common compilation unit

2025-03-07 Thread Pierrick Bouvier
Reviewed-by: Richard Henderson Signed-off-by: Pierrick Bouvier --- hw/hyperv/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/hyperv/meson.build b/hw/hyperv/meson.build index 5acd709bdd5..ef5a596c8ab 100644 --- a/hw/hyperv/meson.build +++ b/hw/hyperv/meson.build

Re: [PATCH v4 0/7] hw/hyperv: remove duplication compilation units

2025-03-07 Thread Pierrick Bouvier
On 3/7/25 14:06, Philippe Mathieu-Daudé wrote: On 7/3/25 22:56, Pierrick Bouvier wrote: Work towards having a single binary, by removing duplicated object files. hw/hyperv/hyperv.c was excluded at this time, because it depends on target dependent symbols: - from system/kvm.h - kvm_check_e

Re: [PATCH v4 0/7] hw/hyperv: remove duplication compilation units

2025-03-07 Thread Philippe Mathieu-Daudé
On 7/3/25 23:31, Maciej S. Szmigiero wrote: Hi Philippe, On 7.03.2025 23:25, Philippe Mathieu-Daudé wrote: Hi Maciej, On 7/3/25 22:56, Pierrick Bouvier wrote: Work towards having a single binary, by removing duplicated object files. Pierrick Bouvier (7):    hw/hyperv/hv-balloon-stub: commo

[PATCH v2 6/7] hw/virtio/virtio-mem: Convert VIRTIO_MEM_HAS_LEGACY_GUESTS to runtime

2025-03-07 Thread Philippe Mathieu-Daudé
Use qemu_arch_available() to check at runtime if a target architecture is built in. Register virtio_mem_legacy_guests_properties[] at runtime. Code churn in virtio_mem_device_realize() is due to re-indentation. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: David Hildenbrand --- hw/virtio/v

[PATCH v2 3/7] hw/i386/fw_cfg: Check ACPI availability with acpi_builtin()

2025-03-07 Thread Philippe Mathieu-Daudé
Replace the compile-time CONFIG_ACPI check by a runtime one. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Ani Sinha --- hw/i386/fw_cfg.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/hw/i386/fw_cfg.c b/hw/i386/fw_cfg.c index d08aefa0291..a7f1b60b98c 100644 --

[PATCH v2 5/7] hw/virtio/virtio-mem: Convert VIRTIO_MEM_USABLE_EXTENT to runtime

2025-03-07 Thread Philippe Mathieu-Daudé
Use qemu_arch_available() to check at runtime if a target architecture is built in. Consider the maximum extent size of any architecture built in. Signed-off-by: Philippe Mathieu-Daudé --- hw/virtio/virtio-mem.c | 28 1 file changed, 20 insertions(+), 8 deletions(-)

Re: [PATCH 11/57] docs/qapi-domain: add qapi:command directive

2025-03-07 Thread John Snow
On Fri, Mar 7, 2025 at 1:34 AM Markus Armbruster wrote: > John Snow writes: > > > This commit adds a stubbed version of QAPICommand that utilizes the > > QAPIObject class, the qapi:command directive, the :qapi:cmd: > > cross-reference role, and the "command" object type in the QAPI object > > re

[PATCH v2 1/7] system: Replace arch_type global by qemu_arch_available() helper

2025-03-07 Thread Philippe Mathieu-Daudé
qemu_arch_available() is a bit simpler to understand while reviewing than the undocumented arch_type variable. Reviewed-by: Richard Henderson Signed-off-by: Philippe Mathieu-Daudé --- include/system/arch_init.h | 2 +- hw/scsi/scsi-disk.c| 2 +- system/arch_init.c | 5 - sys

[PATCH v2 2/7] hw/acpi: Introduce acpi_builtin() helper

2025-03-07 Thread Philippe Mathieu-Daudé
acpi_builtin() can be used to check at runtime whether the ACPI subsystem is built in a qemu-system binary. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Ani Sinha --- include/hw/acpi/acpi.h | 3 +++ hw/acpi/acpi-stub.c| 5 + hw/acpi/core.c | 5 + 3 files changed, 13 in

Re: [PATCH 12/57] docs/qapi-domain: add :since: directive option

2025-03-07 Thread John Snow
On Fri, Mar 7, 2025 at 1:59 AM Markus Armbruster wrote: > John Snow writes: > > > Add a little special markup for registering "Since:" information. Adding > > it as an option instead of generic content lets us hoist the information > > into the Signature bar, optionally put it in the index, etc.

[PATCH v2 7/7] hw/virtio: Compile virtio-mem.c once

2025-03-07 Thread Philippe Mathieu-Daudé
Remove unused "exec/ram_addr.h" header. This file doesn't use any target specific definitions anymore, compile it once by moving it to system_virtio_ss[]. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: David Hildenbrand --- hw/virtio/virtio-mem.c | 1 - hw/virtio/meson.build | 2 +- 2 file

  1   2   3   4   5   >