test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V3 0/2] v3 pmdrss_hash: replace legacy filter with rte_flow
@ 2021-11-08  7:09 Yu Jiang
  2021-11-08  7:09 ` [dts] [PATCH V3 1/2] test_plans/pmdrss_hash: " Yu Jiang
  2021-11-08  7:09 ` [dts] [PATCH V3 2/2] tests/pmdrss_hash: " Yu Jiang
  0 siblings, 2 replies; 9+ messages in thread
From: Yu Jiang @ 2021-11-08  7:09 UTC (permalink / raw)
  To: yuan.peng, dts; +Cc: Yu Jiang

pmdrss_hash: replace legacy filter with rte_flow

Yu Jiang (2):
  test_plans/pmdrss_hash: replace legacy filter with rte_flow
  tests/pmdrss_hash: replace legacy filter with rte_flow

 test_plans/pmdrss_hash_test_plan.rst | 46 ++++++++++++++++++++++++++-----
 tests/TestSuite_pmdrss_hash.py       | 52 +++++++++++++++++++++---------------
 2 files changed, 71 insertions(+), 27 deletions(-)

-- 
2.7.4


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

* [dts] [PATCH V3 1/2] test_plans/pmdrss_hash: replace legacy filter with rte_flow
  2021-11-08  7:09 [dts] [PATCH V3 0/2] v3 pmdrss_hash: replace legacy filter with rte_flow Yu Jiang
@ 2021-11-08  7:09 ` Yu Jiang
  2021-11-08  7:09 ` [dts] [PATCH V3 2/2] tests/pmdrss_hash: " Yu Jiang
  1 sibling, 0 replies; 9+ messages in thread
From: Yu Jiang @ 2021-11-08  7:09 UTC (permalink / raw)
  To: yuan.peng, dts; +Cc: Yu Jiang

According to dpdk commit 81db321da("ethdev: remove legacy HASH filter type support"),
remove legacy hash filter to use rte_flow api replace.

Signed-off-by: Yu Jiang <yux.jiang@intel.com>
---
 test_plans/pmdrss_hash_test_plan.rst | 46 +++++++++++++++++++++++++++++++-----
 1 file changed, 40 insertions(+), 6 deletions(-)

diff --git a/test_plans/pmdrss_hash_test_plan.rst b/test_plans/pmdrss_hash_test_plan.rst
index c0430a0..48a61e5 100644
--- a/test_plans/pmdrss_hash_test_plan.rst
+++ b/test_plans/pmdrss_hash_test_plan.rst
@@ -95,9 +95,20 @@ Testpmd configuration - 16 RX/TX queues per port
 
        testpmd command: set verbose 8
 
-#. set hash functions, can choose symmetric or not, choose port and packet type::
-
-       testpmd command: set_hash_global_config 0 toeplitz ipv4-other enable
+#. set hash functions, can choose symmetric or not, choose port, rss type and packet type::
+
+       Toeplitz hash function: toeplitz
+       testpmd command:
+         flow create <port_id> ingress pattern eth / ipv4 / sctp / end actions rss types ipv4-sctp end queues end func toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / end actions rss types ipv4-other end queues end func toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / end actions rss types ipv4-frag end queues end func toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / udp / end actions rss types ipv4-udp end queues end func toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / tcp / end actions rss types ipv4-tcp end queues end func toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / end actions rss types ipv6-other end queues end func toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / sctp / end actions rss types ipv6-sctp end queues end func toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / udp / end actions rss types ipv6-udp end queues end func toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / tcp / end actions rss types ipv6-tcp end queues end func toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / end actions rss types ipv6-frag end queues end func toeplitz queues end / end
 
 #. start packet receive::
 
@@ -129,8 +140,18 @@ Test Case:  test_toeplitz_symmetric
 
 The same with the above steps, pay attention to "set hash function", should use::
 
-  set_hash_global_config 0 toeplitz ipv4-other enable
-  set_sym_hash_ena_per_port 0 enable
+       Symmetric Toeplitz hash function: symmetric_toeplitz
+       testpmd command:
+         flow create <port_id> ingress pattern eth / ipv4 / sctp / end actions rss types ipv4-sctp end queues end func symmetric_toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / end actions rss types ipv4-other end queues end func symmetric_toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / end actions rss types ipv4-frag end queues end func symmetric_toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / udp / end actions rss types ipv4-udp end queues end func symmetric_toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / tcp / end actions rss types ipv4-tcp end queues end func symmetric_toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / end actions rss types ipv6-other end queues end func symmetric_toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / sctp / end actions rss types ipv6-sctp end queues end func symmetric_toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / udp / end actions rss types ipv6-udp end queues end func symmetric_toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / tcp / end actions rss types ipv6-tcp end queues end func symmetric_toeplitz queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / end actions rss types ipv6-frag end queues end func symmetric_toeplitz queues end / end
 
 And send packets with the same flow in different direction::
 
