a41c76
From 2527410246c5d9f0642087fbd894fdba8d8fa906 Mon Sep 17 00:00:00 2001
a41c76
Message-Id: <2527410246c5d9f0642087fbd894fdba8d8fa906@dist-git>
a41c76
From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= <berrange@redhat.com>
a41c76
Date: Mon, 3 Feb 2020 18:07:24 +0000
a41c76
Subject: [PATCH] qemu: add support for specifying CPU "dies" topology
a41c76
 parameter
a41c76
MIME-Version: 1.0
a41c76
Content-Type: text/plain; charset=UTF-8
a41c76
Content-Transfer-Encoding: 8bit
a41c76
a41c76
QEMU since 4.1.0 supports the "dies" parameter for -smp
a41c76
a41c76
Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>
a41c76
Reviewed-by: Jiri Denemark <jdenemar@redhat.com>
a41c76
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
a41c76
(cherry picked from commit 4cf8dd0c57330e9141333a1b9f4e318e3c83a289)
a41c76
a41c76
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1785207
a41c76
Message-Id: <20200203180726.2203691-4-berrange@redhat.com>
a41c76
Reviewed-by: Jiri Denemark <jdenemar@redhat.com>
a41c76
---
a41c76
 src/qemu/qemu_capabilities.c                  |  2 ++
a41c76
 src/qemu/qemu_capabilities.h                  |  1 +
a41c76
 src/qemu/qemu_command.c                       |  9 +++--
a41c76
 .../caps_4.1.0.x86_64.xml                     |  1 +
a41c76
 .../caps_4.2.0.aarch64.xml                    |  1 +
a41c76
 .../qemucapabilitiesdata/caps_4.2.0.ppc64.xml |  1 +
a41c76
 .../qemucapabilitiesdata/caps_4.2.0.s390x.xml |  1 +
a41c76
 .../caps_4.2.0.x86_64.xml                     |  1 +
a41c76
 .../hugepages-nvdimm.x86_64-latest.args       |  2 +-
a41c76
 ...memory-default-hugepage.x86_64-latest.args |  2 +-
a41c76
 .../memfd-memory-numa.x86_64-latest.args      |  2 +-
a41c76
 ...y-hotplug-nvdimm-access.x86_64-latest.args |  2 +-
a41c76
 ...ry-hotplug-nvdimm-align.x86_64-latest.args |  2 +-
a41c76
 ...ry-hotplug-nvdimm-label.x86_64-latest.args |  2 +-
a41c76
 ...ory-hotplug-nvdimm-pmem.x86_64-latest.args |  2 +-
a41c76
 ...hotplug-nvdimm-readonly.x86_64-latest.args |  2 +-
a41c76
 .../memory-hotplug-nvdimm.x86_64-latest.args  |  2 +-
a41c76
 tests/qemuxml2argvdata/smp-dies.args          | 29 ++++++++++++++++
a41c76
 tests/qemuxml2argvdata/smp-dies.xml           | 33 +++++++++++++++++++
a41c76
 tests/qemuxml2argvtest.c                      |  1 +
a41c76
 20 files changed, 86 insertions(+), 12 deletions(-)
a41c76
 create mode 100644 tests/qemuxml2argvdata/smp-dies.args
a41c76
 create mode 100644 tests/qemuxml2argvdata/smp-dies.xml
a41c76
a41c76
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
a41c76
index edefb70309..893fc5a1bb 100644
a41c76
--- a/src/qemu/qemu_capabilities.c
a41c76
+++ b/src/qemu/qemu_capabilities.c
a41c76
@@ -553,6 +553,7 @@ VIR_ENUM_IMPL(virQEMUCaps,
a41c76
               "blockdev-file-dynamic-auto-read-only",
a41c76
               "savevm-monitor-nodes",
a41c76
               "drive-nvme",
a41c76
+              "smp-dies",
a41c76
     );
a41c76
 
a41c76
 
