edecca
From bafa9aa0fb88ceb6ed672d534195f268b8da5e51 Mon Sep 17 00:00:00 2001
edecca
Message-Id: <bafa9aa0fb88ceb6ed672d534195f268b8da5e51@dist-git>
edecca
From: Pavel Hrdina <phrdina@redhat.com>
edecca
Date: Mon, 13 Aug 2018 19:21:48 +0200
edecca
Subject: [PATCH] tests: rename hugepages-pages7 into pages-dimm-discard
edecca
MIME-Version: 1.0
edecca
Content-Type: text/plain; charset=UTF-8
edecca
Content-Transfer-Encoding: 8bit
edecca
edecca
Remove unnecessary XML elements as well.
edecca
edecca
Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
edecca
(cherry picked from commit 7e0ac26867da8b3b1464b7649a53fdd6dde8d9ab)
edecca
edecca
Conflicts:
edecca
    tests/qemuxml2argvdata/pages-dimm-discard.args
edecca
        - missing upstream commit <caccbba64a>
edecca
edecca
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1615461
edecca
edecca
Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
edecca
Reviewed-by: Ján Tomko <jtomko@redhat.com>
edecca
---
edecca
 ...ges-pages7.args => pages-dimm-discard.args} | 16 +++++-----------
edecca
 ...pages-pages7.xml => pages-dimm-discard.xml} | 18 +-----------------
edecca
 tests/qemuxml2argvtest.c                       |  7 ++++---
edecca
 tests/qemuxml2xmloutdata/hugepages-pages7.xml  |  1 -
edecca
 .../qemuxml2xmloutdata/pages-dimm-discard.xml  |  1 +
edecca
 tests/qemuxml2xmltest.c                        |  2 +-
edecca
 6 files changed, 12 insertions(+), 33 deletions(-)
edecca
 rename tests/qemuxml2argvdata/{hugepages-pages7.args => pages-dimm-discard.args} (65%)
edecca
 rename tests/qemuxml2argvdata/{hugepages-pages7.xml => pages-dimm-discard.xml} (73%)
edecca
 delete mode 120000 tests/qemuxml2xmloutdata/hugepages-pages7.xml
edecca
 create mode 120000 tests/qemuxml2xmloutdata/pages-dimm-discard.xml
edecca
edecca
diff --git a/tests/qemuxml2argvdata/hugepages-pages7.args b/tests/qemuxml2argvdata/pages-dimm-discard.args
edecca
similarity index 65%
edecca
rename from tests/qemuxml2argvdata/hugepages-pages7.args
edecca
rename to tests/qemuxml2argvdata/pages-dimm-discard.args
edecca
index 02a98026eb..97184e074a 100644
edecca
--- a/tests/qemuxml2argvdata/hugepages-pages7.args
edecca
+++ b/tests/qemuxml2argvdata/pages-dimm-discard.args
edecca
@@ -10,16 +10,14 @@ QEMU_AUDIO_DRV=none \
edecca
 -machine pc-i440fx-2.3,accel=tcg,usb=off,dump-guest-core=off \
edecca
 -m size=1048576k,slots=16,maxmem=1099511627776k \
edecca
 -smp 2,sockets=2,cores=1,threads=1 \
edecca
--mem-prealloc \
edecca
--mem-path /dev/hugepages2M/libvirt/qemu/-1-fedora \
edecca
 -numa node,nodeid=0,cpus=0-1,mem=1024 \
edecca
 -object memory-backend-file,id=memdimm0,prealloc=yes,\
edecca
 mem-path=/dev/hugepages1G/libvirt/qemu/-1-fedora,size=1073741824,\
edecca
 host-nodes=1-3,policy=bind \
edecca
 -device pc-dimm,node=0,memdev=memdimm0,id=dimm0,slot=0 \
edecca
--object memory-backend-file,id=memdimm1,prealloc=yes,\
edecca
-mem-path=/dev/hugepages2M/libvirt/qemu/-1-fedora,discard-data=yes,share=no,\
edecca
-size=536870912 \
edecca
+-object memory-backend-file,id=memdimm1,\
edecca
+mem-path=/var/lib/libvirt/qemu/ram/libvirt/qemu/-1-fedora/dimm1,\
edecca
+discard-data=yes,share=no,size=536870912 \
edecca
 -device pc-dimm,node=0,memdev=memdimm1,id=dimm1,slot=1 \
