Re: [PATCH 19/22] vfio/igd.c: use QOM casts where appropriate

2025-07-18 Thread Tomita Moeko
On 7/15/2025 5:25 PM, Mark Cave-Ayland wrote: > Use QOM casts to convert between VFIOPCIDevice and PCIDevice instead of > accessing pdev directly. > > Signed-off-by: Mark Cave-Ayland > --- > hw/vfio/igd.c | 38 +- > 1 file changed,

[PULL 2/6] python: use qom-list-get

2025-07-16 Thread Markus Armbruster
From: Steve Sistare Use qom-list-get to speed up the qom-tree command. Signed-off-by: Steve Sistare Acked-by: Markus Armbruster Message-ID: <1752248703-217318-3-git-send-email-steven.sist...@oracle.com> Tested-by: Markus Armbruster [Lint picked off to mollify make check-minreqs] Sign

[PULL 3/6] tests/qtest/qom-test: unit test for qom-list-get

2025-07-16 Thread Markus Armbruster
From: Steve Sistare Add a unit test for qom-list-get. Signed-off-by: Steve Sistare Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Markus Armbruster Message-ID: <1752248703-217318-4-git-send-email-steven.sist...@oracle.com> Signed-off-by: Markus Armbruster --- tests/qtest/qom-test.c

[PULL 1/6] qom: qom-list-get

2025-07-16 Thread Markus Armbruster
From: Steve Sistare Using qom-list and qom-get to get all the nodes and property values in a QOM tree can take multiple seconds because it requires 1000's of individual QOM requests. Some managers fetch the entire tree or a large subset of it when starting a new VM, and this cost

Re: [PATCH V4 2/3] python: use qom-list-get

2025-07-16 Thread Markus Armbruster
27;s old/bad code, I don't want > python-qemu-qmp faulting like this. Reproducer for Fedora 41, current master c079d3a31e4: Run $ qemu-system-x86_64 -S -display none -chardev socket,id=mon1,path=test-qmp,server=on,wait=off -mon mode=control,id=qmp,chardev=mon1 and $ scripts/qmp/qom-tree -s test-qmp >/dev/null Questions?

Re: [PATCH 06/22] vfio/spapr.c: use QOM casts where appropriate

2025-07-16 Thread Harsh Prateek Bora
On 7/15/25 14:55, Mark Cave-Ayland wrote: Use QOM casts to convert between VFIOContainer and VFIOContainerBase instead of accessing bcontainer directly. Signed-off-by: Mark Cave-Ayland Reviewed-by: Harsh Prateek Bora --- hw/vfio/spapr.c | 16 ++-- 1 file changed, 6

Re: [PATCH 05/22] ppc/spapr_pci_vfio.c: use QOM casts where appropriate

2025-07-16 Thread Harsh Prateek Bora
On 7/15/25 14:55, Mark Cave-Ayland wrote: Use a QOM cast to convert to VFIOContainer instead of accessing bcontainer directly. Signed-off-by: Mark Cave-Ayland Reviewed-by: Harsh Prateek Bora --- hw/ppc/spapr_pci_vfio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PULL 04/17] qapi/machine: Add @qom-type field to CpuInfoFast structure

2025-07-15 Thread Philippe Mathieu-Daudé
Knowing the QOM type name of a CPU can be useful, in particular to infer its model name. Reviewed-by: Richard Henderson Reviewed-by: Zhao Liu Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Xiaoyao Li Message-Id: <20250715090624.52377-2-phi...@linaro.org> --- qapi/machin

[PULL 30/57] iotests: add test for changing the 'drive' property via 'qom-set'

2025-07-15 Thread Kevin Wolf
From: Fiona Ebner Signed-off-by: Fiona Ebner Message-ID: <20250605100938.43133-1-f.eb...@proxmox.com> [kwolf: Fixed up pylint warnings flagged by 297] Reviewed-by: Kevin Wolf Signed-off-by: Kevin Wolf --- tests/qemu-iotests/tests/qom-set-drive | 75 ++ test

Re: [PATCH V4 2/3] python: use qom-list-get

2025-07-15 Thread John Snow
On Fri, Jul 11, 2025, 10:47 AM Markus Armbruster wrote: > Steve Sistare writes: > > > Use qom-list-get to speed up the qom-tree command. > > > > Signed-off-by: Steve Sistare > > Quick test... Differences in output before and after match expectations > (se

Re: [PATCH 00/22] vfio: improve use of QOM and coding guidelines

2025-07-15 Thread Cédric Le Goater
On 7/15/25 11:25, Mark Cave-Ayland wrote: This series attempts to improve use of QOM within the vfio subsystem which appears to have been added at a later date. It's mostly mechanical changes that do the following: 1) Format the QOM structs per our coding guidelines 2) Ensure the p

Re: [PATCH 21/22] s390x/s390-pci-vfio.c: use QOM casts where appropriate

2025-07-15 Thread Matthew Rosato
On 7/15/25 5:26 AM, Mark Cave-Ayland wrote: > Use QOM casts to cast to VFIOPCIDevice instead of using container_of(). > > Signed-off-by: Mark Cave-Ayland > --- > hw/s390x/s390-pci-vfio.c | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) Reviewed-by: Matthew Rosato

