* [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id
@ 2024-08-19 14:43 Peter Maydell
2024-09-06 15:13 ` Peter Maydell
2024-09-10 9:03 ` Igor Mammedov
0 siblings, 2 replies; 8+ messages in thread
From: Peter Maydell @ 2024-08-19 14:43 UTC (permalink / raw)
To: qemu-devel
At some point the way we allocate socket-id and core-id to CPUs
by default changed; update the example of how to do CPU hotplug
and unplug so the example commands work again. The differences
in the sample input and output are:
* the second CPU is now socket-id=0 core-id=1,
not socket-id=1 core-id=0
* the order of fields from the qmp_shell is different (it seems
to now always be in alphabetical order)
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
I noticed this while I was playing around with vcpu hotplug trying to
demonstrate a memory leak I want to fix...
docs/system/cpu-hotplug.rst | 54 ++++++++++++++++++-------------------
1 file changed, 26 insertions(+), 28 deletions(-)
diff --git a/docs/system/cpu-hotplug.rst b/docs/system/cpu-hotplug.rst
index 015ce2b6ec3..443ff226b90 100644
--- a/docs/system/cpu-hotplug.rst
+++ b/docs/system/cpu-hotplug.rst
@@ -33,23 +33,23 @@ vCPU hotplug
{
"return": [
{
- "type": "IvyBridge-IBRS-x86_64-cpu",
- "vcpus-count": 1,
"props": {
- "socket-id": 1,
- "core-id": 0,
+ "core-id": 1,
+ "socket-id": 0,
"thread-id": 0
- }
+ },
+ "type": "IvyBridge-IBRS-x86_64-cpu",
+ "vcpus-count": 1
},
{
+ "props": {
+ "core-id": 0,
+ "socket-id": 0,
+ "thread-id": 0
+ },
"qom-path": "/machine/unattached/device[0]",
"type": "IvyBridge-IBRS-x86_64-cpu",
- "vcpus-count": 1,
- "props": {
- "socket-id": 0,
- "core-id": 0,
- "thread-id": 0
- }
+ "vcpus-count": 1
}
]
}
@@ -58,18 +58,18 @@ vCPU hotplug
(4) The ``query-hotpluggable-cpus`` command returns an object for CPUs
that are present (containing a "qom-path" member) or which may be
hot-plugged (no "qom-path" member). From its output in step (3), we
- can see that ``IvyBridge-IBRS-x86_64-cpu`` is present in socket 0,
- while hot-plugging a CPU into socket 1 requires passing the listed
+ can see that ``IvyBridge-IBRS-x86_64-cpu`` is present in socket 0 core 0,
+ while hot-plugging a CPU into socket 0 core 1 requires passing the listed
properties to QMP ``device_add``::
(QEMU) device_add id=cpu-2 driver=IvyBridge-IBRS-x86_64-cpu socket-id=1 core-id=0 thread-id=0
{
"execute": "device_add",
"arguments": {
- "socket-id": 1,
+ "core-id": 1,
"driver": "IvyBridge-IBRS-x86_64-cpu",
"id": "cpu-2",
- "core-id": 0,
+ "socket-id": 0,
"thread-id": 0
}
}
@@ -83,34 +83,32 @@ vCPU hotplug
(QEMU) query-cpus-fast
{
- "execute": "query-cpus-fast",
"arguments": {}
+ "execute": "query-cpus-fast",
}
{
"return": [
{
- "qom-path": "/machine/unattached/device[0]",
- "target": "x86_64",
- "thread-id": 11534,
"cpu-index": 0,
"props": {
- "socket-id": 0,
"core-id": 0,
+ "socket-id": 0,
"thread-id": 0
},
- "arch": "x86"
+ "qom-path": "/machine/unattached/device[0]",
+ "target": "x86_64",
+ "thread-id": 28957
},
{
- "qom-path": "/machine/peripheral/cpu-2",
- "target": "x86_64",
- "thread-id": 12106,
"cpu-index": 1,
"props": {
- "socket-id": 1,
- "core-id": 0,
+ "core-id": 1,
+ "socket-id": 0,
"thread-id": 0
},
- "arch": "x86"
+ "qom-path": "/machine/peripheral/cpu-2",
+ "target": "x86_64",
+ "thread-id": 29095
}
]
}
@@ -123,10 +121,10 @@ From the 'qmp-shell', invoke the QMP ``device_del`` command::
(QEMU) device_del id=cpu-2
{
- "execute": "device_del",
"arguments": {
"id": "cpu-2"
}
+ "execute": "device_del",
}
{
"return": {}
--
2.34.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id
2024-08-19 14:43 [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id Peter Maydell
@ 2024-09-06 15:13 ` Peter Maydell
2024-09-10 9:03 ` Igor Mammedov
1 sibling, 0 replies; 8+ messages in thread
From: Peter Maydell @ 2024-09-06 15:13 UTC (permalink / raw)
To: qemu-devel
On Mon, 19 Aug 2024 at 15:43, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> At some point the way we allocate socket-id and core-id to CPUs
> by default changed; update the example of how to do CPU hotplug
> and unplug so the example commands work again. The differences
> in the sample input and output are:
> * the second CPU is now socket-id=0 core-id=1,
> not socket-id=1 core-id=0
> * the order of fields from the qmp_shell is different (it seems
> to now always be in alphabetical order)
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
> I noticed this while I was playing around with vcpu hotplug trying to
> demonstrate a memory leak I want to fix...
Ping for review on this one, please?
thanks
-- PMM
>
> docs/system/cpu-hotplug.rst | 54 ++++++++++++++++++-------------------
> 1 file changed, 26 insertions(+), 28 deletions(-)
>
> diff --git a/docs/system/cpu-hotplug.rst b/docs/system/cpu-hotplug.rst
> index 015ce2b6ec3..443ff226b90 100644
> --- a/docs/system/cpu-hotplug.rst
> +++ b/docs/system/cpu-hotplug.rst
> @@ -33,23 +33,23 @@ vCPU hotplug
> {
> "return": [
> {
> - "type": "IvyBridge-IBRS-x86_64-cpu",
> - "vcpus-count": 1,
> "props": {
> - "socket-id": 1,
> - "core-id": 0,
> + "core-id": 1,
> + "socket-id": 0,
> "thread-id": 0
> - }
> + },
> + "type": "IvyBridge-IBRS-x86_64-cpu",
> + "vcpus-count": 1
> },
> {
> + "props": {
> + "core-id": 0,
> + "socket-id": 0,
> + "thread-id": 0
> + },
> "qom-path": "/machine/unattached/device[0]",
> "type": "IvyBridge-IBRS-x86_64-cpu",
> - "vcpus-count": 1,
> - "props": {
> - "socket-id": 0,
> - "core-id": 0,
> - "thread-id": 0
> - }
> + "vcpus-count": 1
> }
> ]
> }
> @@ -58,18 +58,18 @@ vCPU hotplug
> (4) The ``query-hotpluggable-cpus`` command returns an object for CPUs
> that are present (containing a "qom-path" member) or which may be
> hot-plugged (no "qom-path" member). From its output in step (3), we
> - can see that ``IvyBridge-IBRS-x86_64-cpu`` is present in socket 0,
> - while hot-plugging a CPU into socket 1 requires passing the listed
> + can see that ``IvyBridge-IBRS-x86_64-cpu`` is present in socket 0 core 0,
> + while hot-plugging a CPU into socket 0 core 1 requires passing the listed
> properties to QMP ``device_add``::
>
> (QEMU) device_add id=cpu-2 driver=IvyBridge-IBRS-x86_64-cpu socket-id=1 core-id=0 thread-id=0
> {
> "execute": "device_add",
> "arguments": {
> - "socket-id": 1,
> + "core-id": 1,
> "driver": "IvyBridge-IBRS-x86_64-cpu",
> "id": "cpu-2",
> - "core-id": 0,
> + "socket-id": 0,
> "thread-id": 0
> }
> }
> @@ -83,34 +83,32 @@ vCPU hotplug
>
> (QEMU) query-cpus-fast
> {
> - "execute": "query-cpus-fast",
> "arguments": {}
> + "execute": "query-cpus-fast",
> }
> {
> "return": [
> {
> - "qom-path": "/machine/unattached/device[0]",
> - "target": "x86_64",
> - "thread-id": 11534,
> "cpu-index": 0,
> "props": {
> - "socket-id": 0,
> "core-id": 0,
> + "socket-id": 0,
> "thread-id": 0
> },
> - "arch": "x86"
> + "qom-path": "/machine/unattached/device[0]",
> + "target": "x86_64",
> + "thread-id": 28957
> },
> {
> - "qom-path": "/machine/peripheral/cpu-2",
> - "target": "x86_64",
> - "thread-id": 12106,
> "cpu-index": 1,
> "props": {
> - "socket-id": 1,
> - "core-id": 0,
> + "core-id": 1,
> + "socket-id": 0,
> "thread-id": 0
> },
> - "arch": "x86"
> + "qom-path": "/machine/peripheral/cpu-2",
> + "target": "x86_64",
> + "thread-id": 29095
> }
> ]
> }
> @@ -123,10 +121,10 @@ From the 'qmp-shell', invoke the QMP ``device_del`` command::
>
> (QEMU) device_del id=cpu-2
> {
> - "execute": "device_del",
> "arguments": {
> "id": "cpu-2"
> }
> + "execute": "device_del",
> }
> {
> "return": {}
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id
2024-08-19 14:43 [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id Peter Maydell
2024-09-06 15:13 ` Peter Maydell
@ 2024-09-10 9:03 ` Igor Mammedov
2024-09-10 9:38 ` Peter Maydell
1 sibling, 1 reply; 8+ messages in thread
From: Igor Mammedov @ 2024-09-10 9:03 UTC (permalink / raw)
To: Peter Maydell; +Cc: qemu-devel
On Mon, 19 Aug 2024 15:43:03 +0100
Peter Maydell <peter.maydell@linaro.org> wrote:
> At some point the way we allocate socket-id and core-id to CPUs
> by default changed; update the example of how to do CPU hotplug
> and unplug so the example commands work again. The differences
> in the sample input and output are:
> * the second CPU is now socket-id=0 core-id=1,
> not socket-id=1 core-id=0
> * the order of fields from the qmp_shell is different (it seems
> to now always be in alphabetical order)
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
> I noticed this while I was playing around with vcpu hotplug trying to
> demonstrate a memory leak I want to fix...
>
> docs/system/cpu-hotplug.rst | 54 ++++++++++++++++++-------------------
> 1 file changed, 26 insertions(+), 28 deletions(-)
>
> diff --git a/docs/system/cpu-hotplug.rst b/docs/system/cpu-hotplug.rst
> index 015ce2b6ec3..443ff226b90 100644
> --- a/docs/system/cpu-hotplug.rst
> +++ b/docs/system/cpu-hotplug.rst
> @@ -33,23 +33,23 @@ vCPU hotplug
> {
> "return": [
> {
> - "type": "IvyBridge-IBRS-x86_64-cpu",
> - "vcpus-count": 1,
> "props": {
> - "socket-id": 1,
> - "core-id": 0,
> + "core-id": 1,
> + "socket-id": 0,
> "thread-id": 0
> - }
> + },
> + "type": "IvyBridge-IBRS-x86_64-cpu",
> + "vcpus-count": 1
> },
> {
> + "props": {
> + "core-id": 0,
> + "socket-id": 0,
> + "thread-id": 0
> + },
> "qom-path": "/machine/unattached/device[0]",
> "type": "IvyBridge-IBRS-x86_64-cpu",
> - "vcpus-count": 1,
> - "props": {
> - "socket-id": 0,
> - "core-id": 0,
> - "thread-id": 0
> - }
> + "vcpus-count": 1
> }
> ]
> }
> @@ -58,18 +58,18 @@ vCPU hotplug
> (4) The ``query-hotpluggable-cpus`` command returns an object for CPUs
> that are present (containing a "qom-path" member) or which may be
> hot-plugged (no "qom-path" member). From its output in step (3), we
> - can see that ``IvyBridge-IBRS-x86_64-cpu`` is present in socket 0,
> - while hot-plugging a CPU into socket 1 requires passing the listed
> + can see that ``IvyBridge-IBRS-x86_64-cpu`` is present in socket 0 core 0,
> + while hot-plugging a CPU into socket 0 core 1 requires passing the listed
> properties to QMP ``device_add``::
>
> (QEMU) device_add id=cpu-2 driver=IvyBridge-IBRS-x86_64-cpu socket-id=1 core-id=0 thread-id=0
> {
> "execute": "device_add",
> "arguments": {
> - "socket-id": 1,
> + "core-id": 1,
> "driver": "IvyBridge-IBRS-x86_64-cpu",
> "id": "cpu-2",
> - "core-id": 0,
> + "socket-id": 0,
> "thread-id": 0
after above changes device_add doesn't match comment nor 'execute' output
> }
> }
> @@ -83,34 +83,32 @@ vCPU hotplug
>
> (QEMU) query-cpus-fast
> {
> - "execute": "query-cpus-fast",
> "arguments": {}
> + "execute": "query-cpus-fast",
> }
> {
> "return": [
> {
> - "qom-path": "/machine/unattached/device[0]",
> - "target": "x86_64",
> - "thread-id": 11534,
> "cpu-index": 0,
> "props": {
> - "socket-id": 0,
> "core-id": 0,
> + "socket-id": 0,
> "thread-id": 0
> },
> - "arch": "x86"
> + "qom-path": "/machine/unattached/device[0]",
> + "target": "x86_64",
> + "thread-id": 28957
> },
> {
> - "qom-path": "/machine/peripheral/cpu-2",
> - "target": "x86_64",
> - "thread-id": 12106,
> "cpu-index": 1,
> "props": {
> - "socket-id": 1,
> - "core-id": 0,
> + "core-id": 1,
> + "socket-id": 0,
> "thread-id": 0
> },
> - "arch": "x86"
> + "qom-path": "/machine/peripheral/cpu-2",
> + "target": "x86_64",
> + "thread-id": 29095
> }
beside reordering, which seems fine, this hunk also introduces target change
perhaps a separate patch for that?
> ]
> }
> @@ -123,10 +121,10 @@ From the 'qmp-shell', invoke the QMP ``device_del`` command::
>
> (QEMU) device_del id=cpu-2
> {
> - "execute": "device_del",
> "arguments": {
> "id": "cpu-2"
> }
> + "execute": "device_del",
> }
> {
> "return": {}
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id
2024-09-10 9:03 ` Igor Mammedov
@ 2024-09-10 9:38 ` Peter Maydell
2024-09-19 12:34 ` Peter Maydell
0 siblings, 1 reply; 8+ messages in thread
From: Peter Maydell @ 2024-09-10 9:38 UTC (permalink / raw)
To: Igor Mammedov; +Cc: qemu-devel
On Tue, 10 Sept 2024 at 10:03, Igor Mammedov <imammedo@redhat.com> wrote:
>
> On Mon, 19 Aug 2024 15:43:03 +0100
> Peter Maydell <peter.maydell@linaro.org> wrote:
>
> > At some point the way we allocate socket-id and core-id to CPUs
> > by default changed; update the example of how to do CPU hotplug
> > and unplug so the example commands work again. The differences
> > in the sample input and output are:
> > * the second CPU is now socket-id=0 core-id=1,
> > not socket-id=1 core-id=0
> > * the order of fields from the qmp_shell is different (it seems
> > to now always be in alphabetical order)
> >
> > Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> > ---
> > I noticed this while I was playing around with vcpu hotplug trying to
> > demonstrate a memory leak I want to fix...
> >
> > docs/system/cpu-hotplug.rst | 54 ++++++++++++++++++-------------------
> > 1 file changed, 26 insertions(+), 28 deletions(-)
> >
> > diff --git a/docs/system/cpu-hotplug.rst b/docs/system/cpu-hotplug.rst
> > index 015ce2b6ec3..443ff226b90 100644
> > --- a/docs/system/cpu-hotplug.rst
> > +++ b/docs/system/cpu-hotplug.rst
> > @@ -33,23 +33,23 @@ vCPU hotplug
> > {
> > "return": [
> > {
> > - "type": "IvyBridge-IBRS-x86_64-cpu",
> > - "vcpus-count": 1,
> > "props": {
> > - "socket-id": 1,
> > - "core-id": 0,
> > + "core-id": 1,
> > + "socket-id": 0,
> > "thread-id": 0
> > - }
> > + },
> > + "type": "IvyBridge-IBRS-x86_64-cpu",
> > + "vcpus-count": 1
> > },
> > {
> > + "props": {
> > + "core-id": 0,
> > + "socket-id": 0,
> > + "thread-id": 0
> > + },
> > "qom-path": "/machine/unattached/device[0]",
> > "type": "IvyBridge-IBRS-x86_64-cpu",
> > - "vcpus-count": 1,
> > - "props": {
> > - "socket-id": 0,
> > - "core-id": 0,
> > - "thread-id": 0
> > - }
> > + "vcpus-count": 1
> > }
> > ]
> > }
> > @@ -58,18 +58,18 @@ vCPU hotplug
> > (4) The ``query-hotpluggable-cpus`` command returns an object for CPUs
> > that are present (containing a "qom-path" member) or which may be
> > hot-plugged (no "qom-path" member). From its output in step (3), we
> > - can see that ``IvyBridge-IBRS-x86_64-cpu`` is present in socket 0,
> > - while hot-plugging a CPU into socket 1 requires passing the listed
> > + can see that ``IvyBridge-IBRS-x86_64-cpu`` is present in socket 0 core 0,
> > + while hot-plugging a CPU into socket 0 core 1 requires passing the listed
> > properties to QMP ``device_add``::
> >
>
> > (QEMU) device_add id=cpu-2 driver=IvyBridge-IBRS-x86_64-cpu socket-id=1 core-id=0 thread-id=0
>
> > {
> > "execute": "device_add",
> > "arguments": {
> > - "socket-id": 1,
> > + "core-id": 1,
> > "driver": "IvyBridge-IBRS-x86_64-cpu",
> > "id": "cpu-2",
> > - "core-id": 0,
> > + "socket-id": 0,
> > "thread-id": 0
>
> after above changes device_add doesn't match comment nor 'execute' output
Oops, yes. The device_add line should be changed too
("socket-id=0 core-id=1 thread-id=0").
>
> > }
> > }
> > @@ -83,34 +83,32 @@ vCPU hotplug
> >
> > (QEMU) query-cpus-fast
> > {
> > - "execute": "query-cpus-fast",
> > "arguments": {}
> > + "execute": "query-cpus-fast",
> > }
> > {
> > "return": [
> > {
> > - "qom-path": "/machine/unattached/device[0]",
> > - "target": "x86_64",
> > - "thread-id": 11534,
> > "cpu-index": 0,
> > "props": {
> > - "socket-id": 0,
> > "core-id": 0,
> > + "socket-id": 0,
> > "thread-id": 0
> > },
> > - "arch": "x86"
> > + "qom-path": "/machine/unattached/device[0]",
> > + "target": "x86_64",
> > + "thread-id": 28957
> > },
> > {
> > - "qom-path": "/machine/peripheral/cpu-2",
> > - "target": "x86_64",
> > - "thread-id": 12106,
> > "cpu-index": 1,
> > "props": {
> > - "socket-id": 1,
> > - "core-id": 0,
> > + "core-id": 1,
> > + "socket-id": 0,
> > "thread-id": 0
> > },
> > - "arch": "x86"
> > + "qom-path": "/machine/peripheral/cpu-2",
> > + "target": "x86_64",
> > + "thread-id": 29095
> > }
>
> beside reordering, which seems fine, this hunk also introduces target change
> perhaps a separate patch for that?
What target change? It all says "target": "x86_64" both before
and after.
thanks
-- PMM
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id
2024-09-10 9:38 ` Peter Maydell
@ 2024-09-19 12:34 ` Peter Maydell
2024-09-20 10:45 ` Igor Mammedov
0 siblings, 1 reply; 8+ messages in thread
From: Peter Maydell @ 2024-09-19 12:34 UTC (permalink / raw)
To: Igor Mammedov; +Cc: qemu-devel
On Tue, 10 Sept 2024 at 10:38, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> On Tue, 10 Sept 2024 at 10:03, Igor Mammedov <imammedo@redhat.com> wrote:
> >
> > On Mon, 19 Aug 2024 15:43:03 +0100
> > Peter Maydell <peter.maydell@linaro.org> wrote:
> > > @@ -83,34 +83,32 @@ vCPU hotplug
> > >
> > > (QEMU) query-cpus-fast
> > > {
> > > - "execute": "query-cpus-fast",
> > > "arguments": {}
> > > + "execute": "query-cpus-fast",
> > > }
> > > {
> > > "return": [
> > > {
> > > - "qom-path": "/machine/unattached/device[0]",
> > > - "target": "x86_64",
> > > - "thread-id": 11534,
> > > "cpu-index": 0,
> > > "props": {
> > > - "socket-id": 0,
> > > "core-id": 0,
> > > + "socket-id": 0,
> > > "thread-id": 0
> > > },
> > > - "arch": "x86"
> > > + "qom-path": "/machine/unattached/device[0]",
> > > + "target": "x86_64",
> > > + "thread-id": 28957
> > > },
> > > {
> > > - "qom-path": "/machine/peripheral/cpu-2",
> > > - "target": "x86_64",
> > > - "thread-id": 12106,
> > > "cpu-index": 1,
> > > "props": {
> > > - "socket-id": 1,
> > > - "core-id": 0,
> > > + "core-id": 1,
> > > + "socket-id": 0,
> > > "thread-id": 0
> > > },
> > > - "arch": "x86"
> > > + "qom-path": "/machine/peripheral/cpu-2",
> > > + "target": "x86_64",
> > > + "thread-id": 29095
> > > }
> >
> > beside reordering, which seems fine, this hunk also introduces target change
> > perhaps a separate patch for that?
>
> What target change? It all says "target": "x86_64" both before
> and after.
Hi Igor, would you mind clarifying what you meant here?
I'm happy to respin, but I don't understand what change you're
asking for.
thanks
-- PMM
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id
2024-09-19 12:34 ` Peter Maydell
@ 2024-09-20 10:45 ` Igor Mammedov
2024-10-03 14:54 ` Peter Maydell
0 siblings, 1 reply; 8+ messages in thread
From: Igor Mammedov @ 2024-09-20 10:45 UTC (permalink / raw)
To: Peter Maydell; +Cc: qemu-devel
On Thu, 19 Sep 2024 13:34:54 +0100
Peter Maydell <peter.maydell@linaro.org> wrote:
> On Tue, 10 Sept 2024 at 10:38, Peter Maydell <peter.maydell@linaro.org> wrote:
> >
> > On Tue, 10 Sept 2024 at 10:03, Igor Mammedov <imammedo@redhat.com> wrote:
> > >
> > > On Mon, 19 Aug 2024 15:43:03 +0100
> > > Peter Maydell <peter.maydell@linaro.org> wrote:
> > > > @@ -83,34 +83,32 @@ vCPU hotplug
> > > >
> > > > (QEMU) query-cpus-fast
> > > > {
> > > > - "execute": "query-cpus-fast",
> > > > "arguments": {}
> > > > + "execute": "query-cpus-fast",
> > > > }
> > > > {
> > > > "return": [
> > > > {
> > > > - "qom-path": "/machine/unattached/device[0]",
> > > > - "target": "x86_64",
> > > > - "thread-id": 11534,
> > > > "cpu-index": 0,
> > > > "props": {
> > > > - "socket-id": 0,
> > > > "core-id": 0,
> > > > + "socket-id": 0,
> > > > "thread-id": 0
> > > > },
> > > > - "arch": "x86"
> > > > + "qom-path": "/machine/unattached/device[0]",
> > > > + "target": "x86_64",
> > > > + "thread-id": 28957
> > > > },
> > > > {
> > > > - "qom-path": "/machine/peripheral/cpu-2",
> > > > - "target": "x86_64",
> > > > - "thread-id": 12106,
> > > > "cpu-index": 1,
> > > > "props": {
> > > > - "socket-id": 1,
> > > > - "core-id": 0,
> > > > + "core-id": 1,
> > > > + "socket-id": 0,
> > > > "thread-id": 0
> > > > },
> > > > - "arch": "x86"
> > > > + "qom-path": "/machine/peripheral/cpu-2",
> > > > + "target": "x86_64",
> > > > + "thread-id": 29095
> > > > }
> > >
> > > beside reordering, which seems fine, this hunk also introduces target change
> > > perhaps a separate patch for that?
> >
> > What target change? It all says "target": "x86_64" both before
> > and after.
my mistake,
I should've said '"arch": "x86"' instead, which is gone after the patch
>
> Hi Igor, would you mind clarifying what you meant here?
> I'm happy to respin, but I don't understand what change you're
> asking for.
>
> thanks
> -- PMM
>
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id
2024-09-20 10:45 ` Igor Mammedov
@ 2024-10-03 14:54 ` Peter Maydell
2024-10-07 12:04 ` Igor Mammedov
0 siblings, 1 reply; 8+ messages in thread
From: Peter Maydell @ 2024-10-03 14:54 UTC (permalink / raw)
To: Igor Mammedov; +Cc: qemu-devel
On Fri, 20 Sept 2024 at 11:45, Igor Mammedov <imammedo@redhat.com> wrote:
>
> On Thu, 19 Sep 2024 13:34:54 +0100
> Peter Maydell <peter.maydell@linaro.org> wrote:
>
> > On Tue, 10 Sept 2024 at 10:38, Peter Maydell <peter.maydell@linaro.org> wrote:
> > >
> > > On Tue, 10 Sept 2024 at 10:03, Igor Mammedov <imammedo@redhat.com> wrote:
> > > >
> > > > On Mon, 19 Aug 2024 15:43:03 +0100
> > > > Peter Maydell <peter.maydell@linaro.org> wrote:
> > > > > @@ -83,34 +83,32 @@ vCPU hotplug
> > > > >
> > > > > (QEMU) query-cpus-fast
> > > > > {
> > > > > - "execute": "query-cpus-fast",
> > > > > "arguments": {}
> > > > > + "execute": "query-cpus-fast",
> > > > > }
> > > > > {
> > > > > "return": [
> > > > > {
> > > > > - "qom-path": "/machine/unattached/device[0]",
> > > > > - "target": "x86_64",
> > > > > - "thread-id": 11534,
> > > > > "cpu-index": 0,
> > > > > "props": {
> > > > > - "socket-id": 0,
> > > > > "core-id": 0,
> > > > > + "socket-id": 0,
> > > > > "thread-id": 0
> > > > > },
> > > > > - "arch": "x86"
> > > > > + "qom-path": "/machine/unattached/device[0]",
> > > > > + "target": "x86_64",
> > > > > + "thread-id": 28957
> > > > > },
> > > > > {
> > > > > - "qom-path": "/machine/peripheral/cpu-2",
> > > > > - "target": "x86_64",
> > > > > - "thread-id": 12106,
> > > > > "cpu-index": 1,
> > > > > "props": {
> > > > > - "socket-id": 1,
> > > > > - "core-id": 0,
> > > > > + "core-id": 1,
> > > > > + "socket-id": 0,
> > > > > "thread-id": 0
> > > > > },
> > > > > - "arch": "x86"
> > > > > + "qom-path": "/machine/peripheral/cpu-2",
> > > > > + "target": "x86_64",
> > > > > + "thread-id": 29095
> > > > > }
> > > >
> > > > beside reordering, which seems fine, this hunk also introduces target change
> > > > perhaps a separate patch for that?
> > >
> > > What target change? It all says "target": "x86_64" both before
> > > and after.
>
> my mistake,
> I should've said '"arch": "x86"' instead, which is gone after the patch
This is because the "arch" output member was removed
from the query-cpus-fast output in QEMU 6.0. If we
mention this also in the commit message, is that OK?
======
docs/system/cpu-hotplug: Update example to match current QEMU
The example of how to do vCPU hotplug and hot-unlpug in the
cpu-hotplug documentation no longer works, because the way
we allocate socket-id and core-id to CPUs by default has
changed at some point. The output also no longer matches what
current QEMU produces in some more cosmetic ways.
Update the example to match current QEMU. The differences are:
* the second CPU is now socket-id=0 core-id=1,
not socket-id=1 core-id=0
* the order of fields from the qmp_shell is now in
alphabetical order
* the "arch" member is no longer present in the query-cpus-fast
output (it was removed in QEMU 6.0)
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
======
If that seems OK to you I'll send out a v2 with the updated
commit message and the fix to the device_add line.
thanks
-- PMM
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id
2024-10-03 14:54 ` Peter Maydell
@ 2024-10-07 12:04 ` Igor Mammedov
0 siblings, 0 replies; 8+ messages in thread
From: Igor Mammedov @ 2024-10-07 12:04 UTC (permalink / raw)
To: Peter Maydell; +Cc: qemu-devel
On Thu, 3 Oct 2024 15:54:45 +0100
Peter Maydell <peter.maydell@linaro.org> wrote:
> On Fri, 20 Sept 2024 at 11:45, Igor Mammedov <imammedo@redhat.com> wrote:
> >
> > On Thu, 19 Sep 2024 13:34:54 +0100
> > Peter Maydell <peter.maydell@linaro.org> wrote:
> >
> > > On Tue, 10 Sept 2024 at 10:38, Peter Maydell <peter.maydell@linaro.org> wrote:
> > > >
> > > > On Tue, 10 Sept 2024 at 10:03, Igor Mammedov <imammedo@redhat.com> wrote:
> > > > >
> > > > > On Mon, 19 Aug 2024 15:43:03 +0100
> > > > > Peter Maydell <peter.maydell@linaro.org> wrote:
> > > > > > @@ -83,34 +83,32 @@ vCPU hotplug
> > > > > >
> > > > > > (QEMU) query-cpus-fast
> > > > > > {
> > > > > > - "execute": "query-cpus-fast",
> > > > > > "arguments": {}
> > > > > > + "execute": "query-cpus-fast",
> > > > > > }
> > > > > > {
> > > > > > "return": [
> > > > > > {
> > > > > > - "qom-path": "/machine/unattached/device[0]",
> > > > > > - "target": "x86_64",
> > > > > > - "thread-id": 11534,
> > > > > > "cpu-index": 0,
> > > > > > "props": {
> > > > > > - "socket-id": 0,
> > > > > > "core-id": 0,
> > > > > > + "socket-id": 0,
> > > > > > "thread-id": 0
> > > > > > },
> > > > > > - "arch": "x86"
> > > > > > + "qom-path": "/machine/unattached/device[0]",
> > > > > > + "target": "x86_64",
> > > > > > + "thread-id": 28957
> > > > > > },
> > > > > > {
> > > > > > - "qom-path": "/machine/peripheral/cpu-2",
> > > > > > - "target": "x86_64",
> > > > > > - "thread-id": 12106,
> > > > > > "cpu-index": 1,
> > > > > > "props": {
> > > > > > - "socket-id": 1,
> > > > > > - "core-id": 0,
> > > > > > + "core-id": 1,
> > > > > > + "socket-id": 0,
> > > > > > "thread-id": 0
> > > > > > },
> > > > > > - "arch": "x86"
> > > > > > + "qom-path": "/machine/peripheral/cpu-2",
> > > > > > + "target": "x86_64",
> > > > > > + "thread-id": 29095
> > > > > > }
> > > > >
> > > > > beside reordering, which seems fine, this hunk also introduces target change
> > > > > perhaps a separate patch for that?
> > > >
> > > > What target change? It all says "target": "x86_64" both before
> > > > and after.
> >
> > my mistake,
> > I should've said '"arch": "x86"' instead, which is gone after the patch
>
> This is because the "arch" output member was removed
> from the query-cpus-fast output in QEMU 6.0. If we
> mention this also in the commit message, is that OK?
>
> ======
> docs/system/cpu-hotplug: Update example to match current QEMU
>
> The example of how to do vCPU hotplug and hot-unlpug in the
> cpu-hotplug documentation no longer works, because the way
> we allocate socket-id and core-id to CPUs by default has
> changed at some point. The output also no longer matches what
> current QEMU produces in some more cosmetic ways.
>
> Update the example to match current QEMU. The differences are:
> * the second CPU is now socket-id=0 core-id=1,
> not socket-id=1 core-id=0
> * the order of fields from the qmp_shell is now in
> alphabetical order
> * the "arch" member is no longer present in the query-cpus-fast
> output (it was removed in QEMU 6.0)
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ======
Acked-by: Igor Mammedov <imammedo@redhat.com>
>
> If that seems OK to you I'll send out a v2 with the updated
> commit message and the fix to the device_add line.
>
> thanks
> -- PMM
>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2024-10-07 12:04 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-19 14:43 [PATCH] docs/system/cpu-hotplug: Update example's socket-id/core-id Peter Maydell
2024-09-06 15:13 ` Peter Maydell
2024-09-10 9:03 ` Igor Mammedov
2024-09-10 9:38 ` Peter Maydell
2024-09-19 12:34 ` Peter Maydell
2024-09-20 10:45 ` Igor Mammedov
2024-10-03 14:54 ` Peter Maydell
2024-10-07 12:04 ` Igor Mammedov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).