[dts][PATCH V1] tests/dynamic_queue: cancel support for ice

[dts][PATCH V1] tests/dynamic_queue: cancel support for ice

[Hongbo Li]

confirm on dpdk doc:doc/guides/nics/features/iavf.ini, ice not support runtime to setup rx/tx queue, and remove "port 0 rxq <id> setup" is invalid cases.

Signed-off-by: Hongbo Li <hongbox.li@intel.com>
---
 conf/test_case_checklist.json    | 36 --------------------------------
 tests/TestSuite_dynamic_queue.py |  5 ++---
 2 files changed, 2 insertions(+), 39 deletions(-)

diff --git a/conf/test_case_checklist.json b/conf/test_case_checklist.json
index 670e4ec8..5b34828f 100644
--- a/conf/test_case_checklist.json
+++ b/conf/test_case_checklist.json
@@ -3807,41 +3807,5 @@
             "Bug ID": "",
             "Comments": "testpmd not support socket-mem param in freebsd"
         }
-    ],
-    "rxq_chgring_setup":[
-        {
-            "OS": [
-                "ALL"
-            ],
-            "NIC": [
-                "ICE_100G-E810C_QSFP",
-                "ICE_25G-E823C_QSFP",
-                "ICE_25G-E810C_SFP",
-                "ICE_25G-E810_XXV_SFP"
-            ],
-            "Target": [
-                "ALL"
-            ],
-            "Bug ID": "",
-            "Comments": "this NIC not support this case"
-        }
-    ],
-    "txq_chgring_setup":[
-        {
-            "OS": [
-                "ALL"
-            ],
-            "NIC": [
-                "ICE_100G-E810C_QSFP",
-                "ICE_25G-E823C_QSFP",
-                "ICE_25G-E810C_SFP",
-                "ICE_25G-E810_XXV_SFP"
-            ],
-            "Target": [
-                "ALL"
-            ],
-            "Bug ID": "",
-            "Comments": "this NIC not support this case"
-        }
     ]
 }
diff --git a/tests/TestSuite_dynamic_queue.py b/tests/TestSuite_dynamic_queue.py
index a42cd075..76319646 100644
--- a/tests/TestSuite_dynamic_queue.py
+++ b/tests/TestSuite_dynamic_queue.py
@@ -27,7 +27,6 @@ class TestDynamicQueue(TestCase):
         tester_port = self.tester.get_local_port(self.used_dut_port)
         self.tester_intf = self.tester.get_interface(tester_port)
         self.dut_testpmd = PmdOutput(self.dut)
-        self.is_800_series = self.is_eth_series_nic(800)
 
     def set_up(self):
         # Intel® Ethernet Converged Network Adapter XL710-QDA1 needs more cores to run properly
@@ -129,7 +128,7 @@ class TestDynamicQueue(TestCase):
                 )
 
         for i in range(test_loop):
-            if chgflag == 0 and not self.is_800_series:
+            if chgflag == 0:
                 self.dut_testpmd.execute_cmd("port 0 rxq %d setup" % queue[i])
             self.dut_testpmd.execute_cmd("port 0 rxq %d start" % queue[i])
 
@@ -187,7 +186,7 @@ class TestDynamicQueue(TestCase):
                     chk_qringsize == chg_qringsize,
                     "Fail to change ring size at runtime!",
                 )
-            if chgflag == 0 and not self.is_800_series:
+            if chgflag == 0:
                 self.dut_testpmd.execute_cmd("port 0 txq %d setup" % queue)
 
             self.dut_testpmd.execute_cmd("port 0 txq %d start" % queue)
-- 
2.17.1

RE: [dts][PATCH V1] tests/dynamic_queue: cancel support for ice

[Peng, Yuan]

> -----Original Message-----
> From: Hongbo Li <hongbox.li@intel.com>
> Sent: Saturday, April 15, 2023 2:44 AM
> To: dts@dpdk.org
> Cc: Li, HongboX <hongbox.li@intel.com>
> Subject: [dts][PATCH V1] tests/dynamic_queue: cancel support for ice
> 
> confirm on dpdk doc:doc/guides/nics/features/iavf.ini, ice not support
> runtime to setup rx/tx queue, and remove "port 0 rxq <id> setup" is invalid
> cases.
> 
> Signed-off-by: Hongbo Li <hongbox.li@intel.com>
> ---

Acked-by: Yuan Peng <yuan.peng@intel.com>

[dts][PATCH V1] tests/dynamic_queue: cancel support for ice

[lijuan.tu]

On Sat, 15 Apr 2023 02:43:35 +0800, Hongbo Li <hongbox.li@intel.com> wrote:
> confirm on dpdk doc:doc/guides/nics/features/iavf.ini, ice not support runtime to setup rx/tx queue, and remove "port 0 rxq <id> setup" is invalid cases.
> 
> Signed-off-by: Hongbo Li <hongbox.li@intel.com>


Applied, thanks