a41c76
@@ -3097,6 +3098,7 @@ static struct virQEMUCapsCommandLineProps virQEMUCapsCommandLine[] = {
a41c76
     { "sandbox", "elevateprivileges", QEMU_CAPS_SECCOMP_BLACKLIST },
a41c76
     { "chardev", "fd", QEMU_CAPS_CHARDEV_FD_PASS },
a41c76
     { "overcommit", NULL, QEMU_CAPS_OVERCOMMIT },
a41c76
+    { "smp-opts", "dies", QEMU_CAPS_SMP_DIES },
a41c76
 };
a41c76
 
a41c76
 static int
a41c76
diff --git a/src/qemu/qemu_capabilities.h b/src/qemu/qemu_capabilities.h
a41c76
index d76c1dbfa9..5ec8901bb3 100644
a41c76
--- a/src/qemu/qemu_capabilities.h
a41c76
+++ b/src/qemu/qemu_capabilities.h
a41c76
@@ -534,6 +534,7 @@ typedef enum { /* virQEMUCapsFlags grouping marker for syntax-check */
a41c76
     QEMU_CAPS_BLOCK_FILE_AUTO_READONLY_DYNAMIC, /* the auto-read-only property of block backends for files is dynamic */
a41c76
     QEMU_CAPS_SAVEVM_MONITOR_NODES, /* 'savevm' handles monitor-owned nodes properly */
a41c76
     QEMU_CAPS_DRIVE_NVME, /* -drive file.driver=nvme */
a41c76
+    QEMU_CAPS_SMP_DIES, /*  -smp dies= */
a41c76
 
a41c76
     QEMU_CAPS_LAST /* this must always be the last item */
a41c76
 } virQEMUCapsFlags;
