* [dts] [PATCH V1] fix vxlan failed case
@ 2016-08-24 6:51 xu,huilong
2016-08-25 6:09 ` Liu, Yong
0 siblings, 1 reply; 6+ messages in thread
From: xu,huilong @ 2016-08-24 6:51 UTC (permalink / raw)
To: dts; +Cc: xu,huilong
update list:
1. packet type detect must use without VEC PMD
2. out ip bab checksum count should include
Signed-off-by: xu,huilong <huilongx.xu@intel.com>
---
tests/TestSuite_nvgre.py | 26 +++++++++++++++++++++++---
tests/TestSuite_vxlan.py | 44 ++++++++++++++++++++++++++++++++++++++++----
2 files changed, 63 insertions(+), 7 deletions(-)
diff --git a/tests/TestSuite_nvgre.py b/tests/TestSuite_nvgre.py
index 1da551b..bcbf41b 100644
--- a/tests/TestSuite_nvgre.py
+++ b/tests/TestSuite_nvgre.py
@@ -349,7 +349,12 @@ class TestNvgre(TestCase):
nvgre Prerequisites
"""
# this feature only enable in FVL now
- self.verify(self.nic in ["fortville_eagle", "fortville_spirit", "fortville_spirit_single", "fortpark_TLV", "sageville", "sagepond"], "NVGRE Only supported by Fortville and Sageville")
+ if self.nic in ["fortville_eagle", "fortville_spirit", "fortville_spirit_single", "fortpark_TLV"]:
+ self.compile_switch = 'CONFIG_RTE_LIBRTE_I40E_INC_VECTOR'
+ elif self.nic in ["sageville", "sagepond"]:
+ self.compile_switch = 'CONFIG_RTE_IXGBE_INC_VECTOR'
+ else:
+ self.verify(False, "%s not support NVGRE case" % self.nic)
# Based on h/w type, choose how many ports to use
ports = self.dut.get_ports(self.nic)
self.portmask = dts.create_mask(self.dut.get_ports(self.nic))
@@ -457,7 +462,7 @@ class TestNvgre(TestCase):
self.dut.send_expect("show port stats all", "testpmd>", 10)
self.dut.send_expect("stop", "testpmd>", 10)
self.dut.send_expect("quit", "#", 10)
-
+
def nvgre_filter(self, filter_type="omac-imac-tenid", queue_id=3, vlan=False, remove=False):
"""
send nvgre packet and check whether receive packet in assigned queue
@@ -602,6 +607,15 @@ class TestNvgre(TestCase):
"""
verify nvgre packet with ipv4
"""
+ # packet type detect must used without VECTOR pmd
+ out = self.dut.send_expect("cat config/common_base", "]# ", 10)
+ src_vec_model = re.findall("%s=." % self.compile_switch, out)[0][-1]
+ if src_vec_model == 'y':
+ self.dut.send_expect("sed -i -e 's/%s=.*$/" % self.compile_switch
+ + "%s=n/' config/common_base" % self.compile_switch, "# ", 30)
+ self.dut.skip_setup = False
+ self.dut.build_install_dpdk(self.target)
+
# check no nvgre packet
self.nvgre_detect(outer_ip_proto=0xFF)
# check nvgre + IP inner packet
@@ -614,7 +628,13 @@ class TestNvgre(TestCase):
self.nvgre_detect(outer_vlan=1)
# check vlan nvgre + vlan inner packet
self.nvgre_detect(outer_vlan=1, inner_vlan=1)
-
+ out = self.dut.send_expect("cat config/common_base", "]# ", 10)
+ dst_vec_model = re.findall("%s=." % self.compile_switch, out)[0][-1]
+ if src_vec_model != dst_vec_model:
+ self.dut.send_expect("sed -i -e 's/%s=.*$/" % self.compile_switch
+ + "%s=%s/' config/common_base" % (self.compile_switch, src_vec_model), "# ", 30)
+ self.dut.skip_setup = False
+ self.dut.build_install_dpdk(self.target)
def test_tunnel_filter(self):
# verify tunnel filter feature
diff --git a/tests/TestSuite_vxlan.py b/tests/TestSuite_vxlan.py
index c1d0200..f94dc24 100644
--- a/tests/TestSuite_vxlan.py
+++ b/tests/TestSuite_vxlan.py
@@ -257,9 +257,12 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
vxlan Prerequisites
"""
# this feature only enable in FVL now
- self.verify(self.nic in ["fortville_eagle", "fortville_spirit",
- "fortville_spirit_single", "sagepond","fortpark_TLV"],
- "Vxlan Only supported by Fortville and Sageville")
+ if self.nic in ["fortville_eagle", "fortville_spirit", "fortville_spirit_single", "fortpark_TLV"]:
+ self.compile_switch = 'CONFIG_RTE_LIBRTE_I40E_INC_VECTOR'
+ elif self.nic in ["sageville", "sagepond"]:
+ self.compile_switch = 'CONFIG_RTE_IXGBE_INC_VECTOR'
+ else:
+ self.verify(False, "%s not support this vxlan" % self.nic)
# Based on h/w type, choose how many ports to use
ports = self.dut.get_ports()
@@ -466,7 +469,7 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
self.l4err_num += 1
# verify detected l3 invalid checksum
- if "inner_ip_invalid" in kwargs:
+ if "ip_invalid" in kwargs:
self.verify(self.pmdout.get_pmd_value("Bad-ipcsum:", out)
== self.iperr_num + 1, "Failed to count inner ip chksum error")
self.iperr_num += 1
@@ -528,6 +531,15 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
"""
verify vxlan packet detection
"""
+ out = self.dut.send_expect("cat config/common_base", "]# ", 10)
+ src_vec_model = re.findall("%s=." % self.compile_switch, out)[0][-1]
+ if src_vec_model == 'y':
+ self.dut.send_expect("sed -i -e 's/%s=.*$/" % self.compile_switch
+ + "%s=n/' config/common_base" % self.compile_switch, "# ", 30)
+ self.dut.skip_setup = False
+ self.dut.build_install_dpdk(self.target)
+
+
pmd_temp = "./%(TARGET)s/app/testpmd -c %(COREMASK)s -n " + \
"%(CHANNEL)d -- -i --disable-rss --rxq=4 --txq=4" + \
" --nb-cores=4 --portmask=%(PORT)s --txqflags=0x0"
@@ -557,11 +569,27 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
out = self.dut.send_expect("stop", "testpmd>", 10)
self.dut.send_expect("quit", "#", 10)
+
+ out = self.dut.send_expect("cat config/common_base", "]# ", 10)
+ dst_vec_model = re.findall("%s=." % self.compile_switch, out)[0][-1]
+ if src_vec_model != dst_vec_model:
+ self.dut.send_expect("sed -i -e 's/%s=.*$/" % self.compile_switch
+ + "%s=%s/' config/common_base" % (self.compile_switch, src_vec_model), "# ", 30)
+ self.dut.skip_setup = False
+ self.dut.build_install_dpdk(self.target)
def test_vxlan_ipv6_detect(self):
"""
verify vxlan packet detection with ipv6 header
"""
+ out = self.dut.send_expect("cat config/common_base", "]# ", 10)
+ src_vec_model = re.findall("%s=." % self.compile_switch, out)[0][-1]
+ if src_vec_model == 'y':
+ self.dut.send_expect("sed -i -e 's/%s=.*$/" % self.compile_switch
+ + "%s=n/' config/common_base" % self.compile_switch, "# ", 30)
+ self.dut.skip_setup = False
+ self.dut.build_install_dpdk(self.target)
+
pmd_temp = "./%(TARGET)s/app/testpmd -c %(COREMASK)s -n " + \
"%(CHANNEL)d -- -i --disable-rss --rxq=4 --txq=4" + \
" --nb-cores=4 --portmask=%(PORT)s --txqflags=0x0"
@@ -596,6 +624,14 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
out = self.dut.send_expect("stop", "testpmd>", 10)
self.dut.send_expect("quit", "#", 10)
+ out = self.dut.send_expect("cat config/common_base", "]# ", 10)
+ dst_vec_model = re.findall("%s=." % self.compile_switch, out)[0][-1]
+ if src_vec_model != dst_vec_model:
+ self.dut.send_expect("sed -i -e 's/%s=.*$/" % self.compile_switch
+ + "%s=%s/' config/common_base" % (self.compile_switch, src_vec_model), "# ", 30)
+ self.dut.skip_setup = False
+ self.dut.build_install_dpdk(self.target)
+
def test_vxlan_ipv4_checksum_offload(self):
"""
verify vxlan packet checksum offload
--
1.9.3
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dts] [PATCH V1] fix vxlan failed case
2016-08-24 6:51 [dts] [PATCH V1] fix vxlan failed case xu,huilong
@ 2016-08-25 6:09 ` Liu, Yong
2016-08-25 6:42 ` Xu, HuilongX
0 siblings, 1 reply; 6+ messages in thread
From: Liu, Yong @ 2016-08-25 6:09 UTC (permalink / raw)
To: xu,huilong, dts
Thanks huilong, two comments.
1. no need to change the setting in all cases. Only one time in
set_up_all is enough.
2. Nvgre patch has been merged, please send out patch only for vxlan suite.
On 08/24/2016 02:51 PM, xu,huilong wrote:
> def test_vxlan_ipv6_detect(self):
> """
> verify vxlan packet detection with ipv6 header
> """
> + out = self.dut.send_expect("cat config/common_base", "]# ", 10)
> + src_vec_model = re.findall("%s=." % self.compile_switch, out)[0][-1]
> + if src_vec_model == 'y':
> + self.dut.send_expect("sed -i -e 's/%s=.*$/" % self.compile_switch
> + + "%s=n/' config/common_base" % self.compile_switch, "# ", 30)
> + self.dut.skip_setup = False
> + self.dut.build_install_dpdk(self.target)
> +
> pmd_temp = "./%(TARGET)s/app/testpmd -c %(COREMASK)s -n " + \
> "%(CHANNEL)d -- -i --disable-rss --rxq=4 --txq=4" + \
> " --nb-cores=4 --portmask=%(PORT)s --txqflags=0x0"
> @@ -596,6 +624,14 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
> out = self.dut.send_expect("stop", "testpmd>", 10)
> self.dut.send_expect("quit", "#", 10)
>
> + out = self.dut.send_expect("cat config/common_base", "]# ", 10)
> + dst_vec_model = re.findall("%s=." % self.compile_switch, out)[0][-1]
> + if src_vec_model != dst_vec_model:
> + self.dut.send_expect("sed -i -e 's/%s=.*$/" % self.compile_switch
> + + "%s=%s/' config/common_base" % (self.compile_switch, src_vec_model), "# ", 30)
> + self.dut.skip_setup = False
> + self.dut.build_install_dpdk(self.target)
> +
> def test_vxlan_ipv4_checksum_offload(self):
> """
> verify vxlan packet checksum offload
> -- 1.9.3
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dts] [PATCH V1] fix vxlan failed case
2016-08-25 6:09 ` Liu, Yong
@ 2016-08-25 6:42 ` Xu, HuilongX
2016-08-25 7:00 ` Liu, Yong
0 siblings, 1 reply; 6+ messages in thread
From: Xu, HuilongX @ 2016-08-25 6:42 UTC (permalink / raw)
To: Liu, Yong, dts
Hi yong,
Only packet type detect case need without VEC PMD. Other cases support with VEC PMD and without VEC PMD.
So for every packet type detect case need re-compile dpdk when we test with VEC PMD.
Thanks a lot
> -----Original Message-----
> From: Liu, Yong
> Sent: Thursday, August 25, 2016 2:09 PM
> To: Xu, HuilongX; dts@dpdk.org
> Subject: Re: [dts] [PATCH V1] fix vxlan failed case
>
> Thanks huilong, two comments.
> 1. no need to change the setting in all cases. Only one time in
> set_up_all is enough.
> 2. Nvgre patch has been merged, please send out patch only for vxlan
> suite.
>
> On 08/24/2016 02:51 PM, xu,huilong wrote:
> > def test_vxlan_ipv6_detect(self):
> > """
> > verify vxlan packet detection with ipv6 header
> > """
> > + out = self.dut.send_expect("cat config/common_base", "]# ",
> 10)
> > + src_vec_model = re.findall("%s=." % self.compile_switch,
> out)[0][-1]
> > + if src_vec_model == 'y':
> > + self.dut.send_expect("sed -i -e 's/%s=.*$/" %
> self.compile_switch
> > + + "%s=n/' config/common_base" %
> self.compile_switch, "# ", 30)
> > + self.dut.skip_setup = False
> > + self.dut.build_install_dpdk(self.target)
> > +
> > pmd_temp = "./%(TARGET)s/app/testpmd -c %(COREMASK)s -n " +
> \
> > "%(CHANNEL)d -- -i --disable-rss --rxq=4 --txq=4" + \
> > " --nb-cores=4 --portmask=%(PORT)s --txqflags=0x0"
> > @@ -596,6 +624,14 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
> > out = self.dut.send_expect("stop", "testpmd>", 10)
> > self.dut.send_expect("quit", "#", 10)
> >
> > + out = self.dut.send_expect("cat config/common_base", "]# ",
> 10)
> > + dst_vec_model = re.findall("%s=." % self.compile_switch,
> out)[0][-1]
> > + if src_vec_model != dst_vec_model:
> > + self.dut.send_expect("sed -i -e 's/%s=.*$/" %
> self.compile_switch
> > + + "%s=%s/' config/common_base" %
> (self.compile_switch, src_vec_model), "# ", 30)
> > + self.dut.skip_setup = False
> > + self.dut.build_install_dpdk(self.target)
> > +
> > def test_vxlan_ipv4_checksum_offload(self):
> > """
> > verify vxlan packet checksum offload
> > -- 1.9.3
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dts] [PATCH V1] fix vxlan failed case
2016-08-25 6:42 ` Xu, HuilongX
@ 2016-08-25 7:00 ` Liu, Yong
2016-08-25 7:09 ` Xu, HuilongX
0 siblings, 1 reply; 6+ messages in thread
From: Liu, Yong @ 2016-08-25 7:00 UTC (permalink / raw)
To: Xu, HuilongX, dts
Thanks for clear this.
There's one open discussion for vector mode switch in DTS.
One option is use testpmd parameter "enable-rx-cksum". It work on ixgbe and i40e, but this method maybe invalid in later release.
Another option is just like patch, we switch between vector and scatter pmd by configuration file. Between the switch, it will take long time for recompile dpdk.
There's another way is that we build different kinds of testpmd applications like vector/legacy/scatter. It can skip build between switch modes.
Marvin,
> -----Original Message-----
> From: Xu, HuilongX
> Sent: Thursday, August 25, 2016 2:42 PM
> To: Liu, Yong; dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix vxlan failed case
>
> Hi yong,
> Only packet type detect case need without VEC PMD. Other cases support
> with VEC PMD and without VEC PMD.
> So for every packet type detect case need re-compile dpdk when we test
> with VEC PMD.
> Thanks a lot
>
>
> > -----Original Message-----
> > From: Liu, Yong
> > Sent: Thursday, August 25, 2016 2:09 PM
> > To: Xu, HuilongX; dts@dpdk.org
> > Subject: Re: [dts] [PATCH V1] fix vxlan failed case
> >
> > Thanks huilong, two comments.
> > 1. no need to change the setting in all cases. Only one time in
> > set_up_all is enough.
> > 2. Nvgre patch has been merged, please send out patch only for vxlan
> > suite.
> >
> > On 08/24/2016 02:51 PM, xu,huilong wrote:
> > > def test_vxlan_ipv6_detect(self):
> > > """
> > > verify vxlan packet detection with ipv6 header
> > > """
> > > + out = self.dut.send_expect("cat config/common_base", "]# ",
> > 10)
> > > + src_vec_model = re.findall("%s=." % self.compile_switch,
> > out)[0][-1]
> > > + if src_vec_model == 'y':
> > > + self.dut.send_expect("sed -i -e 's/%s=.*$/" %
> > self.compile_switch
> > > + + "%s=n/' config/common_base" %
> > self.compile_switch, "# ", 30)
> > > + self.dut.skip_setup = False
> > > + self.dut.build_install_dpdk(self.target)
> > > +
> > > pmd_temp = "./%(TARGET)s/app/testpmd -c %(COREMASK)s -n " +
> > \
> > > "%(CHANNEL)d -- -i --disable-rss --rxq=4 --txq=4" + \
> > > " --nb-cores=4 --portmask=%(PORT)s --txqflags=0x0"
> > > @@ -596,6 +624,14 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
> > > out = self.dut.send_expect("stop", "testpmd>", 10)
> > > self.dut.send_expect("quit", "#", 10)
> > >
> > > + out = self.dut.send_expect("cat config/common_base", "]# ",
> > 10)
> > > + dst_vec_model = re.findall("%s=." % self.compile_switch,
> > out)[0][-1]
> > > + if src_vec_model != dst_vec_model:
> > > + self.dut.send_expect("sed -i -e 's/%s=.*$/" %
> > self.compile_switch
> > > + + "%s=%s/' config/common_base" %
> > (self.compile_switch, src_vec_model), "# ", 30)
> > > + self.dut.skip_setup = False
> > > + self.dut.build_install_dpdk(self.target)
> > > +
> > > def test_vxlan_ipv4_checksum_offload(self):
> > > """
> > > verify vxlan packet checksum offload
> > > -- 1.9.3
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dts] [PATCH V1] fix vxlan failed case
2016-08-25 7:00 ` Liu, Yong
@ 2016-08-25 7:09 ` Xu, HuilongX
2016-09-08 8:04 ` Liu, Yong
0 siblings, 1 reply; 6+ messages in thread
From: Xu, HuilongX @ 2016-08-25 7:09 UTC (permalink / raw)
To: Liu, Yong, dts
Hi yong,
I think, dts framework should compile many times for different compile switch.
Then case should choose different dpdk APP for test.
Could you merge this patch fist, and we can submit a new patch for dts framework compile many dpdk with different compile switch.
Thanks a lot
> -----Original Message-----
> From: Liu, Yong
> Sent: Thursday, August 25, 2016 3:00 PM
> To: Xu, HuilongX; dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix vxlan failed case
>
> Thanks for clear this.
> There's one open discussion for vector mode switch in DTS.
> One option is use testpmd parameter "enable-rx-cksum". It work on ixgbe
> and i40e, but this method maybe invalid in later release.
> Another option is just like patch, we switch between vector and scatter
> pmd by configuration file. Between the switch, it will take long time
> for recompile dpdk.
> There's another way is that we build different kinds of testpmd
> applications like vector/legacy/scatter. It can skip build between
> switch modes.
>
> Marvin,
>
> > -----Original Message-----
> > From: Xu, HuilongX
> > Sent: Thursday, August 25, 2016 2:42 PM
> > To: Liu, Yong; dts@dpdk.org
> > Subject: RE: [dts] [PATCH V1] fix vxlan failed case
> >
> > Hi yong,
> > Only packet type detect case need without VEC PMD. Other cases support
> > with VEC PMD and without VEC PMD.
> > So for every packet type detect case need re-compile dpdk when we test
> > with VEC PMD.
> > Thanks a lot
> >
> >
> > > -----Original Message-----
> > > From: Liu, Yong
> > > Sent: Thursday, August 25, 2016 2:09 PM
> > > To: Xu, HuilongX; dts@dpdk.org
> > > Subject: Re: [dts] [PATCH V1] fix vxlan failed case
> > >
> > > Thanks huilong, two comments.
> > > 1. no need to change the setting in all cases. Only one time in
> > > set_up_all is enough.
> > > 2. Nvgre patch has been merged, please send out patch only for vxlan
> > > suite.
> > >
> > > On 08/24/2016 02:51 PM, xu,huilong wrote:
> > > > def test_vxlan_ipv6_detect(self):
> > > > """
> > > > verify vxlan packet detection with ipv6 header
> > > > """
> > > > + out = self.dut.send_expect("cat config/common_base", "]#
> ",
> > > 10)
> > > > + src_vec_model = re.findall("%s=." % self.compile_switch,
> > > out)[0][-1]
> > > > + if src_vec_model == 'y':
> > > > + self.dut.send_expect("sed -i -e 's/%s=.*$/" %
> > > self.compile_switch
> > > > + + "%s=n/' config/common_base" %
> > > self.compile_switch, "# ", 30)
> > > > + self.dut.skip_setup = False
> > > > + self.dut.build_install_dpdk(self.target)
> > > > +
> > > > pmd_temp = "./%(TARGET)s/app/testpmd -c %(COREMASK)s -n
> " +
> > > \
> > > > "%(CHANNEL)d -- -i --disable-rss --rxq=4 --txq=4" +
> \
> > > > " --nb-cores=4 --portmask=%(PORT)s --txqflags=0x0"
> > > > @@ -596,6 +624,14 @@ class TestVxlan(TestCase,
> IxiaPacketGenerator):
> > > > out = self.dut.send_expect("stop", "testpmd>", 10)
> > > > self.dut.send_expect("quit", "#", 10)
> > > >
> > > > + out = self.dut.send_expect("cat config/common_base", "]#
> ",
> > > 10)
> > > > + dst_vec_model = re.findall("%s=." % self.compile_switch,
> > > out)[0][-1]
> > > > + if src_vec_model != dst_vec_model:
> > > > + self.dut.send_expect("sed -i -e 's/%s=.*$/" %
> > > self.compile_switch
> > > > + + "%s=%s/' config/common_base" %
> > > (self.compile_switch, src_vec_model), "# ", 30)
> > > > + self.dut.skip_setup = False
> > > > + self.dut.build_install_dpdk(self.target)
> > > > +
> > > > def test_vxlan_ipv4_checksum_offload(self):
> > > > """
> > > > verify vxlan packet checksum offload
> > > > -- 1.9.3
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dts] [PATCH V1] fix vxlan failed case
2016-08-25 7:09 ` Xu, HuilongX
@ 2016-09-08 8:04 ` Liu, Yong
0 siblings, 0 replies; 6+ messages in thread
From: Liu, Yong @ 2016-09-08 8:04 UTC (permalink / raw)
To: Xu, HuilongX, dts
OK, merged in this temporary solution.
On 08/25/2016 03:09 PM, Xu, HuilongX wrote:
> Hi yong,
> I think, dts framework should compile many times for different compile switch.
> Then case should choose different dpdk APP for test.
> Could you merge this patch fist, and we can submit a new patch for dts framework compile many dpdk with different compile switch.
> Thanks a lot
>
>> -----Original Message-----
>> From: Liu, Yong
>> Sent: Thursday, August 25, 2016 3:00 PM
>> To: Xu, HuilongX; dts@dpdk.org
>> Subject: RE: [dts] [PATCH V1] fix vxlan failed case
>>
>> Thanks for clear this.
>> There's one open discussion for vector mode switch in DTS.
>> One option is use testpmd parameter "enable-rx-cksum". It work on ixgbe
>> and i40e, but this method maybe invalid in later release.
>> Another option is just like patch, we switch between vector and scatter
>> pmd by configuration file. Between the switch, it will take long time
>> for recompile dpdk.
>> There's another way is that we build different kinds of testpmd
>> applications like vector/legacy/scatter. It can skip build between
>> switch modes.
>>
>> Marvin,
>>
>>> -----Original Message-----
>>> From: Xu, HuilongX
>>> Sent: Thursday, August 25, 2016 2:42 PM
>>> To: Liu, Yong; dts@dpdk.org
>>> Subject: RE: [dts] [PATCH V1] fix vxlan failed case
>>>
>>> Hi yong,
>>> Only packet type detect case need without VEC PMD. Other cases support
>>> with VEC PMD and without VEC PMD.
>>> So for every packet type detect case need re-compile dpdk when we test
>>> with VEC PMD.
>>> Thanks a lot
>>>
>>>
>>>> -----Original Message-----
>>>> From: Liu, Yong
>>>> Sent: Thursday, August 25, 2016 2:09 PM
>>>> To: Xu, HuilongX; dts@dpdk.org
>>>> Subject: Re: [dts] [PATCH V1] fix vxlan failed case
>>>>
>>>> Thanks huilong, two comments.
>>>> 1. no need to change the setting in all cases. Only one time in
>>>> set_up_all is enough.
>>>> 2. Nvgre patch has been merged, please send out patch only for vxlan
>>>> suite.
>>>>
>>>> On 08/24/2016 02:51 PM, xu,huilong wrote:
>>>>> def test_vxlan_ipv6_detect(self):
>>>>> """
>>>>> verify vxlan packet detection with ipv6 header
>>>>> """
>>>>> + out = self.dut.send_expect("cat config/common_base", "]#
>> ",
>>>> 10)
>>>>> + src_vec_model = re.findall("%s=." % self.compile_switch,
>>>> out)[0][-1]
>>>>> + if src_vec_model == 'y':
>>>>> + self.dut.send_expect("sed -i -e 's/%s=.*$/" %
>>>> self.compile_switch
>>>>> + + "%s=n/' config/common_base" %
>>>> self.compile_switch, "# ", 30)
>>>>> + self.dut.skip_setup = False
>>>>> + self.dut.build_install_dpdk(self.target)
>>>>> +
>>>>> pmd_temp = "./%(TARGET)s/app/testpmd -c %(COREMASK)s -n
>> " +
>>>> \
>>>>> "%(CHANNEL)d -- -i --disable-rss --rxq=4 --txq=4" +
>> \
>>>>> " --nb-cores=4 --portmask=%(PORT)s --txqflags=0x0"
>>>>> @@ -596,6 +624,14 @@ class TestVxlan(TestCase,
>> IxiaPacketGenerator):
>>>>> out = self.dut.send_expect("stop", "testpmd>", 10)
>>>>> self.dut.send_expect("quit", "#", 10)
>>>>>
>>>>> + out = self.dut.send_expect("cat config/common_base", "]#
>> ",
>>>> 10)
>>>>> + dst_vec_model = re.findall("%s=." % self.compile_switch,
>>>> out)[0][-1]
>>>>> + if src_vec_model != dst_vec_model:
>>>>> + self.dut.send_expect("sed -i -e 's/%s=.*$/" %
>>>> self.compile_switch
>>>>> + + "%s=%s/' config/common_base" %
>>>> (self.compile_switch, src_vec_model), "# ", 30)
>>>>> + self.dut.skip_setup = False
>>>>> + self.dut.build_install_dpdk(self.target)
>>>>> +
>>>>> def test_vxlan_ipv4_checksum_offload(self):
>>>>> """
>>>>> verify vxlan packet checksum offload
>>>>> -- 1.9.3
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-09-08 8:02 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-24 6:51 [dts] [PATCH V1] fix vxlan failed case xu,huilong
2016-08-25 6:09 ` Liu, Yong
2016-08-25 6:42 ` Xu, HuilongX
2016-08-25 7:00 ` Liu, Yong
2016-08-25 7:09 ` Xu, HuilongX
2016-09-08 8:04 ` 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).