Re: [PATCH 20/22] vfio-user/pci.c: use QOM casts where appropriate

2025-07-15 Thread John Levon
On Tue, Jul 15, 2025 at 10:26:00AM +0100, Mark Cave-Ayland wrote: > Use QOM casts to convert between VFIOPCIDevice and PCIDevice instead of > accessing pdev directly. > > Signed-off-by: Mark Cave-Ayland > --- > hw/vfio-user/pci.c | 6 +++--- > 1 file changed, 3 inser

Re: [PATCH 09/22] vfio/container.c: use QOM casts where appropriate

2025-07-15 Thread John Levon
On Tue, Jul 15, 2025 at 10:25:49AM +0100, Mark Cave-Ayland wrote: > Use QOM casts to convert between VFIOUserContainer and VFIOContainerBase > instead > of accessing bcontainer directly. > > Signed-off-by: Mark Cave-Ayland Patch subject should be "vfio-user/container.c&q

Re: [PATCH 21/22] s390x/s390-pci-vfio.c: use QOM casts where appropriate

2025-07-15 Thread Eric Farman
On Tue, 2025-07-15 at 10:26 +0100, Mark Cave-Ayland wrote: > Use QOM casts to cast to VFIOPCIDevice instead of using container_of(). > > Signed-off-by: Mark Cave-Ayland > --- > hw/s390x/s390-pci-vfio.c | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-)

Re: [PATCH 12/22] vfio-user/pci.c: use QOM casts where appropriate

2025-07-15 Thread John Levon
On Tue, Jul 15, 2025 at 10:25:52AM +0100, Mark Cave-Ayland wrote: > Use QOM casts to convert between VFIOUserPCIDevice and VFIOPCIDevice instead > of accessing device directly. > > Signed-off-by: Mark Cave-Ayland > --- > hw/vfio-user/pci.c | 7 --- > 1 file change

Re: [PATCH 16/22] vfio/pci.c: use QOM casts where appropriate

2025-07-15 Thread BALATON Zoltan
On Tue, 15 Jul 2025, Mark Cave-Ayland wrote: Use QOM casts to convert between VFIOPCIDevice and PCIDevice instead of accessing pdev directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio/pci.c | 202 ++ 1 file changed, 120 insertions(+), 82

Re: [PATCH 18/22] vfio/cpr.c: use QOM casts where appropriate

2025-07-15 Thread Steven Sistare
On 7/15/2025 5:25 AM, Mark Cave-Ayland wrote: Use QOM casts to convert between VFIOPCIDevice and PCIDevice instead of accessing pdev directly. Signed-off-by: Mark Cave-Ayland Reviewed-by: Steve Sistare

Re: [PATCH 03/22] hw/vfio/cpr-legacy.c: use QOM casts where appropriate

2025-07-15 Thread Steven Sistare
On 7/15/2025 5:25 AM, Mark Cave-Ayland wrote: Use QOM casts to convert between VFIOContainer and VFIOContainerBase instead of accessing bcontainer directly. Signed-off-by: Mark Cave-Ayland Reviewed-by: Steve Sistare

[PATCH 04/22] hw/vfio/container.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOContainer and VFIOContainerBase instead of accessing bcontainer directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio/container.c | 31 +-- 1 file changed, 13 insertions(+), 18 deletions(-) diff --git a/hw/vfio/container.c b/hw

[PATCH 16/22] vfio/pci.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOPCIDevice and PCIDevice instead of accessing pdev directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio/pci.c | 202 ++ 1 file changed, 120 insertions(+), 82 deletions(-) diff --git a/hw/vfio/pci.c b/hw/vfio

[PATCH 15/22] vfio/pci.h: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOPCIDevice and PCIDevice instead of accessing pdev directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio/pci.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/vfio/pci.h b/hw/vfio/pci.h index 16f092a0c6..c0c3d68742 100644 --- a/hw/vfio

[PATCH 18/22] vfio/cpr.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOPCIDevice and PCIDevice instead of accessing pdev directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio/cpr.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/vfio/cpr.c b/hw/vfio/cpr.c index af0f12a7ad..3e3f4035ab 100644 --- a/hw

[PATCH 17/22] vfio/pci-quirks.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOPCIDevice and PCIDevice instead of accessing pdev directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio/pci-quirks.c | 48 ++-- 1 file changed, 29 insertions(+), 19 deletions(-) diff --git a/hw/vfio/pci-quirks.c b/hw

