[PATCH 0/2] Add support for Unix mode to serial port dev
by Praveen K Paladugu
This patchset adds support for Unix mode to serial port device in ch guests.
Praveen K Paladugu (2):
ch: Add Cap checks for unix backend of serial port
ch: Add support for `Unix` mode to serial port dev
src/ch/ch_capabilities.c | 6 ++++++
src/ch/ch_capabilities.h | 1 +
src/ch/ch_domain.c | 21 +++++++++++++++------
src/ch/ch_monitor.c | 35 ++++++++++++++++++++++++-----------
src/ch/ch_process.c | 7 +++++++
5 files changed, 53 insertions(+), 17 deletions(-)
--
2.43.0
8 months, 2 weeks
[PATCH] tests: Don't advertise features for edk2-loongarch64
by Andrea Bolognani
The "acpi" feature is not part of the specification[1],
so it will just get ignored.
[1] https://gitlab.com/qemu-project/qemu/-/blob/master/docs/interop/firmware....
Signed-off-by: Andrea Bolognani <abologna(a)redhat.com>
---
.../usr/share/qemu/firmware/50-edk2-loongarch64.json | 1 -
1 file changed, 1 deletion(-)
diff --git a/tests/qemufirmwaredata/usr/share/qemu/firmware/50-edk2-loongarch64.json b/tests/qemufirmwaredata/usr/share/qemu/firmware/50-edk2-loongarch64.json
index 8251137ba9..c5a7ec5f7b 100644
--- a/tests/qemufirmwaredata/usr/share/qemu/firmware/50-edk2-loongarch64.json
+++ b/tests/qemufirmwaredata/usr/share/qemu/firmware/50-edk2-loongarch64.json
@@ -24,6 +24,5 @@
}
],
"features": [
- "acpi"
]
}
--
2.44.0
8 months, 2 weeks
[PATCH] docs: Update documentation of CPU models in domain caps
by Jiri Denemark
Using check='none' when starting a domain with a CPU model marked as
usable is no longer needed as libvirt will do the right thing even with
check='partial'.
Signed-off-by: Jiri Denemark <jdenemar(a)redhat.com>
---
docs/formatdomaincaps.rst | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/docs/formatdomaincaps.rst b/docs/formatdomaincaps.rst
index ef752a0f3a..1999112944 100644
--- a/docs/formatdomaincaps.rst
+++ b/docs/formatdomaincaps.rst
@@ -237,10 +237,12 @@ more details about it:
using the CPU model and no additional feature elements. Models marked as
usable (``usable='yes'``) can be safely used in domain XMLs with
``check='none'`` as the hypervisor guarantees the model can be used on the
- current host and additional checks done by libvirt are redundant. In fact,
- disabling libvirt checks via ``check='none'`` for such models is recommended
- to avoid needless issues with starting domains when libvirt's definition of
- a particular model differs from hypervisor's definition. The
+ current host and additional checks done by libvirt are redundant.
+ :since:`Since 10.2.0` libvirt automatically detects this situation and
+ avoids the redundant checks, with older releases disabling libvirt checks
+ via ``check='none'`` for such models is recommended to avoid needless
+ issues with starting domains when libvirt's definition of a particular model
+ differs from hypervisor's definition. The
``deprecated`` attribute reflects the hypervisor's policy on usage of this
model :since:`(since 7.1.0)`. The ``vendor`` attribute :since:`(since 8.9.0)`
contains the vendor of the CPU model for users who want to use CPU models
--
2.44.0
8 months, 2 weeks
[PATCH 02/14] hw/core/machine-smp: Deprecate unsupported "parameter=1" SMP configurations
by Zhao Liu
From: Zhao Liu <zhao1.liu(a)intel.com>
Currentlt, it was allowed for users to specify the unsupported
topology parameter as "1". For example, x86 PC machine doesn't
support drawer/book/cluster topology levels, but user could specify
"-smp drawers=1,books=1,clusters=1".
This is meaningless and confusing, so that the support for this kind of
configurations is marked depresated since 9.0. And report warning
message for such case like:
qemu-system-x86_64: warning: Deprecated CPU topology (considered invalid):
Unsupported clusters parameter mustn't be specified as 1
qemu-system-x86_64: warning: Deprecated CPU topology (considered invalid):
Unsupported books parameter mustn't be specified as 1
qemu-system-x86_64: warning: Deprecated CPU topology (considered invalid):
Unsupported drawers parameter mustn't be specified as 1
Users have to ensure that all the topology members described with -smp
are supported by the target machine.
Cc: devel(a)lists.libvirt.org
Signed-off-by: Zhao Liu <zhao1.liu(a)intel.com>
---
docs/about/deprecated.rst | 14 +++++++++
hw/core/machine-smp.c | 63 +++++++++++++++++++++++++++++----------
2 files changed, 61 insertions(+), 16 deletions(-)
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index 872974640252..2e782e83e952 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -47,6 +47,20 @@ as short-form boolean values, and passed to plugins as ``arg_name=on``.
However, short-form booleans are deprecated and full explicit ``arg_name=on``
form is preferred.
+``-smp`` (Unsopported "parameter=1" SMP configurations) (since 9.0)
+'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+
+Specified CPU topology parameters must be supported by the machine.
+
+In the SMP configuration, users should provide the CPU topology parameters that
+are supported by the target machine.
+
+However, historically it was allowed for users to specify the unsupported
+topology parameter as "1", which is meaningless. So support for this kind of
+configurations (e.g. -smp drawers=1,books=1,clusters=1 for x86 PC machine) is
+marked depresated since 9.0, users have to ensure that all the topology members
+described with -smp are supported by the target machine.
+
QEMU Machine Protocol (QMP) commands
------------------------------------
diff --git a/hw/core/machine-smp.c b/hw/core/machine-smp.c
index 96533886b14e..50a5a40dbc3d 100644
--- a/hw/core/machine-smp.c
+++ b/hw/core/machine-smp.c
@@ -112,30 +112,61 @@ void machine_parse_smp_config(MachineState *ms,
/*
* If not supported by the machine, a topology parameter must be
- * omitted or specified equal to 1.
+ * omitted.
*/
- if (!mc->smp_props.dies_supported && dies > 1) {
- error_setg(errp, "dies not supported by this machine's CPU topology");
- return;
- }
- if (!mc->smp_props.clusters_supported && clusters > 1) {
- error_setg(errp, "clusters not supported by this machine's CPU topology");
- return;
+ if (!mc->smp_props.clusters_supported && config->has_clusters) {
+ if (config->clusters > 1) {
+ error_setg(errp, "clusters not supported by this "
+ "machine's CPU topology");
+ return;
+ } else {
+ /* Here clusters only equals 1 since we've checked zero case. */
+ warn_report("Deprecated CPU topology (considered invalid): "
+ "Unsupported clusters parameter mustn't be "
+ "specified as 1");
+ }
}
+ clusters = clusters > 0 ? clusters : 1;
+ if (!mc->smp_props.dies_supported && config->has_dies) {
+ if (config->dies > 1) {
+ error_setg(errp, "dies not supported by this "
+ "machine's CPU topology");
+ return;
+ } else {
+ /* Here dies only equals 1 since we've checked zero case. */
+ warn_report("Deprecated CPU topology (considered invalid): "
+ "Unsupported dies parameter mustn't be "
+ "specified as 1");
+ }
+ }
dies = dies > 0 ? dies : 1;
- clusters = clusters > 0 ? clusters : 1;
- if (!mc->smp_props.books_supported && books > 1) {
- error_setg(errp, "books not supported by this machine's CPU topology");
- return;
+ if (!mc->smp_props.books_supported && config->has_books) {
+ if (config->books > 1) {
+ error_setg(errp, "books not supported by this "
+ "machine's CPU topology");
+ return;
+ } else {
+ /* Here books only equals 1 since we've checked zero case. */
+ warn_report("Deprecated CPU topology (considered invalid): "
+ "Unsupported books parameter mustn't be "
+ "specified as 1");
+ }
}
books = books > 0 ? books : 1;
- if (!mc->smp_props.drawers_supported && drawers > 1) {
- error_setg(errp,
- "drawers not supported by this machine's CPU topology");
- return;
+ if (!mc->smp_props.drawers_supported && config->has_drawers) {
+ if (config->drawers > 1) {
+ error_setg(errp, "drawers not supported by this "
+ "machine's CPU topology");
+ return;
+ } else {
+ /* Here drawers only equals 1 since we've checked zero case. */
+ warn_report("Deprecated CPU topology (considered invalid): "
+ "Unsupported drawers parameter mustn't be "
+ "specified as 1");
+ }
}
drawers = drawers > 0 ? drawers : 1;
--
2.34.1
8 months, 2 weeks
[PATCH 0/5] multiple memory backend support for CPR Live Updates
by mgalaxy＠akamai.com
From: Michael Galaxy <mgalaxy(a)akamai.com>
CPR-based support for whole-hypervisor kexec-based live updates is
making progress on qemu-devel. In support of this, we need NUMA to be
support in these kinds of environments. To do this we use a technology
called PMEM (persistent memory), which underpin the ability for
CPR Live Updates to work so that QEMU memory can remain in RAM and
be recovered after the kexec has completed.
Our systems are highly NUMA-aware, and so this patch series enables
NUMA awareness for live update and also allows live migrations to work.
Further, we make a small change that allows live migrations to work
between *non* PMEM-based systems and PMEM-based systems (and
vice-versa). This allows for seemless upgrades from non-live-compatible
systems to live-update-compatible sytems without any downtime.
Michael Galaxy (5):
qemu.conf changes to support multiple memory backend directories
Update cleanup routines to handle multiple memory backing paths
instead of just one.
Implement multiple memory backing paths
Support live migration between file-backed memory and anonymous
memory.
Update unit test to support multiple memory backends
src/qemu/qemu_command.c | 8 ++-
src/qemu/qemu_conf.c | 140 ++++++++++++++++++++++++++++++++++++----
src/qemu/qemu_conf.h | 14 ++--
src/qemu/qemu_domain.c | 24 +++++--
src/qemu/qemu_driver.c | 31 +++++----
src/qemu/qemu_hotplug.c | 6 +-
src/qemu/qemu_process.c | 45 +++++++------
src/qemu/qemu_process.h | 13 ++--
tests/testutilsqemu.c | 5 +-
9 files changed, 217 insertions(+), 69 deletions(-)
--
2.25.1
8 months, 2 weeks
[libvirt PATCH 0/5] implement virDomainGraphicsReload
by Ján Tomko
https://issues.redhat.com/browse/RHEL-16333
Based on the patches Huawei sent back in 2021:
https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/LD...
* rebased
* renamed
* simplified (removed virTypedParams from the API and the requirement to
specify the type)
Ján Tomko (1):
remote: implement virDomainGraphicsReload
Zheng Yan (4):
qemu_capabilities: Add QEMU_CAPS_DISPLAY_RELOAD
libvirt: Introduce virDomainGraphicsReload API
qemu: implement qemuDomainGraphicsReload
virsh: Introduce domdisplay-reload command
docs/manpages/virsh.rst | 15 ++++++
include/libvirt/libvirt-domain.h | 18 +++++++
src/driver-hypervisor.h | 6 +++
src/libvirt-domain.c | 50 +++++++++++++++++
src/libvirt_public.syms | 5 ++
src/qemu/qemu_capabilities.c | 2 +
src/qemu/qemu_capabilities.h | 1 +
src/qemu/qemu_driver.c | 54 +++++++++++++++++++
src/qemu/qemu_monitor.c | 10 ++++
src/qemu/qemu_monitor.h | 5 ++
src/qemu/qemu_monitor_json.c | 23 ++++++++
src/qemu/qemu_monitor_json.h | 4 ++
src/remote/remote_driver.c | 1 +
src/remote/remote_protocol.x | 13 ++++-
src/remote_protocol-structs | 6 +++
.../caps_6.0.0_aarch64.xml | 1 +
.../qemucapabilitiesdata/caps_6.0.0_s390x.xml | 1 +
.../caps_6.0.0_x86_64.xml | 1 +
.../caps_6.1.0_x86_64.xml | 1 +
.../caps_6.2.0_aarch64.xml | 1 +
.../qemucapabilitiesdata/caps_6.2.0_ppc64.xml | 1 +
.../caps_6.2.0_x86_64.xml | 1 +
.../caps_7.0.0_aarch64+hvf.xml | 1 +
.../caps_7.0.0_aarch64.xml | 1 +
.../qemucapabilitiesdata/caps_7.0.0_ppc64.xml | 1 +
.../caps_7.0.0_x86_64.xml | 1 +
.../qemucapabilitiesdata/caps_7.1.0_ppc64.xml | 1 +
.../caps_7.1.0_x86_64.xml | 1 +
tests/qemucapabilitiesdata/caps_7.2.0_ppc.xml | 1 +
.../caps_7.2.0_x86_64+hvf.xml | 1 +
.../caps_7.2.0_x86_64.xml | 1 +
.../caps_8.0.0_riscv64.xml | 1 +
.../caps_8.0.0_x86_64.xml | 1 +
.../qemucapabilitiesdata/caps_8.1.0_s390x.xml | 1 +
.../caps_8.1.0_x86_64.xml | 1 +
.../caps_8.2.0_aarch64.xml | 1 +
.../qemucapabilitiesdata/caps_8.2.0_s390x.xml | 1 +
.../caps_8.2.0_x86_64.xml | 1 +
.../caps_9.0.0_x86_64.xml | 1 +
tools/virsh-domain.c | 45 ++++++++++++++++
40 files changed, 281 insertions(+), 1 deletion(-)
--
2.43.2
8 months, 2 weeks
[PATCH 00/25] qemu: Remove use of '-usb' to instantiate controllers
by Peter Krempa
This patchset removes the use of '-usb' legacy controller specification.
For the vast majority of platforms/machines (most of the irrelevant
ones) which don't actually have a concept of implicit USB qemu ignores
the '-usb' argument.
For the relevant platforms libvirt would already pick exactly the
same controller and thus '-usb' wouldn't be used at all.
There's a small handful of ARM boards 'realview' and 'versatilepb' where
libvirt would pick the wrong one as default if some controllers were
compiled out and thus reallistically '-usb' would have effect for those
although would be unusable as libvirt's bus name for the bus is incorret
and thus no devices would be usable.
For the two mostly obsolete board types this series can break ABI in the
aforementioned corner case.
Other than that this removes all the legacy code which makes USB
controller selection utterly confusing.
Peter Krempa (25):
tests: Replace mentions of the removed 'qemucapabilitiesnumbering'
tool
qemucapabilitiesdata: Document faked caps dumps
testutilshostcpus: Add support for VIR_ARCH_ARMV7L
qemucapabilitiestest: Add test dump for 'armv7l' architecture
qemu: domain: Add checkers for 'microvm' machine type
qemu: Add basic framework for 'microvm' machine type
qemuxmlconfdata: Add tests for USB controller selection with
'versatilepb' machine on aarch64 qemu binary
qemuxmlconfdata: Add tests for USB controller selection with
'versatilepb' machine on arm qemu binary
qemuDomainDefAddDefaultDevices: Handle defaults for all ARM arches
together
qemuDomainControllerDefPostParse: Use proper enum value for default
USB controller model
qemuDomainDefAddDefaultDevices: Populate default USB for 'versatilepb'
and 'realview' ARM machines
qemuDomainControllerDefPostParse: Use 'pci-ohci' as last-resort
fallback USB controller
qemuxmlconftest: Add test for 'realview' machine
qemuxmlconftest: Add test data for an ARM machine where qemu doesn't
honour -usb
qemuxml2conftest: Consolidate usb controller tests for 'pseries'
machine
qemuxmlconftest: Add test case when default USB controller for
'pseries' is compiled out
qemuxmlconftest: Add test data for USB selection with 'g3beige'
machine on ppc64
qemuxmlconftest: Add test data for USB selection with 'mac99' machine
on ppc64
qemuxmlconftest: Add test data for USB selection with 'powernv'
machine on ppc64
qemuxmlconftest: Add test data for USB selection with 'mac99' machine
on ppc
qemu: command: Don't downgrade to '-usb' for x86 based machines
qemu: command: Don't downgrade to '-usb' with 'pseries' machines
qemu: command: Don't downgrade to '-usb' for ppc based machines
qemu: command: Don't downgrade to '-usb' for arm based machines
qemu: command: Remove fallback '-usb' handling
src/qemu/qemu_command.c | 87 +-
src/qemu/qemu_domain.c | 63 +-
src/qemu/qemu_domain.h | 1 +
tests/domaincapsdata/qemu_8.2.0.armv7l.xml | 175 +
tests/domaincapstest.c | 5 +-
tests/qemucapabilitiesdata/README.rst | 18 +-
.../caps_8.2.0_armv7l.replies | 35062 ++++++++++++++++
.../caps_8.2.0_armv7l.xml | 326 +
tests/qemucaps2xmloutdata/caps.armv7l.xml | 26 +
...usb-minimal.aarch64-latest.abi-update.args | 30 +
...ousb-minimal.aarch64-latest.abi-update.xml | 21 +
.../aarch64-nousb-minimal.aarch64-latest.args | 30 +
.../aarch64-nousb-minimal.aarch64-latest.xml | 21 +
.../qemuxmlconfdata/aarch64-nousb-minimal.xml | 15 +
...iew-minimal.aarch64-latest.abi-update.args | 31 +
...view-minimal.aarch64-latest.abi-update.xml | 22 +
...rch64-realview-minimal.aarch64-latest.args | 31 +
...arch64-realview-minimal.aarch64-latest.xml | 22 +
.../aarch64-realview-minimal.xml | 14 +
...epb-minimal.aarch64-latest.abi-update.args | 32 +
...lepb-minimal.aarch64-latest.abi-update.xml | 25 +
...64-versatilepb-minimal.aarch64-latest.args | 32 +
...h64-versatilepb-minimal.aarch64-latest.xml | 25 +
.../aarch64-versatilepb-minimal.xml | 14 +
.../arm-vexpressa9-basic.aarch64-latest.args | 1 -
.../arm-vexpressa9-nodevs.aarch64-latest.args | 1 -
.../arm-vexpressa9-virtio.aarch64-latest.args | 1 -
...lepb-minimal.armv7l-latest.abi-update.args | 32 +
...ilepb-minimal.armv7l-latest.abi-update.xml | 27 +
...v7l-versatilepb-minimal.armv7l-latest.args | 32 +
...mv7l-versatilepb-minimal.armv7l-latest.xml | 27 +
.../armv7l-versatilepb-minimal.xml | 13 +
.../disk-arm-virtio-sd.aarch64-latest.args | 1 -
.../microvm.x86_64-latest.args | 34 +
.../qemuxmlconfdata/microvm.x86_64-latest.xml | 29 +
tests/qemuxmlconfdata/microvm.xml | 20 +
...c-mac99-minimal.ppc-latest.abi-update.args | 32 +
...pc-mac99-minimal.ppc-latest.abi-update.xml | 26 +
.../ppc-mac99-minimal.ppc-latest.args | 32 +
.../ppc-mac99-minimal.ppc-latest.xml | 26 +
tests/qemuxmlconfdata/ppc-mac99-minimal.xml | 12 +
...beige-minimal.ppc64-latest.abi-update.args | 33 +
...3beige-minimal.ppc64-latest.abi-update.xml | 29 +
.../ppc64-g3beige-minimal.ppc64-latest.args | 33 +
.../ppc64-g3beige-minimal.ppc64-latest.xml | 29 +
.../qemuxmlconfdata/ppc64-g3beige-minimal.xml | 12 +
...mac99-minimal.ppc64-latest.abi-update.args | 33 +
...-mac99-minimal.ppc64-latest.abi-update.xml | 29 +
.../ppc64-mac99-minimal.ppc64-latest.args | 33 +
.../ppc64-mac99-minimal.ppc64-latest.xml | 29 +
tests/qemuxmlconfdata/ppc64-mac99-minimal.xml | 12 +
...ernv9-minimal.ppc64-latest.abi-update.args | 33 +
...wernv9-minimal.ppc64-latest.abi-update.xml | 29 +
.../ppc64-powernv9-minimal.ppc64-latest.args | 33 +
.../ppc64-powernv9-minimal.ppc64-latest.xml | 29 +
.../ppc64-powernv9-minimal.xml | 12 +
.../sparc-minimal.sparc-latest.args | 1 -
...default-fallback-g3beige.ppc64-latest.args | 33 +
...-default-fallback-g3beige.ppc64-latest.xml | 34 +
...sb-controller-default-fallback-g3beige.xml | 1 +
...r-default-fallback-mac99.ppc64-latest.args | 33 +
...er-default-fallback-mac99.ppc64-latest.xml | 34 +
.../usb-controller-default-fallback-mac99.xml | 1 +
...default-fallback-mac99ppc.ppc-latest.args} | 9 +-
...r-default-fallback-mac99ppc.ppc-latest.xml | 31 +
...b-controller-default-fallback-mac99ppc.xml | 1 +
...default-fallback-nousb.aarch64-latest.args | 31 +
...-default-fallback-nousb.aarch64-latest.xml | 23 +
.../usb-controller-default-fallback-nousb.xml | 1 +
...efault-fallback-powernv9.ppc64-latest.args | 33 +
...default-fallback-powernv9.ppc64-latest.xml | 34 +
...b-controller-default-fallback-powernv9.xml | 1 +
...ault-fallback-realview.aarch64-latest.args | 31 +
...fault-fallback-realview.aarch64-latest.xml | 23 +
...b-controller-default-fallback-realview.xml | 1 +
...ck-versatilepb-aarch64.aarch64-latest.args | 32 +
...ack-versatilepb-aarch64.aarch64-latest.xml | 25 +
...r-default-fallback-versatilepb-aarch64.xml | 1 +
...back-versatilepb-armv7l.armv7l-latest.args | 32 +
...lback-versatilepb-armv7l.armv7l-latest.xml | 27 +
...er-default-fallback-versatilepb-armv7l.xml | 1 +
...ntroller-default-g3beige.ppc64-latest.args | 33 +
...ontroller-default-g3beige.ppc64-latest.xml | 34 +
.../usb-controller-default-g3beige.xml | 23 +
...controller-default-mac99.ppc64-latest.args | 33 +
...-controller-default-mac99.ppc64-latest.xml | 34 +
.../usb-controller-default-mac99.xml | 23 +
...ontroller-default-mac99ppc.ppc-latest.args | 32 +
...controller-default-mac99ppc.ppc-latest.xml | 31 +
.../usb-controller-default-mac99ppc.xml | 23 +
...ntroller-default-nousb.aarch64-latest.args | 31 +
...ontroller-default-nousb.aarch64-latest.xml | 23 +
.../usb-controller-default-nousb.xml | 17 +
...troller-default-powernv9.ppc64-latest.args | 33 +
...ntroller-default-powernv9.ppc64-latest.xml | 34 +
.../usb-controller-default-powernv9.xml | 23 +
...ault-pseries.ppc64-latest.abi-update.args} | 0
...fault-pseries.ppc64-latest.abi-update.xml} | 0
...troller-default-pseries.ppc64-latest.args} | 0
...ntroller-default-pseries.ppc64-latest.xml} | 0
...xml => usb-controller-default-pseries.xml} | 0
...oller-default-realview.aarch64-latest.args | 31 +
...roller-default-realview.aarch64-latest.xml | 23 +
.../usb-controller-default-realview.xml | 16 +
...fault-unavailable-g3beige.ppc64-latest.err | 1 +
...fault-unavailable-g3beige.ppc64-latest.xml | 34 +
...controller-default-unavailable-g3beige.xml | 1 +
...fault-unavailable-i440fx.x86_64-latest.err | 1 +
...default-unavailable-mac99.ppc64-latest.err | 1 +
...default-unavailable-mac99.ppc64-latest.xml | 34 +
...b-controller-default-unavailable-mac99.xml | 1 +
...efault-unavailable-mac99ppc.ppc-latest.err | 1 +
...efault-unavailable-mac99ppc.ppc-latest.xml | 31 +
...ontroller-default-unavailable-mac99ppc.xml | 1 +
...ault-unavailable-nousb.aarch64-latest.args | 31 +
...fault-unavailable-nousb.aarch64-latest.xml | 23 +
...b-controller-default-unavailable-nousb.xml | 1 +
...ault-unavailable-powernv9.ppc64-latest.err | 1 +
...ault-unavailable-powernv9.ppc64-latest.xml | 34 +
...ontroller-default-unavailable-powernv9.xml | 1 +
...fault-unavailable-pseries.ppc64-latest.err | 1 +
...fault-unavailable-pseries.ppc64-latest.xml | 38 +
...controller-default-unavailable-pseries.xml | 1 +
...t-unavailable-realview.aarch64-latest.args | 31 +
...lt-unavailable-realview.aarch64-latest.xml | 23 +
...ontroller-default-unavailable-realview.xml | 1 +
...le-versatilepb-aarch64.aarch64-latest.args | 32 +
...ble-versatilepb-aarch64.aarch64-latest.xml | 25 +
...efault-unavailable-versatilepb-aarch64.xml | 1 +
...lable-versatilepb-armv7l.armv7l-latest.err | 1 +
...lable-versatilepb-armv7l.armv7l-latest.xml | 27 +
...default-unavailable-versatilepb-armv7l.xml | 1 +
...lt-versatilepb-aarch64.aarch64-latest.args | 32 +
...ult-versatilepb-aarch64.aarch64-latest.xml | 25 +
...controller-default-versatilepb-aarch64.xml | 16 +
...ault-versatilepb-armv7l.armv7l-latest.args | 32 +
...fault-versatilepb-armv7l.armv7l-latest.xml | 27 +
...-controller-default-versatilepb-armv7l.xml | 15 +
...ntroller-implicit-isapc.x86_64-latest.args | 1 -
...troller-implicit-microvm.x86_64-latest.err | 1 +
...troller-implicit-microvm.x86_64-latest.xml | 25 +
.../usb-controller-implicit-microvm.xml | 17 +
...rovm-minimal.x86_64-latest.abi-update.args | 31 +
...crovm-minimal.x86_64-latest.abi-update.xml | 23 +
.../x86_64-microvm-minimal.x86_64-latest.args | 31 +
.../x86_64-microvm-minimal.x86_64-latest.xml | 23 +
.../x86_64-microvm-minimal.xml | 14 +
tests/qemuxmlconftest.c | 140 +-
tests/testutilshostcpus.h | 10 +
149 files changed, 38588 insertions(+), 107 deletions(-)
create mode 100644 tests/domaincapsdata/qemu_8.2.0.armv7l.xml
create mode 100644 tests/qemucapabilitiesdata/caps_8.2.0_armv7l.replies
create mode 100644 tests/qemucapabilitiesdata/caps_8.2.0_armv7l.xml
create mode 100644 tests/qemucaps2xmloutdata/caps.armv7l.xml
create mode 100644 tests/qemuxmlconfdata/aarch64-nousb-minimal.aarch64-latest.abi-update.args
create mode 100644 tests/qemuxmlconfdata/aarch64-nousb-minimal.aarch64-latest.abi-update.xml
create mode 100644 tests/qemuxmlconfdata/aarch64-nousb-minimal.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/aarch64-nousb-minimal.aarch64-latest.xml
create mode 100644 tests/qemuxmlconfdata/aarch64-nousb-minimal.xml
create mode 100644 tests/qemuxmlconfdata/aarch64-realview-minimal.aarch64-latest.abi-update.args
create mode 100644 tests/qemuxmlconfdata/aarch64-realview-minimal.aarch64-latest.abi-update.xml
create mode 100644 tests/qemuxmlconfdata/aarch64-realview-minimal.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/aarch64-realview-minimal.aarch64-latest.xml
create mode 100644 tests/qemuxmlconfdata/aarch64-realview-minimal.xml
create mode 100644 tests/qemuxmlconfdata/aarch64-versatilepb-minimal.aarch64-latest.abi-update.args
create mode 100644 tests/qemuxmlconfdata/aarch64-versatilepb-minimal.aarch64-latest.abi-update.xml
create mode 100644 tests/qemuxmlconfdata/aarch64-versatilepb-minimal.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/aarch64-versatilepb-minimal.aarch64-latest.xml
create mode 100644 tests/qemuxmlconfdata/aarch64-versatilepb-minimal.xml
create mode 100644 tests/qemuxmlconfdata/armv7l-versatilepb-minimal.armv7l-latest.abi-update.args
create mode 100644 tests/qemuxmlconfdata/armv7l-versatilepb-minimal.armv7l-latest.abi-update.xml
create mode 100644 tests/qemuxmlconfdata/armv7l-versatilepb-minimal.armv7l-latest.args
create mode 100644 tests/qemuxmlconfdata/armv7l-versatilepb-minimal.armv7l-latest.xml
create mode 100644 tests/qemuxmlconfdata/armv7l-versatilepb-minimal.xml
create mode 100644 tests/qemuxmlconfdata/microvm.x86_64-latest.args
create mode 100644 tests/qemuxmlconfdata/microvm.x86_64-latest.xml
create mode 100644 tests/qemuxmlconfdata/microvm.xml
create mode 100644 tests/qemuxmlconfdata/ppc-mac99-minimal.ppc-latest.abi-update.args
create mode 100644 tests/qemuxmlconfdata/ppc-mac99-minimal.ppc-latest.abi-update.xml
create mode 100644 tests/qemuxmlconfdata/ppc-mac99-minimal.ppc-latest.args
create mode 100644 tests/qemuxmlconfdata/ppc-mac99-minimal.ppc-latest.xml
create mode 100644 tests/qemuxmlconfdata/ppc-mac99-minimal.xml
create mode 100644 tests/qemuxmlconfdata/ppc64-g3beige-minimal.ppc64-latest.abi-update.args
create mode 100644 tests/qemuxmlconfdata/ppc64-g3beige-minimal.ppc64-latest.abi-update.xml
create mode 100644 tests/qemuxmlconfdata/ppc64-g3beige-minimal.ppc64-latest.args
create mode 100644 tests/qemuxmlconfdata/ppc64-g3beige-minimal.ppc64-latest.xml
create mode 100644 tests/qemuxmlconfdata/ppc64-g3beige-minimal.xml
create mode 100644 tests/qemuxmlconfdata/ppc64-mac99-minimal.ppc64-latest.abi-update.args
create mode 100644 tests/qemuxmlconfdata/ppc64-mac99-minimal.ppc64-latest.abi-update.xml
create mode 100644 tests/qemuxmlconfdata/ppc64-mac99-minimal.ppc64-latest.args
create mode 100644 tests/qemuxmlconfdata/ppc64-mac99-minimal.ppc64-latest.xml
create mode 100644 tests/qemuxmlconfdata/ppc64-mac99-minimal.xml
create mode 100644 tests/qemuxmlconfdata/ppc64-powernv9-minimal.ppc64-latest.abi-update.args
create mode 100644 tests/qemuxmlconfdata/ppc64-powernv9-minimal.ppc64-latest.abi-update.xml
create mode 100644 tests/qemuxmlconfdata/ppc64-powernv9-minimal.ppc64-latest.args
create mode 100644 tests/qemuxmlconfdata/ppc64-powernv9-minimal.ppc64-latest.xml
create mode 100644 tests/qemuxmlconfdata/ppc64-powernv9-minimal.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-g3beige.ppc64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-g3beige.ppc64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-fallback-g3beige.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-mac99.ppc64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-mac99.ppc64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-fallback-mac99.xml
rename tests/qemuxmlconfdata/{usb-controller-default-unavailable-i440fx.x86_64-latest.args => usb-controller-default-fallback-mac99ppc.ppc-latest.args} (80%)
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-mac99ppc.ppc-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-fallback-mac99ppc.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-nousb.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-nousb.aarch64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-fallback-nousb.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-powernv9.ppc64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-powernv9.ppc64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-fallback-powernv9.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-realview.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-realview.aarch64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-fallback-realview.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-versatilepb-aarch64.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-versatilepb-aarch64.aarch64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-fallback-versatilepb-aarch64.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-versatilepb-armv7l.armv7l-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-fallback-versatilepb-armv7l.armv7l-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-fallback-versatilepb-armv7l.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-g3beige.ppc64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-g3beige.ppc64-latest.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-g3beige.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-mac99.ppc64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-mac99.ppc64-latest.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-mac99.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-mac99ppc.ppc-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-mac99ppc.ppc-latest.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-mac99ppc.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-nousb.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-nousb.aarch64-latest.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-nousb.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-powernv9.ppc64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-powernv9.ppc64-latest.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-powernv9.xml
rename tests/qemuxmlconfdata/{ppc64-usb-controller.ppc64-latest.abi-update.args => usb-controller-default-pseries.ppc64-latest.abi-update.args} (100%)
rename tests/qemuxmlconfdata/{ppc64-usb-controller.ppc64-latest.abi-update.xml => usb-controller-default-pseries.ppc64-latest.abi-update.xml} (100%)
rename tests/qemuxmlconfdata/{ppc64-usb-controller.ppc64-latest.args => usb-controller-default-pseries.ppc64-latest.args} (100%)
rename tests/qemuxmlconfdata/{ppc64-usb-controller.ppc64-latest.xml => usb-controller-default-pseries.ppc64-latest.xml} (100%)
rename tests/qemuxmlconfdata/{ppc64-usb-controller.xml => usb-controller-default-pseries.xml} (100%)
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-realview.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-realview.aarch64-latest.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-realview.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-g3beige.ppc64-latest.err
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-g3beige.ppc64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-unavailable-g3beige.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-i440fx.x86_64-latest.err
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-mac99.ppc64-latest.err
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-mac99.ppc64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-unavailable-mac99.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-mac99ppc.ppc-latest.err
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-mac99ppc.ppc-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-unavailable-mac99ppc.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-nousb.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-nousb.aarch64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-unavailable-nousb.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-powernv9.ppc64-latest.err
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-powernv9.ppc64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-unavailable-powernv9.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-pseries.ppc64-latest.err
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-pseries.ppc64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-unavailable-pseries.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-realview.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-realview.aarch64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-unavailable-realview.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-versatilepb-aarch64.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-versatilepb-aarch64.aarch64-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-unavailable-versatilepb-aarch64.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-versatilepb-armv7l.armv7l-latest.err
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-unavailable-versatilepb-armv7l.armv7l-latest.xml
create mode 120000 tests/qemuxmlconfdata/usb-controller-default-unavailable-versatilepb-armv7l.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-versatilepb-aarch64.aarch64-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-versatilepb-aarch64.aarch64-latest.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-versatilepb-aarch64.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-versatilepb-armv7l.armv7l-latest.args
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-versatilepb-armv7l.armv7l-latest.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-default-versatilepb-armv7l.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-implicit-microvm.x86_64-latest.err
create mode 100644 tests/qemuxmlconfdata/usb-controller-implicit-microvm.x86_64-latest.xml
create mode 100644 tests/qemuxmlconfdata/usb-controller-implicit-microvm.xml
create mode 100644 tests/qemuxmlconfdata/x86_64-microvm-minimal.x86_64-latest.abi-update.args
create mode 100644 tests/qemuxmlconfdata/x86_64-microvm-minimal.x86_64-latest.abi-update.xml
create mode 100644 tests/qemuxmlconfdata/x86_64-microvm-minimal.x86_64-latest.args
create mode 100644 tests/qemuxmlconfdata/x86_64-microvm-minimal.x86_64-latest.xml
create mode 100644 tests/qemuxmlconfdata/x86_64-microvm-minimal.xml
--
2.43.0
8 months, 2 weeks
[PATCH 01/14] hw/core/machine-smp: Remove deprecated "parameter=0" SMP configurations
by Zhao Liu
From: Zhao Liu <zhao1.liu(a)intel.com>
The "parameter=0" SMP configurations have been marked as deprecated
since v6.2.
For these cases, -smp currently returns the warning and adjusts the
zeroed parameters to 1 by default.
Remove the above compatibility logic in v9.0, and return error directly
if any -smp parameter is set as 0.
Cc: devel(a)lists.libvirt.org
Signed-off-by: Zhao Liu <zhao1.liu(a)intel.com>
Reviewed-by: Thomas Huth <thuth(a)redhat.com>
Reviewed-by: Prasad Pandit <pjp(a)fedoraproject.org>
---
docs/about/deprecated.rst | 16 ----------------
docs/about/removed-features.rst | 15 +++++++++++++++
hw/core/machine-smp.c | 5 +++--
3 files changed, 18 insertions(+), 18 deletions(-)
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index 36bd3e15ef06..872974640252 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -36,22 +36,6 @@ and will cause a warning.
The replacement for the ``nodelay`` short-form boolean option is ``nodelay=on``
rather than ``delay=off``.
-``-smp`` ("parameter=0" SMP configurations) (since 6.2)
-'''''''''''''''''''''''''''''''''''''''''''''''''''''''
-
-Specified CPU topology parameters must be greater than zero.
-
-In the SMP configuration, users should either provide a CPU topology
-parameter with a reasonable value (greater than zero) or just omit it
-and QEMU will compute the missing value.
-
-However, historically it was implicitly allowed for users to provide
-a parameter with zero value, which is meaningless and could also possibly
-cause unexpected results in the -smp parsing. So support for this kind of
-configurations (e.g. -smp 8,sockets=0) is deprecated since 6.2 and will
-be removed in the near future, users have to ensure that all the topology
-members described with -smp are greater than zero.
-
Plugin argument passing through ``arg=<string>`` (since 6.1)
''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
index 417a0e4fa1d9..f9cf874f7b1f 100644
--- a/docs/about/removed-features.rst
+++ b/docs/about/removed-features.rst
@@ -489,6 +489,21 @@ The ``-singlestep`` option has been turned into an accelerator property,
and given a name that better reflects what it actually does.
Use ``-accel tcg,one-insn-per-tb=on`` instead.
+``-smp`` ("parameter=0" SMP configurations) (removed in 9.0)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+
+Specified CPU topology parameters must be greater than zero.
+
+In the SMP configuration, users should either provide a CPU topology
+parameter with a reasonable value (greater than zero) or just omit it
+and QEMU will compute the missing value.
+
+However, historically it was implicitly allowed for users to provide
+a parameter with zero value, which is meaningless and could also possibly
+cause unexpected results in the -smp parsing. So support for this kind of
+configurations (e.g. -smp 8,sockets=0) is removed since 9.0, users have
+to ensure that all the topology members described with -smp are greater
+than zero.
User-mode emulator command line arguments
-----------------------------------------
diff --git a/hw/core/machine-smp.c b/hw/core/machine-smp.c
index 25019c91ee36..96533886b14e 100644
--- a/hw/core/machine-smp.c
+++ b/hw/core/machine-smp.c
@@ -105,8 +105,9 @@ void machine_parse_smp_config(MachineState *ms,
(config->has_cores && config->cores == 0) ||
(config->has_threads && config->threads == 0) ||
(config->has_maxcpus && config->maxcpus == 0)) {
- warn_report("Deprecated CPU topology (considered invalid): "
- "CPU topology parameters must be greater than zero");
+ error_setg(errp, "Invalid CPU topology: "
+ "CPU topology parameters must be greater than zero");
+ return;
}
/*
--
2.34.1
8 months, 2 weeks
[PATCH] hw/core/machine-smp: Remove deprecated "parameter=0" SMP configurations
by Zhao Liu
From: Zhao Liu <zhao1.liu(a)intel.com>
The "parameter=0" SMP configurations have been marked as deprecated
since v6.2.
For these cases, -smp currently returns the warning and adjusts the
zeroed parameters to 1 by default.
Remove the above compatibility logic in v9.0, and return error directly
if any -smp parameter is set as 0.
Signed-off-by: Zhao Liu <zhao1.liu(a)intel.com>
---
docs/about/deprecated.rst | 16 ----------------
docs/about/removed-features.rst | 15 +++++++++++++++
hw/core/machine-smp.c | 5 +++--
3 files changed, 18 insertions(+), 18 deletions(-)
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index 36bd3e15ef06..872974640252 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -36,22 +36,6 @@ and will cause a warning.
The replacement for the ``nodelay`` short-form boolean option is ``nodelay=on``
rather than ``delay=off``.
-``-smp`` ("parameter=0" SMP configurations) (since 6.2)
-'''''''''''''''''''''''''''''''''''''''''''''''''''''''
-
-Specified CPU topology parameters must be greater than zero.
-
-In the SMP configuration, users should either provide a CPU topology
-parameter with a reasonable value (greater than zero) or just omit it
-and QEMU will compute the missing value.
-
-However, historically it was implicitly allowed for users to provide
-a parameter with zero value, which is meaningless and could also possibly
-cause unexpected results in the -smp parsing. So support for this kind of
-configurations (e.g. -smp 8,sockets=0) is deprecated since 6.2 and will
-be removed in the near future, users have to ensure that all the topology
-members described with -smp are greater than zero.
-
Plugin argument passing through ``arg=<string>`` (since 6.1)
''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
index 417a0e4fa1d9..f9cf874f7b1f 100644
--- a/docs/about/removed-features.rst
+++ b/docs/about/removed-features.rst
@@ -489,6 +489,21 @@ The ``-singlestep`` option has been turned into an accelerator property,
and given a name that better reflects what it actually does.
Use ``-accel tcg,one-insn-per-tb=on`` instead.
+``-smp`` ("parameter=0" SMP configurations) (removed in 9.0)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+
+Specified CPU topology parameters must be greater than zero.
+
+In the SMP configuration, users should either provide a CPU topology
+parameter with a reasonable value (greater than zero) or just omit it
+and QEMU will compute the missing value.
+
+However, historically it was implicitly allowed for users to provide
+a parameter with zero value, which is meaningless and could also possibly
+cause unexpected results in the -smp parsing. So support for this kind of
+configurations (e.g. -smp 8,sockets=0) is removed since 9.0, users have
+to ensure that all the topology members described with -smp are greater
+than zero.
User-mode emulator command line arguments
-----------------------------------------
diff --git a/hw/core/machine-smp.c b/hw/core/machine-smp.c
index 25019c91ee36..96533886b14e 100644
--- a/hw/core/machine-smp.c
+++ b/hw/core/machine-smp.c
@@ -105,8 +105,9 @@ void machine_parse_smp_config(MachineState *ms,
(config->has_cores && config->cores == 0) ||
(config->has_threads && config->threads == 0) ||
(config->has_maxcpus && config->maxcpus == 0)) {
- warn_report("Deprecated CPU topology (considered invalid): "
- "CPU topology parameters must be greater than zero");
+ error_setg(errp, "Invalid CPU topology: "
+ "CPU topology parameters must be greater than zero");
+ return;
}
/*
--
2.34.1
8 months, 2 weeks