@@ -142,7 +163,20 @@ And the hash value and queue should be the same for these two flow .
 Test Case:  test_simple
 =======================
 
-The same as the above test case "test_toeplitz". Just pay attention to set the hash function to "simple xor"
+The same as the above test case "test_toeplitz". Just pay attention to set the hash function to "simple xor"::
+
+       simple XOR hash function: simple_xor
+       testpmd command:
+         flow create <port_id> ingress pattern eth / ipv4 / sctp / end actions rss types ipv4-sctp end queues end func simple_xor queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / end actions rss types ipv4-other end queues end func simple_xor queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / end actions rss types ipv4-frag end queues end func simple_xor queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / udp / end actions rss types ipv4-udp end queues end func simple_xor queues end / end
+         flow create <port_id> ingress pattern eth / ipv4 / tcp / end actions rss types ipv4-tcp end queues end func simple_xor queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / end actions rss types ipv6-other end queues end func simple_xor queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / sctp / end actions rss types ipv6-sctp end queues end func simple_xor queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / udp / end actions rss types ipv6-udp end queues end func simple_xor queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / tcp / end actions rss types ipv6-tcp end queues end func simple_xor queues end / end
+         flow create <port_id> ingress pattern eth / ipv6 / end actions rss types ipv6-frag end queues end func simple_xor queues end / end
 
 Test Case:  test_simple_symmetric
 =================================
-- 
2.7.4


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

* [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with rte_flow
  2021-11-08  7:09 [dts] [PATCH V3 0/2] v3 pmdrss_hash: replace legacy filter with rte_flow Yu Jiang
  2021-11-08  7:09 ` [dts] [PATCH V3 1/2] test_plans/pmdrss_hash: " Yu Jiang
@ 2021-11-08  7:09 ` Yu Jiang
  2021-11-08  7:21   ` Jiang, YuX
  2021-11-09  9:11   ` Tu, Lijuan
  1 sibling, 2 replies; 9+ messages in thread
From: Yu Jiang @ 2021-11-08  7:09 UTC (permalink / raw)
  To: yuan.peng, dts; +Cc: Yu Jiang

According to dpdk commit 81db321da("ethdev: remove legacy HASH filter type support"),
remove legacy hash filter to use rte_flow api replace.

Signed-off-by: Yu Jiang <yux.jiang@intel.com>
---
 tests/TestSuite_pmdrss_hash.py | 52 +++++++++++++++++++++++++-----------------
 1 file changed, 31 insertions(+), 21 deletions(-)

diff --git a/tests/TestSuite_pmdrss_hash.py b/tests/TestSuite_pmdrss_hash.py
index 2ff98cb..9eb17cf 100644
--- a/tests/TestSuite_pmdrss_hash.py
+++ b/tests/TestSuite_pmdrss_hash.py
@@ -41,6 +41,7 @@ import re
 import time
 
 import framework.utils as utils
+from framework.test_case import skip_unsupported_nic
 
 queue = 16
 reta_entries = []
@@ -411,33 +412,18 @@ class TestPmdrssHash(TestCase):
         self.verify(
             sum(result) == 0, "the symmetric RSS hash function failed!")
 
+    @skip_unsupported_nic(["columbiaville_25g", "columbiaville_100g", "niantic", "foxville"])
     def set_up_all(self):
         """
         Run at the start of each test suite.
         """
 
-        self.verify(self.nic in ["columbiaville_25g", "columbiaville_100g","fortville_eagle", "fortville_spirit",
-                    "fortville_spirit_single", "fortpark_TLV", "fortpark_BASE-T","fortville_25g", "niantic", "carlsville", "foxville"],
-                    "NIC Unsupported: " + str(self.nic))
         global reta_num
         global iptypes
         global queue
-        if self.nic in ["foxville"]:
-            queue = 4
 
         if self.nic in ["fortville_eagle", "fortville_spirit", "fortville_spirit_single", "fortpark_TLV", "fortpark_BASE-T","fortville_25g", "carlsville"]:
             reta_num = 512
-        elif self.nic in ["niantic", "foxville"]:
-            reta_num = 128
-            iptypes = {'ipv4-other': 'ip',
-                       'ipv4-frag': 'ip',
-                       'ipv4-udp': 'udp',
-                       'ipv4-tcp': 'tcp',
-                       'ipv6-other': 'ip',
-                       'ipv6-udp': 'udp',
-                       'ipv6-tcp': 'tcp',
-                       'ipv6-frag': 'ip'
-                       }
         else:
             self.verify(False, "NIC Unsupported:%s" % str(self.nic))
         ports = self.dut.get_ports(self.nic)
@@ -453,9 +439,13 @@ class TestPmdrssHash(TestCase):
         self.coremask = utils.create_mask(cores)
 
     def test_toeplitz(self):
+        """
+            Test Case:  test_toeplitz
+        """
         dutPorts = self.dut.get_ports(self.nic)
         localPort = self.tester.get_local_port(dutPorts[0])
         itf = self.tester.get_interface(localPort)
+        rule_action = 'func toeplitz queues end / end'
         global reta_num
         global iptypes
 
@@ -474,8 +464,14 @@ class TestPmdrssHash(TestCase):
                 "set nbcore %d" % (queue + 1), "testpmd> ")
 
             self.dut.send_expect("port stop all", "testpmd> ")
