test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts][PATCH V1] tests/large_vf:modify test case to adapt ice change
@ 2022-07-18 15:27 Zhimin Huang
  2022-08-24 10:13 ` lijuan.tu
  0 siblings, 1 reply; 2+ messages in thread
From: Zhimin Huang @ 2022-07-18 15:27 UTC (permalink / raw)
  To: dts; +Cc: Zhimin Huang

For ice kernel update, the available queue is 767 in 2 port E810 nic.
it's only support 2 vfs start testpmd with 256 queue.

Signed-off-by: Zhimin Huang <zhiminx.huang@intel.com>
---
 test_plans/large_vf_test_plan.rst | 11 ++++++++---
 tests/TestSuite_large_vf.py       | 10 ++++++----
 2 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/test_plans/large_vf_test_plan.rst b/test_plans/large_vf_test_plan.rst
index c7f0d119..e043a660 100644
--- a/test_plans/large_vf_test_plan.rst
+++ b/test_plans/large_vf_test_plan.rst
@@ -36,7 +36,7 @@ Note::
      --total-num-mbufs=N, N is mbuf number, usually allocate 512 mbuf for one
      queue, if use 3 VFs, N >= 512*256*3=393216.
 
-Test case: 3 Max VFs + 256 queues
+Test case: multi VFs + 256 queues
 =================================
 
 Subcase 1 : multi fdir for 256 queues of consistent queue group
@@ -308,12 +308,17 @@ or::
 Fail to setup test.
 
 
-Subcase 7: negative: fail to setup 256 queues when more than 3 VFs
+Subcase 7: negative: fail to setup 256 queues when more than 2 VFs
 ------------------------------------------------------------------
-Create 4 VFs.
+Create 3 VFs.
 Bind all VFs to vfio-pci.
 Fail to start testpmd with "--txq=256 --rxq=256".
 
+.. note::
+
+    For SW4.0 + ice-1.9.5, the available queue is 767 in 2 port E810 nic, it support 2 vfs start testpmd with 256 queue
+
+    For SW3.2 + ice-1.8.3, the available queue is 943 in 2 port E810 nic, it support 3 vfs start testpmd with 256 queue
 
 Test case: 128 Max VFs + 4 queues (default)
 ===========================================
diff --git a/tests/TestSuite_large_vf.py b/tests/TestSuite_large_vf.py
index 00991790..4e2ff1d6 100644
--- a/tests/TestSuite_large_vf.py
+++ b/tests/TestSuite_large_vf.py
@@ -355,7 +355,9 @@ class TestLargeVf(TestCase):
                 elif subcase_name == "test_more_than_3_vfs_256_queues":
                     self.pmd_output.execute_cmd("quit", "#")
                     # start testpmd use 256 queues
-                    for i in range(self.vf_num + 1):
+                    # for CVL 2 ports, the available queue is 767 for ice-1.9.5(SW4.0)
+                    _vfs_num = self.vf_num - 1
+                    for i in range(_vfs_num + 1):
                         if self.max_vf_num == 64:
                             self.pmdout_list[0].start_testpmd(
                                 param=tv["param"],
@@ -379,7 +381,7 @@ class TestLargeVf(TestCase):
                             self.pmdout_list[0].execute_cmd("quit", "# ")
                             break
                         else:
-                            if i < self.vf_num:
+                            if i < _vfs_num:
                                 self.pmdout_list[i].start_testpmd(
                                     param=tv["param"],
                                     ports=[self.sriov_vfs_port[i].pci],
@@ -406,7 +408,7 @@ class TestLargeVf(TestCase):
                                 self.pmdout_list[0].execute_cmd("quit", "# ")
                                 self.pmdout_list[1].execute_cmd("quit", "# ")
                                 self.pmdout_list[2].execute_cmd("quit", "# ")
-                                if self.vf_num > 3:
+                                if _vfs_num > 3:
                                     self.pmdout_list[3].execute_cmd("quit", "# ")
                                     self.pmdout_list[4].execute_cmd("quit", "# ")
                                     self.pmdout_list[5].execute_cmd("quit", "# ")
@@ -645,7 +647,7 @@ class TestLargeVf(TestCase):
             )
             self.check_txonly_pkts(rxtx_num)
 
-    def test_3_vfs_256_queues(self):
+    def test_multi_vfs_256_queues(self):
         self.create_iavf(self.vf_num + 1)
         self.launch_testpmd("--rxq=256 --txq=256", total=True)
         self.config_testpmd()
-- 
2.17.1


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

* [dts][PATCH V1] tests/large_vf:modify test case to adapt ice change
  2022-07-18 15:27 [dts][PATCH V1] tests/large_vf:modify test case to adapt ice change Zhimin Huang
@ 2022-08-24 10:13 ` lijuan.tu
  0 siblings, 0 replies; 2+ messages in thread
From: lijuan.tu @ 2022-08-24 10:13 UTC (permalink / raw)
  To: dts, Zhimin Huang; +Cc: Zhimin Huang

On Mon, 18 Jul 2022 23:27:29 +0800, Zhimin Huang <zhiminx.huang@intel.com> wrote:
> For ice kernel update, the available queue is 767 in 2 port E810 nic.
> it's only support 2 vfs start testpmd with 256 queue.
> 
> Signed-off-by: Zhimin Huang <zhiminx.huang@intel.com>

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

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

end of thread, other threads:[~2022-08-24 10:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-18 15:27 [dts][PATCH V1] tests/large_vf:modify test case to adapt ice change Zhimin Huang
2022-08-24 10:13 ` lijuan.tu

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).