[PATCH 02/33] tests: Add aarch64-panic-no-model

Andrea Bolognani posted 33 patches 8 months, 2 weeks ago
There is a newer version of this series
[PATCH 02/33] tests: Add aarch64-panic-no-model
Posted by Andrea Bolognani 8 months, 2 weeks ago
This demonstrates that on aarch64, where a native panic device
doesn't exist, it's necessary for the user to specify the model
explicitly.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>
---
 .../aarch64-panic-no-model.aarch64-latest.err       |  1 +
 tests/qemuxmlconfdata/aarch64-panic-no-model.xml    | 13 +++++++++++++
 tests/qemuxmlconftest.c                             |  1 +
 3 files changed, 15 insertions(+)
 create mode 100644 tests/qemuxmlconfdata/aarch64-panic-no-model.aarch64-latest.err
 create mode 100644 tests/qemuxmlconfdata/aarch64-panic-no-model.xml

diff --git a/tests/qemuxmlconfdata/aarch64-panic-no-model.aarch64-latest.err b/tests/qemuxmlconfdata/aarch64-panic-no-model.aarch64-latest.err
new file mode 100644
index 0000000000..8e3f2c194d
--- /dev/null
+++ b/tests/qemuxmlconfdata/aarch64-panic-no-model.aarch64-latest.err
@@ -0,0 +1 @@
+unsupported configuration: the QEMU binary does not support the ISA panic device
diff --git a/tests/qemuxmlconfdata/aarch64-panic-no-model.xml b/tests/qemuxmlconfdata/aarch64-panic-no-model.xml
new file mode 100644
index 0000000000..5207e48bbd
--- /dev/null
+++ b/tests/qemuxmlconfdata/aarch64-panic-no-model.xml
@@ -0,0 +1,13 @@
+<domain type='qemu'>
+  <name>guest</name>
+  <uuid>1ccfd97d-5eb4-478a-bbe6-88d254c16db7</uuid>
+  <memory>4194304</memory>
+  <vcpu>4</vcpu>
+  <os>
+    <type arch='aarch64' machine='virt'>hvm</type>
+  </os>
+  <devices>
+    <emulator>/usr/bin/qemu-system-aarch64</emulator>
+    <panic/>
+  </devices>
+</domain>
diff --git a/tests/qemuxmlconftest.c b/tests/qemuxmlconftest.c
index 301c683448..33c4448414 100644
--- a/tests/qemuxmlconftest.c
+++ b/tests/qemuxmlconftest.c
@@ -2480,6 +2480,7 @@ mymain(void)
     DO_TEST_CAPS_LATEST("panic");
     DO_TEST_CAPS_LATEST("panic-double");
     DO_TEST_CAPS_LATEST("panic-no-address");
+    DO_TEST_CAPS_ARCH_LATEST_PARSE_ERROR("aarch64-panic-no-model", "aarch64");
 
     DO_TEST_CAPS_LATEST("pvpanic-pci-x86_64");
     DO_TEST_CAPS_ARCH_LATEST("pvpanic-pci-aarch64", "aarch64");
-- 
2.43.0
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org
Re: [PATCH 02/33] tests: Add aarch64-panic-no-model
Posted by Peter Krempa 8 months, 2 weeks ago
On Wed, Jan 24, 2024 at 20:37:22 +0100, Andrea Bolognani wrote:
> This demonstrates that on aarch64, where a native panic device
> doesn't exist, it's necessary for the user to specify the model
> explicitly.
> 
> Signed-off-by: Andrea Bolognani <abologna@redhat.com>
> ---
>  .../aarch64-panic-no-model.aarch64-latest.err       |  1 +
>  tests/qemuxmlconfdata/aarch64-panic-no-model.xml    | 13 +++++++++++++
>  tests/qemuxmlconftest.c                             |  1 +
>  3 files changed, 15 insertions(+)
>  create mode 100644 tests/qemuxmlconfdata/aarch64-panic-no-model.aarch64-latest.err
>  create mode 100644 tests/qemuxmlconfdata/aarch64-panic-no-model.xml

Reviewed-by: Peter Krempa <pkrempa@redhat.com>
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org