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
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
[-- 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 --]
[-- 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 --]
> 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.