test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1 1/2] tests/TestSuite_runtime_vf_queue_number:the queue pairs number up to 256 for iavf
@ 2020-12-28  6:57 Xie wei
  2020-12-28  6:57 ` [dts] [PATCH V1 2/2] tests/TestSuite_runtime_vf_queue_number_kernel:the " Xie wei
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Xie wei @ 2020-12-28  6:57 UTC (permalink / raw)
  To: dts; +Cc: Xie wei

as the queue pairs number up to 256 queue pairs per VF, so change 17 to
257 for testing.

Signed-off-by: Xie wei <weix.xie@intel.com>
---
 tests/TestSuite_runtime_vf_queue_number.py | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/tests/TestSuite_runtime_vf_queue_number.py b/tests/TestSuite_runtime_vf_queue_number.py
index 8d8a6f7b..1d7d536c 100644
--- a/tests/TestSuite_runtime_vf_queue_number.py
+++ b/tests/TestSuite_runtime_vf_queue_number.py
@@ -302,10 +302,10 @@ class TestRuntimeVfQn(TestCase):
         outstring = self.vm0_testpmd.execute_cmd(command_0, expected='# ')
         self.verify("Either rx or tx queues should be non-zero" in outstring, "The output of testpmd start is different from expect when set invalid VF queue number 0.")
         time.sleep(2)
-        command_17 = app_name + "-c %s -n %d %s -- -i %s" % ('0xf', self.dut.get_memory_channels(), gest_eal_param, ' --rxq=17 --txq=17')
-        outstring1 = self.vm0_testpmd.execute_cmd(command_17, expected='# ')
-        self.verify("rxq 17 invalid - must be >= 0 && <= 16" in outstring1,
-                    "The output of testpmd start is different from expect when set invalid VF queue number 17.")
+        command_257 = app_name + "-c %s -n %d %s -- -i %s" % ('0xf', self.dut.get_memory_channels(), gest_eal_param, ' --rxq=257 --txq=257')
+        outstring1 = self.vm0_testpmd.execute_cmd(command_257, expected='# ')
+        self.verify("rxq 257 invalid - must be >= 0 && <= 256" in outstring1,
+                    "The output of testpmd start is different from expect when set invalid VF queue number 257.")
 
     def test_set_valid_vf_qn_with_testpmd_func_cmd(self):
         """
@@ -341,14 +341,14 @@ class TestRuntimeVfQn(TestCase):
         :return:
         """
         expect_str = ["Warning: Either rx or tx queues should be non zero",
-                      "Fail: input rxq (17) can't be greater than max_rx_queues (16) of port 0",
-                      "Fail: input txq (17) can't be greater than max_tx_queues (16) of port 0"]
+                      "Fail: input rxq (257) can't be greater than max_rx_queues (256) of port 0",
+                      "Fail: input txq (257) can't be greater than max_tx_queues (256) of port 0"]
         host_eal_param = '-w %s --file-prefix=test1 --socket-mem 1024,1024' % self.pf_pci
         self.host_testpmd.start_testpmd(self.pmdout.default_cores, param='', eal_param=host_eal_param)
         gest_eal_param = '-w %s --file-prefix=test2' % self.vm_dut_0.ports_info[0]['pci']
         self.vm0_testpmd = PmdOutput(self.vm_dut_0)
         self.vm0_testpmd.start_testpmd(self.pmdout.default_cores, eal_param=gest_eal_param, param='')
-        for invalid_qn in [0, 17]:
+        for invalid_qn in [0, 257]:
             self.vm0_testpmd.execute_cmd('port stop all')
             rxq_output = self.vm0_testpmd.execute_cmd('port config all rxq %d' % invalid_qn)
             txq_output = self.vm0_testpmd.execute_cmd('port config all txq %d' % invalid_qn)
-- 
2.17.1


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

* [dts] [PATCH V1 2/2] tests/TestSuite_runtime_vf_queue_number_kernel:the queue pairs number up to 256 for iavf
  2020-12-28  6:57 [dts] [PATCH V1 1/2] tests/TestSuite_runtime_vf_queue_number:the queue pairs number up to 256 for iavf Xie wei
@ 2020-12-28  6:57 ` Xie wei
  2020-12-28  7:00   ` Xie, WeiX
  2020-12-28  7:00 ` [dts] [PATCH V1 1/2] tests/TestSuite_runtime_vf_queue_number:the " Xie, WeiX
  2020-12-28  7:45 ` Tu, Lijuan
  2 siblings, 1 reply; 5+ messages in thread
From: Xie wei @ 2020-12-28  6:57 UTC (permalink / raw)
  To: dts; +Cc: Xie wei

as the queue pairs number up to 256 queue pairs per VF, so change 17 to
257 for testing.

Signed-off-by: Xie wei <weix.xie@intel.com>
---
 tests/TestSuite_runtime_vf_queue_number_kernel.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/TestSuite_runtime_vf_queue_number_kernel.py b/tests/TestSuite_runtime_vf_queue_number_kernel.py
index fe4bea07..3daf2347 100644
--- a/tests/TestSuite_runtime_vf_queue_number_kernel.py
+++ b/tests/TestSuite_runtime_vf_queue_number_kernel.py
@@ -204,7 +204,7 @@ class TestRuntimeVfQueueNumberKernel(TestCase):
             self.check_result(nr_queue, out, out2, pkts_num, count, misc)
 
     def test_set_invalid_vf_queue_num_command_line(self):
-        invalid_queue_num = [0, 17]
+        invalid_queue_num = [0, 257]
         for i in invalid_queue_num:
             self.vm0_testpmd = PmdOutput(self.vm_dut_0)
             self.vm_dut_0.session_secondary = self.vm_dut_0.new_session()
@@ -215,7 +215,7 @@ class TestRuntimeVfQueueNumberKernel(TestCase):
                 self.verify('Either rx or tx queues should be non-zero' in out, "queue number can't be zero")
             else:
                 # the dpdk output non-zero conflict with >=0, to be fixed...
-                self.verify('txq 17 invalid - must be >= 0 && <= 16' in out, "queue number is too big")
+                self.verify('txq 257 invalid - must be >= 0 && <= 256' in out, "queue number is too big")
 
     def test_set_valid_vf_queue_num_with_function(self):
         random_queue = random.randint(2, 15)
@@ -270,7 +270,7 @@ class TestRuntimeVfQueueNumberKernel(TestCase):
             self.check_result(nr_queue, out, out2, pkts_num, count, misc)
 
     def test_set_invalid_vf_queue_num_with_testpmd_command(self):
-        invalid_queue_num = [0, 17]
+        invalid_queue_num = [0, 257]
         self.vm0_testpmd = PmdOutput(self.vm_dut_0)
         eal_param = '-w %(vf0)s' % {'vf0': self.vm_dut_0.vm_pci0}
         self.vm0_testpmd.start_testpmd("all", eal_param=eal_param)
@@ -287,7 +287,7 @@ class TestRuntimeVfQueueNumberKernel(TestCase):
                 self.verify('Either rx or tx queues should be non zero' in out, "queue number can't be zero")
             else:
                 out = self.vm0_testpmd.execute_cmd("port config all rxq %s" % i)
-                self.verify("input rxq (17) can't be greater than max_rx_queues (16) of port 0" in out,
+                self.verify("input rxq (257) can't be greater than max_rx_queues (256) of port 0" in out,
                             "queue number is too big")
             self.vm0_testpmd.execute_cmd("clear port stats all")
             time.sleep(1)
-- 
2.17.1


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

* Re: [dts] [PATCH V1 1/2] tests/TestSuite_runtime_vf_queue_number:the queue pairs number up to 256 for iavf
  2020-12-28  6:57 [dts] [PATCH V1 1/2] tests/TestSuite_runtime_vf_queue_number:the queue pairs number up to 256 for iavf Xie wei
  2020-12-28  6:57 ` [dts] [PATCH V1 2/2] tests/TestSuite_runtime_vf_queue_number_kernel:the " Xie wei
