test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH v2]tests uni_pkt: disable vector ixgbe driver and rename i350 to powerville
@ 2015-12-01  7:52 Lijuan Tu
  2015-12-02  1:12 ` Liu, Yong
  0 siblings, 1 reply; 4+ messages in thread
From: Lijuan Tu @ 2015-12-01  7:52 UTC (permalink / raw)
  To: dts

Signed-off-by: Lijuan Tu <lijuanx.a.tu@intel.com>
---
 tests/TestSuite_uni_pkt.py | 32 +++++++++++++++++++++++++++-----
 1 file changed, 27 insertions(+), 5 deletions(-)

diff --git a/tests/TestSuite_uni_pkt.py b/tests/TestSuite_uni_pkt.py
index ca72e75..60756c9 100644
--- a/tests/TestSuite_uni_pkt.py
+++ b/tests/TestSuite_uni_pkt.py
@@ -63,6 +63,17 @@ class TestUniPacket(TestCase):
         self.dut_port = valports[0]
         tester_port = self.tester.get_local_port(self.dut_port)
         self.tester_iface = self.tester.get_interface(tester_port)
+        """
+        Change config file to disable vector pmd and recompile the package.
+        """
+        [arch, machine, env, toolchain] = self.target.split('-')
+        if "fortville" in self.nic.lower():
+            self.dut.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
+        elif "niantic" in self.nic.lower():
+            self.dut.send_expect("sed -i -e 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
+        self.dut.build_install_dpdk(self.target)
         self.dut.send_expect(
             "./%s/app/testpmd -c f -n 4 -- -i --txqflags=0x0" % self.target, "testpmd>", 20)
         self.dut.send_expect("set fwd rxonly", "testpmd>")
@@ -117,7 +128,7 @@ class TestUniPacket(TestCase):
         """
         if "fortville" in self.nic.lower():
             outerL4Type = "(outer) L4 type: L4_NONFRAG"
-        elif "niantic" in self.nic.lower() or "i350" in self.nic.lower():
+        elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower():
             outerL4Type = "(outer) L4 type: Unknown"
 
         pktType = {
@@ -135,7 +146,7 @@ class TestUniPacket(TestCase):
         if "fortville" in self.nic.lower():
             pktType.pop("MAC_IPihl_PKT")
             pktType.pop("MAC_IPihl_SCTP_PKT")
-        elif "niantic" in self.nic.lower() or "i350" in self.nic.lower():
+        elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower():
             pktType.pop("MAC_IP_ICMP_PKT")
             pktType.pop("MAC_IPFRAG_TCP_PKT")
 
@@ -148,7 +159,7 @@ class TestUniPacket(TestCase):
         if "fortville" in self.nic.lower():
             outerL4Type = "(outer) L4 type: L4_NONFRAG"
             outerL3Type = "(outer) L3 type: IPV6_EXT_UNKNOWN"
-        elif "niantic" in self.nic.lower() or "i350" in self.nic.lower():
+        elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower():
             outerL4Type = "(outer) L4 type: Unknown"
             outerL3Type = "(outer) L3 type: IPV6"
 
@@ -163,7 +174,7 @@ class TestUniPacket(TestCase):
         # delete the unsupported packet based on nic type
         if "fortville" in self.nic.lower():
             pktType.pop("MAC_IPv6FRAG_PKT_N")
-        elif "niantic" in self.nic.lower() or "i350" in self.nic.lower():
+        elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower():
             pktType.pop("MAC_IPv6FRAG_PKT_F")
 
         self.run_test(pktType)
@@ -198,7 +209,7 @@ class TestUniPacket(TestCase):
         checked that whether IPv4 in IPv6 tunnel packet can be normally
         detected by Niantic and i350.
         """
-        if "niantic" not in self.nic.lower() and "i350" not in self.nic.lower():
+        if "niantic" not in self.nic.lower() and "powerville" not in self.nic.lower():
             return
 
         pktType = {
@@ -361,4 +372,15 @@ class TestUniPacket(TestCase):
         Nothing to do.
         """
         self.dut.kill_all()
+        """
+        Restore the config file and recompile the package.
+        """
+        [arch, machine, env, toolchain] = self.target.split('-')
+        if "fortville" in self.nic.lower():
+            self.dut.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
+        elif "niantic" in self.nic.lower():
+            self.dut.send_expect("sed -i -e 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
+                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
+        self.dut.build_install_dpdk(self.target)
         pass
-- 
1.9.1

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

* Re: [dts] [PATCH v2]tests uni_pkt: disable vector ixgbe driver and rename i350 to powerville
  2015-12-01  7:52 [dts] [PATCH v2]tests uni_pkt: disable vector ixgbe driver and rename i350 to powerville Lijuan Tu
@ 2015-12-02  1:12 ` Liu, Yong
  2015-12-02  1:19   ` Tu, LijuanX A
  0 siblings, 1 reply; 4+ messages in thread
From: Liu, Yong @ 2015-12-02  1:12 UTC (permalink / raw)
  To: Lijuan Tu, dts

Hi Lijuan,
One comment about change rx/tx code path.

On 12/01/2015 03:52 PM, Lijuan Tu wrote:
> Signed-off-by: Lijuan Tu <lijuanx.a.tu@intel.com>
> ---
>   tests/TestSuite_uni_pkt.py | 32 +++++++++++++++++++++++++++-----
>   1 file changed, 27 insertions(+), 5 deletions(-)
>
> diff --git a/tests/TestSuite_uni_pkt.py b/tests/TestSuite_uni_pkt.py
> index ca72e75..60756c9 100644
> --- a/tests/TestSuite_uni_pkt.py
> +++ b/tests/TestSuite_uni_pkt.py
> @@ -63,6 +63,17 @@ class TestUniPacket(TestCase):
>           self.dut_port = valports[0]
>           tester_port = self.tester.get_local_port(self.dut_port)
>           self.tester_iface = self.tester.get_interface(tester_port)
> +        """
> +        Change config file to disable vector pmd and recompile the package.
> +        """
> +        [arch, machine, env, toolchain] = self.target.split('-')
> +        if "fortville" in self.nic.lower():
> +            self.dut.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=.*$/"
> +                             + "CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
> +        elif "niantic" in self.nic.lower():
> +            self.dut.send_expect("sed -i -e 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
> +                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
> +        self.dut.build_install_dpdk(self.target)
>           self.dut.send_expect(
>               "./%s/app/testpmd -c f -n 4 -- -i --txqflags=0x0" % self.target, "testpmd>", 20)
>           self.dut.send_expect("set fwd rxonly", "testpmd>")
> @@ -117,7 +128,7 @@ class TestUniPacket(TestCase):
>           """
>           if "fortville" in self.nic.lower():
>               outerL4Type = "(outer) L4 type: L4_NONFRAG"
> -        elif "niantic" in self.nic.lower() or "i350" in self.nic.lower():
> +        elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower():
>               outerL4Type = "(outer) L4 type: Unknown"
>   
>           pktType = {
> @@ -135,7 +146,7 @@ class TestUniPacket(TestCase):
>           if "fortville" in self.nic.lower():
>               pktType.pop("MAC_IPihl_PKT")
>               pktType.pop("MAC_IPihl_SCTP_PKT")
> -        elif "niantic" in self.nic.lower() or "i350" in self.nic.lower():
> +        elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower():
>               pktType.pop("MAC_IP_ICMP_PKT")
>               pktType.pop("MAC_IPFRAG_TCP_PKT")
>   
> @@ -148,7 +159,7 @@ class TestUniPacket(TestCase):
>           if "fortville" in self.nic.lower():
>               outerL4Type = "(outer) L4 type: L4_NONFRAG"
>               outerL3Type = "(outer) L3 type: IPV6_EXT_UNKNOWN"
> -        elif "niantic" in self.nic.lower() or "i350" in self.nic.lower():
> +        elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower():
>               outerL4Type = "(outer) L4 type: Unknown"
>               outerL3Type = "(outer) L3 type: IPV6"
>   
> @@ -163,7 +174,7 @@ class TestUniPacket(TestCase):
>           # delete the unsupported packet based on nic type
>           if "fortville" in self.nic.lower():
>               pktType.pop("MAC_IPv6FRAG_PKT_N")
> -        elif "niantic" in self.nic.lower() or "i350" in self.nic.lower():
> +        elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower():
>               pktType.pop("MAC_IPv6FRAG_PKT_F")
>   
>           self.run_test(pktType)
> @@ -198,7 +209,7 @@ class TestUniPacket(TestCase):
>           checked that whether IPv4 in IPv6 tunnel packet can be normally
>           detected by Niantic and i350.
>           """
> -        if "niantic" not in self.nic.lower() and "i350" not in self.nic.lower():
> +        if "niantic" not in self.nic.lower() and "powerville" not in self.nic.lower():
>               return
>   
>           pktType = {
> @@ -361,4 +372,15 @@ class TestUniPacket(TestCase):
>           Nothing to do.
>           """
>           self.dut.kill_all()
> +        """
> +        Restore the config file and recompile the package.
> +        """
> +        [arch, machine, env, toolchain] = self.target.split('-')
> +        if "fortville" in self.nic.lower():
> +            self.dut.send_expect("sed -i -e 's/CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=.*$/"
> +                             + "CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
> +        elif "niantic" in self.nic.lower():
> +            self.dut.send_expect("sed -i -e 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
> +                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/' config/common_%s" % env, "# ", 30)
> +        self.dut.build_install_dpdk(self.target)
Here is still disable vector mode. There's other approach is that use 
dts.rx_mode which is stripped from execution configuration file.
Only when dts.rx_mode is "default" or "vector", this suite need change 
dpdk configuration.


>           pass

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

* Re: [dts] [PATCH v2]tests uni_pkt: disable vector ixgbe driver and rename i350 to powerville
  2015-12-02  1:12 ` Liu, Yong
