Re: [PATCH 29/42] qapi: Add "Details:" disambiguation marker

2025-02-17 Thread Markus Armbruster
John Snow writes: > This clarifies sections that are mistaken by the parser as "intro" > sections to be "details" sections instead. > > Signed-off-by: John Snow Is this missing announce-self in net.json? diff --git a/qapi/net.json b/qapi/net.json index 49bc7de64e..44ed72dbe9 100644 --- a/qapi/

Re: [PATCH v4 26/33] vfio/migration: Multifd device state transfer support - receive init/cleanup

2025-02-17 Thread Cédric Le Goater
On 2/14/25 21:55, Maciej S. Szmigiero wrote: On 12.02.2025 11:55, Cédric Le Goater wrote: On 1/30/25 11:08, Maciej S. Szmigiero wrote: From: "Maciej S. Szmigiero" Add support for VFIOMultifd data structure that will contain most of the receive-side data together with its init/cleanup methods.

Re: [PATCH 4/5] target/rx: Load reset vector from memory after first run

2025-02-17 Thread Peter Maydell
On Sat, 15 Feb 2025 at 18:24, Richard Henderson wrote: > > On 2/14/25 18:16, Keith Packard via wrote: > > The ROM images all get deleted as they've been loaded to memory, so we > > can't go fetch the reset vector from there. Instead, fetch it from > > memory. To make that work, we need to execute

[PULL 4/4] tests/qtest/ufs-test: Add test code for MCQ functionality

2025-02-17 Thread Jeuk Kim
This patch tests whether MCQ initialization and basic read-write operations work correctly when the MCQ parameter of hw/ufs is enabled. Acked-by: Fabiano Rosas Signed-off-by: Jeuk Kim --- tests/qtest/ufs-test.c | 171 ++--- 1 file changed, 142 insertions(+),

[PULL 1/4] hw/ufs: Fix legacy single doorbell support bit

2025-02-17 Thread Jeuk Kim
QEMU UFS has supported both legacy single doorbell and MCQ, but the LSDBS value was incorrectly set. This change corrects the LSDBS value to 0. Signed-off-by: Jeuk Kim --- hw/ufs/ufs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/ufs/ufs.c b/hw/ufs/ufs.c index 428fe927a

Re: [PATCH 29/42] qapi: Add "Details:" disambiguation marker

2025-02-17 Thread Markus Armbruster
John Snow writes: > This clarifies sections that are mistaken by the parser as "intro" > sections to be "details" sections instead. > > Signed-off-by: John Snow > --- > qapi/machine.json | 2 ++ > qapi/migration.json| 4 > qapi/qom.json | 4 > qapi/yank.json

Re: [PATCH v7] target/riscv: Add support to access ctrsource, ctrtarget, ctrdata regs.

2025-02-17 Thread Rajnesh Kanwal
On Mon, Feb 17, 2025 at 5:25 AM Alistair Francis wrote: > > On Wed, Feb 12, 2025 at 8:20 PM Rajnesh Kanwal wrote: > > > > CTR entries are accessed using ctrsource, ctrtarget and ctrdata > > registers using smcsrind/sscsrind extension. This commits extends > > the csrind extension to support CTR r

Re: [PATCH 03/11] tcg/arm: Drop addrhi from prepare_host_addr

2025-02-17 Thread Philippe Mathieu-Daudé
On 5/2/25 05:03, Richard Henderson wrote: The guest address will now always be TCG_TYPE_I32. Signed-off-by: Richard Henderson --- tcg/arm/tcg-target.c.inc | 63 ++-- 1 file changed, 21 insertions(+), 42 deletions(-) /* * Add the tl

Re: [PATCH 29/42] qapi: Add "Details:" disambiguation marker

2025-02-17 Thread Markus Armbruster
John Snow writes: > This clarifies sections that are mistaken by the parser as "intro" > sections to be "details" sections instead. > > Signed-off-by: John Snow This is rather terse. Why does the boundary between "intro" (previously "body") and "details" matter? As far as I understand, it mat

Re: [PATCH] physmem: replace assertion with error

2025-02-17 Thread Philippe Mathieu-Daudé
Hi Paolo, On 17/2/25 13:08, Paolo Bonzini wrote: It is possible to start QEMU with a confidential-guest-support object even in TCG mode. While there is already a check in qemu_machine_creation_done: if (machine->cgs && !machine->cgs->ready) { error_setg(errp, "accelerator does no

Re: [PATCH] physmem: replace assertion with error

2025-02-17 Thread Daniel P . Berrangé
On Mon, Feb 17, 2025 at 01:08:12PM +0100, Paolo Bonzini wrote: > It is possible to start QEMU with a confidential-guest-support object > even in TCG mode. While there is already a check in > qemu_machine_creation_done: > > if (machine->cgs && !machine->cgs->ready) { > error_setg(errp

<    1   2   3   4