@ 2020-12-28  7:00 ` Xie, WeiX
  2020-12-28  7:45 ` Tu, Lijuan
  2 siblings, 0 replies; 5+ messages in thread
From: Xie, WeiX @ 2020-12-28  7:00 UTC (permalink / raw)
  To: dts

[-- Attachment #1: Type: text/plain, Size: 363 bytes --]

Tested-by:  Xie,WeiX < weix.xie@intel.com>

Regards,
Xie Wei

> -----Original Message-----
> From: Xie wei [mailto:weix.xie@intel.com]
> Sent: Monday, December 28, 2020 2:57 PM
> To: dts@dpdk.org
> Cc: Xie, WeiX <weix.xie@intel.com>
> Subject: [dts][PATCH V1 1/2]
> tests/TestSuite_runtime_vf_queue_number:the queue pairs number up to
> 256 for iavf

[-- Attachment #2: TestRuntimeVfQn.log --]
[-- Type: application/octet-stream, Size: 217050 bytes --]

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

* Re: [dts] [PATCH V1 2/2] tests/TestSuite_runtime_vf_queue_number_kernel:the queue pairs number up to 256 for iavf
  2020-12-28  6:57 ` [dts] [PATCH V1 2/2] tests/TestSuite_runtime_vf_queue_number_kernel:the " Xie wei
@ 2020-12-28  7:00   ` Xie, WeiX
  0 siblings, 0 replies; 5+ messages in thread
From: Xie, WeiX @ 2020-12-28  7:00 UTC (permalink / raw)
  To: dts

[-- Attachment #1: Type: text/plain, Size: 372 bytes --]

Tested-by:  Xie,WeiX < weix.xie@intel.com>

Regards,
Xie Wei


> -----Original Message-----
> From: Xie wei [mailto:weix.xie@intel.com]
> Sent: Monday, December 28, 2020 2:57 PM
> To: dts@dpdk.org
> Cc: Xie, WeiX <weix.xie@intel.com>
> Subject: [dts][PATCH V1 2/2]
> tests/TestSuite_runtime_vf_queue_number_kernel:the queue pairs
> number up to 256 for iavf

[-- Attachment #2: TestRuntimeVfQueueNumberKernel.log --]
[-- Type: application/octet-stream, Size: 74712 bytes --]

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

* Re: [dts] [PATCH V1 1/2] tests/TestSuite_runtime_vf_queue_number:the queue pairs number up to 256 for iavf
  2020-12-28  6:57 [dts] [PATCH V1 1/2] tests/TestSuite_runtime_vf_queue_number:the queue pairs number up to 256 for iavf Xie wei
  2020-12-28  6:57 ` [dts] [PATCH V1 2/2] tests/TestSuite_runtime_vf_queue_number_kernel:the " Xie wei
  2020-12-28  7:00 ` [dts] [PATCH V1 1/2] tests/TestSuite_runtime_vf_queue_number:the " Xie, WeiX
@ 2020-12-28  7:45 ` Tu, Lijuan
  2 siblings, 0 replies; 5+ messages in thread
From: Tu, Lijuan @ 2020-12-28  7:45 UTC (permalink / raw)
  To: Xie, WeiX, dts; +Cc: Xie, WeiX

> as the queue pairs number up to 256 queue pairs per VF, so change 17 to
> 257 for testing.
> 
> Signed-off-by: Xie wei <weix.xie@intel.com>

Applied the series.

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

end of thread, other threads:[~2020-12-28  7:45 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-28  6:57 [dts] [PATCH V1 1/2] tests/TestSuite_runtime_vf_queue_number:the queue pairs number up to 256 for iavf Xie wei
2020-12-28  6:57 ` [dts] [PATCH V1 2/2] tests/TestSuite_runtime_vf_queue_number_kernel:the " Xie wei
2020-12-28  7:00   ` Xie, WeiX
2020-12-28  7:00 ` [dts] [PATCH V1 1/2] tests/TestSuite_runtime_vf_queue_number:the " Xie, WeiX
2020-12-28  7:45 ` Tu, Lijuan

test suite reviews and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dts/0 dts/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dts dts/ https://inbox.dpdk.org/dts \
		dts@dpdk.org
	public-inbox-index dts

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dts


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git