test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts][PATCH V2 2/2] tests/dynamic_queue: skip the dynamic_queue/test_rxq_chgring_setup and test_txq_chgring_setup on ice
  2023-01-10 15:53 ` [dts][PATCH V2 2/2] tests/dynamic_queue: " Hongbo Li
@ 2023-01-10  8:02   ` lijuan.tu
  0 siblings, 0 replies; 4+ messages in thread
From: lijuan.tu @ 2023-01-10  8:02 UTC (permalink / raw)
  To: dts, Hongbo Li; +Cc: Hongbo Li

On Tue, 10 Jan 2023 23:53:41 +0800, Hongbo Li <hongbox.li@intel.com> wrote:
> skip the "setup" command on ice, according to the conclusion of the developer, ice does not support the setup queue command
> 
> Signed-off-by: Hongbo Li <hongbox.li@intel.com>

Acked-by: Lijuan Tu <lijuan.tu@intel.com>
Series applied, thanks

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [dts][PATCH V2 1/2] conf/test_case_checklist: skip the dynamic_queue/test_rxq_chgring_setup and test_txq_chgring_setup on ice
@ 2023-01-10 15:53 Hongbo Li
  2023-01-10 15:53 ` [dts][PATCH V2 2/2] tests/dynamic_queue: " Hongbo Li
  0 siblings, 1 reply; 4+ messages in thread
From: Hongbo Li @ 2023-01-10 15:53 UTC (permalink / raw)
  To: dts; +Cc: Hongbo Li

skip the dynamic_queue/test_rxq_chgring_setup and test_txq_chgring_setup on ice, according to the conclusion of the developer, ice does not support the setup queue command

Signed-off-by: Hongbo Li <hongbox.li@intel.com>
---

v2:modify comments in test_case_checklist.json

 conf/test_case_checklist.json | 36 +++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/conf/test_case_checklist.json b/conf/test_case_checklist.json
index f31daf62..a8dce02c 100644
--- a/conf/test_case_checklist.json
+++ b/conf/test_case_checklist.json
@@ -3807,5 +3807,41 @@
             "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"
+        }
     ]
 }
-- 
2.17.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [dts][PATCH V2 2/2] tests/dynamic_queue: skip the dynamic_queue/test_rxq_chgring_setup and test_txq_chgring_setup on ice
  2023-01-10 15:53 [dts][PATCH V2 1/2] conf/test_case_checklist: skip the dynamic_queue/test_rxq_chgring_setup and test_txq_chgring_setup on ice Hongbo Li
@ 2023-01-10 15:53 ` Hongbo Li
  2023-01-10  8:02   ` lijuan.tu
  0 siblings, 1 reply; 4+ messages in thread
From: Hongbo Li @ 2023-01-10 15:53 UTC (permalink / raw)
  To: dts; +Cc: Hongbo Li

skip the "setup" command on ice, according to the conclusion of the developer, ice does not support the setup queue command

Signed-off-by: Hongbo Li <hongbox.li@intel.com>
---
 tests/TestSuite_dynamic_queue.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tests/TestSuite_dynamic_queue.py b/tests/TestSuite_dynamic_queue.py
index 5a826bd1..e6bad6c9 100644
--- a/tests/TestSuite_dynamic_queue.py
+++ b/tests/TestSuite_dynamic_queue.py
@@ -27,6 +27,7 @@ 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
@@ -127,7 +128,7 @@ class TestDynamicQueue(TestCase):
                 )
 
         for i in range(test_loop):
-            if chgflag == 0:
+            if chgflag == 0 and not self.is_800_series:
                 self.dut_testpmd.execute_cmd("port 0 rxq %d setup" % queue[i])
             self.dut_testpmd.execute_cmd("port 0 rxq %d start" % queue[i])
 
@@ -185,7 +186,7 @@ class TestDynamicQueue(TestCase):
                     chk_qringsize == chg_qringsize,
                     "Fail to change ring size at runtime!",
                 )
-            if chgflag == 0:
+            if chgflag == 0 and not self.is_800_series:
                 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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [dts][PATCH V2 2/2] tests/dynamic_queue: skip the dynamic_queue/test_rxq_chgring_setup and test_txq_chgring_setup on ice
  2023-01-10 15:48 [dts][PATCH V2 1/2] conf/test_case_checklist: " Hongbo Li
@ 2023-01-10 15:48 ` Hongbo Li
  0 siblings, 0 replies; 4+ messages in thread
From: Hongbo Li @ 2023-01-10 15:48 UTC (permalink / raw)
  To: dts; +Cc: Hongbo Li

skip the "setup" command on ice, according to the conclusion of the developer, ice does not support the setup queue command

Signed-off-by: Hongbo Li <hongbox.li@intel.com>
---
 tests/TestSuite_dynamic_queue.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tests/TestSuite_dynamic_queue.py b/tests/TestSuite_dynamic_queue.py
index 5a826bd1..e6bad6c9 100644
--- a/tests/TestSuite_dynamic_queue.py
+++ b/tests/TestSuite_dynamic_queue.py
@@ -27,6 +27,7 @@ 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
@@ -127,7 +128,7 @@ class TestDynamicQueue(TestCase):
                 )
 
         for i in range(test_loop):
-            if chgflag == 0:
+            if chgflag == 0 and not self.is_800_series:
                 self.dut_testpmd.execute_cmd("port 0 rxq %d setup" % queue[i])
             self.dut_testpmd.execute_cmd("port 0 rxq %d start" % queue[i])
 
@@ -185,7 +186,7 @@ class TestDynamicQueue(TestCase):
                     chk_qringsize == chg_qringsize,
                     "Fail to change ring size at runtime!",
                 )
-            if chgflag == 0:
+            if chgflag == 0 and not self.is_800_series:
                 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


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-01-10  8:02 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-10 15:53 [dts][PATCH V2 1/2] conf/test_case_checklist: skip the dynamic_queue/test_rxq_chgring_setup and test_txq_chgring_setup on ice Hongbo Li
2023-01-10 15:53 ` [dts][PATCH V2 2/2] tests/dynamic_queue: " Hongbo Li
2023-01-10  8:02   ` lijuan.tu
  -- strict thread matches above, loose matches on Subject: below --
2023-01-10 15:48 [dts][PATCH V2 1/2] conf/test_case_checklist: " Hongbo Li
2023-01-10 15:48 ` [dts][PATCH V2 2/2] tests/dynamic_queue: " Hongbo Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).