@ 2015-12-02  1:19   ` Tu, LijuanX A
  2015-12-02  1:26     ` Liu, Yong
  0 siblings, 1 reply; 4+ messages in thread
From: Tu, LijuanX A @ 2015-12-02  1:19 UTC (permalink / raw)
  To: Liu, Yong, dts

Hi yong,


> -----Original Message-----
> From: Liu, Yong
> Sent: Wednesday, December 02, 2015 9:12 AM
> To: Tu, LijuanX A; dts@dpdk.org
> Subject: Re: [dts] [PATCH v2]tests uni_pkt: disable vector ixgbe driver
> and rename i350 to powerville
> 
> Hi Lijuan,
> One comment about change rx/tx code path.
> 
> On 12/01/2015 03:52 PM, Lijuan Tu wrote:
> > Signed-off-by: Lijuan Tu <lijuanx.a.tu@intel.com>
> > ---
> >   tests/TestSuite_uni_pkt.py | 32 +++++++++++++++++++++++++++-----
> >   1 file changed, 27 insertions(+), 5 deletions(-)
> >
> > diff --git a/tests/TestSuite_uni_pkt.py b/tests/TestSuite_uni_pkt.py
> > index ca72e75..60756c9 100644
> > --- a/tests/TestSuite_uni_pkt.py
> > +++ b/tests/TestSuite_uni_pkt.py
> > @@ -63,6 +63,17 @@ class TestUniPacket(TestCase):
> >           self.dut_port = valports[0]
> >           tester_port = self.tester.get_local_port(self.dut_port)
> >           self.tester_iface = self.tester.get_interface(tester_port)
> > +        """
> > +        Change config file to disable vector pmd and recompile the
> package.
> > +        """
> > +        [arch, machine, env, toolchain] = self.target.split('-')
> > +        if "fortville" in self.nic.lower():
> > +            self.dut.send_expect("sed -i -e
> 's/CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=.*$/"
> > +                             +
> "CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=n/' config/common_%s" % env, "# ",
> 30)
> > +        elif "niantic" in self.nic.lower():
> > +            self.dut.send_expect("sed -i -e
> 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
> > +                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/'
> config/common_%s" % env, "# ", 30)
> > +        self.dut.build_install_dpdk(self.target)
> >           self.dut.send_expect(
> >               "./%s/app/testpmd -c f -n 4 -- -i --txqflags=0x0" %
> self.target, "testpmd>", 20)
> >           self.dut.send_expect("set fwd rxonly", "testpmd>") @@ -
> 117,7
> > +128,7 @@ class TestUniPacket(TestCase):
> >           """
> >           if "fortville" in self.nic.lower():
> >               outerL4Type = "(outer) L4 type: L4_NONFRAG"
> > -        elif "niantic" in self.nic.lower() or "i350" in
> self.nic.lower():
> > +        elif "niantic" in self.nic.lower() or "powerville" in
> self.nic.lower():
> >               outerL4Type = "(outer) L4 type: Unknown"
> >
> >           pktType = {
> > @@ -135,7 +146,7 @@ class TestUniPacket(TestCase):
> >           if "fortville" in self.nic.lower():
> >               pktType.pop("MAC_IPihl_PKT")
> >               pktType.pop("MAC_IPihl_SCTP_PKT")
> > -        elif "niantic" in self.nic.lower() or "i350" in
> self.nic.lower():
> > +        elif "niantic" in self.nic.lower() or "powerville" in
> self.nic.lower():
> >               pktType.pop("MAC_IP_ICMP_PKT")
> >               pktType.pop("MAC_IPFRAG_TCP_PKT")
> >
> > @@ -148,7 +159,7 @@ class TestUniPacket(TestCase):
> >           if "fortville" in self.nic.lower():
> >               outerL4Type = "(outer) L4 type: L4_NONFRAG"
> >               outerL3Type = "(outer) L3 type: IPV6_EXT_UNKNOWN"
> > -        elif "niantic" in self.nic.lower() or "i350" in
> self.nic.lower():
> > +        elif "niantic" in self.nic.lower() or "powerville" in
> self.nic.lower():
> >               outerL4Type = "(outer) L4 type: Unknown"
> >               outerL3Type = "(outer) L3 type: IPV6"
> >
> > @@ -163,7 +174,7 @@ class TestUniPacket(TestCase):
> >           # delete the unsupported packet based on nic type
> >           if "fortville" in self.nic.lower():
> >               pktType.pop("MAC_IPv6FRAG_PKT_N")
> > -        elif "niantic" in self.nic.lower() or "i350" in
> self.nic.lower():
> > +        elif "niantic" in self.nic.lower() or "powerville" in
> self.nic.lower():
> >               pktType.pop("MAC_IPv6FRAG_PKT_F")
> >
> >           self.run_test(pktType)
> > @@ -198,7 +209,7 @@ class TestUniPacket(TestCase):
> >           checked that whether IPv4 in IPv6 tunnel packet can be
> normally
> >           detected by Niantic and i350.
> >           """
> > -        if "niantic" not in self.nic.lower() and "i350" not in
> self.nic.lower():
> > +        if "niantic" not in self.nic.lower() and "powerville" not in
> self.nic.lower():
> >               return
> >
> >           pktType = {
> > @@ -361,4 +372,15 @@ class TestUniPacket(TestCase):
> >           Nothing to do.
> >           """
> >           self.dut.kill_all()
> > +        """
> > +        Restore the config file and recompile the package.
> > +        """
> > +        [arch, machine, env, toolchain] = self.target.split('-')
> > +        if "fortville" in self.nic.lower():
> > +            self.dut.send_expect("sed -i -e
> 's/CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=.*$/"
> > +                             +
> "CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=n/' config/common_%s" % env, "# ",
> 30)
> > +        elif "niantic" in self.nic.lower():
> > +            self.dut.send_expect("sed -i -e
> 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
> > +                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/'
> config/common_%s" % env, "# ", 30)
> > +        self.dut.build_install_dpdk(self.target)
> Here is still disable vector mode. There's other approach is that use
> dts.rx_mode which is stripped from execution configuration file.
> Only when dts.rx_mode is "default" or "vector", this suite need change
> dpdk configuration.
In  i40e driver , the default of " vector "  is "n" in dpdk configuration file
And in ixgbe driver, the default of " vector "  is "y" in dpdk configuration file
> 
> 
> >           pass

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

* Re: [dts] [PATCH v2]tests uni_pkt: disable vector ixgbe driver and rename i350 to powerville
  2015-12-02  1:19   ` Tu, LijuanX A
@ 2015-12-02  1:26     ` Liu, Yong
  0 siblings, 0 replies; 4+ messages in thread
From: Liu, Yong @ 2015-12-02  1:26 UTC (permalink / raw)
  To: Tu, LijuanX A, dts

Lijuan, dpdk has default setting for Rx code method. But its mode can be changed by dts execution file.
So I mentioned that suite should depend on dts.rx_mode not dpdk default setting.

> -----Original Message-----
> From: Tu, LijuanX A
> Sent: Wednesday, December 02, 2015 9:20 AM
> To: Liu, Yong; dts@dpdk.org
> Subject: RE: [dts] [PATCH v2]tests uni_pkt: disable vector ixgbe driver
> and rename i350 to powerville
> 
> Hi yong,
> 
> 
> > -----Original Message-----
> > From: Liu, Yong
> > Sent: Wednesday, December 02, 2015 9:12 AM
> > To: Tu, LijuanX A; dts@dpdk.org
> > Subject: Re: [dts] [PATCH v2]tests uni_pkt: disable vector ixgbe driver
> > and rename i350 to powerville
> >
> > Hi Lijuan,
> > One comment about change rx/tx code path.
> >
> > On 12/01/2015 03:52 PM, Lijuan Tu wrote:
> > > Signed-off-by: Lijuan Tu <lijuanx.a.tu@intel.com>
> > > ---
> > >   tests/TestSuite_uni_pkt.py | 32 +++++++++++++++++++++++++++-----
> > >   1 file changed, 27 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/tests/TestSuite_uni_pkt.py b/tests/TestSuite_uni_pkt.py
> > > index ca72e75..60756c9 100644
> > > --- a/tests/TestSuite_uni_pkt.py
> > > +++ b/tests/TestSuite_uni_pkt.py
> > > @@ -63,6 +63,17 @@ class TestUniPacket(TestCase):
> > >           self.dut_port = valports[0]
> > >           tester_port = self.tester.get_local_port(self.dut_port)
> > >           self.tester_iface = self.tester.get_interface(tester_port)
> > > +        """
> > > +        Change config file to disable vector pmd and recompile the
> > package.
> > > +        """
> > > +        [arch, machine, env, toolchain] = self.target.split('-')
> > > +        if "fortville" in self.nic.lower():
> > > +            self.dut.send_expect("sed -i -e
> > 's/CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=.*$/"
> > > +                             +
> > "CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=n/' config/common_%s" % env, "# ",
> > 30)
> > > +        elif "niantic" in self.nic.lower():
> > > +            self.dut.send_expect("sed -i -e
> > 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
> > > +                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/'
> > config/common_%s" % env, "# ", 30)
> > > +        self.dut.build_install_dpdk(self.target)
> > >           self.dut.send_expect(
> > >               "./%s/app/testpmd -c f -n 4 -- -i --txqflags=0x0" %
> > self.target, "testpmd>", 20)
> > >           self.dut.send_expect("set fwd rxonly", "testpmd>") @@ -
> > 117,7
> > > +128,7 @@ class TestUniPacket(TestCase):
> > >           """
> > >           if "fortville" in self.nic.lower():
> > >               outerL4Type = "(outer) L4 type: L4_NONFRAG"
> > > -        elif "niantic" in self.nic.lower() or "i350" in
> > self.nic.lower():
> > > +        elif "niantic" in self.nic.lower() or "powerville" in
> > self.nic.lower():
> > >               outerL4Type = "(outer) L4 type: Unknown"
> > >
> > >           pktType = {
> > > @@ -135,7 +146,7 @@ class TestUniPacket(TestCase):
> > >           if "fortville" in self.nic.lower():
> > >               pktType.pop("MAC_IPihl_PKT")
> > >               pktType.pop("MAC_IPihl_SCTP_PKT")
> > > -        elif "niantic" in self.nic.lower() or "i350" in
> > self.nic.lower():
> > > +        elif "niantic" in self.nic.lower() or "powerville" in
> > self.nic.lower():
> > >               pktType.pop("MAC_IP_ICMP_PKT")
> > >               pktType.pop("MAC_IPFRAG_TCP_PKT")
> > >
> > > @@ -148,7 +159,7 @@ class TestUniPacket(TestCase):
> > >           if "fortville" in self.nic.lower():
> > >               outerL4Type = "(outer) L4 type: L4_NONFRAG"
> > >               outerL3Type = "(outer) L3 type: IPV6_EXT_UNKNOWN"
> > > -        elif "niantic" in self.nic.lower() or "i350" in
> > self.nic.lower():
> > > +        elif "niantic" in self.nic.lower() or "powerville" in
> > self.nic.lower():
> > >               outerL4Type = "(outer) L4 type: Unknown"
> > >               outerL3Type = "(outer) L3 type: IPV6"
> > >
> > > @@ -163,7 +174,7 @@ class TestUniPacket(TestCase):
> > >           # delete the unsupported packet based on nic type
> > >           if "fortville" in self.nic.lower():
> > >               pktType.pop("MAC_IPv6FRAG_PKT_N")
> > > -        elif "niantic" in self.nic.lower() or "i350" in
> > self.nic.lower():
> > > +        elif "niantic" in self.nic.lower() or "powerville" in
> > self.nic.lower():
> > >               pktType.pop("MAC_IPv6FRAG_PKT_F")
> > >
> > >           self.run_test(pktType)
> > > @@ -198,7 +209,7 @@ class TestUniPacket(TestCase):
> > >           checked that whether IPv4 in IPv6 tunnel packet can be
> > normally
> > >           detected by Niantic and i350.
> > >           """
> > > -        if "niantic" not in self.nic.lower() and "i350" not in
> > self.nic.lower():
> > > +        if "niantic" not in self.nic.lower() and "powerville" not in
> > self.nic.lower():
> > >               return
> > >
> > >           pktType = {
> > > @@ -361,4 +372,15 @@ class TestUniPacket(TestCase):
> > >           Nothing to do.
> > >           """
> > >           self.dut.kill_all()
> > > +        """
> > > +        Restore the config file and recompile the package.
> > > +        """
> > > +        [arch, machine, env, toolchain] = self.target.split('-')
> > > +        if "fortville" in self.nic.lower():
> > > +            self.dut.send_expect("sed -i -e
> > 's/CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=.*$/"
> > > +                             +
> > "CONFIG_RTE_LIBRTE_I40E_INC_VECTOR=n/' config/common_%s" % env, "# ",
> > 30)
> > > +        elif "niantic" in self.nic.lower():
> > > +            self.dut.send_expect("sed -i -e
> > 's/CONFIG_RTE_IXGBE_INC_VECTOR=.*$/"
> > > +                             + "CONFIG_RTE_IXGBE_INC_VECTOR=n/'
> > config/common_%s" % env, "# ", 30)
> > > +        self.dut.build_install_dpdk(self.target)
> > Here is still disable vector mode. There's other approach is that use
> > dts.rx_mode which is stripped from execution configuration file.
> > Only when dts.rx_mode is "default" or "vector", this suite need change
> > dpdk configuration.
> In  i40e driver , the default of " vector "  is "n" in dpdk configuration
> file
> And in ixgbe driver, the default of " vector "  is "y" in dpdk
> configuration file
> >
> >
> > >           pass

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

end of thread, other threads:[~2015-12-02  1:26 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-01  7:52 [dts] [PATCH v2]tests uni_pkt: disable vector ixgbe driver and rename i350 to powerville Lijuan Tu
2015-12-02  1:12 ` Liu, Yong
2015-12-02  1:19   ` Tu, LijuanX A
2015-12-02  1:26     ` 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).