-            self.dut.send_expect(
-                "set_hash_global_config  0 toeplitz %s enable" % iptype, "testpmd> ")
+            self.dut.send_expect("flow flush 0", "testpmd> ")
+            rule_cmd = f'flow create 0 ingress pattern eth / ipv4 / end actions rss types {iptype} end queues end {rule_action}'
+            if 'sctp' in iptype or 'udp' in iptype or 'tcp' in iptype:
+                rule_cmd = rule_cmd.replace('/ ipv4 /', f'/ ipv4 / {rsstype} /')
+            if 'ipv6' in iptype:
+                rule_cmd = rule_cmd.replace('ipv4', 'ipv6')
+            outx = self.dut.send_expect(rule_cmd, "testpmd> ")
+            self.verify("created" in outx, "Create flow failed")
             self.dut.send_expect("port start all", "testpmd> ")
             out = self.dut.send_expect(
                 "port config all rss %s" % rsstype, "testpmd> ")
@@ -491,6 +487,9 @@ class TestPmdrssHash(TestCase):
         self.dut.send_expect("quit", "# ", 30)
 
     def test_toeplitz_symmetric(self):
+        """
+            Test Case:  test_toeplitz_symmetric
+        """
         dutPorts = self.dut.get_ports(self.nic)
         localPort = self.tester.get_local_port(dutPorts[0])
         itf = self.tester.get_interface(localPort)
@@ -519,7 +518,8 @@ class TestPmdrssHash(TestCase):
                 rule_cmd = rule_cmd.replace('/ ipv4 /', f'/ ipv4 / {rsstype} /')
             if 'ipv6' in iptype:
                 rule_cmd = rule_cmd.replace('ipv4', 'ipv6')
-            self.dut.send_expect(rule_cmd, "testpmd> ")
+            outx = self.dut.send_expect(rule_cmd, "testpmd> ")
+            self.verify("created" in outx, "Create flow failed")
             self.dut.send_expect("port start all", "testpmd> ")
             out = self.dut.send_expect(
                 "port config all rss %s" % rsstype, "testpmd> ")
@@ -537,9 +537,13 @@ class TestPmdrssHash(TestCase):
         self.dut.send_expect("quit", "# ", 30)
 
     def test_simple(self):
+        """
+            Test Case:  test_simple
+        """
         dutPorts = self.dut.get_ports(self.nic)
         localPort = self.tester.get_local_port(dutPorts[0])
         itf = self.tester.get_interface(localPort)
+        rule_action = 'func simple_xor queues end / end'
         global reta_num
         global iptypes
 
@@ -560,8 +564,14 @@ class TestPmdrssHash(TestCase):
 
             self.dut.send_expect("port stop all", "testpmd> ")
             # some nic not support change hash algorithm
