test suite reviews and discussions
 help / color / mirror / Atom feed
* Re: [dts] [PATCH V1] tests/ddp_mpls: compatible with iavf and i40evf for vf
  2021-03-25 17:16 [dts] [PATCH V1] tests/ddp_mpls: compatible with iavf and i40evf for vf xiewei
@ 2021-03-25  9:09 ` Xie, WeiX
  2021-03-30  1:53 ` Tu, Lijuan
  1 sibling, 0 replies; 3+ messages in thread
From: Xie, WeiX @ 2021-03-25  9:09 UTC (permalink / raw)
  To: dts

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

> -----Original Message-----
> From: Xie, WeiX
> Sent: Friday, March 26, 2021 1:17 AM
> To: dts@dpdk.org
> Cc: Xie, WeiX <weix.xie@intel.com>
> Subject: [PATCH V1] tests/ddp_mpls: compatible with iavf and i40evf for vf

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

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

[-- Attachment #3: Testddp_mpls_i40evf.log --]
[-- Type: application/octet-stream, Size: 383929 bytes --]

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

* [dts] [PATCH V1] tests/ddp_mpls: compatible with iavf and i40evf for vf
@ 2021-03-25 17:16 xiewei
  2021-03-25  9:09 ` Xie, WeiX
  2021-03-30  1:53 ` Tu, Lijuan
  0 siblings, 2 replies; 3+ messages in thread
From: xiewei @ 2021-03-25 17:16 UTC (permalink / raw)
  To: dts; +Cc: xiewei

iavf has rss hash rx offload capability by default, but i40evf not have.
So the hash value and rx queue number will be random when receive wrong label MPLS packets.
If want to wrong label packets received by default queue 0, just launch test-pmd with parameter "--disable-rss".

Signed-off-by: xiewei <weix.xie@intel.com>
---
 tests/TestSuite_ddp_mpls.py | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/tests/TestSuite_ddp_mpls.py b/tests/TestSuite_ddp_mpls.py
index 53fd4a76..f0c28386 100644
--- a/tests/TestSuite_ddp_mpls.py
+++ b/tests/TestSuite_ddp_mpls.py
@@ -39,6 +39,7 @@ from test_case import TestCase
 from pmd_output import PmdOutput
 from settings import get_nic_name
 import random
+import re
 
 VM_CORES_MASK = 'all'
 PF_MAX_QUEUE = 64
@@ -131,7 +132,12 @@ class Testddp_mpls(TestCase):
         self.vm0_testpmd.start_testpmd(
             VM_CORES_MASK,"--port-topology=chained --txq=%s --rxq=%s" 
             % (VF_MAX_QUEUE, VF_MAX_QUEUE))
-        
+
+        output = self.vm0_testpmd.execute_cmd('show port info 0')
+        vf_driver = re.findall("Driver\s*name:\s*(\w+)", output)
+        if vf_driver[0] == "net_iavf":
+            self.vm0_testpmd.execute_cmd('quit', '# ')
+            self.vm0_testpmd.start_testpmd(VM_CORES_MASK, "--port-topology=chained --txq=%s --rxq=%s --disable-rss" % (VF_MAX_QUEUE, VF_MAX_QUEUE))
 
     def destroy_vm_env(self):
         
-- 
2.17.1


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

* Re: [dts] [PATCH V1] tests/ddp_mpls: compatible with iavf and i40evf for vf
  2021-03-25 17:16 [dts] [PATCH V1] tests/ddp_mpls: compatible with iavf and i40evf for vf xiewei
  2021-03-25  9:09 ` Xie, WeiX
@ 2021-03-30  1:53 ` Tu, Lijuan
  1 sibling, 0 replies; 3+ messages in thread
From: Tu, Lijuan @ 2021-03-30  1:53 UTC (permalink / raw)
  To: Xie, WeiX, dts; +Cc: Xie, WeiX



> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of xiewei
> Sent: 2021年3月26日 1:17
> To: dts@dpdk.org
> Cc: Xie, WeiX <weix.xie@intel.com>
> Subject: [dts] [PATCH V1] tests/ddp_mpls: compatible with iavf and i40evf for vf
> 
> iavf has rss hash rx offload capability by default, but i40evf not have.
> So the hash value and rx queue number will be random when receive wrong label
> MPLS packets.
> If want to wrong label packets received by default queue 0, just launch test-pmd
> with parameter "--disable-rss".
> 
> Signed-off-by: xiewei <weix.xie@intel.com>
> ---
>  tests/TestSuite_ddp_mpls.py | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/TestSuite_ddp_mpls.py b/tests/TestSuite_ddp_mpls.py index
> 53fd4a76..f0c28386 100644
> --- a/tests/TestSuite_ddp_mpls.py
> +++ b/tests/TestSuite_ddp_mpls.py
> @@ -39,6 +39,7 @@ from test_case import TestCase  from pmd_output import
> PmdOutput  from settings import get_nic_name  import random
> +import re
> 
>  VM_CORES_MASK = 'all'
>  PF_MAX_QUEUE = 64
> @@ -131,7 +132,12 @@ class Testddp_mpls(TestCase):
>          self.vm0_testpmd.start_testpmd(
>              VM_CORES_MASK,"--port-topology=chained --txq=%s --rxq=%s"
>              % (VF_MAX_QUEUE, VF_MAX_QUEUE))
> -
> +
> +        output = self.vm0_testpmd.execute_cmd('show port info 0')
> +        vf_driver = re.findall("Driver\s*name:\s*(\w+)", output)
> +        if vf_driver[0] == "net_iavf":
> +            self.vm0_testpmd.execute_cmd('quit', '# ')
> +            self.vm0_testpmd.start_testpmd(VM_CORES_MASK,
> + "--port-topology=chained --txq=%s --rxq=%s --disable-rss" %
> + (VF_MAX_QUEUE, VF_MAX_QUEUE))

I think re-launch testpmd is a waste of time, prefer to start testpmd with 'disable-rss` directly.

> 
>      def destroy_vm_env(self):
> 
> --
> 2.17.1


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

end of thread, other threads:[~2021-03-30  1:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-25 17:16 [dts] [PATCH V1] tests/ddp_mpls: compatible with iavf and i40evf for vf xiewei
2021-03-25  9:09 ` Xie, WeiX
2021-03-30  1:53 ` Tu, Lijuan

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