test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts][PATCH V2] tests/vf_vlan: case add_pvid_vf restart testpmd after remove vlan according to testplan
@ 2022-09-08  2:47 Weiyuan Li
  2022-09-15  7:30 ` Chen, LingliX
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Weiyuan Li @ 2022-09-08  2:47 UTC (permalink / raw)
  To: dts; +Cc: Weiyuan Li

Case add_pvid_vf modify to restart testpmd after remove vlan according to testplan.

Signed-off-by: Weiyuan Li <weiyuanx.li@intel.com>
---

v2:
-Patch conflict modification

 tests/TestSuite_vf_vlan.py | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/tests/TestSuite_vf_vlan.py b/tests/TestSuite_vf_vlan.py
index c4904c55..31826db5 100644
--- a/tests/TestSuite_vf_vlan.py
+++ b/tests/TestSuite_vf_vlan.py
@@ -233,16 +233,18 @@ class TestVfVlan(TestCase):
 
         # remove vlan
         self.vm0_testpmd.execute_cmd("stop")
-        self.vm0_testpmd.execute_cmd("port stop all")
+        self.vm0_testpmd.quit()
         self.dut.send_expect("ip link set %s vf 0 vlan 0" % self.host_intf0, "# ")
         out = self.dut.send_expect("ip link show %s" % self.host_intf0, "# ")
         self.verify("vlan %d" % random_vlan not in out, "Failed to remove pvid on VF0")
 
-        # send packet with vlan
-        self.vm0_testpmd.execute_cmd("port reset 0", "testpmd> ", 120)
-        self.vm0_testpmd.execute_cmd("port start all")
+        # restart testpmd
+        self.vm0_testpmd = PmdOutput(self.vm_dut_0)
+        self.vm0_testpmd.start_testpmd(VM_CORES_MASK)
+        self.vm0_testpmd.execute_cmd("set fwd rxonly")
+        self.vm0_testpmd.execute_cmd("set verbose 1")
         self.vm0_testpmd.execute_cmd("start")
-
+        # send packet with vlan
         out = self.send_and_getout(vlan=random_vlan, pkt_type="VLAN_UDP")
         if (
             (self.kdriver == "i40e" and self.driver_version < "2.13.10")
-- 
2.25.1


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

* RE: [dts][PATCH V2] tests/vf_vlan: case add_pvid_vf restart testpmd after remove vlan according to testplan
  2022-09-08  2:47 [dts][PATCH V2] tests/vf_vlan: case add_pvid_vf restart testpmd after remove vlan according to testplan Weiyuan Li
@ 2022-09-15  7:30 ` Chen, LingliX
  2022-09-20  7:34 ` Fu, Qi
  2022-09-20  8:04 ` lijuan.tu
  2 siblings, 0 replies; 4+ messages in thread
From: Chen, LingliX @ 2022-09-15  7:30 UTC (permalink / raw)
  To: dts; +Cc: Li, WeiyuanX


> -----Original Message-----
> From: Weiyuan Li <weiyuanx.li@intel.com>
> Sent: Thursday, September 8, 2022 10:47 AM
> To: dts@dpdk.org
> Cc: Li, WeiyuanX <weiyuanx.li@intel.com>
> Subject: [dts][PATCH V2] tests/vf_vlan: case add_pvid_vf restart testpmd after
> remove vlan according to testplan
> 
> Case add_pvid_vf modify to restart testpmd after remove vlan according to
> testplan.
> 
> Signed-off-by: Weiyuan Li <weiyuanx.li@intel.com>
> ---

Tested-by: Lingli Chen <linglix.chen@intel.com>

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

* RE: [dts][PATCH V2] tests/vf_vlan: case add_pvid_vf restart testpmd after remove vlan according to testplan
  2022-09-08  2:47 [dts][PATCH V2] tests/vf_vlan: case add_pvid_vf restart testpmd after remove vlan according to testplan Weiyuan Li
  2022-09-15  7:30 ` Chen, LingliX
@ 2022-09-20  7:34 ` Fu, Qi
  2022-09-20  8:04 ` lijuan.tu
  2 siblings, 0 replies; 4+ messages in thread
From: Fu, Qi @ 2022-09-20  7:34 UTC (permalink / raw)
  To: Li, WeiyuanX, dts; +Cc: Li, WeiyuanX

> -----Original Message-----
> From: Weiyuan Li <weiyuanx.li@intel.com>
> Sent: Thursday, September 8, 2022 10:47 AM
> To: dts@dpdk.org
> Cc: Li, WeiyuanX <weiyuanx.li@intel.com>
> Subject: [dts][PATCH V2] tests/vf_vlan: case add_pvid_vf restart testpmd after
> remove vlan according to testplan
> 
> Case add_pvid_vf modify to restart testpmd after remove vlan according to
> testplan.
> 
> Signed-off-by: Weiyuan Li <weiyuanx.li@intel.com>
> ---
> 
> v2:
> -Patch conflict modification
> 
>  tests/TestSuite_vf_vlan.py | 12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
> 
> 2.25.1
Acked-by: Fu, Qi <qi.fu@intel.com>


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

* [dts][PATCH V2] tests/vf_vlan: case add_pvid_vf restart testpmd after remove vlan according to testplan
  2022-09-08  2:47 [dts][PATCH V2] tests/vf_vlan: case add_pvid_vf restart testpmd after remove vlan according to testplan Weiyuan Li
  2022-09-15  7:30 ` Chen, LingliX
  2022-09-20  7:34 ` Fu, Qi
@ 2022-09-20  8:04 ` lijuan.tu
  2 siblings, 0 replies; 4+ messages in thread
From: lijuan.tu @ 2022-09-20  8:04 UTC (permalink / raw)
  To: dts, Weiyuan Li; +Cc: Weiyuan Li

On Thu,  8 Sep 2022 10:47:19 +0800, Weiyuan Li <weiyuanx.li@intel.com> wrote:
> Case add_pvid_vf modify to restart testpmd after remove vlan according to testplan.
> 
> Signed-off-by: Weiyuan Li <weiyuanx.li@intel.com>

Acked-by: Lijuan Tu <lijuan.tu@intel.com>
Applied, thanks

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

end of thread, other threads:[~2022-09-20  8:04 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-08  2:47 [dts][PATCH V2] tests/vf_vlan: case add_pvid_vf restart testpmd after remove vlan according to testplan Weiyuan Li
2022-09-15  7:30 ` Chen, LingliX
2022-09-20  7:34 ` Fu, Qi
2022-09-20  8:04 ` lijuan.tu

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