[libvirt PATCH 1/2] tests: Sync some ppc64 tests

Daniel Henrique Barboza danielhb413 at gmail.com
Thu Dec 3 17:15:06 UTC 2020



On 12/3/20 2:04 PM, Andrea Bolognani wrote:
> The ppc64 tests
> 
>    memory-hotplug-ppc64-nonuma
>    memory-hotplug-nvdimm-ppc64
> 
> are not passed the same information for qemuxml2argv and
> qemuxml2xml tests; the former, in particular, doesn't show up
> at all in qemuxml2xml. Address this inconsistency.
> 
> Note that one of the new output files had been introduced with
> 5540acb9a2bd despite not being actually used as of that commit.
> 
> Signed-off-by: Andrea Bolognani <abologna at redhat.com>
> ---

Reviewed-by: Daniel Henrique Barboza <danielhb413 at gmail.com>


>   .../memory-hotplug-nvdimm-ppc64.args          |  2 +-
>   ...ory-hotplug-nvdimm-ppc64.ppc64-latest.args | 38 ----------------
>   tests/qemuxml2argvtest.c                      |  4 +-
>   .../memory-hotplug-ppc64-nonuma.xml           | 45 +++++++++++++++++++
>   tests/qemuxml2xmltest.c                       |  5 +++
>   5 files changed, 54 insertions(+), 40 deletions(-)
>   delete mode 100644 tests/qemuxml2argvdata/memory-hotplug-nvdimm-ppc64.ppc64-latest.args
>   create mode 100644 tests/qemuxml2xmloutdata/memory-hotplug-ppc64-nonuma.xml
> 
> diff --git a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-ppc64.args b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-ppc64.args
> index 92e6c538fb..94bd86ada0 100644
> --- a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-ppc64.args
> +++ b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-ppc64.args
> @@ -15,7 +15,7 @@ QEMU_AUDIO_DRV=none \
>   -realtime mlock=off \
>   -smp 2,sockets=2,cores=1,threads=1 \
>   -numa node,nodeid=0,cpus=0-1,mem=1024 \
> --object memory-backend-file,id=memnvdimm0,prealloc=yes,mem-path=/tmp/nvdimm,\
> +-object memory-backend-file,id=memnvdimm0,mem-path=/tmp/nvdimm,prealloc=yes,\
>   size=537001984 \
>   -device nvdimm,node=0,label-size=131072,\
>   uuid=49545eb3-75e1-2d0a-acdd-f0294406c99e,memdev=memnvdimm0,id=nvdimm0,slot=0 \
> diff --git a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-ppc64.ppc64-latest.args b/tests/qemuxml2argvdata/memory-hotplug-nvdimm-ppc64.ppc64-latest.args
> deleted file mode 100644
> index e7be7216c1..0000000000
> --- a/tests/qemuxml2argvdata/memory-hotplug-nvdimm-ppc64.ppc64-latest.args
> +++ /dev/null
> @@ -1,38 +0,0 @@
> -LC_ALL=C \
> -PATH=/bin \
> -HOME=/tmp/lib/domain--1-QEMUGuest1 \
> -USER=test \
> -LOGNAME=test \
> -XDG_DATA_HOME=/tmp/lib/domain--1-QEMUGuest1/.local/share \
> -XDG_CACHE_HOME=/tmp/lib/domain--1-QEMUGuest1/.cache \
> -XDG_CONFIG_HOME=/tmp/lib/domain--1-QEMUGuest1/.config \
> -QEMU_AUDIO_DRV=none \
> -/usr/bin/qemu-system-ppc64 \
> --name guest=QEMUGuest1,debug-threads=on \
> --S \
> --object secret,id=masterKey0,format=raw,\
> -file=/tmp/lib/domain--1-QEMUGuest1/master-key.aes \
> --machine pseries,accel=tcg,usb=off,dump-guest-core=off,nvdimm=on \
> --cpu POWER9 \
> --m size=1048576k,slots=16,maxmem=1099511627776k \
> --overcommit mem-lock=off \
> --smp 2,sockets=2,dies=1,cores=1,threads=1 \
> --numa node,nodeid=0,cpus=0-1,mem=1024 \
> --object memory-backend-file,id=memnvdimm0,mem-path=/tmp/nvdimm,prealloc=yes,\
> -size=537001984 \
> --device nvdimm,node=0,label-size=131072,\
> -uuid=49545eb3-75e1-2d0a-acdd-f0294406c99e,memdev=memnvdimm0,id=nvdimm0,slot=0 \
> --uuid c7a5fdbd-edaf-9455-926a-d65c16db1809 \
> --display none \
> --no-user-config \
> --nodefaults \
> --chardev socket,id=charmonitor,fd=1729,server,nowait \
> --mon chardev=charmonitor,id=monitor,mode=control \
> --rtc base=utc \
> --no-shutdown \
> --boot strict=on \
> --device pci-ohci,id=usb,bus=pci.0,addr=0x1 \
> --device virtio-balloon-pci,id=balloon0,bus=pci.0,addr=0x2 \
> --sandbox on,obsolete=deny,elevateprivileges=deny,spawn=deny,\
> -resourcecontrol=deny \
> --msg timestamp=on
> diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
> index 409680c84e..8c8426e699 100644
> --- a/tests/qemuxml2argvtest.c
> +++ b/tests/qemuxml2argvtest.c
> @@ -3011,7 +3011,9 @@ mymain(void)
>       DO_TEST_CAPS_LATEST("memory-hotplug-nvdimm-align");
>       DO_TEST_CAPS_LATEST("memory-hotplug-nvdimm-pmem");
>       DO_TEST_CAPS_LATEST("memory-hotplug-nvdimm-readonly");
> -    DO_TEST_CAPS_ARCH_LATEST("memory-hotplug-nvdimm-ppc64", "ppc64");
> +    DO_TEST("memory-hotplug-nvdimm-ppc64", QEMU_CAPS_DEVICE_SPAPR_PCI_HOST_BRIDGE,
> +                                           QEMU_CAPS_OBJECT_MEMORY_FILE,
> +                                           QEMU_CAPS_DEVICE_NVDIMM);
>   
>       DO_TEST("machine-aeskeywrap-on-caps",
>               QEMU_CAPS_AES_KEY_WRAP,
> diff --git a/tests/qemuxml2xmloutdata/memory-hotplug-ppc64-nonuma.xml b/tests/qemuxml2xmloutdata/memory-hotplug-ppc64-nonuma.xml
> new file mode 100644
> index 0000000000..a5905e64b5
> --- /dev/null
> +++ b/tests/qemuxml2xmloutdata/memory-hotplug-ppc64-nonuma.xml
> @@ -0,0 +1,45 @@
> +<domain type='kvm'>
> +  <name>QEMUGuest1</name>
> +  <uuid>49545eb3-75e1-2d0a-acdd-f0294406c99e</uuid>
> +  <maxMemory slots='16' unit='KiB'>4194304</maxMemory>
> +  <memory unit='KiB'>2097152</memory>
> +  <currentMemory unit='KiB'>2097152</currentMemory>
> +  <vcpu placement='static'>1</vcpu>
> +  <os>
> +    <type arch='ppc64' machine='pseries'>hvm</type>
> +    <kernel>/media/ram/uImage</kernel>
> +    <initrd>/media/ram/ramdisk</initrd>
> +    <cmdline>root=/dev/ram rw console=ttyS0,115200</cmdline>
> +    <boot dev='hd'/>
> +  </os>
> +  <clock offset='utc'/>
> +  <on_poweroff>destroy</on_poweroff>
> +  <on_reboot>restart</on_reboot>
> +  <on_crash>destroy</on_crash>
> +  <devices>
> +    <emulator>/usr/bin/qemu-system-ppc64</emulator>
> +    <controller type='usb' index='0'>
> +      <address type='pci' domain='0x0000' bus='0x00' slot='0x01' function='0x0'/>
> +    </controller>
> +    <controller type='pci' index='0' model='pci-root'>
> +      <model name='spapr-pci-host-bridge'/>
> +      <target index='0'/>
> +    </controller>
> +    <memballoon model='virtio'>
> +      <address type='pci' domain='0x0000' bus='0x00' slot='0x02' function='0x0'/>
> +    </memballoon>
> +    <panic model='pseries'/>
> +    <memory model='dimm'>
> +      <target>
> +        <size unit='KiB'>523264</size>
> +      </target>
> +      <address type='dimm' slot='0'/>
> +    </memory>
> +    <memory model='dimm'>
> +      <target>
> +        <size unit='KiB'>524287</size>
> +      </target>
> +      <address type='dimm' slot='1'/>
> +    </memory>
> +  </devices>
> +</domain>
> diff --git a/tests/qemuxml2xmltest.c b/tests/qemuxml2xmltest.c
> index c006719dee..376464a925 100644
> --- a/tests/qemuxml2xmltest.c
> +++ b/tests/qemuxml2xmltest.c
> @@ -1227,6 +1227,9 @@ mymain(void)
>       /* SVE aarch64 CPU features work on modern QEMU */
>       DO_TEST_CAPS_ARCH_LATEST("aarch64-features-sve", "aarch64");
>   
> +    DO_TEST("memory-hotplug-ppc64-nonuma", QEMU_CAPS_KVM, QEMU_CAPS_DEVICE_PC_DIMM, QEMU_CAPS_NUMA,
> +            QEMU_CAPS_DEVICE_SPAPR_PCI_HOST_BRIDGE,
> +            QEMU_CAPS_OBJECT_MEMORY_RAM, QEMU_CAPS_OBJECT_MEMORY_FILE);
>       DO_TEST("memory-hotplug", NONE);
>       DO_TEST("memory-hotplug-dimm", QEMU_CAPS_DEVICE_PC_DIMM);
>       DO_TEST("memory-hotplug-nvdimm", QEMU_CAPS_DEVICE_NVDIMM);
> @@ -1237,7 +1240,9 @@ mymain(void)
>       DO_TEST("memory-hotplug-nvdimm-readonly", QEMU_CAPS_DEVICE_NVDIMM,
>                                                 QEMU_CAPS_DEVICE_NVDIMM_UNARMED);
>       DO_TEST("memory-hotplug-nvdimm-ppc64", QEMU_CAPS_DEVICE_SPAPR_PCI_HOST_BRIDGE,
> +                                           QEMU_CAPS_OBJECT_MEMORY_FILE,
>                                              QEMU_CAPS_DEVICE_NVDIMM);
> +
>       DO_TEST("net-udp", NONE);
>   
>       DO_TEST("video-virtio-gpu-device", QEMU_CAPS_DEVICE_VIRTIO_GPU);
> 




More information about the libvir-list mailing list