[PATCH 12/22] vfio-user/pci.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOUserPCIDevice and VFIOPCIDevice instead of accessing device directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio-user/pci.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/hw/vfio-user/pci.c b/hw/vfio-user/pci.c index da6fe51809

[PATCH 20/22] vfio-user/pci.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOPCIDevice and PCIDevice instead of accessing pdev directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio-user/pci.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/vfio-user/pci.c b/hw/vfio-user/pci.c index 9380766548..25a1dc0529

[PATCH 19/22] vfio/igd.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOPCIDevice and PCIDevice instead of accessing pdev directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio/igd.c | 38 +- 1 file changed, 21 insertions(+), 17 deletions(-) diff --git a/hw/vfio/igd.c b/hw/vfio/igd.c index

[PATCH 21/22] s390x/s390-pci-vfio.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to cast to VFIOPCIDevice instead of using container_of(). Signed-off-by: Mark Cave-Ayland --- hw/s390x/s390-pci-vfio.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/hw/s390x/s390-pci-vfio.c b/hw/s390x/s390-pci-vfio.c index aaf91319b4

[PATCH 06/22] vfio/spapr.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOContainer and VFIOContainerBase instead of accessing bcontainer directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio/spapr.c | 16 ++-- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/hw/vfio/spapr.c b/hw/vfio/spapr.c index

Re: [PATCH-for-10.1 v7 1/2] qapi/machine: Add @qom-type field to CpuInfoFast structure

2025-07-15 Thread Xiaoyao Li
On 7/15/2025 5:06 PM, Philippe Mathieu-Daudé wrote: Knowing the QOM type name of a CPU can be useful, in particular to infer its model name. Reviewed-by: Richard Henderson Reviewed-by: Zhao Liu Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Xiaoyao Li --- qapi/machine.json

[PATCH 09/22] vfio/container.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOUserContainer and VFIOContainerBase instead of accessing bcontainer directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio-user/container.c | 26 +++--- 1 file changed, 11 insertions(+), 15 deletions(-) diff --git a/hw/vfio-user

[PATCH 00/22] vfio: improve use of QOM and coding guidelines

2025-07-15 Thread Mark Cave-Ayland
This series attempts to improve use of QOM within the vfio subsystem which appears to have been added at a later date. It's mostly mechanical changes that do the following: 1) Format the QOM structs per our coding guidelines 2) Ensure the parent object is called parent_obj 3) Us

[PATCH 05/22] ppc/spapr_pci_vfio.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use a QOM cast to convert to VFIOContainer instead of accessing bcontainer directly. Signed-off-by: Mark Cave-Ayland --- hw/ppc/spapr_pci_vfio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/ppc/spapr_pci_vfio.c b/hw/ppc/spapr_pci_vfio.c index e318d0d912..7e1c71ef59

[PATCH 03/22] hw/vfio/cpr-legacy.c: use QOM casts where appropriate

2025-07-15 Thread Mark Cave-Ayland
Use QOM casts to convert between VFIOContainer and VFIOContainerBase instead of accessing bcontainer directly. Signed-off-by: Mark Cave-Ayland --- hw/vfio/cpr-legacy.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/hw/vfio/cpr-legacy.c b/hw/vfio/cpr-legacy.c

[PATCH-for-10.1 v7 1/2] qapi/machine: Add @qom-type field to CpuInfoFast structure

2025-07-15 Thread Philippe Mathieu-Daudé
Knowing the QOM type name of a CPU can be useful, in particular to infer its model name. Reviewed-by: Richard Henderson Reviewed-by: Zhao Liu Signed-off-by: Philippe Mathieu-Daudé --- qapi/machine.json | 3 +++ hw/core/machine-qmp-cmds.c | 1 + 2 files changed, 4 insertions(+) diff

Re: [PATCH] iotests: add test for changing the 'drive' property via 'qom-set'

2025-07-14 Thread Kevin Wolf
Am 05.06.2025 um 12:09 hat Fiona Ebner geschrieben: > Signed-off-by: Fiona Ebner Thanks, applied to the block branch. Kevin

[PULL 31/32] python: use qom-list-get

2025-07-14 Thread Markus Armbruster
From: Steve Sistare Use qom-list-get to speed up the qom-tree command. Signed-off-by: Steve Sistare Acked-by: Markus Armbruster Message-ID: <1752248703-217318-3-git-send-email-steven.sist...@oracle.com> Tested-by: Markus Armbruster Signed-off-by: Markus Armbruster --- python/qemu

[PULL 30/32] qom: qom-list-get

2025-07-14 Thread Markus Armbruster
From: Steve Sistare Using qom-list and qom-get to get all the nodes and property values in a QOM tree can take multiple seconds because it requires 1000's of individual QOM requests. Some managers fetch the entire tree or a large subset of it when starting a new VM, and this cost

