test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] update rxmode test config
@ 2016-12-29  6:39 xu,huilong
  2017-01-04  2:15 ` Liu, Yong
  0 siblings, 1 reply; 2+ messages in thread
From: xu,huilong @ 2016-12-29  6:39 UTC (permalink / raw)
  To: dts; +Cc: xu,huilong

dpdk ixgbe/i40e/fm10k update rx/tx model alrady. so need update rx model test config

Signed-off-by: xu,huilong <huilongx.xu@intel.com>
---
 framework/project_dpdk.py | 32 ++++++++++++++++++--------------
 1 file changed, 18 insertions(+), 14 deletions(-)

diff --git a/framework/project_dpdk.py b/framework/project_dpdk.py
index 067590b..5f10c8b 100644
--- a/framework/project_dpdk.py
+++ b/framework/project_dpdk.py
@@ -132,26 +132,30 @@ class DPDKdut(Dut):
 
     def set_rxtx_mode(self):
         """
-        Set default RX/TX PMD function, now only take effect on ixgbe.
+        Set default RX/TX PMD function,
+        only i40e support scalar/full RX/TX model.
+        ixgbe and fm10k only support vector and no vector model
+        all NIC default rx/tx model is vector PMD
         """
-        [arch, machine, env, toolchain] = self.target.split('-')
 
         mode = load_global_setting(DPDK_RXMODE_SETTING)
         if mode == 'scalar':
-            self.send_expect("sed -i -e 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
-                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
-            self.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=.*$/"
-                             + "CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=y/' config/common_%s" % env, "# ", 30)
+            self.send_expect("sed -i -e 's/CONFIG_RTE_I40E_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_I40E_INC_VECTOR=n/' config/common_base", "# ", 30)
+            self.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC=.*$/"
+                             + "CONFIG_RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC=y/' config/common_base", "# ", 30)
         if mode == 'full':
+            self.send_expect("sed -i -e 's/CONFIG_RTE_I40E_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_I40E_INC_VECTOR=n/' config/common_base", "# ", 30)
+            self.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC=.*$/"
+                             + "CONFIG_RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC=n/' config/common_base", "# ", 30)
+        if mode == 'novector':
             self.send_expect("sed -i -e 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
-                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
-            self.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=.*$/"
-                             + "CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=n/' config/common_%s" % env, "# ", 30)
-        if mode == 'vector':
-            self.send_expect("sed -i -e 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
-                             + "CONFIG_RTE_IXGBE_INC_VECTOR=y/' config/common_%s" % env, "# ", 30)
-            self.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=.*$/"
-                             + "CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=y/' config/common_%s" % env, "# ", 30)
+                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/' config/common_base", "# ", 30)
+            self.send_expect("sed -i -e 's/CONFIG_RTE_I40E_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_I40E_INC_VECTOR=n/' config/common_base", "# ", 30)
+            self.send_expect("sed -i -e 's/CONFIG_RTE_FM10K_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_FM10K_INC_VECTOR=n/' config/common_base", "# ", 30)
 
     def set_package(self, pkg_name="", patch_list=[]):
         self.package = pkg_name
-- 
1.9.3

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

* Re: [dts] [PATCH V1] update rxmode test config
  2016-12-29  6:39 [dts] [PATCH V1] update rxmode test config xu,huilong
@ 2017-01-04  2:15 ` Liu, Yong
  0 siblings, 0 replies; 2+ messages in thread
From: Liu, Yong @ 2017-01-04  2:15 UTC (permalink / raw)
  To: Xu, HuilongX, dts; +Cc: Xu, HuilongX

Thanks, applied with comment modified.

-----Original Message-----
From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of xu,huilong
Sent: Thursday, December 29, 2016 2:40 PM
To: dts@dpdk.org
Cc: Xu, HuilongX <huilongx.xu@intel.com>
Subject: [dts] [PATCH V1] update rxmode test config

dpdk ixgbe/i40e/fm10k update rx/tx model alrady. so need update rx model test config

Signed-off-by: xu,huilong <huilongx.xu@intel.com>
---
 framework/project_dpdk.py | 32 ++++++++++++++++++--------------
 1 file changed, 18 insertions(+), 14 deletions(-)

diff --git a/framework/project_dpdk.py b/framework/project_dpdk.py index 067590b..5f10c8b 100644
--- a/framework/project_dpdk.py
+++ b/framework/project_dpdk.py
@@ -132,26 +132,30 @@ class DPDKdut(Dut):
 
     def set_rxtx_mode(self):
         """
-        Set default RX/TX PMD function, now only take effect on ixgbe.
+        Set default RX/TX PMD function,
+        only i40e support scalar/full RX/TX model.
+        ixgbe and fm10k only support vector and no vector model
+        all NIC default rx/tx model is vector PMD
         """
-        [arch, machine, env, toolchain] = self.target.split('-')
 
         mode = load_global_setting(DPDK_RXMODE_SETTING)
         if mode == 'scalar':
-            self.send_expect("sed -i -e 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
-                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
-            self.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=.*$/"
-                             + "CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=y/' config/common_%s" % env, "# ", 30)
+            self.send_expect("sed -i -e 's/CONFIG_RTE_I40E_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_I40E_INC_VECTOR=n/' config/common_base", "# ", 30)
+            self.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC=.*$/"
+                             + 
+ "CONFIG_RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC=y/' config/common_base", 
+ "# ", 30)
         if mode == 'full':
+            self.send_expect("sed -i -e 's/CONFIG_RTE_I40E_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_I40E_INC_VECTOR=n/' config/common_base", "# ", 30)
+            self.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC=.*$/"
+                             + "CONFIG_RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC=n/' config/common_base", "# ", 30)
+        if mode == 'novector':
             self.send_expect("sed -i -e 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
-                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
-            self.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=.*$/"
-                             + "CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=n/' config/common_%s" % env, "# ", 30)
-        if mode == 'vector':
-            self.send_expect("sed -i -e 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
-                             + "CONFIG_RTE_IXGBE_INC_VECTOR=y/' config/common_%s" % env, "# ", 30)
-            self.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=.*$/"
-                             + "CONFIG_RTE_LIBRTE_IXGBE_RX_ALLOW_BULK_ALLOC=y/' config/common_%s" % env, "# ", 30)
+                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/' config/common_base", "# ", 30)
+            self.send_expect("sed -i -e 's/CONFIG_RTE_I40E_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_I40E_INC_VECTOR=n/' config/common_base", "# ", 30)
+            self.send_expect("sed -i -e 's/CONFIG_RTE_FM10K_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_FM10K_INC_VECTOR=n/' 
+ config/common_base", "# ", 30)
 
     def set_package(self, pkg_name="", patch_list=[]):
         self.package = pkg_name
--
1.9.3

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

end of thread, other threads:[~2017-01-04  2:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-29  6:39 [dts] [PATCH V1] update rxmode test config xu,huilong
2017-01-04  2:15 ` Liu, Yong

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