Re: [PATCH 0/2] Add property to support writing ERSTBA in high-low order

2025-04-05 Thread Guenter Roeck
On 4/5/25 12:28, Bernhard Beschow wrote: Am 5. April 2025 17:26:14 UTC schrieb Guenter Roeck : On 4/5/25 07:25, Philippe Mathieu-Daudé wrote: Hi Guenter, On 5/4/25 16:00, Guenter Roeck wrote: This series is needed to support the USB interface on imx8mp-evk when booting the Linux kernel. Ac

[PATCH v5] block/file-posix.c: Use pwritev2() with RWF_DSYNC for FUA

2025-04-05 Thread Pinku Deb Nath
Full Unit Access (FUA) is an optimization where a disk write with the flag set will be persisted to disk immediately instead of potentially remaining in the disk's write cache. This commit address the todo task for using pwritev2() with RWF_DSYNC in the thread pool section of raw_co_prw(), if pwri

[PATCH-for-10.0 1/3] tests/functional: Add test for imx8mp-evk board with USDHC coverage

2025-04-05 Thread Bernhard Beschow
Introduce a functional test which boots Debian 12 on the imx8mp-evk board. Since the root filesystem resides on an SD card, the test also verifies the basic operation of the USDHC. Signed-off-by: Bernhard Beschow --- MAINTAINERS | 1 + tests/functional/meson.buil

[PATCH-for-10.0 0/3] More imx8mp-evk improvements

2025-04-05 Thread Bernhard Beschow
This series adds a functional test for the imx8mp-evk board with USDHC coverage. Thanks to Guenter Roeck the usage of the board gets simplified by not asking the user to remove the cpu-idle-states properties from the devicetree since that is now done automatically. Likewise, a stack trace during bo

[PATCH-for-10.0 3/3] hw/arm/imx8mp-evk: Remove unimplemented nxp, imx8mp-fspi node from devicetree

2025-04-05 Thread Bernhard Beschow
From: Guenter Roeck The nxp,imx8mp-fspi node triggers a warning backtrace. Remove it from the devicetree file. Signed-off-by: Guenter Roeck Inspired-by: commit bf1da4b308 ("hw/arm/raspi4b: Temporarily disable unimplemented rpi4b devices") Signed-off-by: Bernhard Beschow [Bernhard: split patch,

[PATCH-for-10.0 2/3] hw/arm/imx8mp-evk: Remove unimplemented cpu-idle-states properties from devicetree

2025-04-05 Thread Bernhard Beschow
From: Guenter Roeck The cpu-idle-states property causes a hard boot hang. Rather than documenting the workaround, perform the removal from the devicetree automatically. Signed-off-by: Guenter Roeck Signed-off-by: Bernhard Beschow [Bernhard: split patch, update documentation, adapt commit messa

Re: [PATCH 0/2] Add property to support writing ERSTBA in high-low order

2025-04-05 Thread Guenter Roeck
Hi Bernhard, On 4/5/25 12:28, Bernhard Beschow wrote: Am 5. April 2025 17:26:14 UTC schrieb Guenter Roeck : On 4/5/25 07:25, Philippe Mathieu-Daudé wrote: Hi Guenter, On 5/4/25 16:00, Guenter Roeck wrote: This series is needed to support the USB interface on imx8mp-evk when booting the Lin

Re: [PATCH 0/2] Add property to support writing ERSTBA in high-low order

2025-04-05 Thread Bernhard Beschow
Am 5. April 2025 17:26:14 UTC schrieb Guenter Roeck : >On 4/5/25 07:25, Philippe Mathieu-Daudé wrote: >> Hi Guenter, >> >> On 5/4/25 16:00, Guenter Roeck wrote: >>> This series is needed to support the USB interface on imx8mp-evk when >>> booting the Linux kernel. >>> >>> According to the XHCI

[PATCH v3] hw/arm/virt: Allow additions to the generated device tree

2025-04-05 Thread Simon Glass
At present qemu creates a device tree automatically with the 'virt' generic virtual platform. This is very convenient in most cases but there is not much control over what is generated. Add a way to provide a device tree binary file with additional properties to add before booting. This provides f

[PATCH v3 09/15] rust/vmstate: Re-implement VMState trait for timer binding

2025-04-05 Thread Zhao Liu
At present, Rust side has a timer binding "timer::Timer", so the vmstate for timer should base on that binding instead of the raw "binding::QEMUTimer". It's possible to apply impl_vmstate_transparent for cell::Opaque and then impl_vmstate_forward for timer::Timer. But binding::QEMUTimer shouldn't

Re: [PATCH v2 4/5] ipmi/bmc-sim: implement watchdog dont log flag

2025-04-05 Thread Nicholas Piggin
On Tue Apr 1, 2025 at 11:05 PM AEST, Corey Minyard wrote: > On Tue, Apr 01, 2025 at 09:44:11PM +1000, Nicholas Piggin wrote: >> If the dont-log flag is set in the 'timer use' field for the >> 'set watchdog' command, a watchdog timeout will not get logged as >> a timer use expiration. >> >> Signed-

Re: [PATCH v2] rust: assertions: add static_assert

2025-04-05 Thread Pierrick Bouvier
On 3/20/25 04:55, Paolo Bonzini wrote: Add a new assertion that is similar to "const { assert!(...) }" but can be used outside functions and with older versions of Rust. A similar macro is found in Linux, whereas the "static_assertions" crate has a const_assert macro that produces worse error me

Re: Central repo for VirtIO conformance tests?

2025-04-05 Thread Stefan Hajnoczi
On Mon, Mar 31, 2025 at 6:39 AM Alex Bennée wrote: > The unikernel utilizes rcore-os's no_std VirtIO drivers to discover and > initialize a range of VirtIO devices. https://github.com/rcore-os/virtio-drivers I noticed that VIRTIO_F_VERSION_1, VIRTIO_F_RING_PACKED, and hardware device support are

[Stable-9.2.3 54/69] target/loongarch: Fix vldi inst

2025-04-05 Thread Michael Tokarev
From: Guo Hongyu Refer to the link below for a description of the vldi instructions: https://jia.je/unofficial-loongarch-intrinsics-guide/lsx/misc/#synopsis_88 Fixed errors in vldi instruction implementation. Signed-off-by: Guo Hongyu Tested-by: Xianglai Li Signed-off-by: Xianglai Li Reviewed

Re: [PATCH] smbios: Fix buffer overrun when using path= option

2025-04-05 Thread Philippe Mathieu-Daudé
Hi Daan, On 3/4/25 21:29, Daan De Meyer wrote: Hi, Unless I'm missing something, I don't think the patch has been merged yet. Any chance it might have been missed? I have it tagged, as sensible enough, in case nobody else takes it. IIRC it was sent the same day I posted my latest pull request

Re: [PATCH 1/2] migration: Add some documentation for multifd

2025-04-05 Thread Prasad Pandit
On Thu, 20 Mar 2025 at 20:15, Fabiano Rosas wrote: > Technically both can happen. But that would just be the case of > file:fdset migration which requires an extra fd for O_DIRECT. So > "multiple" in the usual sense of "more is better" is only > fd-per-thread. IOW, using multiple fds is an impleme

Re: [PATCH-for-10.1 v2 6/7] target/s390x: Register CPUClass:list_cpus

2025-04-05 Thread Philippe Mathieu-Daudé
On 24/3/25 19:46, Philippe Mathieu-Daudé wrote: Register s390_cpu_list() as CPUClass:list_cpus callback and remove the cpu_list definition. Signed-off-by: Philippe Mathieu-Daudé --- target/s390x/cpu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/target/s390x/cpu.c b/target/s390x/cpu.c

[PATCH-for-10.0 v3 5/5] hw/arm: Do not build VMapple machine by default

2025-04-05 Thread Philippe Mathieu-Daudé
Unfortunately as of v10.0.0-rc2 the VMapple machine is unusable: $ qemu-system-aarch64 -M vmapple [...] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[PGIOSurfaceHostDeviceDescriptor setMapMemory:]: unrecognized selector sent to instance 0x61e

Re: [PATCH v6] hw/misc/vmfwupdate: Introduce hypervisor fw-cfg interface support

2025-04-05 Thread Alexander Graf
On 26.03.25 13:27, Gerd Hoffman wrote: Hi, The problem is that add-ons are 1) Separate binaries. So you need to match multiple files. 2) In this case, get generated out of the vendor (RH)'s control in a one-off fashion. I don't think "signing" is the correct way to address the latt