[PULL 32/32] tests/qtest/qom-test: unit test for qom-list-get

2025-07-14 Thread Markus Armbruster
From: Steve Sistare Add a unit test for qom-list-get. Signed-off-by: Steve Sistare Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Markus Armbruster Message-ID: <1752248703-217318-4-git-send-email-steven.sist...@oracle.com> Signed-off-by: Markus Armbruster --- tests/qtest/qom-test.c

[PULL 22/32] qapi: add cross-references to QOM

2025-07-14 Thread Markus Armbruster
--- a/qapi/qom.json +++ b/qapi/qom.json @@ -52,7 +52,7 @@ # # List properties of a object given a path in the object model. # -# @path: the path within the object model. See @qom-get for a +# @path: the path within the object model. See `qom-get` for a # description of this parameter. # # R

Re: [PATCH V5 0/3] fast qom tree get

2025-07-11 Thread Markus Armbruster
Steve Sistare writes: > Using qom-list and qom-get to get all the nodes and property values in a > QOM tree can take multiple seconds because it requires 1000's of individual > QOM requests. Some managers fetch the entire tree or a large subset > of it when starting a new VM, a

Re: [PATCH V4 2/3] python: use qom-list-get

2025-07-11 Thread Steven Sistare
On 7/11/2025 12:50 PM, Markus Armbruster wrote: Steven Sistare writes: On 7/11/2025 10:47 AM, Markus Armbruster wrote: Steve Sistare writes: Use qom-list-get to speed up the qom-tree command. Signed-off-by: Steve Sistare Quick test... Differences in output before and after match

Re: [PATCH V4 2/3] python: use qom-list-get

2025-07-11 Thread Markus Armbruster
Steven Sistare writes: > On 7/11/2025 10:47 AM, Markus Armbruster wrote: >> Steve Sistare writes: >> >>> Use qom-list-get to speed up the qom-tree command. >>> >>> Signed-off-by: Steve Sistare >> >> Quick test... Differences in outpu

Re: [PATCH V4 0/3] fast qom tree get

2025-07-11 Thread Steven Sistare
On 7/11/2025 11:06 AM, Markus Armbruster wrote: This feels ready. Have a look at my (few) comments, and tell me what you think. Happy to apply finishing touches in my tree, without another respin. Thanks your your patience! I'll send V5 with the final changes. Thank you for your time Markus

[PATCH V5 2/3] python: use qom-list-get

2025-07-11 Thread Steve Sistare
Use qom-list-get to speed up the qom-tree command. Signed-off-by: Steve Sistare Acked-by: Markus Armbruster --- python/qemu/utils/qom.py| 43 +++-- python/qemu/utils/qom_common.py | 53 + 2 files changed, 78 insertions

[PATCH V5 0/3] fast qom tree get

2025-07-11 Thread Steve Sistare
Using qom-list and qom-get to get all the nodes and property values in a QOM tree can take multiple seconds because it requires 1000's of individual QOM requests. Some managers fetch the entire tree or a large subset of it when starting a new VM, and this cost is a substantial fraction of

[PATCH V5 1/3] qom: qom-list-get

2025-07-11 Thread Steve Sistare
Using qom-list and qom-get to get all the nodes and property values in a QOM tree can take multiple seconds because it requires 1000's of individual QOM requests. Some managers fetch the entire tree or a large subset of it when starting a new VM, and this cost is a substantial fraction of

[PATCH V5 3/3] tests/qtest/qom-test: unit test for qom-list-get

2025-07-11 Thread Steve Sistare
Add a unit test for qom-list-get. Signed-off-by: Steve Sistare Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Markus Armbruster --- tests/qtest/qom-test.c | 116 - 1 file changed, 115 insertions(+), 1 deletion(-) diff --git a/tests/qtest/qom

Re: [PATCH V4 2/3] python: use qom-list-get

2025-07-11 Thread Steven Sistare
On 7/11/2025 10:47 AM, Markus Armbruster wrote: Steve Sistare writes: Use qom-list-get to speed up the qom-tree command. Signed-off-by: Steve Sistare Quick test... Differences in output before and after match expectations (see appended diff). New version: real0m0.446s

Re: [PATCH V4 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-11 Thread Steven Sistare
On 7/11/2025 11:02 AM, Markus Armbruster wrote: Steve Sistare writes: Add a unit test for qom-list-getv. qom-list-get here and in subject. Could scratch this line, subject suffices. Doh, I missed it. Will fix. (re)names are hard :) Signed-off-by: Steve Sistare Reviewed-by: Philippe

Re: [PATCH V4 1/3] qom: qom-list-get

2025-07-11 Thread Steven Sistare
On 7/11/2025 10:35 AM, Markus Armbruster wrote: Steve Sistare writes: Define the qom-list-get command, which fetches all the properties and values for a list of paths. This is faster than qom-list plus qom-get, especially when fetching a large subset of the QOM tree. Some managers do so

