[dts][PATCH V2] tests/dpdk_gro_lib:fix start QEMU with wrong command issue

[dts][PATCH V2] tests/dpdk_gro_lib:fix start QEMU with wrong command issue

[Wei Ling]

When start QEMU with multi-queues, the command must include
`mq=on,vectors=x` parameter.

Signed-off-by: Wei Ling <weix.ling@intel.com>
---

v2:Handle the queues=1 scene.

 tests/TestSuite_dpdk_gro_lib.py | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/tests/TestSuite_dpdk_gro_lib.py b/tests/TestSuite_dpdk_gro_lib.py
index eaaf757e..f2948e76 100644
--- a/tests/TestSuite_dpdk_gro_lib.py
+++ b/tests/TestSuite_dpdk_gro_lib.py
@@ -180,7 +180,7 @@ class TestDPDKGROLib(TestCase):
                 vm_config.params[i]["cpu"][0]["number"] = 1
                 vm_config.params[i]["cpu"][0]["cpupin"] = self.qemu_cpupin
 
-    def start_vm(self, mode=1, queue=1):
+    def start_vm(self, queue=1):
         self.vm1 = VM(self.dut, "vm0", "vhost_sample")
         self.vm1.load_config()
         vm_params_1 = {}
@@ -188,14 +188,13 @@ class TestDPDKGROLib(TestCase):
         vm_params_1["opt_path"] = self.base_dir + "/vhost-net"
         vm_params_1["opt_mac"] = self.virtio_mac1
         vm_params_1["opt_queue"] = queue
-        if mode == 5:
-            vm_params_1[
-                "opt_settings"
-            ] = "mrg_rxbuf=on,csum=on,gso=on,host_tso4=on,guest_tso4=on,mq=on,vectors=15"
+        if int(queue) > 1:
+            mq_pram = ",mq=on,vectors=%s" % (2 + 2 * int(queue))
         else:
-            vm_params_1[
-                "opt_settings"
-            ] = "mrg_rxbuf=on,csum=on,gso=on,host_tso4=on,guest_tso4=on"
+            mq_pram = ""
+        vm_params_1["opt_settings"] = (
+            "mrg_rxbuf=on,csum=on,gso=on,host_tso4=on,guest_tso4=on%s" % mq_pram
+        )
         self.vm1.set_vm_device(**vm_params_1)
         self.set_vm_cpu_number(self.vm1)
         try:
@@ -347,7 +346,7 @@ class TestDPDKGROLib(TestCase):
         """
         self.config_kernel_nic_host(0)
         self.launch_testpmd_gro_on(mode=1, queue=2)
-        self.start_vm(mode=1, queue=2)
+        self.start_vm(queue=2)
         time.sleep(5)
         self.dut.get_session_output(timeout=2)
         # Get the virtio-net device name
-- 
2.25.1

[dts][PATCH V2] tests/dpdk_gro_lib:fix start QEMU with wrong command issue

[lijuan.tu]

On Mon, 13 Feb 2023 15:18:31 +0800, Wei Ling <weix.ling@intel.com> wrote:
> When start QEMU with multi-queues, the command must include
> `mq=on,vectors=x` parameter.
> 
> Signed-off-by: Wei Ling <weix.ling@intel.com>


Applied, thanks