test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] tests/vf_offload : stop port before set csum
@ 2018-01-26  2:36 zhao,meijuan
  2018-01-29  5:33 ` Liu, Yong
  0 siblings, 1 reply; 2+ messages in thread
From: zhao,meijuan @ 2018-01-26  2:36 UTC (permalink / raw)
  To: dts; +Cc: zhao,meijuan

Need to stop port before configure csum.
you can refer dpdk commit : 597f9fafe13b


Signed-off-by: zhao,meijuan <meijuanx.zhao@intel.com>
---
 tests/TestSuite_vf_offload.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/TestSuite_vf_offload.py b/tests/TestSuite_vf_offload.py
index 2dfc424..c1983ca 100644
--- a/tests/TestSuite_vf_offload.py
+++ b/tests/TestSuite_vf_offload.py
@@ -118,10 +118,12 @@ class TestVfOffload(TestCase):
         self.setup_2pf_2vf_1vm_env_flag = 0
 
     def checksum_enablehw(self, port, dut):
+        dut.send_expect("port stop all", "testpmd>")
         dut.send_expect("csum set ip hw %d" % port, "testpmd>")
         dut.send_expect("csum set udp hw %d" % port, "testpmd>")
         dut.send_expect("csum set tcp hw %d" % port, "testpmd>")
         dut.send_expect("csum set sctp hw %d" % port, "testpmd>")
+        dut.send_expect("port start all", "testpmd>")
 
     def checksum_enablesw(self, port, dut):
         dut.send_expect("csum set ip sw %d" % port, "testpmd>")
-- 
1.9.3

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

* Re: [dts] [PATCH V1] tests/vf_offload : stop port before set csum
  2018-01-26  2:36 [dts] [PATCH V1] tests/vf_offload : stop port before set csum zhao,meijuan
@ 2018-01-29  5:33 ` Liu, Yong
  0 siblings, 0 replies; 2+ messages in thread
From: Liu, Yong @ 2018-01-29  5:33 UTC (permalink / raw)
  To: Zhao, MeijuanX, dts; +Cc: Zhao, MeijuanX

Thanks, Meijuan. Applied.

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of zhao,meijuan
> Sent: Friday, January 26, 2018 10:36 AM
> To: dts@dpdk.org
> Cc: Zhao, MeijuanX <meijuanx.zhao@intel.com>
> Subject: [dts] [PATCH V1] tests/vf_offload : stop port before set csum
> 
> Need to stop port before configure csum.
> you can refer dpdk commit : 597f9fafe13b
> 
> 
> Signed-off-by: zhao,meijuan <meijuanx.zhao@intel.com>
> ---
>  tests/TestSuite_vf_offload.py | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/TestSuite_vf_offload.py b/tests/TestSuite_vf_offload.py
> index 2dfc424..c1983ca 100644
> --- a/tests/TestSuite_vf_offload.py
> +++ b/tests/TestSuite_vf_offload.py
> @@ -118,10 +118,12 @@ class TestVfOffload(TestCase):
>          self.setup_2pf_2vf_1vm_env_flag = 0
> 
>      def checksum_enablehw(self, port, dut):
> +        dut.send_expect("port stop all", "testpmd>")
>          dut.send_expect("csum set ip hw %d" % port, "testpmd>")
>          dut.send_expect("csum set udp hw %d" % port, "testpmd>")
>          dut.send_expect("csum set tcp hw %d" % port, "testpmd>")
>          dut.send_expect("csum set sctp hw %d" % port, "testpmd>")
> +        dut.send_expect("port start all", "testpmd>")
> 
>      def checksum_enablesw(self, port, dut):
>          dut.send_expect("csum set ip sw %d" % port, "testpmd>")
> --
> 1.9.3

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

end of thread, other threads:[~2018-01-29  5:33 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-26  2:36 [dts] [PATCH V1] tests/vf_offload : stop port before set csum zhao,meijuan
2018-01-29  5:33 ` 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).