edecca
 -uuid 63840878-0deb-4095-97e6-fc444d9bc9fa \
edecca
 -display none \
edecca
@@ -30,10 +28,6 @@ server,nowait \
edecca
 -mon chardev=charmonitor,id=monitor,mode=control \
edecca
 -rtc base=utc \
edecca
 -no-shutdown \
edecca
+-no-acpi \
edecca
 -boot c \
edecca
--usb \
edecca
--drive file=/var/lib/libvirt/images/fedora.qcow2,format=qcow2,if=none,\
edecca
-id=drive-virtio-disk0 \
edecca
--device virtio-blk-pci,bus=pci.0,addr=0x7,drive=drive-virtio-disk0,\
edecca
-id=virtio-disk0 \
edecca
--device virtio-balloon-pci,id=balloon0,bus=pci.0,addr=0x3
edecca
+-usb
edecca
diff --git a/tests/qemuxml2argvdata/hugepages-pages7.xml b/tests/qemuxml2argvdata/pages-dimm-discard.xml
edecca
similarity index 73%
edecca
rename from tests/qemuxml2argvdata/hugepages-pages7.xml
edecca
rename to tests/qemuxml2argvdata/pages-dimm-discard.xml
edecca
index 28c72f85a7..3d233687e1 100644
edecca
--- a/tests/qemuxml2argvdata/hugepages-pages7.xml
edecca
+++ b/tests/qemuxml2argvdata/pages-dimm-discard.xml
edecca
@@ -4,19 +4,11 @@
edecca
   <maxMemory slots='16' unit='KiB'>1099511627776</maxMemory>
edecca
   <memory unit='KiB'>2621439</memory>
edecca
   <currentMemory unit='KiB'>2621439</currentMemory>
edecca
-  <memoryBacking>
edecca
-    <hugepages/>
edecca
-  </memoryBacking>
edecca
   <vcpu placement='static'>2</vcpu>
edecca
   <os>
edecca
     <type arch='x86_64' machine='pc-i440fx-2.3'>hvm</type>
edecca
     <boot dev='hd'/>
edecca
   </os>
edecca
-  <features>
edecca
-    <acpi/>
edecca
-    <apic/>
edecca
-    <pae/>
edecca
-  </features>
edecca
   <cpu>
edecca
     <numa>
edecca
       <cell id='0' cpus='0-1' memory='1048576' unit='KiB'/>
edecca
@@ -28,21 +20,13 @@
edecca
   <on_crash>restart</on_crash>
edecca
   <devices>
edecca
     <emulator>/usr/bin/qemu-system-x86_64</emulator>
edecca
-    <disk type='file' device='disk'>
edecca
-      <driver name='qemu' type='qcow2'/>
edecca
-      <source file='/var/lib/libvirt/images/fedora.qcow2'/>
edecca
-      <target dev='vda' bus='virtio'/>
edecca
-      <address type='pci' domain='0x0000' bus='0x00' slot='0x07' function='0x0'/>
edecca
-    </disk>
edecca
     <controller type='usb' index='0'>
edecca
       <address type='pci' domain='0x0000' bus='0x00' slot='0x01' function='0x2'/>
edecca
     </controller>
edecca
     <controller type='pci' index='0' model='pci-root'/>
edecca
     <input type='mouse' bus='ps2'/>
edecca
     <input type='keyboard' bus='ps2'/>
edecca
-    <memballoon model='virtio'>
edecca
-      <address type='pci' domain='0x0000' bus='0x00' slot='0x03' function='0x0'/>
edecca
-    </memballoon>
edecca
+    <memballoon model='none'/>
edecca
     <memory model='dimm' discard='no'>
edecca
       <source>
edecca
         <nodemask>1-3</nodemask>
edecca
diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
edecca
index 1aab23408d..f8fbca1806 100644
edecca
--- a/tests/qemuxml2argvtest.c
edecca
+++ b/tests/qemuxml2argvtest.c
edecca
@@ -952,6 +952,10 @@ mymain(void)
edecca
             QEMU_CAPS_OBJECT_MEMORY_RAM,