-            self.dut.send_expect(
-                "set_hash_global_config 0 simple_xor %s enable" % iptype, "testpmd> ")
+            self.dut.send_expect("flow flush 0", "testpmd> ")
+            rule_cmd = f'flow create 0 ingress pattern eth / ipv4 / end actions rss types {iptype} end queues end {rule_action}'
+            if 'sctp' in iptype or 'udp' in iptype or 'tcp' in iptype:
+                rule_cmd = rule_cmd.replace('/ ipv4 /', f'/ ipv4 / {rsstype} /')
+            if 'ipv6' in iptype:
+                rule_cmd = rule_cmd.replace('ipv4', 'ipv6')
+            outx = self.dut.send_expect(rule_cmd, "testpmd> ")
+            self.verify("created" in outx, "Create flow failed")
             self.dut.send_expect("port start all", "testpmd> ")
             out = self.dut.send_expect(
                 "port config all rss %s" % rsstype, "testpmd> ")
-- 
2.7.4


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

* Re: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with rte_flow
  2021-11-08  7:09 ` [dts] [PATCH V3 2/2] tests/pmdrss_hash: " Yu Jiang
@ 2021-11-08  7:21   ` Jiang, YuX
  2021-11-09  9:11   ` Tu, Lijuan
  1 sibling, 0 replies; 9+ messages in thread
From: Jiang, YuX @ 2021-11-08  7:21 UTC (permalink / raw)
  To: Peng, Yuan, dts

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

> -----Original Message-----
> From: Jiang, YuX <yux.jiang@intel.com>
> Sent: Monday, November 8, 2021 3:10 PM
> To: Peng, Yuan <yuan.peng@intel.com>; dts@dpdk.org
> Cc: Jiang, YuX <yux.jiang@intel.com>
> Subject: [dts][PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with
> rte_flow
> 
> According to dpdk commit 81db321da("ethdev: remove legacy HASH filter
> type support"), remove legacy hash filter to use rte_flow api replace.
> 
> Signed-off-by: Yu Jiang <yux.jiang@intel.com>
Tested-by: Yu Jiang <yux.jiang@intel.com>

"pmdrss_hash/test_dynamic_rss_bond_config" Failed due to dpdk commit 
81db321da "ethdev: remove legacy HASH filter type support" 
do not support get_hash_global_config <port-id> cmd.

"x86_64-native-linuxapp-gcc": {
            "pmdrss_hash/test_dynamic_rss_bond_config": "failed",
            "pmdrss_hash/test_simple": "passed",
            "pmdrss_hash/test_simple_symmetric": "n/a",
            "pmdrss_hash/test_toeplitz": "passed",
            "pmdrss_hash/test_toeplitz_symmetric": "passed"
        }

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

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

* Re: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with rte_flow
  2021-11-08  7:09 ` [dts] [PATCH V3 2/2] tests/pmdrss_hash: " Yu Jiang
  2021-11-08  7:21   ` Jiang, YuX
@ 2021-11-09  9:11   ` Tu, Lijuan
  2021-11-09  9:22     ` Jiang, YuX
  1 sibling, 1 reply; 9+ messages in thread
From: Tu, Lijuan @ 2021-11-09  9:11 UTC (permalink / raw)
  To: Jiang, YuX, Peng, Yuan, dts; +Cc: Jiang, YuX

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Yu Jiang
> Sent: 2021年11月8日 15:10
> To: Peng, Yuan <yuan.peng@intel.com>; dts@dpdk.org
> Cc: Jiang, YuX <yux.jiang@intel.com>
> Subject: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with
> rte_flow
> 
> According to dpdk commit 81db321da("ethdev: remove legacy HASH filter type
> support"), remove legacy hash filter to use rte_flow api replace.
> 
> Signed-off-by: Yu Jiang <yux.jiang@intel.com>

Is there any other case impact by the dpdk commit ? if yes, please fix them too.

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

* Re: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with rte_flow
  2021-11-09  9:11   ` Tu, Lijuan
@ 2021-11-09  9:22     ` Jiang, YuX
  2021-11-09 12:03       ` Tu, Lijuan
  0 siblings, 1 reply; 9+ messages in thread
From: Jiang, YuX @ 2021-11-09  9:22 UTC (permalink / raw)
  To: Tu, Lijuan, Peng, Yuan, dts

