Re: [PATCH 09/23] qapi/source: allow multi-line QAPISourceInfo advancing

2025-01-09 Thread Markus Armbruster
John Snow writes: > On Fri, Dec 20, 2024 at 8:22 AM Markus Armbruster wrote: > >> John Snow writes: >> >> > This is for the sake of the new rST generator (the "transmogrifier") so >> > we can advance multiple lines on occasion while keeping the >> > generated<-->source mappings accurate. >> > >

Re: [PATCH 5/7] memory: Register the RamDiscardManager instance upon guest_memfd creation

2025-01-09 Thread Chenyi Qiang
Thanks Zhao for your review! On 1/9/2025 4:14 PM, Zhao Liu wrote: >> #ifdef CONFIG_FALLOCATE_PUNCH_HOLE >> @@ -1885,6 +1886,9 @@ static void ram_block_add(RAMBlock *new_block, Error >> **errp) >> qemu_mutex_unlock_ramlist(); >> goto out_free; >> } >> + >> +

Re: [PATCH 0/7] Enable shared device assignment

2025-01-09 Thread Alexey Kardashevskiy
On 9/1/25 18:52, Chenyi Qiang wrote: On 1/8/2025 7:38 PM, Alexey Kardashevskiy wrote: On 8/1/25 17:28, Chenyi Qiang wrote: Thanks Alexey for your review! On 1/8/2025 12:47 PM, Alexey Kardashevskiy wrote: On 13/12/24 18:08, Chenyi Qiang wrote: Commit 852f0048f3 ("RAMBlock: make guest_m

Re: [PATCH 0/7] Enable shared device assignment

2025-01-09 Thread Chenyi Qiang
On 1/10/2025 9:42 AM, Alexey Kardashevskiy wrote: > > > On 9/1/25 19:49, Chenyi Qiang wrote: >> >> >> On 1/9/2025 4:18 PM, Alexey Kardashevskiy wrote: >>> >>> >>> On 9/1/25 18:52, Chenyi Qiang wrote: On 1/8/2025 7:38 PM, Alexey Kardashevskiy wrote: > > > On 8/1/25 17

Re: [PATCH v7 1/5] hw/core/machine: Reject thread level cache

2025-01-09 Thread Zhao Liu
Hi Jonathon, Thanks for more explaination! Based on your clarification, I think the commit message for Patch 1 needs to be updated since I used the same wrods as the cover letter... What about the following change? On Wed, Jan 08, 2025 at 11:01:46PM +0800, Zhao Liu wrote: > Date: Wed, 8 Jan 202

Re: [PATCH v7 0/9] target/riscv: Add support for Smdbltrp and Ssdbltrp extensions

2025-01-09 Thread Clément Léger
On 10/01/2025 03:12, Alistair Francis wrote: > On Tue, Dec 17, 2024 at 7:10 PM Clément Léger wrote: >> >> A double trap typically arises during a sensitive phase in trap handling >> operations — when an exception or interrupt occurs while the trap >> handler (the component responsible for manag

Re: [PATCH] contrib/plugins/bbv.c: Start bb index from 1

2025-01-09 Thread Pierrick Bouvier
On 12/27/24 15:50, Alex Bennée wrote: Michael Tokarev writes: 17.12.2024 17:24, ckf104 wrote: Standard simpoint tool reqeusts that index of basic block index starts from 1. While this patch is a trivial one-liner, but the underlying issue requires at least a minimal understanding of what i

Re: [PATCH v2 40/81] tcg/ppc: Fold the ext{8,16,32}[us] cases into {s}extract

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Accept byte and word extensions with the extract opcodes. This is preparatory to removing the specialized extracts. Signed-off-by: Richard Henderson --- tcg/ppc/tcg-target-has.h | 16 ++-- tcg/ppc/tcg-target.c.inc | 30 +++

RE: [PATCH] plugins: add plugin API to get args passed to binary

2025-01-09 Thread Demin Han
Hi, > -Original Message- > From: Alex Bennée > Sent: 2025年1月9日 19:59 > To: Demin Han > Cc: qemu-devel@nongnu.org; erdn...@crans.org; ma.mando...@gmail.com; > pierrick.bouv...@linaro.org > Subject: Re: [PATCH] plugins: add plugin API to get args passed to binary > > "demin.han" writes:

Re: [PATCH v4 10/11] target/riscv: Add implied rule for counter delegation extensions

2025-01-09 Thread Alistair Francis
On Wed, Dec 4, 2024 at 9:17 AM Atish Patra wrote: > > The counter delegation/configuration extensions depend on the following > extensions. > > 1. Smcdeleg - To enable counter delegation from M to S > 2. S[m|s]csrind - To enable indirect access CSRs > > Add an implied rule so that these extensions

Re: Subject: [PATCH] loader: Add register setting support via cli

2025-01-09 Thread Sam Price
Yes that is true a boot loader will do more than just set registers. Ill rework the text a bit on the next update. In my case i need to set the r5 register that specifies the memory location to the device tree. I also use the device loader to load in a elf file to ram, and the device-loader to load

CXL emulation on aarch64

2025-01-09 Thread Itaru Kitayama
Hi, Is anybody working on the CXL emulation on aarch64? If there’s a WIP branch, a pointer would be appreciated. Itaru.

Re: [PATCH] virtio-net: Fix num_buffers for version 1

2025-01-09 Thread Lei Yang
I tested this patch with virtio-net regression tests and tests "mrg_rxbuf" on/off status with netperf tests, everything works fine. Tested-by: Lei Yang On Wed, Jan 8, 2025 at 8:23 PM Akihiko Odaki wrote: > > The specification says the device MUST set num_buffers to 1 if > VIRTIO_NET_F_MRG_RXBUF

Re: [PATCH 04/23] qapi: expand tags to all doc sections

2025-01-09 Thread Markus Armbruster
John Snow writes: > On Fri, Dec 20, 2024, 8:13 AM Markus Armbruster wrote: > >> John Snow writes: >> >> > This patch adds an explicit section tag to all QAPIDoc >> > sections. Members/Features are now explicitly tagged as such, with the >> > name now being stored in a dedicated "name" field (wh

Re: [PATCH 08/22] system/vl: more error exit into config enumeration code

2025-01-09 Thread Philippe Mathieu-Daudé
On 9/1/25 18:06, Alex Bennée wrote: All of the failures to configure devices will result in QEMU exiting with an error code. In preparation for passing Error * down the chain re-name the iterator to foreach_device_config_or_exit and exit using &error_fatal instead of returning a failure indicatio

Re: [PATCH 09/23] qapi/source: allow multi-line QAPISourceInfo advancing

2025-01-09 Thread Markus Armbruster
John Snow writes: > On Thu, Jan 9, 2025, 3:00 AM Markus Armbruster wrote: > >> John Snow writes: [...] >> > Modifying freeform syntax to be purely rST that isn't modified or rewritten >> > at all has benefits: >> > >> > - No need to mangle or multiplex source line source information >> > - Less

Re: [PATCH 2/7] guest_memfd: Introduce an object to manage the guest-memfd with RamDiscardManager

2025-01-09 Thread Chenyi Qiang
On 1/10/2025 8:58 AM, Alexey Kardashevskiy wrote: > > > On 9/1/25 15:29, Chenyi Qiang wrote: >> >> >> On 1/9/2025 10:55 AM, Alexey Kardashevskiy wrote: >>> >>> >>> On 9/1/25 13:11, Chenyi Qiang wrote: On 1/8/2025 7:20 PM, Alexey Kardashevskiy wrote: > > > On 8/1/25 2

Re: [PATCH 3/3] accel/tcg: Implement cpu_exec_reset_hold() on user emulation

2025-01-09 Thread Ilya Leoshkevich
On Thu, 2025-01-02 at 19:25 +0100, Philippe Mathieu-Daudé wrote: > Commit bb6cf6f0168 ("accel/tcg: Factor tcg_cpu_reset_hold() > out") wanted to restrict tlb_flush() to system emulation, > but inadvertently also restricted tcg_flush_jmp_cache(), > which was before called on user emulation via: > >

Re: [PATCH v2 38/81] tcg/loongarch64: Fold the ext{8, 16, 32}[us] cases into {s}extract

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Accept byte and word extensions with the extract opcodes. This is preparatory to removing the specialized extracts. Signed-off-by: Richard Henderson --- tcg/loongarch64/tcg-target-has.h | 15 -- tcg/loongarch64/tcg-target.c.inc | 34 +

Re: [PATCH v2 44/81] tcg/sparc64: Use SRA, SRL for {s}extract_i64

2025-01-09 Thread Philippe Mathieu-Daudé
On 10/1/25 00:44, Philippe Mathieu-Daudé wrote: On 10/1/25 00:00, Philippe Mathieu-Daudé wrote: On 7/1/25 09:00, Richard Henderson wrote: Extracts which abut bit 32 may use 32-bit shifts. (Fix typos?) Signed-off-by: Richard Henderson ---   tcg/sparc64/tcg-target-has.h |  9 +++--   tcg

Re: [PATCH v2 44/81] tcg/sparc64: Use SRA, SRL for {s}extract_i64

2025-01-09 Thread Philippe Mathieu-Daudé
On 10/1/25 00:00, Philippe Mathieu-Daudé wrote: On 7/1/25 09:00, Richard Henderson wrote: Extracts which abut bit 32 may use 32-bit shifts. Signed-off-by: Richard Henderson ---   tcg/sparc64/tcg-target-has.h |  9 +++--   tcg/sparc64/tcg-target.c.inc | 11 +++   2 files changed, 18 i

Re: [PATCH] gdbstub/user-target: fix gdbserver int format (%d -> %x)

2025-01-09 Thread Ilya Leoshkevich
On Fri, 2024-12-27 at 18:04 +0100, Dominik 'Disconnect3d' Czarnota wrote: > From: disconnect3d > > This commit fixes an incorrect format string for formatting integers > provided to GDB when debugging a target run in QEMU user mode. > > The correct format is hexadecimal for both success and errn

Re: [PATCH v4 04/11] target/riscv: Support generic CSR indirect access

2025-01-09 Thread Alistair Francis
On Wed, Dec 4, 2024 at 9:17 AM Atish Patra wrote: > > From: Kaiwen Xue > > This adds the indirect access registers required by sscsrind/smcsrind > and the operations on them. Note that xiselect and xireg are used for > both AIA and sxcsrind, and the behavior of accessing them depends on > whether

Re: [PATCH v4 05/11] target/riscv: Add properties for counter delegation ISA extensions

2025-01-09 Thread Alistair Francis
On Wed, Dec 4, 2024 at 9:16 AM Atish Patra wrote: > > This adds the properties for counter delegation ISA extensions > (Smcdeleg/Ssccfg). Definitions of new registers and and implementation > will come in the next set of patches. > > Reviewed-by: Daniel Henrique Barboza > Signed-off-by: Atish Pat

Re: [PATCH] vvfat: fix out of bounds array write

2025-01-09 Thread Pierrick Bouvier
Hi Volker, On 1/5/25 05:59, Volker Rümelin wrote: In function create_long_filname(), the array name[8 + 3] in struct direntry_t is used as if it were defined as name[32]. This is intentional and works. It's nevertheless an out of bounds array access. To avoid this problem, this patch adds a stru

Re: [PATCH 0/7] Enable shared device assignment

2025-01-09 Thread Alexey Kardashevskiy
On 9/1/25 19:49, Chenyi Qiang wrote: On 1/9/2025 4:18 PM, Alexey Kardashevskiy wrote: On 9/1/25 18:52, Chenyi Qiang wrote: On 1/8/2025 7:38 PM, Alexey Kardashevskiy wrote: On 8/1/25 17:28, Chenyi Qiang wrote: Thanks Alexey for your review! On 1/8/2025 12:47 PM, Alexey Kardashevski

Re: [PATCH 5/7] memory: Register the RamDiscardManager instance upon guest_memfd creation

2025-01-09 Thread Chenyi Qiang
On 1/9/2025 5:32 PM, Alexey Kardashevskiy wrote: > > > On 9/1/25 16:34, Chenyi Qiang wrote: >> >> >> On 1/8/2025 12:47 PM, Alexey Kardashevskiy wrote: >>> On 13/12/24 18:08, Chenyi Qiang wrote: Introduce the realize()/unrealize() callbacks to initialize/ uninitialize the new gue

Re: [PATCH v4 2/2] tests/qtest/libqos: add DMA support for writing and reading fw_cfg files

2025-01-09 Thread Ani Sinha
> On 10 Jan 2025, at 2:00 AM, Fabiano Rosas wrote: > > Ani Sinha writes: > >> At present, the libqos/fw_cfg.c library does not support the modern DMA >> interface which is required to write to the fw_cfg files. It only uses the IO >> interface. Implement read and write methods based on DMA.

Re: [PATCH] qmp: update vhost-user protocol feature maps

2025-01-09 Thread Philippe Mathieu-Daudé
On 6/1/25 14:25, Jonah Palmer wrote: On 12/22/24 4:13 AM, Michael Tokarev wrote: 16.12.2024 20:30, Jonah Palmer wrote: ... Signed-off-by: Jonah Palmer Should this be a Reviewed-by instead? Thanks, /mjt Oof, yes, it should've been. My apologies, I was on vacation and only saw this just

Re: [PATCH 2/7] guest_memfd: Introduce an object to manage the guest-memfd with RamDiscardManager

2025-01-09 Thread Alexey Kardashevskiy
On 9/1/25 15:29, Chenyi Qiang wrote: On 1/9/2025 10:55 AM, Alexey Kardashevskiy wrote: On 9/1/25 13:11, Chenyi Qiang wrote: On 1/8/2025 7:20 PM, Alexey Kardashevskiy wrote: On 8/1/25 21:56, Chenyi Qiang wrote: On 1/8/2025 12:48 PM, Alexey Kardashevskiy wrote: On 13/12/24 18:08,

Re: [PATCH v4 1/2] libqos/fw_cfg: refactor file directory iteraton to make it more reusable

2025-01-09 Thread Ani Sinha
> On 10 Jan 2025, at 1:31 AM, Fabiano Rosas wrote: > > Ani Sinha writes: > >> fw-cfg file directory iteration code can be used by other functions that may >> want to implement fw-cfg file operations. Refactor it into a smaller helper >> so that it can be reused. >> >> No functional change.

Re: [PATCH v2 44/81] tcg/sparc64: Use SRA, SRL for {s}extract_i64

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Extracts which abut bit 32 may use 32-bit shifts. Signed-off-by: Richard Henderson --- tcg/sparc64/tcg-target-has.h | 9 +++-- tcg/sparc64/tcg-target.c.inc | 11 +++ 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/tcg/sp

Re: [PATCH v2 39/81] tcg/mips: Fold the ext{8,16,32}[us] cases into {s}extract

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Accept AND, ext32u, ext32s extensions with the extract opcodes. This is preparatory to removing the specialized extracts. Signed-off-by: Richard Henderson --- tcg/mips/tcg-target-has.h | 26 ++ tcg/mips/tcg-target.c.inc | 33 +

Re: [PATCH v4 0/4] virtio: Convert feature properties to OnOffAuto

2025-01-09 Thread Akihiko Odaki
t;[PATCH v2 0/4] hw/pci: Convert rom_bar into OnOffAuto"? https://lore.kernel.org/all/20240714-rombar-v2-0-af1504ef5...@daynix.com/ > They are orthogonal. "[PATCH v2 0/4] hw/pci: Convert rom_bar into OnOffAuto" was also abandoned in favor of "[PATCH v19 13/14] hw/pci: Use

Re: [PATCH 2/3] accel/tcg: Factor out common tcg_exec_reset() helper

2025-01-09 Thread Ilya Leoshkevich
On Thu, 2025-01-02 at 19:25 +0100, Philippe Mathieu-Daudé wrote: > Since tcg_cpu_reset_hold() is a system emulation specific > helper, factor tcg_exec_reset() out so we can use it from > user emulation, similarly to the [un]realize() handlers. > > Signed-off-by: Philippe Mathieu-Daudé > --- >  ac

Re: [PATCH v2 56/81] tcg/optimize: Fold andc with immediate to and

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Signed-off-by: Richard Henderson --- tcg/optimize.c | 19 +++ 1 file changed, 19 insertions(+) Reviewed-by: Philippe Mathieu-Daudé

Re: [PATCH v2 67/81] tcg/optimize: Fold eqv with immediate to xor

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Signed-off-by: Richard Henderson --- tcg/optimize.c | 25 +++-- 1 file changed, 23 insertions(+), 2 deletions(-) Reviewed-by: Philippe Mathieu-Daudé

Re: [PATCH v2 57/81] tcg/optimize: Emit add r,r,-1 in fold_setcond_tst_pow2

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: We canonicalize subtract with constant to add with constant. Fix this missed instance. Signed-off-by: Richard Henderson --- tcg/optimize.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) Reviewed-by: Philippe Mathieu-Daudé

Re: [PULL 04/11] vvfat: fix ubsan issue in create_long_filename

2025-01-09 Thread Pierrick Bouvier
Hi Volker, On 12/29/24 01:24, Volker Rümelin wrote: Found with test sbsaref introduced in [1]. [1] https://patchew.org/QEMU/20241203213629.2482806-1-pierrick.bouv...@linaro.org/ ../block/vvfat.c:433:24: runtime error: index 14 out of bounds for type 'uint8_t [11]' #0 0x56151a66b93a in c

Re: Subject: [PATCH] loader: Add register setting support via cli

2025-01-09 Thread Alistair Francis
On Wed, Jan 8, 2025 at 12:28 PM Sam Price wrote: > > I made the changes, and added documentation. > https://gitlab.com/thesamprice/qemu/-/compare/master...loader?from_project_id=11167699 > > I left it as [PREFIX] > > I can switch this to just RegNumber if desired. > > I am still struggling with th

Re: [PATCH v4 01/11] target/riscv: Add properties for Indirect CSR Access extension

2025-01-09 Thread Alistair Francis
On Wed, Dec 4, 2024 at 9:17 AM Atish Patra wrote: > > From: Kaiwen Xue > > This adds the properties for sxcsrind. Definitions of new registers and > implementations will come with future patches. > > Signed-off-by: Kaiwen Xue > Reviewed-by: Daniel Henrique Barboza > Signed-off-by: Atish Patra

Re: [RFC PATCH v3 04/11] contrib/plugins: add plugin showcasing new dicontinuity related API

2025-01-09 Thread Pierrick Bouvier
On 1/9/25 06:04, Alex Bennée wrote: Julian Ganz writes: We recently introduced new plugin API for registration of discontinuity related callbacks. This change introduces a minimal plugin showcasing the new API. It simply counts the occurances of interrupts, exceptions and host calls per CPU an

Re: [PATCH 1/3] linux-user: Only include 'exec/tb-flush.h' header when necessary

2025-01-09 Thread Ilya Leoshkevich
On Thu, 2025-01-02 at 19:25 +0100, Philippe Mathieu-Daudé wrote: > Very few source files require to access "exec/tb-flush.h" > declarations, and except a pair, they all include it > explicitly. No need to overload the generic "user-internals.h". > > Signed-off-by: Philippe Mathieu-Daudé > --- >  

Re: [PATCH 10/22] system: propagate Error to gdbserver_start (and other device setups)

2025-01-09 Thread Philippe Mathieu-Daudé
On 9/1/25 20:45, Alex Bennée wrote: Pierrick Bouvier writes: On 1/9/25 09:06, Alex Bennée wrote: This started as a clean-up to properly pass a Error handler to the gdbserver_start so we could do the right thing for command line and HMP invocations. Now that we have cleaned up foreach_device_c

Re: [RFC PATCH v3 11/11] tests: add plugin asserting correctness of discon event's to_pc

2025-01-09 Thread Pierrick Bouvier
On 1/9/25 08:33, Alex Bennée wrote: "Julian Ganz" writes: (Add Richard to CC) Hi Pierrick, December 5, 2024 at 12:33 AM, "Pierrick Bouvier" wrote: On 12/2/24 11:41, Julian Ganz wrote: +static void insn_exec(unsigned int vcpu_index, void *userdata) +{ + struct cpu_state *state = qemu_

Re: [RFC PATCH v3 01/11] plugins: add types for callbacks related to certain discontinuities

2025-01-09 Thread Pierrick Bouvier
On 1/9/25 05:52, Alex Bennée wrote: Pierrick Bouvier writes: Hi Julian, thanks for the update! Comments below. On 12/2/24 11:26, Julian Ganz wrote: The plugin API allows registration of callbacks for a variety of VCPU related events, such as VCPU reset, idle and resume. However, traps of an

Re: [PATCH v2 36/81] tcg/aarch64: Expand extract with offset 0 with andi

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: We're about to change canonicalization of masks as extract instead of and. Retain the andi expansion here. Signed-off-by: Richard Henderson --- tcg/aarch64/tcg-target.c.inc | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) Reviewed-by: P

Re: [PATCH v2 62/81] tcg/optimize: Fold orc with immediate to or

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Signed-off-by: Richard Henderson --- tcg/optimize.c | 25 +++-- 1 file changed, 23 insertions(+), 2 deletions(-) Reviewed-by: Philippe Mathieu-Daudé

Re: [PATCH v2 41/81] tcg/riscv64: Fold the ext{8,16,32}[us] cases into {s}extract

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Accept byte and word extensions with the extract opcodes. This is preparatory to removing the specialized extracts. Signed-off-by: Richard Henderson --- tcg/riscv/tcg-target-has.h | 39 ++ tcg/riscv/tcg-target.c.in

Re: [PATCH v3 00/11] Fix 32-bit build for plugins

2025-01-09 Thread Pierrick Bouvier
On 1/9/25 03:44, Alex Bennée wrote: Pierrick Bouvier writes: Since 9.2.0 release, we are building contrib plugins using the QEMU build system (before, it was external makefiles). When building for 32-bit host platform, some warnings are triggered and build fail. Thus, at the time, the decisio

Re: [PATCH v2 37/81] tcg/arm: Add full [US]XT[BH] into {s}extract

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: The armv6 uxt and sxt opcodes have a 2-bit rotate field which supports extractions from ofs = {0,8,16,24}. Special case ofs = 0, len <= 8 as AND. Signed-off-by: Richard Henderson --- tcg/arm/tcg-target-has.h | 17 + tcg/arm/tcg-target.c.i

Re: [PATCH v2 43/81] tcg/s390x: Fold the ext{8, 16, 32}[us] cases into {s}extract

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Accept byte and word extensions with the extract opcodes. This is preparatory to removing the specialized extracts. Signed-off-by: Richard Henderson --- tcg/s390x/tcg-target-has.h | 22 -- tcg/s390x/tcg-target.c.inc | 37 +

Re: [PATCH v2 35/81] tcg/aarch64: Provide TCG_TARGET_{s}extract_valid

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Trivially mirrors TCG_TARGET_HAS_{s}extract_*. Signed-off-by: Richard Henderson --- tcg/aarch64/tcg-target-has.h | 3 +++ 1 file changed, 3 insertions(+) Reviewed-by: Philippe Mathieu-Daudé

Re: [PATCH v2 45/81] tcg/tci: Provide TCG_TARGET_{s}extract_valid

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Trivially mirrors TCG_TARGET_HAS_{s}extract_*. Signed-off-by: Richard Henderson --- tcg/tci/tcg-target-has.h | 3 +++ 1 file changed, 3 insertions(+) Reviewed-by: Philippe Mathieu-Daudé

Re: [PATCH v2 50/81] tcg: Add all_outop[]

2025-01-09 Thread Philippe Mathieu-Daudé
On 7/1/25 09:00, Richard Henderson wrote: Add infrastructure for more consolidated output of opcodes. The base structure allows for constraints to be either static or dynamic, and for the existence of those constraints to replace TCG_TARGET_HAS_* and the bulk of tcg_op_supported. Signed-off-by:

Re: [PATCH v4 08/11] target/riscv: Add counter delegation/configuration support

2025-01-09 Thread Alistair Francis
On Wed, Dec 4, 2024 at 9:16 AM Atish Patra wrote: > > From: Kaiwen Xue > > The Smcdeleg/Ssccfg adds the support for counter delegation via > S*indcsr and Ssccfg. > > It also adds a new shadow CSR scountinhibit and menvcfg enable bit (CDE) > to enable this extension and scountovf virtualization. >

Re: [PATCH v4 00/11] Add RISC-V Counter delegation ISA extension support

2025-01-09 Thread Alistair Francis
On Wed, Dec 4, 2024 at 9:18 AM Atish Patra wrote: > > This series adds the counter delegation extension support. The counter > delegation ISA extension(Smcdeleg/Ssccfg) actually depends on multiple ISA > extensions. > > 1. S[m|s]csrind : The indirect CSR extension[1] which defines additional >

Re: [PATCH v7 0/9] target/riscv: Add support for Smdbltrp and Ssdbltrp extensions

2025-01-09 Thread Alistair Francis
On Tue, Dec 17, 2024 at 7:10 PM Clément Léger wrote: > > A double trap typically arises during a sensitive phase in trap handling > operations — when an exception or interrupt occurs while the trap > handler (the component responsible for managing these events) is in a > non-reentrant state. This

[PATCH v9 1/2] target/riscv: Add RISC-V CSR qtest support

2025-01-09 Thread Ivan Klokov
The RISC-V architecture supports the creation of custom CSR-mapped devices. It would be convenient to test them in the same way as MMIO-mapped devices. To do this, a new call has been added to read/write CSR registers. Signed-off-by: Ivan Klokov Acked-by: Fabiano Rosas Reviewed-by: Daniel Henriq

[PATCH v9 2/2] tests/qtest: QTest example for RISC-V CSR register

2025-01-09 Thread Ivan Klokov
Added demo for reading CSR register from qtest environment. Signed-off-by: Ivan Klokov Reviewed-by: Fabiano Rosas Reviewed-by: Daniel Henrique Barboza --- tests/qtest/meson.build | 2 +- tests/qtest/riscv-csr-test.c | 56 2 files changed, 57 insertion

[PULL 2/2] hw/xen: Check if len is 0 before memcpy()

2025-01-09 Thread David Woodhouse
From: Akihiko Odaki data->data can be NULL when len is 0. Strictly speaking, the behavior of memcpy() in such a scenario is undefined so UBSan complaints. Satisfy UBSan by checking if len is 0 before memcpy(). Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Dav

[PULL 1/2] hw/i386/pc: Fix level interrupt sharing for Xen event channel GSI

2025-01-09 Thread David Woodhouse
From: David Woodhouse The system GSIs are not designed for sharing. One device might assert a shared interrupt with qemu_set_irq() and another might deassert it, and the level from the first device is lost. This could be solved by refactoring the x86 GSI code to use an OrIrq device, but that sti

[PULL 0/2] xenfv queue

2025-01-09 Thread David Woodhouse
sers/dwmw2/qemu.git tags/pull-xenfv-20250109-1 for you to fetch changes up to b6014c5089a313ac84fe74970eee56e3fc87b49b: hw/xen: Check if len is 0 before memcpy() (2025-01-09 10:43:13 +) Xen emula

Re: [PATCH v19 09/14] pcie_sriov: Reuse SR-IOV VF device instances

2025-01-09 Thread Philippe Mathieu-Daudé
Hi Akihiko, On 9/1/25 07:29, Akihiko Odaki wrote: Disable SR-IOV VF devices by reusing code to power down PCI devices instead of removing them when the guest requests to disable VFs. This allows to realize devices and report VF realization errors at PF realization time. Signed-off-by: Akihiko O

Re: [PATCH v4 4/4] virtio: Convert feature properties to OnOffAuto

2025-01-09 Thread Philippe Mathieu-Daudé
On 8/1/25 07:17, Akihiko Odaki wrote: Some features are not always available with vhost. Legacy features are not available with vp_vdpa in particular. virtio devices used to disable them when not available even if the corresponding properties were explicitly set to "on". QEMU already has OnOffAu

Re: [RFC PATCH 0/6] Avoid contention for PCIIOMMUOps between IOMMU and PCIe host

2025-01-09 Thread Jason Chien
Ping. Jason Chien 於 2024年12月16日 週一 上午8:49寫道: > When PCIIOMMUOps.get_address_space() is already registered by the pci host, > e.g. TYPE_DESIGNWARE_PCIE_HOST, IOMMU cannot overwrite this hook without > breaking the PCIe translation rule, which means that IOMMU and the pci host > cannot coexist. >

Re: [PATCH 0/3] linux-user: Call tcg_flush_jmp_cache() before re-using threads

2025-01-09 Thread Philippe Mathieu-Daudé
Hi Ilya, ping? :) On 2/1/25 19:25, Philippe Mathieu-Daudé wrote: Fix a bug reported by Ilya in: https://lore.kernel.org/qemu-devel/uuuk6a2vo24yrrqrchjxaeko3utqshrdu6txcnqziokpg7dkom@4l4kd3dqh6jc/ Commit bb6cf6f0168 ("accel/tcg: Factor tcg_cpu_reset_hold() out") wanted to restrict tlb_flush() t

Re: [PATCH 2/2] xen: do not use '%ms' scanf specifier

2025-01-09 Thread Anthony PERARD
On Tue, Jan 07, 2025 at 10:31:40AM +0100, Roger Pau Monne wrote: > The 'm' parameter used to request auto-allocation of the destination variable > is not supported on FreeBSD, and as such leads to failures to parse. > > What's more, the current usage of '%ms' with xs_node_scanf() is pointless, as

[PATCH 1/3] tests/9p: rename test use_after_unlink -> use_file_after_unlink

2025-01-09 Thread Christian Schoenebeck
To pave the way for adding new test use_dir_after_unlink with subsequent patch, i.e. making it clear that the existing test is just about unlinked files, not unlinked directories. Signed-off-by: Christian Schoenebeck --- tests/qtest/virtio-9p-test.c | 8 1 file changed, 4 insertions(+),

[PATCH 0/3] tests/9p: add use-dir-after-unlink test

2025-01-09 Thread Christian Schoenebeck
Add a test case that verifies that it is possible to do some basic operations on a deleted directory if 9p client opened the directory before directory was removed. Unlike the previous open-unlink-fstat idiom: no fix required here. It already works with 9p server. Christian Schoenebeck (3): tes

[PATCH 2/3] tests/9p: add use_dir_after_unlink test

2025-01-09 Thread Christian Schoenebeck
After removing a directory from the filesystem, it should still be possible to operate on the directory if the directory has been opened before. As a first step this new test will verify whether Tgetattr request works on the unlinked directory. Signed-off-by: Christian Schoenebeck --- tests/qte

Re: [PULL 0/1] vmclock queue

2025-01-09 Thread David Woodhouse
On Thu, 2025-01-09 at 07:52 -0500, Stefan Hajnoczi wrote: > On Wed, 8 Jan 2025 at 06:11, David Woodhouse > wrote: > > > > From: David Woodhouse > > > > The following changes since commit > > 6528013b5f5ba6bb3934b7f5fe57a3110680530f: > > > >   Merge tag 'qga-pull-2025-01-06' of > > https://gith

Re: [PATCH v5] hw/acpi: Add vmclock device

2025-01-09 Thread David Woodhouse
On Thu, 2025-01-09 at 08:00 -0500, Michael S. Tsirkin wrote: > > It's tagged, will be in the next pull. Thanks! Thank you. smime.p7s Description: S/MIME cryptographic signature

[PATCH 3/3] tests/9p: extend use_dir_after_unlink test with Treaddir

2025-01-09 Thread Christian Schoenebeck
Sending a Treaddir request on an unlinked directory should also succeed if the directory was alread opened before unlink. We just check that no error occurs and that we get some kind of Treaddir result, but completely ignore the actual Treaddir result content. In fact, there should be no system as

[PULL 11/11] system: Inline machine_containers[] in qemu_create_machine_containers()

2025-01-09 Thread Philippe Mathieu-Daudé
Only qemu_create_machine_containers() uses the machine_containers[] array, restrict the scope to this single user. Signed-off-by: Philippe Mathieu-Daudé Acked-by: Peter Xu Reviewed-by: Richard Henderson Message-Id: <20250102211800.79235-9-phi...@linaro.org> --- system/vl.c | 16 +++

[PULL 04/11] qdev: Make qdev_get_machine() not use container_get()

2025-01-09 Thread Philippe Mathieu-Daudé
From: Peter Xu Currently, qdev_get_machine() has a slight misuse on container_get(), as the helper says "get a container" but in reality the goal is to get the machine object. It is still a "container" but not strictly. Note that it _may_ get a container (at "/machine") in our current unit test

Re: [PATCH 00/23] docs: add basic sphinx-domain rST generator to qapidoc

2025-01-09 Thread John Snow
On Thu, Jan 9, 2025, 6:48 AM Markus Armbruster wrote: > John Snow writes: > > > On Thu, Dec 19, 2024 at 7:31 AM Markus Armbruster > wrote: > > > >> John Snow writes: > >> > >> > based-on: > >> https://patchew.org/QEMU/20241213011307.2942030-1-js...@redhat.com/ > >> > > >> > Hi! > >> > > >> > T

[PULL 01/11] hw/pci: Use -1 as the default value for rombar

2025-01-09 Thread Philippe Mathieu-Daudé
From: Akihiko Odaki vfio_pci_size_rom() distinguishes whether rombar is explicitly set to 1 by checking dev->opts, bypassing the QOM property infrastructure. Use -1 as the default value for rombar to tell if the user explicitly set it to 1. The property is also converted from unsigned to signed.

[PULL 02/11] qdev: Remove opts member

2025-01-09 Thread Philippe Mathieu-Daudé
From: Akihiko Odaki It is no longer used. Signed-off-by: Akihiko Odaki Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Markus Armbruster Message-ID: <20250104-reuse-v18-14-c349eafd8...@daynix.com> Signed-off-by: Philippe Mathieu-Daudé --- include/hw/qdev-core.h | 4 hw/core/qdev.c

Re: [PATCH 09/22] system: squash usb_parse into a single function

2025-01-09 Thread Philippe Mathieu-Daudé
On 9/1/25 18:06, Alex Bennée wrote: We don't need to wrap usb_device_add as usb_parse is already gated with an if (machine_usb(current_machine)) check. Instead just assert and directly fail if usbdevice_create returns NULL. Signed-off-by: Alex Bennée --- system/vl.c | 22 -

Re: [PATCH 0/2] ARM SMMUv3 StreamID Implementation

2025-01-09 Thread Alex Bennée
Nabih Estefan writes: > This patch series modifies the ARM SMMUv3 to be able to work with an > implementation specific StreamID that does not match exactly the PCIe BDF. > The way to achieve this is by converting the smmu_get_sid and smmu_iommu_mr > functions to virtual functions that can be over

[PULL 10/11] qom: remove unused InterfaceInfo::concrete_class field

2025-01-09 Thread Philippe Mathieu-Daudé
From: Paolo Bonzini The "concrete_class" field of InterfaceClass is only ever written, and as far as I can tell is not particularly useful when debugging either; remove it. Signed-off-by: Paolo Bonzini Reviewed-by: Peter Maydell Message-ID: <20250107111308.21886-1-pbonz...@redhat.com> Signed-o

[PULL 09/11] qom: Remove container_get()

2025-01-09 Thread Philippe Mathieu-Daudé
From: Peter Xu Now there's no user of container_get(), remove it. Signed-off-by: Peter Xu Reviewed-by: Philippe Mathieu-Daudé Message-ID: <20241121192202.4155849-14-pet...@redhat.com> Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson --- include/qom/object.h | 11

[PULL 00/11] QOM / QDev patches for 2025-01-09

2025-01-09 Thread Philippe Mathieu-Daudé
The following changes since commit 3f8bcbba3b320c610689576fc47595f1076198dd: Merge tag 'pull-request-2025-01-08' of https://gitlab.com/thuth/qemu into staging (2025-01-08 11:38:21 -0500) are available in the Git repository at: https://github.com/philmd/qemu.git tags/qom-qdev-202

[PULL 08/11] qom: Use object_get_container()

2025-01-09 Thread Philippe Mathieu-Daudé
From: Peter Xu Use object_get_container() whenever applicable across the tree. Signed-off-by: Peter Xu Reviewed-by: Philippe Mathieu-Daudé Message-ID: <20241121192202.4155849-13-pet...@redhat.com> Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson --- backends/cryptodev.c

[PULL 06/11] qdev: Use machine_get_container()

2025-01-09 Thread Philippe Mathieu-Daudé
From: Peter Xu Use machine_get_container() whenever applicable across the tree. Signed-off-by: Peter Xu Reviewed-by: Philippe Mathieu-Daudé Message-ID: <20241121192202.4155849-11-pet...@redhat.com> Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson --- hw/core/gpio.c

Re: [PATCH 11/23] docs/qapidoc: add preamble() method

2025-01-09 Thread John Snow
On Thu, Jan 9, 2025, 5:34 AM Markus Armbruster wrote: > John Snow writes: > > > On Fri, Dec 20, 2024 at 9:15 AM Markus Armbruster > wrote: > > > >> John Snow writes: > >> > >> > This method adds the options/preamble to each definition block. > Notably, > >> > :since: and :ifcond: are added, as

Re: [PATCH v6 6/8] chardev/char-mux: implement backend chardev multiplexing

2025-01-09 Thread Daniel P . Berrangé
On Thu, Jan 09, 2025 at 01:56:40PM +0100, Roman Penyaev wrote: > Hi, > > On Tue, Jan 7, 2025 at 3:57 PM Marc-André Lureau > wrote: > > Whether we talk about multiplexing front-end or back-end, the issues > > are similar. In general, mixing input will create issues. Teeing > > output is less probl

Re: [PATCH v6 0/2] tests: Add functional tests for HPPA machines

2025-01-09 Thread Fabiano Rosas
Philippe Mathieu-Daudé writes: > On 2/1/25 11:03, Philippe Mathieu-Daudé wrote: >> Since v5: >> - Correct boot-serial-test machine name for stable (th_huth) >> >> Philippe Mathieu-Daudé (2): >>tests/qtest/boot-serial-test: Correct HPPA machine name >>tests: Add functional tests for HPPA

Re: [PULL 11/13] ui/console: Remove dpy_cursor_define_supported()

2025-01-09 Thread Michael Weiser
Hello Hank, Michael, Phil, > > > Remove dpy_cursor_define_supported() as it brings no benefit today and > > > it has a few inherent problems. commit 4bba839808bb1c4f500a11462220a687b4d9ab25 Author: Akihiko Odaki Date: Mon Jul 15 14:25:45 2024 +0900 ui/console: Remove dpy_cursor_define_sup

Re: [RFC PATCH v3 03/11] plugins: add hooks for new discontinuity related callbacks

2025-01-09 Thread Alex Bennée
Julian Ganz writes: > The plugin API allows registration of callbacks for a variety of VCPU > related events, such as VCPU reset, idle and resume. In addition, we > recently introduced API for registering callbacks for discontinuity > events, specifically for interrupts, exceptions and host calls

Re: [RFC PATCH v3 04/11] contrib/plugins: add plugin showcasing new dicontinuity related API

2025-01-09 Thread Alex Bennée
Julian Ganz writes: > We recently introduced new plugin API for registration of discontinuity > related callbacks. This change introduces a minimal plugin showcasing > the new API. It simply counts the occurances of interrupts, exceptions > and host calls per CPU and reports the counts when exitt

Re: [PULL 0/1] vmclock queue

2025-01-09 Thread Stefan Hajnoczi
On Thu, 9 Jan 2025 at 08:18, David Woodhouse wrote: > > On Thu, 2025-01-09 at 07:52 -0500, Stefan Hajnoczi wrote: > > On Wed, 8 Jan 2025 at 06:11, David Woodhouse > > wrote: > > > > > > From: David Woodhouse > > > > > > The following changes since commit > > > 6528013b5f5ba6bb3934b7f5fe57a311068

[PATCH v2 7/7] s390x: Fix CSS migration

2025-01-09 Thread Fabiano Rosas
Commit a55ae46683 ("s390: move css_migration_enabled from machine to css.c") disabled CSS migration globally instead of doing it per-instance. CC: Paolo Bonzini CC: qemu-sta...@nongnu.org #9.1 Fixes: a55ae46683 ("s390: move css_migration_enabled from machine to css.c") Resolves: https://gitlab.co

[PATCH v2 1/7] migration: Add more error handling to analyze-migration.py

2025-01-09 Thread Fabiano Rosas
The analyze-migration script was seen failing in s390x in misterious ways. It seems we're reaching the VMSDFieldStruct constructor without any fields, which would indicate an empty .subsection entry, a VMSTATE_STRUCT with no fields or a vmsd with no fields. We don't have any of those, at least not

[PATCH v2 4/7] migration: Rename vmstate_info_nullptr

2025-01-09 Thread Fabiano Rosas
Rename vmstate_info_nullptr from "uint64_t" to "nullptr". This vmstate actually reads and writes just a byte, so the proper name would be uint8. However, since this is a marker for a NULL pointer, it's convenient to have a more explicit name that can be identified by the consumers of the JSON part

[PATCH v2 0/7] migration: Fix s390 regressions + migration script

2025-01-09 Thread Fabiano Rosas
changes from v1: - dropped comments patch - new patch 4: rename the field to nullptr - patch 6: add a sample JSON, fix the appending code CI run: https://gitlab.com/farosas/qemu/-/pipelines/1616920974 v1: https://lore.kernel.org/r/20250107195025.9951-1-faro...@suse.de Hi, The situation that br

[PATCH v2 3/7] migration: Fix parsing of s390 stream

2025-01-09 Thread Fabiano Rosas
The parsing for the S390StorageAttributes section is currently leaving an unconsumed token that is later interpreted by the generic code as QEMU_VM_EOF, cutting the parsing short. The migration will issue a STATTR_FLAG_DONE between iterations, which the script consumes correctly, but there's a fin

Re: [RFC PATCH v3 08/11] target/mips: call plugin trap callbacks

2025-01-09 Thread Alex Bennée
Julian Ganz writes: > We recently introduced API for registering callbacks for trap related > events as well as the corresponding hook functions. Due to differences > between architectures, the latter need to be called from target specific > code. > > This change places hooks for MIPS targets. >

Re: [RFC PATCH v3 01/11] plugins: add types for callbacks related to certain discontinuities

2025-01-09 Thread Alex Bennée
Pierrick Bouvier writes: > Hi Julian, > > thanks for the update! > Comments below. > > On 12/2/24 11:26, Julian Ganz wrote: >> The plugin API allows registration of callbacks for a variety of VCPU >> related events, such as VCPU reset, idle and resume. However, traps of >> any kind, i.e. interrup

  1   2   3   >