[PATCH v2 16/30] exec/cpu-all: remove this header

2025-04-05 Thread Pierrick Bouvier
Signed-off-by: Pierrick Bouvier --- accel/tcg/tb-internal.h | 1 - include/exec/cpu-all.h | 22 -- include/hw/core/cpu.h | 2 +- include/qemu/bswap.h| 2 +- target/alpha/cpu.h | 2 -- target/arm/cpu.h| 2 -- target/avr/cpu.h| 2 -- target/hexa

[PATCH 03/17] target/avr: Improve decode of LDS, STS

2025-04-05 Thread Richard Henderson
The comment about not being able to define a field with zero bits is out of date since 94597b6146f3 ("decodetree: Allow !function with no input bits"). This fixes the missing load of imm in the disassembler. Cc: qemu-sta...@nongnu.org Fixes: 9d8caa67a24 ("target/avr: Add support for disassembling

[PATCH v1 03/22] hw/misc/aspeed_hace: Improve readability and consistency in variable naming

2025-04-05 Thread Jamin Lin via
Currently, users define multiple local variables within different if-statements. To improve readability and maintain consistency in variable naming, rename the variables accordingly. Introduced "sg_addr" to clearly indicate the scatter-gather mode buffer address. Signed-off-by: Jamin Lin --- hw/

Re: [PATCH-for-10.1 00/13] arm: Spring header cleanups

2025-04-05 Thread Pierrick Bouvier
On 4/3/25 12:31, Philippe Mathieu-Daudé wrote: On 3/4/25 20:22, Pierrick Bouvier wrote: On 4/2/25 15:23, Philippe Mathieu-Daudé wrote: This series is more useful for heterogeneous emulation preparation than single binary, because it allows non-ARM hw/ code to configure ARM cores, so not using t

Re: Central repo for VirtIO conformance tests?

2025-04-05 Thread Stefan Hajnoczi
On Mon, Mar 31, 2025 at 01:14:33PM +0100, Daniel P. Berrangé wrote: > On Mon, Mar 31, 2025 at 07:52:33AM -0400, Stefan Hajnoczi wrote: > > On Mon, Mar 31, 2025 at 6:39 AM Alex Bennée wrote: > > > So what do people think? Where would be a good place for common test > > > repository to live? > > >

[PATCH 1/1] goldfish_rtc: keep time offset when resetting

2025-04-05 Thread Heinrich Schuchardt
Currently resetting leads to resynchronizing the Goldfish RTC with the system clock of the host. In real hardware an RTC reset would not change the wall time. Other RTCs like pl031 do not show this behavior. Move the synchronization of the RTC with the system clock to the instance realization. Re

Re: [PATCH 08/17] target/avr: Add offset-io cpu property

2025-04-05 Thread Pierrick Bouvier
On 3/23/25 10:37, Richard Henderson wrote: Communicate the offset of io within the first page between the board, the cpu, and the translator. So far this is always 0. This will be used to optimize memory layout. Signed-off-by: Richard Henderson --- target/avr/cpu.h | 2 ++ hw/avr/atme

Re: [PATCH-for-10.0?] target/riscv: Do not expose rv128 CPU on user mode emulation

2025-04-05 Thread Philippe Mathieu-Daudé
On 2/4/25 23:22, Daniel Henrique Barboza wrote: On 4/2/25 5:51 PM, Philippe Mathieu-Daudé wrote: As Richard mentioned:    We should allow RV128 in user-mode at all until there's a    kernel abi for it. By the context I suppose Richard said 'We shouldn't allow RV128 ...'. If this was said i

Re: [RFC v5 7/7] vdpa: Support setting vring_base for packed SVQ

2025-04-05 Thread Sahil Siddiq
Hi, On 3/26/25 5:38 PM, Eugenio Perez Martin wrote: On Mon, Mar 24, 2025 at 3:00 PM Sahil Siddiq wrote: [...] Link: https://lists.nongnu.org/archive/html/qemu-devel/2024-10/msg05106.html Link: https://lore.kernel.org/r/20210602021536.39525-4-jasow...@redhat.com Link: 1225c216d954 ("vp_vdpa: al

Re: [PATCH v5 0/6] target/loongarch: Fix some issues reported from coccinelle

2025-04-05 Thread Markus Armbruster
Bibo Mao writes: > This patch set solves errors reported by coccinelle tool with commands: > spatch --sp-file scripts/coccinelle/*.cocci --dir target/loongarch/ > spatch --sp-file scripts/coccinelle/*.cocci --dir hw/loongarch/ > > The main problem is that qemu should fail to run when feature

[PATCH v3 1/4] hw/s390x: add SCLP event type CPI

2025-04-05 Thread Shalini Chellathurai Saroja
Implement the Service-Call Logical Processor (SCLP) event type Control-Program Identification (CPI) in QEMU. This event is used to send CPI identifiers from the guest to the host. The CPI identifiers are: system type, system name, system level and sysplex name. System type: operating system of the

[PATCH 0/5] python: add QAPI and qapidoc et al to python linter tests

2025-04-05 Thread John Snow
Hiya, this series turns on automated linting for scripts/qapi, docs/sphinx/qapidoc.py and docs/sphinx/qapi_domain.py. This includes flake8/isort/pylint/mypy for scripts/qapi, but omits mypy from the Sphinx plugins owing to my inability to strictly type the extensions given the wide versions of Sph

[PATCH-for-10.1] accel/tcg: Extract range_within_page() helper

2025-04-05 Thread Philippe Mathieu-Daudé
To ease code review, factor range_within_page() out. Signed-off-by: Philippe Mathieu-Daudé --- accel/tcg/cputlb.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/accel/tcg/cputlb.c b/accel/tcg/cputlb.c index b6713efdb81..91d185021a4 100644 --- a/accel/tcg/cputlb.c ++

[PULL 06/24] rust: pl011: Check size of state struct at compile time

2025-04-05 Thread Paolo Bonzini
From: Peter Maydell The PL011 device's C implementation exposes its PL011State struct to users of the device, and one common usage pattern is to embed that struct into the user's own state struct. (The internals of the struct are technically visible to the C user of the device, but in practice a

Re: [RFC PATCH-for-10.1 4/4] tcg: Define guest_default_memory_order in TCGCPUOps

2025-04-05 Thread Anton Johansson via
On 21/03/25, Philippe Mathieu-Daudé wrote: > Add the TCGCPUOps::guest_default_memory_order field and have > each target initialize it. > > Use it to set TCGContext::guest_mo in tb_gen_code(), removing > the need for the TCG_GUEST_DEFAULT_MO definition. > > Signed-off-by: Philippe Mathieu-Daudé >

[PATCH-for-10.1 09/24] target/i386: Restrict SoftMMU mmu_index() to TCG

2025-04-05 Thread Philippe Mathieu-Daudé
Move x86_cpu_mmu_index() to tcg-cpu.c, convert CPUClass::mmu_index() to TCGCPUOps::mmu_index(). Signed-off-by: Philippe Mathieu-Daudé --- target/i386/cpu.h| 2 -- target/i386/tcg/tcg-cpu.h| 2 ++ target/i386/cpu.c| 18 -- target/i386/tcg/seg_helper.c

[PATCH-for-10.0 2/2] hw/misc/aspeed_scu: Correct minimum access size for AST2500 / AST2600

2025-04-05 Thread Philippe Mathieu-Daudé
From: Joel Stanley Guest code was performing a byte load to the SCU MMIO region, leading to the guest code crashing (it should be using proper accessors, but that is not Qemu's bug). Hardware and the documentation[1] both agree that byte loads are okay, so change all of the aspeed SCU devices to

Re: [PATCH-for-10.1 01/24] hw/core/cpu: Update CPUClass::mmu_index docstring

2025-04-05 Thread Richard Henderson
On 4/1/25 01:09, Philippe Mathieu-Daudé wrote: Since commits 32a8ea12fab..90b7022e698 (target: "Split out TARGET_env_mmu_index"), target's memory_rw_debug() callbacks use the target's TARGET_env_mmu_index(), not the generic CPUClass::mmu_index() callback. Update the documentation. Signed-off-by:

[PATCH v4 2/3] vhost: return failure if stop virtqueue failed in vhost_dev_stop

2025-04-05 Thread Haoqian He
This patch captures the error of vhost_virtqueue_stop() in vhost_dev_stop() and returns the error upward. Specifically, if QEMU is disconnected from the vhost backend, some actions in vhost_dev_stop() will fail, such as sending vhost-user messages to the backend (GET_VRING_BASE, SET_VRING_ENABLE)

[PATCH v2 12/14] rust/vmstate: Add unit test for vmstate_{of|struct} macro

2025-04-05 Thread Zhao Liu
Add a unit test to cover some patterns accepted by vmstate_of and vmstate_struct macros, which correspond to the following C version macros: * VMSTATE_BOOL_V * VMSTATE_U64 * VMSTATE_STRUCT_VARRAY_UINT8 * (no C version) MULTIPLY variant of VMSTATE_STRUCT_VARRAY_UINT32 * VMSTATE_ARRAY Signed-o

[PATCH for-10.1 30/32] vfio: Rename VFIO dirty tracking services

2025-04-05 Thread Cédric Le Goater
Rename these routines : vfio_devices_all_device_dirty_tracking_started -> vfio_dirty_tracking_devices_is_started_all vfio_devices_all_dirty_tracking_started-> vfio_dirty_tracking_devices_is_started vfio_devices_all_device_dirty_tracking -> vfio_dirty_tracking_devices_is_su

Re: [PATCH-for-10.1 5/6] target/sparc: Register CPUClass:list_cpus

2025-04-05 Thread Thomas Huth
On 24/03/2025 17.32, Philippe Mathieu-Daudé wrote: On 24/3/25 10:30, Thomas Huth wrote: On 23/03/2025 23.40, Philippe Mathieu-Daudé wrote: Register sparc_cpu_list() as CPUClass:list_cpus callback and remove the cpu_list definition. Copy-n-paste error in both, subject and patch description: Th

Re: [PATCH-for-10.1 5/8] target/mips: Replace ldtul_p() -> ldn_p(sizeof(target_ulong))

2025-04-05 Thread Pierrick Bouvier
On 3/25/25 08:40, Philippe Mathieu-Daudé wrote: Replace the single ldtul_p() call by a generic ldn_p() one. No logical change. Signed-off-by: Philippe Mathieu-Daudé --- target/mips/gdbstub.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/target/mips/gdbstub.c b/

[PATCH v1 16/22] test/qtest/hace: Add SHA-384 tests for AST2600

2025-04-05 Thread Jamin Lin via
Introduced "test_sha384_ast2600" to validate SHA-384 hashing. Added "test_sha384_sg_ast2600" for scatter-gather SHA-384 verification. Implemented "test_sha384_accum_ast2600" to test SHA-384 accumulation. Registered new test cases in "main" to ensure execution. Signed-off-by: Jamin Lin --- tests/

Re: [PATCH v1 19/22] test/qtest/hace: Support 64-bit source and digest addresses for AST2700

2025-04-05 Thread Cédric Le Goater
On 3/21/25 10:26, Jamin Lin wrote: Added "HACE_HASH_SRC_HI" and "HACE_HASH_DIGEST_HI", "HACE_HASH_KEY_BUFF_HI" registers to store upper 32 bits. Updated "write_regs" to handle 64-bit source and digest addresses. Signed-off-by: Jamin Lin Reviewed-by: Cédric Le Goater Thanks, C. --- te

[PATCH] virtio-net: Fix the interpretation of max_tx_vq

2025-04-05 Thread Akihiko Odaki
virtio-net uses the max_tx_vq field of struct virtio_net_rss_config to determine the number of queue pairs and emits an error message saying "Can't get queue_pairs". However, the field tells only about tx. Examine the indirection table to determine the number of queues required for rx, and correct

[PATCH v2 2/3] include: Add a header to define PCI MMIO functions

2025-04-05 Thread Farhan Ali
Add a generic QEMU API for PCI MMIO reads/writes. The functions access little endian memory and returns the result in host cpu endianness. Signed-off-by: Farhan Ali --- include/qemu/pci-mmio.h | 116 1 file changed, 116 insertions(+) create mode 100644 i

[PATCH 2/2] rust: Kconfig: Factor out whether HPET is Rust or C

2025-04-05 Thread Peter Maydell
Currently we require everywhere that wants to know if there is an HPET device to check for "CONFIG_HPET || CONFIG_X_HPET_RUST". Factor out whether the HPET device is Rust or C into a separate Kconfig stanza, so that CONFIG_HPET means "there is an HPET", and whether this has pulled in CONFIG_X_HPET_

[PULL 0/3] riscv-to-apply queue

2025-04-05 Thread Alistair Francis
The following changes since commit b876e721f1c939f3e83ac85bd3c1c2821e12b3fa: Update version for v10.0.0-rc1 release (2025-03-25 13:58:14 -0400) are available in the Git repository at: https://github.com/alistair23/qemu.git tags/pull-riscv-to-apply-20250328 for you to fetch changes up to 897

Re: [PATCH v2 2/5] ipmi: add fwinfo to pci ipmi devices

2025-04-05 Thread Corey Minyard
On Tue, Apr 01, 2025 at 09:44:09PM +1000, Nicholas Piggin wrote: > This requires some adjustments to callers to avoid possible behaviour > changes for PCI devices. > > Signed-off-by: Nicholas Piggin > --- > include/hw/ipmi/ipmi.h | 5 + > hw/acpi/ipmi.c | 2 +- > hw/ipmi/is

Re: [PATCH 03/10] target/arm: convert 32 bit gdbstub to new helper

2025-04-05 Thread Pierrick Bouvier
On 3/19/25 11:22, Alex Bennée wrote: For some of the helpers we need a temporary variable to copy from although we could add some helpers to return pointers into env in those cases if we wanted to. Signed-off-by: Alex Bennée --- target/arm/gdbstub.c | 57 --

Re: [PATCH for-10.1 v2 35/37] vfio: Rename vfio-common.h to vfio-device.h

2025-04-05 Thread John Levon
On Wed, Mar 26, 2025 at 08:51:20AM +0100, Cédric Le Goater wrote: > "hw/vfio/vfio-common.h" has been emptied of most of its declarations > by the previous changes and the only declarations left are related to > VFIODevice. Rename it to "hw/vfio/vfio-device.h" and make the > necessary adjustments.

[PATCH v3 1/5] ipmi/pci-ipmi-bt: Rename copy-paste variables

2025-04-05 Thread Nicholas Piggin
IPMI drivers use p/k suffix in variable names depending on bt or kcs. The pci bt driver must have come from the kcs driver because it's still using k suffixes in some cases. Rename. Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Nicholas Piggin --- hw/ipmi/pci_ipmi_bt.c | 38 +++

[PATCH-for-10.1 23/43] target/sparc: Restrict SoftMMU mmu_index() to TCG

2025-04-05 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson --- target/sparc/cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/sparc/cpu.c b/target/sparc/cpu.c index 1bf00407af7..072d5da5736 100644 --- a/target/sparc/cpu.c +++ b/target/sparc/cpu.c @@ -1005

[PULL 6/6] docs/firmware: add feature flag for host uefi variable store

2025-04-05 Thread Gerd Hoffmann
Reviewed-by: Daniel P. Berrangé Signed-off-by: Gerd Hoffmann Message-ID: <20250319141159.1461621-7-kra...@redhat.com> --- docs/interop/firmware.json | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/docs/interop/firmware.json b/docs/interop/firmware.json index 57f5

[PATCH-for-10.1 11/24] target/m68k: Restrict SoftMMU mmu_index() to TCG

2025-04-05 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé --- target/m68k/cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/m68k/cpu.c b/target/m68k/cpu.c index 0065e1c1ca5..4409d8941ce 100644 --- a/target/m68k/cpu.c +++ b/target/m68k/cpu.c @@ -592,6 +592,7 @@ static const TCGCPUOps m6

Re: [PATCH v2 00/11] target/avr: Increase page size

2025-04-05 Thread Philippe Mathieu-Daudé
On 31/3/25 20:48, Philippe Mathieu-Daudé wrote: On 25/3/25 23:43, Richard Henderson wrote: Richard Henderson (11):    target/avr: Fix buffer read in avr_print_insn >>hw/core/cpu: Use size_t for memory_rw_debug len argument Patch 1 queued for 10.1, thanks! Also patch 3 ;)

Re: [PATCH] rust: hpet: fix decoding of timer registers

2025-04-05 Thread Paolo Bonzini
On Fri, Mar 21, 2025 at 3:26 PM Peter Maydell wrote: > Tested-by: Peter Maydell > Reviewed-by: Peter Maydell > > If I understand the code correctly I think you could also > write this as "addr & 0x1f" which might be a little nicer > as it then lines up with the "/ 0x20". Yeah, I was undecided b

[RFC PATCH-for-10.1 0/4] tcg: Move TCG_GUEST_DEFAULT_MO -> TCGCPUOps::guest_default_memory_order

2025-04-05 Thread Philippe Mathieu-Daudé
Hi, In this series we replace the TCG_GUEST_DEFAULT_MO definition from "cpu-param.h" by a 'guest_default_memory_order' field in TCGCPUOps. Since tcg_req_mo() now accesses tcg_ctx, this impact the cpu_req_mo() calls in accel/tcg/{cputlb,user-exec}.c. The long term goal is to be able to use target

Re: [PATCH 05/13] target/arm/cpu: move KVM_HAVE_MCE_INJECTION to kvm-all.c file directly

2025-04-05 Thread Pierrick Bouvier
On 3/18/25 15:19, Richard Henderson wrote: On 3/17/25 21:51, Pierrick Bouvier wrote: diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c index f89568bfa39..28de3990699 100644 --- a/accel/kvm/kvm-all.c +++ b/accel/kvm/kvm-all.c @@ -13,6 +13,10 @@ * */ +#ifdef TARGET_AARCH64 +#defi

[PATCH v1 14/22] test/qtest/hace: Adjust test address range for AST1030 due to SRAM limitations

2025-04-05 Thread Jamin Lin via
The digest_addr is set to "src_addr + 0x100", where src_addr is the DRAM base address. However, the value 0x100 (16MB) is too large because the AST1030 does not support DRAM, and its SRAM size is only 768KB. A range size of 0x1000 (64KB) is sufficient for HACE test cases, as the test vecto

Re: [PATCH v6] hw/misc/vmfwupdate: Introduce hypervisor fw-cfg interface support

2025-04-05 Thread Jörg Rödel
On Tue, Mar 18, 2025 at 12:11:02PM +0100, Gerd Hoffman wrote: > Open questions: > > - Does the idea to use igvm parameters for the kernel hashes makes >sense? Are parameters part of the launch measurement? Parameters itself are fully measured, their presence is, but not their data. This is

[PATCH v3 14/15] rust/vmstate: Add unit test for vmstate_validate

2025-04-05 Thread Zhao Liu
Add a unit test for vmstate_validate, which corresponds to the C version macro: VMSTATE_VALIDATE. Signed-off-by: Zhao Liu --- rust/qemu-api/tests/vmstate_tests.rs | 91 +++- 1 file changed, 89 insertions(+), 2 deletions(-) diff --git a/rust/qemu-api/tests/vmstate_tests.r

[PATCH for-10.1 12/32] vfio: Make vfio_group_list static

2025-04-05 Thread Cédric Le Goater
vfio_group_list is only used in file "container.c". Signed-off-by: Cédric Le Goater --- include/hw/vfio/vfio-common.h | 2 -- hw/vfio/container.c | 3 ++- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/include/hw/vfio/vfio-common.h b/include/hw/vfio/vfio-common.h index

Re: [PATCH-for-10.0 1/1] goldfish_rtc: keep time offset when resetting

2025-04-05 Thread Heinrich Schuchardt
On 21.03.25 17:08, Philippe Mathieu-Daudé wrote: Hi Heinrich, On 21/3/25 09:12, Heinrich Schuchardt wrote: Currently resetting leads to resynchronizing the Goldfish RTC with the system clock of the host. In real hardware an RTC reset would not change the wall time. Other RTCs like pl031 do not

Re: Raspberry Pi 3B energy consumption

2025-04-05 Thread clement . aldebert
Dear Peter Maydell, Thank you for your detailed response. We will take a closer look at why, when using WFI, the affected cores still appear to be at 100% utilization in htop. Additionally, we will investigate whether implementing WFE in QEMU would be necessary to achieve proper CPU core shutd

[PATCH v2 09/14] rust/vmstate: Re-implement VMState trait for timer binding

2025-04-05 Thread Zhao Liu
At present, Rust side has a timer binding "timer::Timer", so the vmstate for timer should base on that binding instead of the raw "binding::QEMUTimer". It's possible to apply impl_vmstate_transparent for cell::Opaque and then impl_vmstate_forward for timer::Timer. But binding::QEMUTimer shouldn't

Re: [PATCH] smbios: Fix buffer overrun when using path= option

2025-04-05 Thread Philippe Mathieu-Daudé
+Valentin On 23/3/25 22:35, Daan De Meyer wrote: We have to make sure the array of bytes read from the path= file is null-terminated, otherwise we run into a buffer overrun later on. Fixes: bb99f4772f54017490e3356ecbb3df25c5d4537f ("hw/smbios: support loading OEM strings values from a file") R

Re: [PATCH v6] hw/misc/vmfwupdate: Introduce hypervisor fw-cfg interface support

2025-04-05 Thread Daniel P . Berrangé
On Fri, Mar 21, 2025 at 11:08:11AM +0100, Gerd Hoffman wrote: > Hi, > > > > While digging around in the igvm spec I've seen there is the > > > concept of 'parameters'. Can this be used to pass on the memory > > > location of kernel + initrd + cmdline? Maybe the kernel hashes too

[PATCH v3 02/29] exec/cpu-all: move cpu_copy to linux-user/qemu.h

2025-04-05 Thread Pierrick Bouvier
Reviewed-by: Richard Henderson Signed-off-by: Pierrick Bouvier --- include/exec/cpu-all.h | 2 -- linux-user/qemu.h | 3 +++ 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/include/exec/cpu-all.h b/include/exec/cpu-all.h index d2895fb55b1..74017a5ce7c 100644 --- a/include/exe

Re: [RFC v2 0/5] accel/kvm: Support KVM PMU filter

2025-04-05 Thread Shaoqin Huang
Hi Zhao, On 3/21/25 11:43 AM, Zhao Liu wrote: Hi Shaoqin, Thank you very much for testing! I tried your series on ARM64, but it reports error at compile time, here is the error output: qapi/kvm.json:59:Unexpected indentation. I guess this is caused by my invalid format and sphinx complains

Re: [PATCH v1 20/22] test/qtest/hace: Support to test upper 32 bits of digest and source addresses

2025-04-05 Thread Cédric Le Goater
On 3/21/25 10:26, Jamin Lin wrote: Added "src_hi" and "dest_hi" fields to "AspeedMasks" for 64-bit addresses test. Updated "aspeed_test_addresses" to validate "HACE_HASH_SRC_HI" and "HACE_HASH_DIGEST_HI". Ensured correct masking of 64-bit addresses by checking both lower and upper 32-bit register

[PATCH for-10.1 v4 10/13] arm/cpu: Store id_dfr0/1 into the idregs array

2025-04-05 Thread Cornelia Huck
From: Eric Auger Reviewed-by: Richard Henderson Reviewed-by: Sebastian Ott Signed-off-by: Eric Auger Signed-off-by: Cornelia Huck --- hw/intc/armv7m_nvic.c | 2 +- target/arm/cpu-features.h | 16 target/arm/cpu.c | 13 + target/arm/cpu.h |

Re: [PATCH 6/6] tests/qtest: Enable bios-tables-test for LoongArch

2025-04-05 Thread Fabiano Rosas
Bibo Mao writes: > Enable bios-tables-test for LoongArch virt machine, it passes to run > with command "make check-qtest-loongarch64". > > Signed-off-by: Bibo Mao > --- > tests/qtest/meson.build | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tests/qtest/meson.build b

Re: [PATCH v2 0/2] RISC-V: ACPI: Add support for RIMT

2025-04-05 Thread Alistair Francis
On Sat, Mar 22, 2025 at 2:32 PM Sunil V L wrote: > > RISC-V IO Mapping Table (RIMT) is a new static ACPI table used to > communicate IOMMU and topology information to the OS. Add support for > creating this table when the IOMMU is present. The specification is > frozen and available at [1]. > > [1

Re: [PATCH v2 23/30] target/arm/cpu: remove inline stubs for aarch32 emulation

2025-04-05 Thread Richard Henderson
On 3/20/25 15:29, Pierrick Bouvier wrote: Directly condition associated calls in target/arm/helper.c for now. Signed-off-by: Pierrick Bouvier --- target/arm/cpu.h| 8 target/arm/helper.c | 6 ++ 2 files changed, 6 insertions(+), 8 deletions(-) Reviewed-by: Richard Henders

Re: [PATCH v3 6/7] target/s390x: Register CPUClass:list_cpus

2025-04-05 Thread Richard Henderson
On 3/24/25 11:58, Philippe Mathieu-Daudé wrote: Register s390_cpu_list() as CPUClass:list_cpus callback and remove the cpu_list definition. Signed-off-by: Philippe Mathieu-Daudé --- target/s390x/cpu.h | 3 --- target/s390x/cpu.c | 1 + 2 files changed, 1 insertion(+), 3 deletions(-) I rea

Re: [PATCH-for-10.0] tcg: Allocate TEMP_VAL_MEM frame in temp_load()

2025-04-05 Thread Richard Henderson
On 4/1/25 10:02, Richard Henderson wrote: On 4/1/25 09:43, Philippe Mathieu-Daudé wrote: Be sure to allocate the temp frame if it wasn't. Fixes: c896fe29d6c ("TCG code generator") Reported-by: Michael Tokarev Reported-by: Helge Konetzka Resolves: https://gitlab.com/qemu-project/qemu/-/issues/

Re: [PATCH for-10.1 09/32] vfio: Introduce a new header file for VFIOIOMMUFD declarations

2025-04-05 Thread John Levon
On Tue, Mar 18, 2025 at 10:53:52AM +0100, Cédric Le Goater wrote: > Gather all VFIOIOMMUFD related declarations into "iommufd.h" to > reduce exposure of VFIO internals in "hw/vfio/vfio-common.h". > > Signed-off-by: Cédric Le Goater Reviewed-by: John Levon regards john

[PATCH-for-10.1 19/43] target/rx: Fix copy/paste typo (riscv -> rx)

2025-04-05 Thread Philippe Mathieu-Daudé
Rename riscv_cpu_mmu_index() -> rx_cpu_mmu_index(). Fixes: ef5cc166da1 ("target/rx: Populate CPUClass.mmu_index") Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson --- target/rx/cpu.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/target/rx/cpu.c b/ta

[PATCH v3 0/4] *** Add SCLP event type CPI ***

2025-04-05 Thread Shalini Chellathurai Saroja
Implement the Service-Call Logical Processor (SCLP) event type Control-Program Identification (CPI) in QEMU. Changed since v2: - Add SPDX license tag in the new file hw/s390x/sclpcpi.c - Store the control-program Identification data in the sclpcpi device - Update the description of CPI attributes

[PULL 10/23] hw/rtc/goldfish: keep time offset when resetting

2025-04-05 Thread Philippe Mathieu-Daudé
From: Heinrich Schuchardt Currently resetting the leads to resynchronizing the Goldfish RTC with the system clock of the host. In real hardware an RTC reset would not change the wall time. Other RTCs like pl031 do not show this behavior. Move the synchronization of the RTC with the system clock

[PATCH 2/2] hvf: only update sysreg from owning thread

2025-04-05 Thread Mads Ynddal
From: Mads Ynddal hv_vcpu_set_sys_reg should only be called from the owning thread of the vCPU, so to avoid crashes, the call to hvf_update_guest_debug is dispatched to the individual threads. Tested-by: Daniel Gomez Signed-off-by: Mads Ynddal --- accel/hvf/hvf-all.c | 7 ++- 1 file chang

Re: Generic way to detect qemu linux-user emulation

2025-04-05 Thread Andreas Schwab
On Mär 18 2025, Helge Deller wrote: > My current (unreliable) way to detect it is using uname. > The kernel string and arch name don't match: > > (sid_hppa)root@paq:/# uname -a > Linux paq 6.1.0-31-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.128-1 (2025-02-07) > parisc GNU/Linux > > (sid_hppa)root@pa

[PATCH v2 03/12] target/riscv: Add vext_check_input_eew to check mismatched input EEWs encoding constraint

2025-04-05 Thread Max Chou
According to the v spec, a vector register cannot be used to provide source operands with more than one EEW for a single instruction. Signed-off-by: Max Chou --- target/riscv/insn_trans/trans_rvv.c.inc | 29 + 1 file changed, 29 insertions(+) diff --git a/target/riscv/in

Re: [PATCH] bsd-user: add option to enable plugins

2025-04-05 Thread Pierrick Bouvier
On 3/31/25 16:42, Pierrick Bouvier wrote: Nothing prevent plugins to be enabled on this platform for user binaries, only the option in the driver is missing. Signed-off-by: Pierrick Bouvier --- bsd-user/main.c | 12 1 file changed, 12 insertions(+) diff --git a/bsd-user/main.c

Re: [PATCH v4 2/4] hw/loongarch/virt: Remove unnecessary NULL pointer

2025-04-05 Thread Markus Armbruster
bibo mao writes: On 2025/3/19 下午2:50, Markus Armbruster wrote: >> Bibo Mao writes: >> >>> There is NULL pointer checking function error_propagate() already, >>> it is not necessary to add checking for function parameter. Here remove >>> NULL pointer checking with function parameter. >> >> I be

[PATCH-for-10.1 6/6] cpus: Remove #ifdef check on cpu_list definition

2025-04-05 Thread Philippe Mathieu-Daudé
Since we removed all definitions of cpu_list, the #ifdef check is always true. Remove it, inlining cpu_list(). Signed-off-by: Philippe Mathieu-Daudé --- cpu-target.c | 19 ++- 1 file changed, 6 insertions(+), 13 deletions(-) diff --git a/cpu-target.c b/cpu-target.c index 5947ca3

Re: [PATCH 1/2] i386/xen: Move KVM_XEN_HVM_CONFIG ioctl to kvm_xen_init_vcpu()

2025-04-05 Thread David Woodhouse
On Fri, 2025-02-07 at 14:37 +, David Woodhouse wrote: > From: David Woodhouse > > At the time kvm_xen_init() is called, hyperv_enabled() doesn't yet work, so > the correct MSR index to use for the hypercall page isn't known. > > Rather than setting it to the default and then shifting it late

Re: [PATCH v2] target/ppc: Deprecate Power8E and Power8NVL

2025-04-05 Thread Aditya Gupta
On 25/03/29 06:42PM, Philippe Mathieu-Daudé wrote: > Hi Aditya, > > On 29/3/25 15:26, Aditya Gupta wrote: > > <...snip...> > > > > --- a/target/ppc/cpu-models.c > > +++ b/target/ppc/cpu-models.c > > @@ -32,17 +32,22 @@ > > /* PowerPC CPU definitions

Re: [PATCH 14/15] fuse: Implement multi-threading

2025-04-05 Thread Stefan Hajnoczi
On Tue, Apr 01, 2025 at 03:36:40PM -0500, Eric Blake wrote: > On Thu, Mar 27, 2025 at 11:55:57AM -0400, Stefan Hajnoczi wrote: > > On Tue, Mar 25, 2025 at 05:06:54PM +0100, Hanna Czenczek wrote: > > > FUSE allows creating multiple request queues by "cloning" /dev/fuse FDs > > > (via open("/dev/fuse

Re: [PATCH for-10.1 30/32] vfio: Rename VFIO dirty tracking services

2025-04-05 Thread Avihai Horon
On 20/03/2025 13:18, Joao Martins wrote: External email: Use caution opening links or attachments On 20/03/2025 11:13, Avihai Horon wrote: On 19/03/2025 14:21, Joao Martins wrote: External email: Use caution opening links or attachments On 18/03/2025 09:54, Cédric Le Goater wrote: Rename

Re: [PATCH 05/17] target/avr: Move cpu register accesses into system memory

2025-04-05 Thread Richard Henderson
On 3/24/25 18:07, Pierrick Bouvier wrote: A simple nit, maybe we could define constant for register names. This way, it can be used in the two switch for read/access. Which constant? - The absolute address (0x58-0x5f) - The i/o port address (0x38-0x3f) - or the region offset (0-7) ? Th

Re: [PATCH-for-10.1 v2 42/43] tcg: Convert TARGET_SUPPORTS_MTTCG to TCGCPUOps::mttcg_supported field

2025-04-05 Thread Richard Henderson
On 4/2/25 14:23, Philippe Mathieu-Daudé wrote: Instead of having a compile-time TARGET_SUPPORTS_MTTCG definition, have each target set the 'mttcg_supported' field in the TCGCPUOps structure. Since so far we only emulate one target architecture at a time, tcg_init_machine() gets whether MTTCG is

Re: [RFC PATCH v2 19/20] hw/arm/virt-acpi-build: Update IORT with multiple smmuv3-accel nodes

2025-04-05 Thread Nicolin Chen
On Wed, Mar 26, 2025 at 07:14:31PM +0100, Eric Auger wrote: > > > On 3/11/25 3:10 PM, Shameer Kolothum wrote: > > Now that we can have multiple user-creatable smmuv3-accel devices, > > each associated with different pci buses, update IORT ID mappings > > accordingly. > > > > Signed-off-by: Shamee

Re: [PATCH v1 12/22] hw/misc/aspeed_hace Support to dump plaintext and digest for better debugging

2025-04-05 Thread Cédric Le Goater
On 3/21/25 10:26, Jamin Lin wrote: 1. Disabled by default. Uncomment "#define DEBUG_HACE 1" to enable it. 2. Uses the "qemu_hexdump" API to dump the digest result. 3. Uses the "iov_hexdump" API to dump the source vector, which contains the source plaintext. Signed-off-by: Jamin Lin --- hw

Re: [PATCH-for-10.1 1/4] target/riscv: Restrict RV128 MTTCG check on system emulation

2025-04-05 Thread Philippe Mathieu-Daudé
On 23/3/25 19:08, Richard Henderson wrote: On 3/21/25 08:59, Philippe Mathieu-Daudé wrote: Multi-threaded TCG only concerns system emulation. That's not really true.  User emulation simply has no option to run in a single-threaded context. I really don't think we should allow RV128 in user-mo

[PATCH 01/15] fuse: Copy write buffer content before polling

2025-04-05 Thread Hanna Czenczek
Polling in I/O functions can lead to nested read_from_fuse_export() calls, overwriting the request buffer's content. The only function affected by this is fuse_write(), which therefore must use a bounce buffer or corruption may occur. Note that in addition we do not know whether libfuse-internal

[PATCH v2 28/30] hw/arm/xlnx-zynqmp: prepare compilation unit to be common

2025-04-05 Thread Pierrick Bouvier
Signed-off-by: Pierrick Bouvier --- hw/arm/xlnx-zynqmp.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/hw/arm/xlnx-zynqmp.c b/hw/arm/xlnx-zynqmp.c index d6022ff2d3d..ec2b3a41eda 100644 --- a/hw/arm/xlnx-zynqmp.c +++ b/hw/arm/xlnx-zynqmp.c @@ -22,9 +22,7 @@ #include "hw/intc/arm_gic_common

[PATCH v3 3/7] target/ppc: Register CPUClass:list_cpus

2025-04-05 Thread Philippe Mathieu-Daudé
Register ppc_cpu_list() as CPUClass:list_cpus callback. Reduce its scope and remove the cpu_list definition. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Thomas Huth Reviewed-by: Richard Henderson --- target/ppc/cpu.h | 4 target/ppc/cpu_init.c | 3 ++- 2 files changed, 2 inser

Re: [PATCH 01/10] include/gdbstub: fix include guard in commands.h

2025-04-05 Thread Pierrick Bouvier
On 3/19/25 11:22, Alex Bennée wrote: Signed-off-by: Alex Bennée --- include/gdbstub/commands.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/gdbstub/commands.h b/include/gdbstub/commands.h index 40f0514fe9..bff3674872 100644 --- a/include/gdbstub/commands.h +++ b

  1   2   3   4   5   >