qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] tests/qtest/virtio-net: fix hotplug test case


From: Thomas Huth
Subject: Re: [PATCH] tests/qtest/virtio-net: fix hotplug test case
Date: Mon, 8 Nov 2021 18:12:23 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0

On 28/10/2021 19.30, Laurent Vivier wrote:
virtio-net-test has an hotplug testcase that is never executed.

This is because the testcase is attached to virtio-pci interface
rather than to virtio-net-pci.

   $ QTEST_QEMU_BINARY=./qemu-system-x86_64 tests/qtest/qos-test -l | grep 
hotplug
   /x86_64/.../pci-ohci-tests/ohci_pci-test-hotplug
   /x86_64/.../e1000e/e1000e-tests/hotplug
   /x86_64/.../virtio-blk-pci/virtio-blk-pci-tests/hotplug
   /x86_64/.../vhost-user-blk-pci/vhost-user-blk-pci-tests/hotplug
   /x86_64/.../virtio-rng-pci/virtio-rng-pci-tests/hotplug
   /x86_64/.../virtio-scsi/virtio-scsi-tests/hotplug
   /x86_64/.../virtio-serial/virtio-serial-tests/hotplug

With this fix:

   $ QTEST_QEMU_BINARY=./qemu-system-x86_64 tests/qtest/qos-test -l | grep 
hotplug
   ...
   /x86_64/.../vhost-user-blk-pci/vhost-user-blk-pci-tests/hotplug
   /x86_64/.../virtio-net-pci/virtio-net-pci-tests/hotplug
   /x86_64/.../virtio-rng-pci/virtio-rng-pci-tests/hotplug
   ...
   $ QTEST_QEMU_BINARY=./qemu-system-x86_64 tests/qtest/qos-test -p 
/x86_64/.../virtio-net-pci-tests/hotplug
   
/x86_64/pc/i440FX-pcihost/pci-bus-pc/pci-bus/virtio-net-pci/virtio-net-pci-tests/hotplug:
 OK

Fixes: 6ae333f91b99 ("qos-test: virtio-net test node")
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
---
  tests/qtest/virtio-net-test.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/qtest/virtio-net-test.c b/tests/qtest/virtio-net-test.c
index a08e2ffe123f..8bf74e516cce 100644
--- a/tests/qtest/virtio-net-test.c
+++ b/tests/qtest/virtio-net-test.c
@@ -319,7 +319,7 @@ static void register_virtio_net_test(void)
          .before = virtio_net_test_setup,
      };
- qos_add_test("hotplug", "virtio-pci", hotplug, &opts);
+    qos_add_test("hotplug", "virtio-net-pci", hotplug, &opts);
  #ifndef _WIN32
      qos_add_test("basic", "virtio-net", send_recv_test, &opts);
      qos_add_test("rx_stop_cont", "virtio-net", stop_cont_test, &opts);


Acked-by: Thomas Huth <thuth@redhat.com>




reply via email to

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