> -----Original Message-----
> From: Tu, Lijuan <lijuan.tu@intel.com>
> Sent: Tuesday, November 9, 2021 5:11 PM
> To: Jiang, YuX <yux.jiang@intel.com>; Peng, Yuan <yuan.peng@intel.com>;
> dts@dpdk.org
> Cc: Jiang, YuX <yux.jiang@intel.com>
> Subject: RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter
> with rte_flow
> 
> > -----Original Message-----
> > From: dts <dts-bounces@dpdk.org> On Behalf Of Yu Jiang
> > Sent: 2021年11月8日 15:10
> > To: Peng, Yuan <yuan.peng@intel.com>; dts@dpdk.org
> > Cc: Jiang, YuX <yux.jiang@intel.com>
> > Subject: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter
> > with rte_flow
> >
> > According to dpdk commit 81db321da("ethdev: remove legacy HASH filter
> > type support"), remove legacy hash filter to use rte_flow api replace.
> >
> > Signed-off-by: Yu Jiang <yux.jiang@intel.com>
> 
> Is there any other case impact by the dpdk commit ? if yes, please fix them
> too.
Other two TCs will be removed from this plan and suite due to no cmd to support them any more.

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

* Re: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with rte_flow
  2021-11-09  9:22     ` Jiang, YuX
@ 2021-11-09 12:03       ` Tu, Lijuan
  2021-11-10 11:03         ` Jiang, YuX
  0 siblings, 1 reply; 9+ messages in thread
From: Tu, Lijuan @ 2021-11-09 12:03 UTC (permalink / raw)
  To: Jiang, YuX, Peng, Yuan, dts

> -----Original Message-----
> From: Jiang, YuX <yux.jiang@intel.com>
> Sent: 2021年11月9日 17:23
> To: Tu, Lijuan <lijuan.tu@intel.com>; Peng, Yuan <yuan.peng@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with
> rte_flow
> 
> > -----Original Message-----
> > From: Tu, Lijuan <lijuan.tu@intel.com>
> > Sent: Tuesday, November 9, 2021 5:11 PM
> > To: Jiang, YuX <yux.jiang@intel.com>; Peng, Yuan
> > <yuan.peng@intel.com>; dts@dpdk.org
> > Cc: Jiang, YuX <yux.jiang@intel.com>
> > Subject: RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy
> > filter with rte_flow
> >
> > > -----Original Message-----
> > > From: dts <dts-bounces@dpdk.org> On Behalf Of Yu Jiang
> > > Sent: 2021年11月8日 15:10
> > > To: Peng, Yuan <yuan.peng@intel.com>; dts@dpdk.org
> > > Cc: Jiang, YuX <yux.jiang@intel.com>
> > > Subject: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy
> > > filter with rte_flow
> > >
> > > According to dpdk commit 81db321da("ethdev: remove legacy HASH
> > > filter type support"), remove legacy hash filter to use rte_flow api replace.
> > >
> > > Signed-off-by: Yu Jiang <yux.jiang@intel.com>
> >
> > Is there any other case impact by the dpdk commit ? if yes, please fix
> > them too.
> Other two TCs will be removed from this plan and suite due to no cmd to
> support them any more.

Series applied


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

* Re: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with rte_flow
  2021-11-09 12:03       ` Tu, Lijuan
@ 2021-11-10 11:03         ` Jiang, YuX
  2021-11-10 15:07           ` Tu, Lijuan
  0 siblings, 1 reply; 9+ messages in thread
From: Jiang, YuX @ 2021-11-10 11:03 UTC (permalink / raw)
  To: Tu, Lijuan, Peng, Yuan, dts

> -----Original Message-----
> From: Tu, Lijuan <lijuan.tu@intel.com>
> Sent: Tuesday, November 9, 2021 8:04 PM
> To: Jiang, YuX <yux.jiang@intel.com>; Peng, Yuan <yuan.peng@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter
> with rte_flow
> 
> > -----Original Message-----
> > From: Jiang, YuX <yux.jiang@intel.com>
> > Sent: 2021年11月9日 17:23
> > To: Tu, Lijuan <lijuan.tu@intel.com>; Peng, Yuan
> > <yuan.peng@intel.com>; dts@dpdk.org
> > Subject: RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy
> > filter with rte_flow
> >
> > > -----Original Message-----
> > > From: Tu, Lijuan <lijuan.tu@intel.com>
> > > Sent: Tuesday, November 9, 2021 5:11 PM
> > > To: Jiang, YuX <yux.jiang@intel.com>; Peng, Yuan
> > > <yuan.peng@intel.com>; dts@dpdk.org
> > > Cc: Jiang, YuX <yux.jiang@intel.com>
> > > Subject: RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy
> > > filter with rte_flow
> > >
> > > > -----Original Message-----
> > > > From: dts <dts-bounces@dpdk.org> On Behalf Of Yu Jiang
> > > > Sent: 2021年11月8日 15:10
> > > > To: Peng, Yuan <yuan.peng@intel.com>; dts@dpdk.org
> > > > Cc: Jiang, YuX <yux.jiang@intel.com>
> > > > Subject: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy
> > > > filter with rte_flow
> > > >
> > > > According to dpdk commit 81db321da("ethdev: remove legacy HASH
> > > > filter type support"), remove legacy hash filter to use rte_flow api
> replace.
> > > >
> > > > Signed-off-by: Yu Jiang <yux.jiang@intel.com>
> > >
> > > Is there any other case impact by the dpdk commit ? if yes, please
> > > fix them too.
> > Other two TCs will be removed from this plan and suite due to no cmd
> > to support them any more.
> 
> Series applied
> 
Seems that series failed to apply, I check the latest dts has no such code....., Could you pls have a look?

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

* RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with rte_flow
  2021-11-10 11:03         ` Jiang, YuX
@ 2021-11-10 15:07           ` Tu, Lijuan
  0 siblings, 0 replies; 9+ messages in thread
From: Tu, Lijuan @ 2021-11-10 15:07 UTC (permalink / raw)
  To: Jiang, YuX, Peng, Yuan, dts



> -----Original Message-----
> From: Jiang, YuX <yux.jiang@intel.com>
> Sent: 2021年11月10日 19:03
> To: Tu, Lijuan <lijuan.tu@intel.com>; Peng, Yuan <yuan.peng@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy filter with
> rte_flow
> 
> > -----Original Message-----
> > From: Tu, Lijuan <lijuan.tu@intel.com>
> > Sent: Tuesday, November 9, 2021 8:04 PM
> > To: Jiang, YuX <yux.jiang@intel.com>; Peng, Yuan
> > <yuan.peng@intel.com>; dts@dpdk.org
> > Subject: RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy
> > filter with rte_flow
> >
> > > -----Original Message-----
> > > From: Jiang, YuX <yux.jiang@intel.com>
> > > Sent: 2021年11月9日 17:23
> > > To: Tu, Lijuan <lijuan.tu@intel.com>; Peng, Yuan
> > > <yuan.peng@intel.com>; dts@dpdk.org
> > > Subject: RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy
> > > filter with rte_flow
> > >
> > > > -----Original Message-----
> > > > From: Tu, Lijuan <lijuan.tu@intel.com>
> > > > Sent: Tuesday, November 9, 2021 5:11 PM
> > > > To: Jiang, YuX <yux.jiang@intel.com>; Peng, Yuan
> > > > <yuan.peng@intel.com>; dts@dpdk.org
> > > > Cc: Jiang, YuX <yux.jiang@intel.com>
> > > > Subject: RE: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace
> > > > legacy filter with rte_flow
> > > >
> > > > > -----Original Message-----
> > > > > From: dts <dts-bounces@dpdk.org> On Behalf Of Yu Jiang
> > > > > Sent: 2021年11月8日 15:10
> > > > > To: Peng, Yuan <yuan.peng@intel.com>; dts@dpdk.org
> > > > > Cc: Jiang, YuX <yux.jiang@intel.com>
> > > > > Subject: [dts] [PATCH V3 2/2] tests/pmdrss_hash: replace legacy
> > > > > filter with rte_flow
> > > > >
> > > > > According to dpdk commit 81db321da("ethdev: remove legacy HASH
> > > > > filter type support"), remove legacy hash filter to use rte_flow
> > > > > api
> > replace.
> > > > >
> > > > > Signed-off-by: Yu Jiang <yux.jiang@intel.com>
> > > >
> > > > Is there any other case impact by the dpdk commit ? if yes, please
> > > > fix them too.
> > > Other two TCs will be removed from this plan and suite due to no cmd
> > > to support them any more.
> >
> > Series applied
> >
> Seems that series failed to apply, I check the latest dts has no such code.....,
> Could you pls have a look?

It show works now, it got some network issues yesterday and now it fixed.


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

end of thread, other threads:[~2021-11-10 15:09 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-08  7:09 [dts] [PATCH V3 0/2] v3 pmdrss_hash: replace legacy filter with rte_flow Yu Jiang
2021-11-08  7:09 ` [dts] [PATCH V3 1/2] test_plans/pmdrss_hash: " Yu Jiang
2021-11-08  7:09 ` [dts] [PATCH V3 2/2] tests/pmdrss_hash: " Yu Jiang
2021-11-08  7:21   ` Jiang, YuX
2021-11-09  9:11   ` Tu, Lijuan
2021-11-09  9:22     ` Jiang, YuX
2021-11-09 12:03       ` Tu, Lijuan
2021-11-10 11:03         ` Jiang, YuX
2021-11-10 15:07           ` Tu, Lijuan

test suite reviews and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://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/ http://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