Re: [PATCH V4 0/3] fast qom tree get

2025-07-11 Thread Markus Armbruster
This feels ready. Have a look at my (few) comments, and tell me what you think. Happy to apply finishing touches in my tree, without another respin. Thanks your your patience!

Re: [PATCH V4 2/3] python: use qom-list-get

2025-07-11 Thread Markus Armbruster
Steve Sistare writes: > Use qom-list-get to speed up the qom-tree command. > > Signed-off-by: Steve Sistare Quick test... Differences in output before and after match expectations (see appended diff). New version: real0m0.446s user0m0.062s sys 0m0.017s Ol

Re: [PATCH V4 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-11 Thread Markus Armbruster
Steve Sistare writes: > Add a unit test for qom-list-getv. qom-list-get here and in subject. Could scratch this line, subject suffices. > > Signed-off-by: Steve Sistare > Reviewed-by: Philippe Mathieu-Daudé > --- > tests/qtes

Re: [PATCH V4 1/3] qom: qom-list-get

2025-07-11 Thread Markus Armbruster
Steve Sistare writes: > Define the qom-list-get command, which fetches all the properties and > values for a list of paths. This is faster than qom-list plus qom-get, > especially when fetching a large subset of the QOM tree. Some managers > do so when starting a new VM, and this c

[PATCH v2 11/18] qapi: add cross-references to QOM

2025-07-10 Thread John Snow
model. # -# @path: the path within the object model. See @qom-get for a +# @path: the path within the object model. See `qom-get` for a # description of this parameter. # # Returns: a list that describe the properties of the object. @@ -129,12 +129,12 @@ # # This command will set a property

[PATCH V4 2/3] python: use qom-list-get

2025-07-10 Thread Steve Sistare
Use qom-list-get to speed up the qom-tree command. Signed-off-by: Steve Sistare --- python/qemu/utils/qom.py| 43 +++-- python/qemu/utils/qom_common.py | 53 + 2 files changed, 78 insertions(+), 18 deletions(-) diff

[PATCH V4 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-10 Thread Steve Sistare
Add a unit test for qom-list-getv. Signed-off-by: Steve Sistare Reviewed-by: Philippe Mathieu-Daudé --- tests/qtest/qom-test.c | 116 - 1 file changed, 115 insertions(+), 1 deletion(-) diff --git a/tests/qtest/qom-test.c b/tests/qtest/qom-test.c

[PATCH V4 0/3] fast qom tree get

2025-07-10 Thread Steve Sistare
Using qom-list and qom-get to get all the nodes and property values in a QOM tree can take multiple seconds because it requires 1000's of individual QOM requests. Some managers fetch the entire tree or a large subset of it when starting a new VM, and this cost is a substantial fraction of

[PATCH V4 1/3] qom: qom-list-get

2025-07-10 Thread Steve Sistare
Define the qom-list-get command, which fetches all the properties and values for a list of paths. This is faster than qom-list plus qom-get, especially when fetching a large subset of the QOM tree. Some managers do so when starting a new VM, and this cost can be a substantial fraction of start

Re: [PATCH V3 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-10 Thread Steven Sistare
On 7/10/2025 4:54 AM, Markus Armbruster wrote: Steven Sistare writes: On 7/8/2025 6:02 PM, Philippe Mathieu-Daudé wrote: Hi Steve, On 8/7/25 19:24, Steve Sistare wrote: Add a unit test for qom-list-getv. Signed-off-by: Steve Sistare ---   tests/qtest/qom-test.c | 64

Re: [PATCH V3 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-10 Thread Markus Armbruster
Steven Sistare writes: > On 7/8/2025 6:02 PM, Philippe Mathieu-Daudé wrote: >> Hi Steve, >> On 8/7/25 19:24, Steve Sistare wrote: >>> Add a unit test for qom-list-getv. >>> >>> Signed-off-by: Steve Sistare

Re: [PATCH V3 1/3] qom: qom-list-getv

2025-07-10 Thread Philippe Mathieu-Daudé
On 8/7/25 23:54, Philippe Mathieu-Daudé wrote: On 8/7/25 19:24, Steve Sistare wrote: Define the qom-list-getv command, which fetches all the properties and values for a list of paths.  This is faster than qom-tree-get when fetching a subset of the QOM tree.  See qom.json for details. Signed

Re: [PATCH V3 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-10 Thread Philippe Mathieu-Daudé
On 9/7/25 17:17, Steven Sistare wrote: On 7/8/2025 6:02 PM, Philippe Mathieu-Daudé wrote: Hi Steve, On 8/7/25 19:24, Steve Sistare wrote: Add a unit test for qom-list-getv. Signed-off-by: Steve Sistare ---   tests/qtest/qom-test.c | 64

Re: [PATCH V3 1/3] qom: qom-list-getv

2025-07-09 Thread Steven Sistare
On 7/9/2025 4:39 AM, Markus Armbruster wrote: Steve Sistare writes: Define the qom-list-getv command, which fetches all the properties and values for a list of paths. This is faster than qom-tree-get when fetching a subset of the QOM tree. See qom.json for details. Signed-off-by: Steve

Re: [PATCH V3 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-09 Thread Steven Sistare
On 7/8/2025 6:02 PM, Philippe Mathieu-Daudé wrote: Hi Steve, On 8/7/25 19:24, Steve Sistare wrote: Add a unit test for qom-list-getv. Signed-off-by: Steve Sistare ---   tests/qtest/qom-test.c | 64 ++   1 file changed, 64 insertions(+) diff

Re: [PATCH-for-10.1 v6 10/14] hw/core/null-machine: Define machine as generic QOM type

2025-07-09 Thread Zhao Liu
On Mon, Jul 07, 2025 at 07:20:04PM +0200, Philippe Mathieu-Daudé wrote: > Date: Mon, 7 Jul 2025 19:20:04 +0200 > From: Philippe Mathieu-Daudé > Subject: [PATCH-for-10.1 v6 10/14] hw/core/null-machine: Define machine as > generic QOM type > X-Mailer: git-send-email 2.

Re: [PATCH V3 1/3] qom: qom-list-getv

2025-07-09 Thread Markus Armbruster
Steve Sistare writes: > Define the qom-list-getv command, which fetches all the properties and > values for a list of paths. This is faster than qom-tree-get when > fetching a subset of the QOM tree. See qom.json for details. > > Signed-off-by: Steve Sistare You cover lette

Re: [PATCH V3 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-08 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > Hi Steve, > > On 8/7/25 19:24, Steve Sistare wrote: >> Add a unit test for qom-list-getv. >> Signed-off-by: Steve Sistare >> --- >> tests/qtest/qom-test.c | 64 >> ++ >&g

Re: [PATCH V3 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-08 Thread Philippe Mathieu-Daudé
Hi Steve, On 8/7/25 19:24, Steve Sistare wrote: Add a unit test for qom-list-getv. Signed-off-by: Steve Sistare --- tests/qtest/qom-test.c | 64 ++ 1 file changed, 64 insertions(+) diff --git a/tests/qtest/qom-test.c b/tests/qtest/qom-test.c

Re: [PATCH V3 1/3] qom: qom-list-getv

2025-07-08 Thread Philippe Mathieu-Daudé
On 8/7/25 19:24, Steve Sistare wrote: Define the qom-list-getv command, which fetches all the properties and values for a list of paths. This is faster than qom-tree-get when fetching a subset of the QOM tree. See qom.json for details. Signed-off-by: Steve Sistare --- qapi/qom.json

Re: [PATCH V2 3/5] tests/qtest/qom-test: unit test for qom-tree-get

2025-07-08 Thread Philippe Mathieu-Daudé
On 12/5/25 15:47, Steve Sistare wrote: Add a unit test for qom-tree-get Signed-off-by: Steve Sistare --- tests/qtest/qom-test.c | 49 + 1 file changed, 49 insertions(+) Reviewed-by: Philippe Mathieu-Daudé

Re: [PATCH V2 1/5] qom: qom-tree-get

2025-07-08 Thread Markus Armbruster
Steven Sistare writes: > On 7/4/2025 8:22 AM, Markus Armbruster wrote: >> Steve Sistare writes: >> >>> Define the qom-tree-get QAPI command, which fetches an entire tree of >>> properties and values with a single QAPI call. This is much faster >>>

Re: [PATCH V2 4/5] qom: qom-list-getv

2025-07-08 Thread Markus Armbruster
Steven Sistare writes: > On 7/4/2025 8:22 AM, Markus Armbruster wrote: >> Steve Sistare writes: >> >>> Define the qom-list-getv command, which fetches all the properties and >>> values for a list of paths. This is faster than qom-tree-get when >>&g

Re: [PATCH V2 1/5] qom: qom-tree-get

2025-07-08 Thread Philippe Mathieu-Daudé
On 12/5/25 15:47, Steve Sistare wrote: Define the qom-tree-get QAPI command, which fetches an entire tree of properties and values with a single QAPI call. This is much faster than using qom-list plus qom-get for every node and property of the tree. See qom.json for details. Signed-off-by

Re: [PATCH V2 1/5] qom: qom-tree-get

2025-07-08 Thread Steven Sistare
On 7/8/2025 3:14 AM, Philippe Mathieu-Daudé wrote: On 12/5/25 15:47, Steve Sistare wrote: Define the qom-tree-get QAPI command, which fetches an entire tree of properties and values with a single QAPI call.  This is much faster than using qom-list plus qom-get for every node and property of the

[PATCH V3 0/3] fast qom tree get

2025-07-08 Thread Steve Sistare
Using qom-list and qom-get to get all the nodes and property values in a QOM tree can take multiple seconds because it requires 1000's of individual QOM requests. Some managers fetch the entire tree or a large subset of it when starting a new VM, and this cost is a substantial fraction of

[PATCH V3 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-08 Thread Steve Sistare
Add a unit test for qom-list-getv. Signed-off-by: Steve Sistare --- tests/qtest/qom-test.c | 64 ++ 1 file changed, 64 insertions(+) diff --git a/tests/qtest/qom-test.c b/tests/qtest/qom-test.c index 27d70bc..4defff1 100644 --- a/tests/qtest/qom

Re: [PATCH V2 1/5] qom: qom-tree-get

2025-07-08 Thread Markus Armbruster
Markus Armbruster writes: > Steven Sistare writes: > >> On 7/4/2025 8:22 AM, Markus Armbruster wrote: >>> Steve Sistare writes: >>> >>>> Define the qom-tree-get QAPI command, which fetches an entire tree of >>>> properties and values w

[PATCH V3 1/3] qom: qom-list-getv

2025-07-08 Thread Steve Sistare
Define the qom-list-getv command, which fetches all the properties and values for a list of paths. This is faster than qom-tree-get when fetching a subset of the QOM tree. See qom.json for details. Signed-off-by: Steve Sistare --- qapi/qom.json | 50

Re: [PATCH V2 1/5] qom: qom-tree-get

2025-07-08 Thread Philippe Mathieu-Daudé
On 8/7/25 13:50, Steven Sistare wrote: On 7/8/2025 3:14 AM, Philippe Mathieu-Daudé wrote: On 12/5/25 15:47, Steve Sistare wrote: Define the qom-tree-get QAPI command, which fetches an entire tree of properties and values with a single QAPI call.  This is much faster than using qom-list plus

[PATCH V3 2/3] python: use qom-list-getv

2025-07-08 Thread Steve Sistare
Use qom-list-getv to speed up the qom-tree command. Signed-off-by: Steve Sistare --- python/qemu/utils/qom.py| 43 +++-- python/qemu/utils/qom_common.py | 53 + 2 files changed, 78 insertions(+), 18 deletions(-) diff

Re: [PATCH V3 3/3] tests/qtest/qom-test: unit test for qom-list-getv

2025-07-08 Thread Fabiano Rosas
Steve Sistare writes: > Add a unit test for qom-list-getv. > > Signed-off-by: Steve Sistare Reviewed-by: Fabiano Rosas

Re: [PATCH-for-10.1 v6 10/14] hw/core/null-machine: Define machine as generic QOM type

2025-07-07 Thread Richard Henderson
On 7/7/25 11:20, Philippe Mathieu-Daudé wrote: While DEFINE_MACHINE() is a succinct macro, it doesn't allow registering QOM interfaces to the defined machine. Convert to the generic DEFINE_TYPES() in preparation to register interfaces. Signed-off-by: Philippe Mathieu-Daudé Review

[PATCH-for-10.1 v6 11/14] hw/arm: Register TYPE_TARGET_ARM/AARCH64_MACHINE QOM interfaces

2025-07-07 Thread Philippe Mathieu-Daudé
Define the TYPE_TARGET_ARM_MACHINE and TYPE_TARGET_AARCH64_MACHINE QOM interface names to allow machines to implement them. Register these interfaces in common code in target_info-qom.c used by all binaries because QOM interfaces must be registered before being checked (see next commit with the

[PATCH-for-10.1 v6 10/14] hw/core/null-machine: Define machine as generic QOM type

2025-07-07 Thread Philippe Mathieu-Daudé
While DEFINE_MACHINE() is a succinct macro, it doesn't allow registering QOM interfaces to the defined machine. Convert to the generic DEFINE_TYPES() in preparation to register interfaces. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Pierrick Bouvier --- hw/core/null-machine.c

Re: [Regression] Re: [PULL 35/35] qom: reverse order of instance_post_init calls

2025-07-07 Thread Paolo Bonzini
Il gio 3 lug 2025, 06:51 Paolo Bonzini ha scritto: > > > Il mer 2 lug 2025, 23:36 Xiaoyao Li ha scritto: > >> The reason why accelerator's instance_init() was moved to post_init, was >> just it needs to consider other factors. Please see commit 4db4385a7ab6 >> ("i386: run accel_cpu_instance_init

Re: [PATCH V2 0/5] fast qom tree get

2025-07-07 Thread Steven Sistare
On 7/4/2025 8:26 AM, Markus Armbruster wrote: Steve Sistare writes: Using qom-list and qom-get to get all the nodes and property values in a QOM tree can take multiple seconds because it requires 1000's of individual QOM requests. Some managers fetch the entire tree or a large subset

Re: [PATCH V2 1/5] qom: qom-tree-get

2025-07-07 Thread Steven Sistare
On 7/4/2025 8:22 AM, Markus Armbruster wrote: Steve Sistare writes: Define the qom-tree-get QAPI command, which fetches an entire tree of properties and values with a single QAPI call. This is much faster than using qom-list plus qom-get for every node and property of the tree. See qom.json

Re: [PATCH V2 4/5] qom: qom-list-getv

2025-07-07 Thread Steven Sistare
On 7/4/2025 8:22 AM, Markus Armbruster wrote: Steve Sistare writes: Define the qom-list-getv command, which fetches all the properties and values for a list of paths. This is faster than qom-tree-get when fetching a subset of the QOM tree. See qom.json for details. Signed-off-by: Steve

Re: [PATCH V2 0/5] fast qom tree get

2025-07-07 Thread Steven Sistare
On 7/4/2025 8:33 AM, Markus Armbruster wrote: Steve, My sincere apologies for the long, long delay. It wasn't just for the usual reasons. It was also because I had a vague feeling of unease about qom-tree, and had trouble figuring out why. I'll try do your work justice before the

[PULL v2 27/36] accel/hvf: Re-use QOM allocated state

2025-07-04 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20250606164418.98655-8-phi...@linaro.org> --- accel/hvf/hvf-all.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/accel/hvf/hvf-all.c b/accel/hvf/hvf-all.c index 9e4012e2d10..df95ba74d6e 1006

Re: [PATCH V2 0/5] fast qom tree get

2025-07-04 Thread Markus Armbruster
Steve, My sincere apologies for the long, long delay. It wasn't just for the usual reasons. It was also because I had a vague feeling of unease about qom-tree, and had trouble figuring out why. I'll try do your work justice before the window for 10.1 closes. QOM maintainers, ple

Re: [PATCH V2 0/5] fast qom tree get

2025-07-04 Thread Markus Armbruster
Steve Sistare writes: > Using qom-list and qom-get to get all the nodes and property values in a > QOM tree can take multiple seconds because it requires 1000's of individual > QOM requests. Some managers fetch the entire tree or a large subset > of it when starting a new VM, a

Re: [PATCH V2 4/5] qom: qom-list-getv

2025-07-04 Thread Markus Armbruster
Steve Sistare writes: > Define the qom-list-getv command, which fetches all the properties and > values for a list of paths. This is faster than qom-tree-get when > fetching a subset of the QOM tree. See qom.json for details. > > Signed-off-by: Steve Sistare > --- > qa

Re: [PATCH V2 1/5] qom: qom-tree-get

2025-07-04 Thread Markus Armbruster
Steve Sistare writes: > Define the qom-tree-get QAPI command, which fetches an entire tree of > properties and values with a single QAPI call. This is much faster > than using qom-list plus qom-get for every node and property of the > tree. See qom.json for details. > > Si

[PATCH v5 12/69] accel/hvf: Re-use QOM allocated state

2025-07-03 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson --- accel/hvf/hvf-accel-ops.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/accel/hvf/hvf-accel-ops.c b/accel/hvf/hvf-accel-ops.c index 6af849450e1..c256cdceffb 100644 --- a/accel/hvf/hvf-accel-ops.c ++

Re: [Regression] Re: [PULL 35/35] qom: reverse order of instance_post_init calls

2025-07-02 Thread Paolo Bonzini
Il mer 2 lug 2025, 23:36 Xiaoyao Li ha scritto: > The reason why accelerator's instance_init() was moved to post_init, was > just it needs to consider other factors. Please see commit 4db4385a7ab6 > ("i386: run accel_cpu_instance_init as post_init") > You're right and this can be a problem with

Re: [Regression] Re: [PULL 35/35] qom: reverse order of instance_post_init calls

2025-07-02 Thread Xiaoyao Li
On 7/3/2025 11:08 AM, Zhao Liu wrote: On Thu, Jul 03, 2025 at 09:03:10AM +0800, Xiaoyao Li wrote: Date: Thu, 3 Jul 2025 09:03:10 +0800 From: Xiaoyao Li Subject: Re: [Regression] Re: [PULL 35/35] qom: reverse order of instance_post_init calls On 7/3/2025 2:54 AM, Paolo Bonzini wrote: Il mer

Re: [Regression] Re: [PULL 35/35] qom: reverse order of instance_post_init calls

2025-07-02 Thread Zhao Liu
On Thu, Jul 03, 2025 at 09:03:10AM +0800, Xiaoyao Li wrote: > Date: Thu, 3 Jul 2025 09:03:10 +0800 > From: Xiaoyao Li > Subject: Re: [Regression] Re: [PULL 35/35] qom: reverse order of > instance_post_init calls > > On 7/3/2025 2:54 AM, Paolo Bonzini wrote: > > Il mer 2

  1   2   3   4   5   6   7   8   9   10   >