edecca
             QEMU_CAPS_OBJECT_MEMORY_FILE,
edecca
             QEMU_CAPS_OBJECT_MEMORY_FILE_DISCARD);
edecca
+    DO_TEST("pages-dimm-discard",
edecca
+            QEMU_CAPS_DEVICE_PC_DIMM,
edecca
+            QEMU_CAPS_OBJECT_MEMORY_FILE,
edecca
+            QEMU_CAPS_OBJECT_MEMORY_FILE_DISCARD);
edecca
     DO_TEST("hugepages-default", NONE);
edecca
     DO_TEST("hugepages-default-2M", NONE);
edecca
     DO_TEST("hugepages-default-system-size", NONE);
edecca
@@ -976,9 +980,6 @@ mymain(void)
edecca
             QEMU_CAPS_OBJECT_MEMORY_RAM,
edecca
             QEMU_CAPS_OBJECT_MEMORY_FILE);
edecca
     DO_TEST_PARSE_ERROR("hugepages-memaccess-invalid", NONE);
edecca
-    DO_TEST("hugepages-pages7",
edecca
-            QEMU_CAPS_DEVICE_PC_DIMM, QEMU_CAPS_OBJECT_MEMORY_FILE,
edecca
-            QEMU_CAPS_OBJECT_MEMORY_FILE_DISCARD);
edecca
     DO_TEST_FAILURE("hugepages-pages8",
edecca
                     QEMU_CAPS_DEVICE_PC_DIMM, QEMU_CAPS_OBJECT_MEMORY_FILE,
edecca
                     QEMU_CAPS_OBJECT_MEMORY_FILE_DISCARD);
edecca
diff --git a/tests/qemuxml2xmloutdata/hugepages-pages7.xml b/tests/qemuxml2xmloutdata/hugepages-pages7.xml
edecca
deleted file mode 120000
edecca
index b4ce4defda..0000000000
edecca
--- a/tests/qemuxml2xmloutdata/hugepages-pages7.xml
edecca
+++ /dev/null
edecca
@@ -1 +0,0 @@
edecca
-../qemuxml2argvdata/hugepages-pages7.xml
edecca
\ No newline at end of file
edecca
diff --git a/tests/qemuxml2xmloutdata/pages-dimm-discard.xml b/tests/qemuxml2xmloutdata/pages-dimm-discard.xml
edecca
new file mode 120000
edecca
index 0000000000..05bbef8d65
edecca
--- /dev/null
edecca
+++ b/tests/qemuxml2xmloutdata/pages-dimm-discard.xml
edecca
@@ -0,0 +1 @@
edecca
+../qemuxml2argvdata/pages-dimm-discard.xml
edecca
\ No newline at end of file
edecca
diff --git a/tests/qemuxml2xmltest.c b/tests/qemuxml2xmltest.c
edecca
index a3555b348e..6afc48f98e 100644
edecca
--- a/tests/qemuxml2xmltest.c
edecca
+++ b/tests/qemuxml2xmltest.c
edecca
@@ -332,6 +332,7 @@ mymain(void)
edecca
 
edecca
     DO_TEST("pages-discard", NONE);
edecca
     DO_TEST("pages-discard-hugepages", NONE);
edecca
+    DO_TEST("pages-dimm-discard", NONE);
edecca
     DO_TEST("hugepages-default", NONE);
edecca
     DO_TEST("hugepages-default-2M", NONE);
edecca
     DO_TEST("hugepages-default-system-size", NONE);
edecca
@@ -340,7 +341,6 @@ mymain(void)
edecca
     DO_TEST("hugepages-numa-nodeset", NONE);
edecca
     DO_TEST("hugepages-numa-nodeset-part", NONE);
edecca
     DO_TEST("hugepages-numa-nodeset-nonexist", NONE);
edecca
-    DO_TEST("hugepages-pages7", NONE);
edecca
     DO_TEST("hugepages-shared", NONE);
edecca
     DO_TEST("hugepages-memaccess", NONE);
edecca
     DO_TEST("hugepages-memaccess2", NONE);
edecca
-- 
edecca
2.18.0
edecca