a41c76
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
a41c76
index d7e8216092..7a184c229e 100644
a41c76
--- a/src/qemu/qemu_command.c
a41c76
+++ b/src/qemu/qemu_command.c
a41c76
@@ -7105,7 +7105,8 @@ qemuBuildTSEGCommandLine(virCommandPtr cmd,
a41c76
 
a41c76
 static int
a41c76
 qemuBuildSmpCommandLine(virCommandPtr cmd,
a41c76
-                        virDomainDefPtr def)
a41c76
+                        virDomainDefPtr def,
a41c76
+                        virQEMUCapsPtr qemuCaps)
a41c76
 {
a41c76
     g_auto(virBuffer) buf = VIR_BUFFER_INITIALIZER;
a41c76
     unsigned int maxvcpus = virDomainDefGetVcpusMax(def);
a41c76
@@ -7130,12 +7131,14 @@ qemuBuildSmpCommandLine(virCommandPtr cmd,
a41c76
     /* sockets, cores, and threads are either all zero
a41c76
      * or all non-zero, thus checking one of them is enough */
a41c76
     if (def->cpu && def->cpu->sockets) {
a41c76
-        if (def->cpu->dies != 1) {
a41c76
+        if (def->cpu->dies != 1 && !virQEMUCapsGet(qemuCaps, QEMU_CAPS_SMP_DIES)) {
a41c76
             virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
a41c76
                            _("Only 1 die per socket is supported"));
a41c76
             return -1;
a41c76
         }
a41c76
         virBufferAsprintf(&buf, ",sockets=%u", def->cpu->sockets);
a41c76
+        if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_SMP_DIES))
a41c76
+            virBufferAsprintf(&buf, ",dies=%u", def->cpu->dies);
a41c76
         virBufferAsprintf(&buf, ",cores=%u", def->cpu->cores);
a41c76
         virBufferAsprintf(&buf, ",threads=%u", def->cpu->threads);
a41c76
     } else {
a41c76
@@ -9823,7 +9826,7 @@ qemuBuildCommandLine(virQEMUDriverPtr driver,
a41c76
     if (qemuBuildMemCommandLine(cmd, cfg, def, qemuCaps, priv) < 0)
a41c76
         return NULL;
a41c76
 
a41c76
-    if (qemuBuildSmpCommandLine(cmd, def) < 0)
a41c76
+    if (qemuBuildSmpCommandLine(cmd, def, qemuCaps) < 0)
a41c76
         return NULL;
a41c76
 
a41c76
     if (qemuBuildIOThreadCommandLine(cmd, def) < 0)
a41c76
diff --git a/tests/qemucapabilitiesdata/caps_4.1.0.x86_64.xml b/tests/qemucapabilitiesdata/caps_4.1.0.x86_64.xml
a41c76
index a98945de0e..54b797a86a 100644
a41c76
--- a/tests/qemucapabilitiesdata/caps_4.1.0.x86_64.xml
a41c76
+++ b/tests/qemucapabilitiesdata/caps_4.1.0.x86_64.xml
a41c76
@@ -215,6 +215,7 @@
a41c76
   <flag name='ramfb'/>
a41c76
   <flag name='blockdev-file-dynamic-auto-read-only'/>
a41c76
   <flag name='drive-nvme'/>
a41c76
+  <flag name='smp-dies'/>
a41c76
   <version>4001000</version>
a41c76
   <kvmVersion>0</kvmVersion>
a41c76
   <microcodeVersion>43100241</microcodeVersion>
a41c76
diff --git a/tests/qemucapabilitiesdata/caps_4.2.0.aarch64.xml b/tests/qemucapabilitiesdata/caps_4.2.0.aarch64.xml
a41c76
index a6469073fd..184bb7ff77 100644
a41c76
--- a/tests/qemucapabilitiesdata/caps_4.2.0.aarch64.xml
a41c76
+++ b/tests/qemucapabilitiesdata/caps_4.2.0.aarch64.xml
a41c76
@@ -175,6 +175,7 @@
a41c76
   <flag name='blockdev-file-dynamic-auto-read-only'/>
a41c76
   <flag name='savevm-monitor-nodes'/>
a41c76
   <flag name='drive-nvme'/>
a41c76
+  <flag name='smp-dies'/>
a41c76
   <version>4001050</version>
a41c76
   <kvmVersion>0</kvmVersion>
a41c76
   <microcodeVersion>61700242</microcodeVersion>
a41c76
diff --git a/tests/qemucapabilitiesdata/caps_4.2.0.ppc64.xml b/tests/qemucapabilitiesdata/caps_4.2.0.ppc64.xml
a41c76
index ee9fb23640..7c62546d74 100644
a41c76
--- a/tests/qemucapabilitiesdata/caps_4.2.0.ppc64.xml
a41c76
+++ b/tests/qemucapabilitiesdata/caps_4.2.0.ppc64.xml
a41c76
@@ -176,6 +176,7 @@
a41c76
   <flag name='machine.pseries.cap-ccf-assist'/>
a41c76
   <flag name='blockdev-file-dynamic-auto-read-only'/>
a41c76
   <flag name='drive-nvme'/>
a41c76
+  <flag name='smp-dies'/>
a41c76
   <version>4001050</version>
a41c76
   <kvmVersion>0</kvmVersion>
a41c76
   <microcodeVersion>42900242</microcodeVersion>
a41c76
diff --git a/tests/qemucapabilitiesdata/caps_4.2.0.s390x.xml b/tests/qemucapabilitiesdata/caps_4.2.0.s390x.xml
a41c76
index fdf5cb4ce9..c8746f05ef 100644
a41c76
--- a/tests/qemucapabilitiesdata/caps_4.2.0.s390x.xml
a41c76
+++ b/tests/qemucapabilitiesdata/caps_4.2.0.s390x.xml
a41c76
@@ -135,6 +135,7 @@
a41c76
   <flag name='query-cpu-model-comparison'/>
a41c76
   <flag name='blockdev-file-dynamic-auto-read-only'/>
a41c76
   <flag name='drive-nvme'/>
a41c76
+  <flag name='smp-dies'/>
a41c76
   <version>4001050</version>
a41c76
   <kvmVersion>0</kvmVersion>
a41c76
   <microcodeVersion>39100242</microcodeVersion>
a41c76
diff --git a/tests/qemucapabilitiesdata/caps_4.2.0.x86_64.xml b/tests/qemucapabilitiesdata/caps_4.2.0.x86_64.xml
a41c76
index 4857e2f5a5..afd59a269d 100644
a41c76
--- a/tests/qemucapabilitiesdata/caps_4.2.0.x86_64.xml
a41c76
+++ b/tests/qemucapabilitiesdata/caps_4.2.0.x86_64.xml
a41c76
@@ -218,6 +218,7 @@
a41c76
   <flag name='blockdev-file-dynamic-auto-read-only'/>
a41c76
   <flag name='savevm-monitor-nodes'/>
a41c76
   <flag name='drive-nvme'/>
a41c76
+  <flag name='smp-dies'/>
a41c76
   <version>4002000</version>
a41c76
   <kvmVersion>0</kvmVersion>
a41c76
   <microcodeVersion>43100242</microcodeVersion>
a41c76
diff --git a/tests/qemuxml2argvdata/hugepages-nvdimm.x86_64-latest.args b/tests/qemuxml2argvdata/hugepages-nvdimm.x86_64-latest.args
a41c76
index 9056e56cb7..0d795dca91 100644
a41c76
--- a/tests/qemuxml2argvdata/hugepages-nvdimm.x86_64-latest.args
a41c76
+++ b/tests/qemuxml2argvdata/hugepages-nvdimm.x86_64-latest.args
a41c76
@@ -15,7 +15,7 @@ file=/tmp/lib/domain--1-QEMUGuest1/master-key.aes \
a41c76
 -machine pc,accel=tcg,usb=off,dump-guest-core=off,nvdimm=on \
a41c76
 -m size=1048576k,slots=16,maxmem=1099511627776k \
a41c76
 -overcommit mem-lock=off \
a41c76
--smp 2,sockets=2,cores=1,threads=1 \
a41c76
+-smp 2,sockets=2,dies=1,cores=1,threads=1 \
a41c76
 -object memory-backend-file,id=ram-node0,prealloc=yes,\
a41c76
 mem-path=/dev/hugepages2M/libvirt/qemu/-1-QEMUGuest1,share=yes,size=1073741824 \
a41c76
 -numa node,nodeid=0,cpus=0-1,memdev=ram-node0 \
a41c76
diff --git a/tests/qemuxml2argvdata/memfd-memory-default-hugepage.x86_64-latest.args b/tests/qemuxml2argvdata/memfd-memory-default-hugepage.x86_64-latest.args
a41c76
index 998c9f98bd..a655fb1f7c 100644
a41c76
--- a/tests/qemuxml2argvdata/memfd-memory-default-hugepage.x86_64-latest.args
a41c76
+++ b/tests/qemuxml2argvdata/memfd-memory-default-hugepage.x86_64-latest.args
a41c76
@@ -16,7 +16,7 @@ file=/tmp/lib/domain--1-instance-00000092/master-key.aes \
a41c76
 -m 14336 \
a41c76
 -mem-prealloc \
a41c76
 -overcommit mem-lock=off \
a41c76
--smp 8,sockets=1,cores=8,threads=1 \
a41c76
+-smp 8,sockets=1,dies=1,cores=8,threads=1 \
a41c76
 -object memory-backend-memfd,id=ram-node0,hugetlb=yes,hugetlbsize=2097152,\
a41c76
 share=yes,size=15032385536,host-nodes=3,policy=preferred \
a41c76
 -numa node,nodeid=0,cpus=0-7,memdev=ram-node0 \
a41c76
diff --git a/tests/qemuxml2argvdata/memfd-memory-numa.x86_64-latest.args b/tests/qemuxml2argvdata/memfd-memory-numa.x86_64-latest.args
a41c76
index 998c9f98bd..a655fb1f7c 100644
a41c76
--- a/tests/qemuxml2argvdata/memfd-memory-numa.x86_64-latest.args
a41c76
+++ b/tests/qemuxml2argvdata/memfd-memory-numa.x86_64-latest.args
a41c76
@@ -16,7 +16,7 @@ file=/tmp/lib/domain--1-instance-00000092/master-key.aes \
a41c76
 -m 14336 \
a41c76
 -mem-prealloc \
a41c76
 -overcommit mem-lock=off \
a41c76
--smp 8,sockets=1,cores=8,threads=1 \
a41c76
+-smp 8,sockets=1,dies=1,cores=8,threads=1 \
a41c76
 -object memory-backend-memfd,id=ram-node0,hugetlb=yes,hugetlbsize=2097152,\
a41c76
 share=yes,size=15032385536,host-nodes=3,policy=preferred \
a41c76
 -numa node,nodeid=0,cpus=0-7,memdev=ram-node0 \
a41c76
diff --git a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-access.x86_64-latest.args b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-access.x86_64-latest.args
a41c76
index beac9ab22a..c8a6ec5755 100644
a41c76
--- a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-access.x86_64-latest.args
a41c76
+++ b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-access.x86_64-latest.args
a41c76
@@ -15,7 +15,7 @@ file=/tmp/lib/domain--1-QEMUGuest1/master-key.aes \
a41c76
 -machine pc,accel=tcg,usb=off,dump-guest-core=off,nvdimm=on \
a41c76
 -m size=219136k,slots=16,maxmem=1099511627776k \
a41c76
 -overcommit mem-lock=off \
a41c76
--smp 2,sockets=2,cores=1,threads=1 \
a41c76
+-smp 2,sockets=2,dies=1,cores=1,threads=1 \
a41c76
 -numa node,nodeid=0,cpus=0-1,mem=214 \
a41c76
 -object memory-backend-file,id=memnvdimm0,prealloc=yes,mem-path=/tmp/nvdimm,\
a41c76
 share=no,size=536870912 \
a41c76
diff --git a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-align.x86_64-latest.args b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-align.x86_64-latest.args
a41c76
index 3e599098f0..60e9e80039 100644
a41c76
--- a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-align.x86_64-latest.args
a41c76
+++ b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-align.x86_64-latest.args
a41c76
@@ -15,7 +15,7 @@ file=/tmp/lib/domain--1-QEMUGuest1/master-key.aes \
a41c76
 -machine pc,accel=tcg,usb=off,dump-guest-core=off,nvdimm=on \
a41c76
 -m size=219136k,slots=16,maxmem=1099511627776k \
a41c76
 -overcommit mem-lock=off \
a41c76
--smp 2,sockets=2,cores=1,threads=1 \
a41c76
+-smp 2,sockets=2,dies=1,cores=1,threads=1 \
a41c76
 -numa node,nodeid=0,cpus=0-1,mem=214 \
a41c76
 -object memory-backend-file,id=memnvdimm0,prealloc=yes,mem-path=/tmp/nvdimm,\
a41c76
 share=no,size=536870912,align=2097152 \
a41c76
diff --git a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-label.x86_64-latest.args b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-label.x86_64-latest.args
a41c76
index 05a473dbcc..8c5e483cbb 100644
a41c76
--- a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-label.x86_64-latest.args
a41c76
+++ b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-label.x86_64-latest.args
a41c76
@@ -15,7 +15,7 @@ file=/tmp/lib/domain--1-QEMUGuest1/master-key.aes \
a41c76
 -machine pc,accel=tcg,usb=off,dump-guest-core=off,nvdimm=on \
a41c76
 -m size=219136k,slots=16,maxmem=1099511627776k \
a41c76
 -overcommit mem-lock=off \
a41c76
--smp 2,sockets=2,cores=1,threads=1 \
a41c76
+-smp 2,sockets=2,dies=1,cores=1,threads=1 \
a41c76
 -numa node,nodeid=0,cpus=0-1,mem=214 \
a41c76
 -object memory-backend-file,id=memnvdimm0,prealloc=yes,mem-path=/tmp/nvdimm,\
a41c76
 share=no,size=536870912 \
a41c76
diff --git a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-pmem.x86_64-latest.args b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-pmem.x86_64-latest.args
a41c76
index c3554ac101..7f77ab9fce 100644
a41c76
--- a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-pmem.x86_64-latest.args
a41c76
+++ b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-pmem.x86_64-latest.args
a41c76
@@ -15,7 +15,7 @@ file=/tmp/lib/domain--1-QEMUGuest1/master-key.aes \
a41c76
 -machine pc,accel=tcg,usb=off,dump-guest-core=off,nvdimm=on \
a41c76
 -m size=219136k,slots=16,maxmem=1099511627776k \
a41c76
 -overcommit mem-lock=off \
a41c76
--smp 2,sockets=2,cores=1,threads=1 \
a41c76
+-smp 2,sockets=2,dies=1,cores=1,threads=1 \
a41c76
 -numa node,nodeid=0,cpus=0-1,mem=214 \
a41c76
 -object memory-backend-file,id=memnvdimm0,prealloc=yes,mem-path=/tmp/nvdimm,\
a41c76
 share=no,size=536870912,pmem=on \
a41c76
diff --git a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-readonly.x86_64-latest.args b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-readonly.x86_64-latest.args
a41c76
index e1d3fc57a4..631835a380 100644
a41c76
--- a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-readonly.x86_64-latest.args
a41c76
+++ b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-readonly.x86_64-latest.args
a41c76
@@ -15,7 +15,7 @@ file=/tmp/lib/domain--1-QEMUGuest1/master-key.aes \
a41c76
 -machine pc,accel=tcg,usb=off,dump-guest-core=off,nvdimm=on \
a41c76
 -m size=219136k,slots=16,maxmem=1099511627776k \
a41c76
 -overcommit mem-lock=off \
a41c76
--smp 2,sockets=2,cores=1,threads=1 \
a41c76
+-smp 2,sockets=2,dies=1,cores=1,threads=1 \
a41c76
 -numa node,nodeid=0,cpus=0-1,mem=214 \
a41c76
 -object memory-backend-file,id=memnvdimm0,prealloc=yes,mem-path=/tmp/nvdimm,\
a41c76
 share=no,size=536870912 \
a41c76
diff --git a/tests/qemuxml2argvdata/memory-hotplug-nvdimm.x86_64-latest.args b/tests/qemuxml2argvdata/memory-hotplug-nvdimm.x86_64-latest.args
a41c76
index dc6ddd3a0e..48221a5526 100644
a41c76
--- a/tests/qemuxml2argvdata/memory-hotplug-nvdimm.x86_64-latest.args
a41c76
+++ b/tests/qemuxml2argvdata/memory-hotplug-nvdimm.x86_64-latest.args
a41c76
@@ -15,7 +15,7 @@ file=/tmp/lib/domain--1-QEMUGuest1/master-key.aes \
a41c76
 -machine pc,accel=tcg,usb=off,dump-guest-core=off,nvdimm=on \
a41c76
 -m size=1048576k,slots=16,maxmem=1099511627776k \
a41c76
 -overcommit mem-lock=off \
a41c76
--smp 2,sockets=2,cores=1,threads=1 \
a41c76
+-smp 2,sockets=2,dies=1,cores=1,threads=1 \
a41c76
 -numa node,nodeid=0,cpus=0-1,mem=1024 \
a41c76
 -object memory-backend-file,id=memnvdimm0,prealloc=yes,mem-path=/tmp/nvdimm,\
a41c76
 size=536870912 \
a41c76
diff --git a/tests/qemuxml2argvdata/smp-dies.args b/tests/qemuxml2argvdata/smp-dies.args
a41c76
new file mode 100644
a41c76
index 0000000000..632e9d8e34
a41c76
--- /dev/null
a41c76
+++ b/tests/qemuxml2argvdata/smp-dies.args
a41c76
@@ -0,0 +1,29 @@
a41c76
+LC_ALL=C \
a41c76
+PATH=/bin \
a41c76
+HOME=/tmp/lib/domain--1-QEMUGuest1 \
a41c76
+USER=test \
a41c76
+LOGNAME=test \
a41c76
+XDG_DATA_HOME=/tmp/lib/domain--1-QEMUGuest1/.local/share \
a41c76
+XDG_CACHE_HOME=/tmp/lib/domain--1-QEMUGuest1/.cache \
a41c76
+XDG_CONFIG_HOME=/tmp/lib/domain--1-QEMUGuest1/.config \
a41c76
+QEMU_AUDIO_DRV=none \
a41c76
+/usr/bin/qemu-system-i386 \
a41c76
+-name QEMUGuest1 \
a41c76
+-S \
a41c76
+-machine pc,accel=tcg,usb=off,dump-guest-core=off \
a41c76
+-m 214 \
a41c76
+-realtime mlock=off \
a41c76
+-smp 1,maxcpus=4,sockets=2,dies=2,cores=1,threads=1 \
a41c76
+-uuid c7a5fdbd-edaf-9455-926a-d65c16db1809 \
a41c76
+-display none \
a41c76
+-no-user-config \
a41c76
+-nodefaults \
a41c76
+-chardev socket,id=charmonitor,path=/tmp/lib/domain--1-QEMUGuest1/monitor.sock,\
a41c76
+server,nowait \
a41c76
+-mon chardev=charmonitor,id=monitor,mode=control \
a41c76
+-rtc base=utc \
a41c76
+-no-shutdown \
a41c76
+-no-acpi \
a41c76
+-usb \
a41c76
+-drive file=/dev/HostVG/QEMUGuest1,format=raw,if=none,id=drive-ide0-0-0 \
a41c76
+-device ide-hd,bus=ide.0,unit=0,drive=drive-ide0-0-0,id=ide0-0-0,bootindex=1
a41c76
diff --git a/tests/qemuxml2argvdata/smp-dies.xml b/tests/qemuxml2argvdata/smp-dies.xml
a41c76
new file mode 100644
a41c76
index 0000000000..caadaef8b5
a41c76
--- /dev/null
a41c76
+++ b/tests/qemuxml2argvdata/smp-dies.xml
a41c76
@@ -0,0 +1,33 @@
a41c76
+<domain type='qemu'>
a41c76
+  <name>QEMUGuest1</name>
a41c76
+  <uuid>c7a5fdbd-edaf-9455-926a-d65c16db1809</uuid>
a41c76
+  <memory unit='KiB'>219136</memory>
a41c76
+  <currentMemory unit='KiB'>219136</currentMemory>
a41c76
+  <vcpu placement='static' current='1'>4</vcpu>
a41c76
+  <os>
a41c76
+    <type arch='i686' machine='pc'>hvm</type>
a41c76
+    <boot dev='hd'/>
a41c76
+  </os>
a41c76
+  <cpu>
a41c76
+    <topology sockets='2' dies='2' cores='1' threads='1'/>
a41c76
+  </cpu>
a41c76
+  <clock offset='utc'/>
a41c76
+  <on_poweroff>destroy</on_poweroff>
a41c76
+  <on_reboot>restart</on_reboot>
a41c76
+  <on_crash>destroy</on_crash>
a41c76
+  <devices>
a41c76
+    <emulator>/usr/bin/qemu-system-i386</emulator>
a41c76
+    <disk type='block' device='disk'>
a41c76
+      <driver name='qemu' type='raw'/>
a41c76
+      <source dev='/dev/HostVG/QEMUGuest1'/>
a41c76
+      <target dev='hda' bus='ide'/>
a41c76
+      <address type='drive' controller='0' bus='0' target='0' unit='0'/>
a41c76
+    </disk>
a41c76
+    <controller type='usb' index='0'/>
a41c76
+    <controller type='ide' index='0'/>
a41c76
+    <controller type='pci' index='0' model='pci-root'/>
a41c76
+    <input type='mouse' bus='ps2'/>
a41c76
+    <input type='keyboard' bus='ps2'/>
a41c76
+    <memballoon model='none'/>
a41c76
+  </devices>
a41c76
+</domain>
a41c76
diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
a41c76
index 58b4deefc6..b923590930 100644
a41c76
--- a/tests/qemuxml2argvtest.c
a41c76
+++ b/tests/qemuxml2argvtest.c
a41c76
@@ -1689,6 +1689,7 @@ mymain(void)
a41c76
     DO_TEST("qemu-ns-alt", NONE);
a41c76
 
a41c76
     DO_TEST("smp", NONE);
a41c76
+    DO_TEST("smp-dies", QEMU_CAPS_SMP_DIES);
a41c76
 
a41c76
     DO_TEST("iothreads", QEMU_CAPS_OBJECT_IOTHREAD);
a41c76
     DO_TEST("iothreads-ids", QEMU_CAPS_OBJECT_IOTHREAD);
a41c76
-- 
a41c76
2.25.0
a41c76