qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v4 5/7] tests/x86: Add 'q35' machine type to override-tests i


From: Thomas Huth
Subject: Re: [PATCH v4 5/7] tests/x86: Add 'q35' machine type to override-tests in hd-geo-test
Date: Tue, 27 Sep 2022 15:00:13 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.13.0

On 20/09/2022 12.48, Michael Labiuk wrote:
Signed-off-by: Michael Labiuk <michael.labiuk@virtuozzo.com>
---
  tests/qtest/hd-geo-test.c | 105 +++++++++++++++++++++++++++++++++++---
  1 file changed, 98 insertions(+), 7 deletions(-)

diff --git a/tests/qtest/hd-geo-test.c b/tests/qtest/hd-geo-test.c
index 413cf964c0..58b1107d64 100644
--- a/tests/qtest/hd-geo-test.c
+++ b/tests/qtest/hd-geo-test.c
...
@@ -799,7 +857,23 @@ static void test_override_virtio_blk(void)
      add_drive_with_mbr(args, empty_mbr, 1);
      add_virtio_disk(args, 0, "pci.0", 3, 10000, 120, 30);
      add_virtio_disk(args, 1, "pci.0", 4, 9000, 120, 30);
-    test_override(args, expected);
+    test_override(args, "pc", expected);
+}
+
+static void test_override_virtio_blk_q35(void)
+{
+    TestArgs *args = create_args();
+    CHSResult expected[] = {
+        {"/pci@i0cf8/pci-bridge@2/scsi@3/disk@0,0", {10000, 120, 30} },
+        {"/pci@i0cf8/pci-bridge@2/scsi@4/disk@0,0", {9000, 120, 30} },
+        {NULL, {0, 0, 0} }
+    };
+    add_drive_with_mbr(args, empty_mbr, 1);
+    add_drive_with_mbr(args, empty_mbr, 1);
+    setup_pci_bridge(args, "pcie.0", "br");
+    add_virtio_disk(args, 0, "br", 3, 10000, 120, 30);
+    add_virtio_disk(args, 1, "br", 4, 9000, 120, 30);
+    test_override(args, "pc", expected);

Shouldn't that use "q35" as machine instead?

 Thomas




reply via email to

[Prev in Thread] Current Thread [Next in Thread]