* [dts] [PATCH V2 0/3] *** modify dts because of i40evf removed ***
@ 2021-10-20 18:10 Yan Xia
2021-10-20 18:10 ` [dts] [PATCH V2 1/3] framework: modify dts because of i40evf removed Yan Xia
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Yan Xia @ 2021-10-20 18:10 UTC (permalink / raw)
To: dts; +Cc: Yan Xia
modify dts because of i40evf removed
dpdk commit-id:c65f9258
*** BLURB HERE ***
Yan Xia (3):
framework: modify dts because of i40evf removed
test_plans: modify dts because of i40evf removed
tests/vf_kernel: modify dts because of i40evf removed
framework/dut.py | 3 +-
framework/settings.py | 4 +-
test_plans/ddp_gtp_test_plan.rst | 2 +-
test_plans/ddp_mpls_test_plan.rst | 2 +-
test_plans/floating_veb_test_plan.rst | 4 +-
.../runtime_vf_queue_number_test_plan.rst | 2 +-
test_plans/veb_switch_test_plan.rst | 4 +-
test_plans/vf_pf_reset_test_plan.rst | 4 +-
tests/TestSuite_vf_kernel.py | 37 ++++++++++++++-----
9 files changed, 40 insertions(+), 22 deletions(-)
--
2.33.1
^ permalink raw reply [flat|nested] 9+ messages in thread* [dts] [PATCH V2 1/3] framework: modify dts because of i40evf removed 2021-10-20 18:10 [dts] [PATCH V2 0/3] *** modify dts because of i40evf removed *** Yan Xia @ 2021-10-20 18:10 ` Yan Xia 2021-10-22 6:29 ` Lin, Xueqin 2021-10-20 18:10 ` [dts] [PATCH V2 2/3] test_plans: " Yan Xia 2021-10-20 18:10 ` [dts] [PATCH V2 3/3] tests/vf_kernel: " Yan Xia 2 siblings, 1 reply; 9+ messages in thread From: Yan Xia @ 2021-10-20 18:10 UTC (permalink / raw) To: dts; +Cc: Yan Xia modify dts because of i40evf removed dpdk commit-id:c65f9258 Signed-off-by: Yan Xia <yanx.xia@intel.com> --- framework/dut.py | 3 +-- framework/settings.py | 4 ++-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/framework/dut.py b/framework/dut.py index f5481d06..f3ef2cbe 100644 --- a/framework/dut.py +++ b/framework/dut.py @@ -507,8 +507,7 @@ class Dut(Crb): break else: # try to bind nic with iavf - if driver == 'i40evf': - driver = 'iavf' + if driver == 'iavf': self.send_expect('modprobe %s' % driver, '# ') self.send_expect('echo %s > /sys/bus/pci/drivers/%s/bind' % (pci_bus, driver), '# ') diff --git a/framework/settings.py b/framework/settings.py index c08b8709..bca4d59f 100644 --- a/framework/settings.py +++ b/framework/settings.py @@ -153,8 +153,8 @@ DRIVERS = { 'fortpark_1g': 'i40e', 'fortpark_TLV': 'i40e', 'fortpark_BASE-T': 'i40e', - 'fortpark_TLV_vf': 'i40evf', - 'fvl10g_vf': 'i40evf', + 'fortpark_TLV_vf': 'iavf', + 'fvl10g_vf': 'iavf', 'ConnectX3_MT4103': 'mlx4_core', 'ConnectX4_MT4115': 'mlx5_core', 'ConnectX4_LX_MT4117': 'mlx5_core', -- 2.33.1 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V2 1/3] framework: modify dts because of i40evf removed 2021-10-20 18:10 ` [dts] [PATCH V2 1/3] framework: modify dts because of i40evf removed Yan Xia @ 2021-10-22 6:29 ` Lin, Xueqin 0 siblings, 0 replies; 9+ messages in thread From: Lin, Xueqin @ 2021-10-22 6:29 UTC (permalink / raw) To: Xia, YanX, dts; +Cc: Xia, YanX > -----Original Message----- > From: dts <dts-bounces@dpdk.org> On Behalf Of Yan Xia > Sent: Thursday, October 21, 2021 2:10 AM > To: dts@dpdk.org > Cc: Xia, YanX <yanx.xia@intel.com> > Subject: [dts] [PATCH V2 1/3] framework: modify dts because of i40evf > removed > > modify dts because of i40evf removed > dpdk commit-id:c65f9258 > > Signed-off-by: Yan Xia <yanx.xia@intel.com> Acked-by: Xueqin Lin <xueqin.lin@intel.com> > --- > framework/dut.py | 3 +-- > framework/settings.py | 4 ++-- > 2 files changed, 3 insertions(+), 4 deletions(-) > > diff --git a/framework/dut.py b/framework/dut.py index f5481d06..f3ef2cbe > 100644 > --- a/framework/dut.py > +++ b/framework/dut.py > @@ -507,8 +507,7 @@ class Dut(Crb): > break > else: > # try to bind nic with iavf > - if driver == 'i40evf': > - driver = 'iavf' > + if driver == 'iavf': > self.send_expect('modprobe %s' % driver, '# ') > self.send_expect('echo %s > /sys/bus/pci/drivers/%s/bind' > % (pci_bus, driver), '# ') diff --git > a/framework/settings.py b/framework/settings.py index c08b8709..bca4d59f > 100644 > --- a/framework/settings.py > +++ b/framework/settings.py > @@ -153,8 +153,8 @@ DRIVERS = { > 'fortpark_1g': 'i40e', > 'fortpark_TLV': 'i40e', > 'fortpark_BASE-T': 'i40e', > - 'fortpark_TLV_vf': 'i40evf', > - 'fvl10g_vf': 'i40evf', > + 'fortpark_TLV_vf': 'iavf', > + 'fvl10g_vf': 'iavf', > 'ConnectX3_MT4103': 'mlx4_core', > 'ConnectX4_MT4115': 'mlx5_core', > 'ConnectX4_LX_MT4117': 'mlx5_core', > -- > 2.33.1 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [dts] [PATCH V2 2/3] test_plans: modify dts because of i40evf removed 2021-10-20 18:10 [dts] [PATCH V2 0/3] *** modify dts because of i40evf removed *** Yan Xia 2021-10-20 18:10 ` [dts] [PATCH V2 1/3] framework: modify dts because of i40evf removed Yan Xia @ 2021-10-20 18:10 ` Yan Xia 2021-10-22 6:31 ` Lin, Xueqin 2021-10-20 18:10 ` [dts] [PATCH V2 3/3] tests/vf_kernel: " Yan Xia 2 siblings, 1 reply; 9+ messages in thread From: Yan Xia @ 2021-10-20 18:10 UTC (permalink / raw) To: dts; +Cc: Yan Xia modify dts because of i40evf removed dpdk commit-id:c65f9258 Signed-off-by: Yan Xia <yanx.xia@intel.com> --- test_plans/ddp_gtp_test_plan.rst | 2 +- test_plans/ddp_mpls_test_plan.rst | 2 +- test_plans/floating_veb_test_plan.rst | 4 ++-- test_plans/runtime_vf_queue_number_test_plan.rst | 2 +- test_plans/veb_switch_test_plan.rst | 4 ++-- test_plans/vf_pf_reset_test_plan.rst | 4 ++-- 6 files changed, 9 insertions(+), 9 deletions(-) diff --git a/test_plans/ddp_gtp_test_plan.rst b/test_plans/ddp_gtp_test_plan.rst index cea2993f..9172b597 100644 --- a/test_plans/ddp_gtp_test_plan.rst +++ b/test_plans/ddp_gtp_test_plan.rst @@ -70,7 +70,7 @@ Prerequisites 3. Detach VF from the host:: - rmmod i40evf + rmmod iavf 4. Pass through VF 81:10.0 to vm0, start vm0. diff --git a/test_plans/ddp_mpls_test_plan.rst b/test_plans/ddp_mpls_test_plan.rst index cce8efb5..d76934c1 100644 --- a/test_plans/ddp_mpls_test_plan.rst +++ b/test_plans/ddp_mpls_test_plan.rst @@ -60,7 +60,7 @@ Prerequisites 2. Detach VF from the host:: - rmmod i40evf + rmmod iavf 3. Pass through VF 81:10.0 to vm0, start vm0. diff --git a/test_plans/floating_veb_test_plan.rst b/test_plans/floating_veb_test_plan.rst index 1522b53b..37f6fc09 100644 --- a/test_plans/floating_veb_test_plan.rst +++ b/test_plans/floating_veb_test_plan.rst @@ -99,8 +99,8 @@ Prerequisites for VEB testing ./dpdk-devbind.py -b igb_uio 05:00.0 echo 2 >/sys/bus/pci/devices/0000:05:00.0/max_vfs ./dpdk-devbind.py --st - 0000:05:02.0 'XL710/X710 Virtual Function' unused=i40evf,igb_uio - 0000:05:02.1 'XL710/X710 Virtual Function' unused=i40evf,igb_uio + 0000:05:02.0 'XL710/X710 Virtual Function' unused=iavf,igb_uio + 0000:05:02.1 'XL710/X710 Virtual Function' unused=iavf,igb_uio 4. Bind the VFs to dpdk driver:: diff --git a/test_plans/runtime_vf_queue_number_test_plan.rst b/test_plans/runtime_vf_queue_number_test_plan.rst index 9d4f9538..b0c2c3a9 100644 --- a/test_plans/runtime_vf_queue_number_test_plan.rst +++ b/test_plans/runtime_vf_queue_number_test_plan.rst @@ -338,7 +338,7 @@ Verify error information:: Test case 7: Reserve VF queue number when VF bind to kernel driver ================================================================== -1. bind vf to kernel driver i40evf:: +1. bind vf to kernel driver iavf:: ./usertools/dpdk-devbind.py -b i40e 0000:18:02.0 diff --git a/test_plans/veb_switch_test_plan.rst b/test_plans/veb_switch_test_plan.rst index ca8bd7e0..1d7d11c2 100644 --- a/test_plans/veb_switch_test_plan.rst +++ b/test_plans/veb_switch_test_plan.rst @@ -86,8 +86,8 @@ Prerequisites for VEB testing ./dpdk-devbind.py -b igb_uio 05:00.0 echo 2 >/sys/bus/pci/devices/0000:05:00.0/max_vfs ./dpdk-devbind.py --st - 0000:05:02.0 'XL710/X710 Virtual Function' unused=i40evf,igb_uio - 0000:05:02.1 'XL710/X710 Virtual Function' unused=i40evf,igb_uio + 0000:05:02.0 'XL710/X710 Virtual Function' unused=iavf,igb_uio + 0000:05:02.1 'XL710/X710 Virtual Function' unused=iavf,igb_uio 4. Bind the VFs to dpdk driver:: diff --git a/test_plans/vf_pf_reset_test_plan.rst b/test_plans/vf_pf_reset_test_plan.rst index 009e99a9..574d4510 100644 --- a/test_plans/vf_pf_reset_test_plan.rst +++ b/test_plans/vf_pf_reset_test_plan.rst @@ -589,11 +589,11 @@ test Case 10: vf reset (two vfs passed through to two VM) using `lspci -nn|grep -i ethernet` got VF device id, for example "8086 154c":: echo "8086 154c" > /sys/bus/pci/drivers/pci-stub/new_id - echo "0000:82:02.0" > /sys/bus/pci/drivers/i40evf/unbind + echo "0000:82:02.0" > /sys/bus/pci/drivers/iavf/unbind echo "0000:82:02.0" > /sys/bus/pci/drivers/pci-stub/bind echo "8086 154c" > /sys/bus/pci/drivers/pci-stub/new_id - echo "0000:82:02.1" > /sys/bus/pci/drivers/i40evf/unbind + echo "0000:82:02.1" > /sys/bus/pci/drivers/iavf/unbind echo "0000:82:02.1" > /sys/bus/pci/drivers/pci-stub/bind 3. Pass through VF0 81:02.0 to vm0, VF1 81:02.1 to vm1:: -- 2.33.1 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V2 2/3] test_plans: modify dts because of i40evf removed 2021-10-20 18:10 ` [dts] [PATCH V2 2/3] test_plans: " Yan Xia @ 2021-10-22 6:31 ` Lin, Xueqin 0 siblings, 0 replies; 9+ messages in thread From: Lin, Xueqin @ 2021-10-22 6:31 UTC (permalink / raw) To: Xia, YanX, dts; +Cc: Xia, YanX > -----Original Message----- > From: dts <dts-bounces@dpdk.org> On Behalf Of Yan Xia > Sent: Thursday, October 21, 2021 2:10 AM > To: dts@dpdk.org > Cc: Xia, YanX <yanx.xia@intel.com> > Subject: [dts] [PATCH V2 2/3] test_plans: modify dts because of i40evf removed > > modify dts because of i40evf removed > dpdk commit-id:c65f9258 > > Signed-off-by: Yan Xia <yanx.xia@intel.com> Acked-by: Xueqin Lin <xueqin.lin@intel.com> > --- > test_plans/ddp_gtp_test_plan.rst | 2 +- > test_plans/ddp_mpls_test_plan.rst | 2 +- > test_plans/floating_veb_test_plan.rst | 4 ++-- > test_plans/runtime_vf_queue_number_test_plan.rst | 2 +- > test_plans/veb_switch_test_plan.rst | 4 ++-- > test_plans/vf_pf_reset_test_plan.rst | 4 ++-- > 6 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/test_plans/ddp_gtp_test_plan.rst > b/test_plans/ddp_gtp_test_plan.rst > index cea2993f..9172b597 100644 > --- a/test_plans/ddp_gtp_test_plan.rst > +++ b/test_plans/ddp_gtp_test_plan.rst > @@ -70,7 +70,7 @@ Prerequisites > > 3. Detach VF from the host:: > > - rmmod i40evf > + rmmod iavf > > 4. Pass through VF 81:10.0 to vm0, start vm0. > > diff --git a/test_plans/ddp_mpls_test_plan.rst > b/test_plans/ddp_mpls_test_plan.rst > index cce8efb5..d76934c1 100644 > --- a/test_plans/ddp_mpls_test_plan.rst > +++ b/test_plans/ddp_mpls_test_plan.rst > @@ -60,7 +60,7 @@ Prerequisites > > 2. Detach VF from the host:: > > - rmmod i40evf > + rmmod iavf > > 3. Pass through VF 81:10.0 to vm0, start vm0. > > diff --git a/test_plans/floating_veb_test_plan.rst > b/test_plans/floating_veb_test_plan.rst > index 1522b53b..37f6fc09 100644 > --- a/test_plans/floating_veb_test_plan.rst > +++ b/test_plans/floating_veb_test_plan.rst > @@ -99,8 +99,8 @@ Prerequisites for VEB testing > ./dpdk-devbind.py -b igb_uio 05:00.0 > echo 2 >/sys/bus/pci/devices/0000:05:00.0/max_vfs > ./dpdk-devbind.py --st > - 0000:05:02.0 'XL710/X710 Virtual Function' unused=i40evf,igb_uio > - 0000:05:02.1 'XL710/X710 Virtual Function' unused=i40evf,igb_uio > + 0000:05:02.0 'XL710/X710 Virtual Function' unused=iavf,igb_uio > + 0000:05:02.1 'XL710/X710 Virtual Function' unused=iavf,igb_uio > > 4. Bind the VFs to dpdk driver:: > > diff --git a/test_plans/runtime_vf_queue_number_test_plan.rst > b/test_plans/runtime_vf_queue_number_test_plan.rst > index 9d4f9538..b0c2c3a9 100644 > --- a/test_plans/runtime_vf_queue_number_test_plan.rst > +++ b/test_plans/runtime_vf_queue_number_test_plan.rst > @@ -338,7 +338,7 @@ Verify error information:: > Test case 7: Reserve VF queue number when VF bind to kernel driver > ================================================================== > > -1. bind vf to kernel driver i40evf:: > +1. bind vf to kernel driver iavf:: > > ./usertools/dpdk-devbind.py -b i40e 0000:18:02.0 > > diff --git a/test_plans/veb_switch_test_plan.rst > b/test_plans/veb_switch_test_plan.rst > index ca8bd7e0..1d7d11c2 100644 > --- a/test_plans/veb_switch_test_plan.rst > +++ b/test_plans/veb_switch_test_plan.rst > @@ -86,8 +86,8 @@ Prerequisites for VEB testing > ./dpdk-devbind.py -b igb_uio 05:00.0 > echo 2 >/sys/bus/pci/devices/0000:05:00.0/max_vfs > ./dpdk-devbind.py --st > - 0000:05:02.0 'XL710/X710 Virtual Function' unused=i40evf,igb_uio > - 0000:05:02.1 'XL710/X710 Virtual Function' unused=i40evf,igb_uio > + 0000:05:02.0 'XL710/X710 Virtual Function' unused=iavf,igb_uio > + 0000:05:02.1 'XL710/X710 Virtual Function' unused=iavf,igb_uio > > 4. Bind the VFs to dpdk driver:: > > diff --git a/test_plans/vf_pf_reset_test_plan.rst > b/test_plans/vf_pf_reset_test_plan.rst > index 009e99a9..574d4510 100644 > --- a/test_plans/vf_pf_reset_test_plan.rst > +++ b/test_plans/vf_pf_reset_test_plan.rst > @@ -589,11 +589,11 @@ test Case 10: vf reset (two vfs passed through to > two VM) > using `lspci -nn|grep -i ethernet` got VF device id, for example "8086 154c":: > > echo "8086 154c" > /sys/bus/pci/drivers/pci-stub/new_id > - echo "0000:82:02.0" > /sys/bus/pci/drivers/i40evf/unbind > + echo "0000:82:02.0" > /sys/bus/pci/drivers/iavf/unbind > echo "0000:82:02.0" > /sys/bus/pci/drivers/pci-stub/bind > > echo "8086 154c" > /sys/bus/pci/drivers/pci-stub/new_id > - echo "0000:82:02.1" > /sys/bus/pci/drivers/i40evf/unbind > + echo "0000:82:02.1" > /sys/bus/pci/drivers/iavf/unbind > echo "0000:82:02.1" > /sys/bus/pci/drivers/pci-stub/bind > > 3. Pass through VF0 81:02.0 to vm0, VF1 81:02.1 to vm1:: > -- > 2.33.1 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [dts] [PATCH V2 3/3] tests/vf_kernel: modify dts because of i40evf removed 2021-10-20 18:10 [dts] [PATCH V2 0/3] *** modify dts because of i40evf removed *** Yan Xia 2021-10-20 18:10 ` [dts] [PATCH V2 1/3] framework: modify dts because of i40evf removed Yan Xia 2021-10-20 18:10 ` [dts] [PATCH V2 2/3] test_plans: " Yan Xia @ 2021-10-20 18:10 ` Yan Xia 2021-10-21 10:21 ` Chen, LingliX 2021-10-22 6:29 ` Lin, Xueqin 2 siblings, 2 replies; 9+ messages in thread From: Yan Xia @ 2021-10-20 18:10 UTC (permalink / raw) To: dts; +Cc: Yan Xia modify dts because of i40evf removed dpdk commit-id: c65f9258 Signed-off-by: Yan Xia <yanx.xia@intel.com> --- tests/TestSuite_vf_kernel.py | 37 +++++++++++++++++++++++++++--------- 1 file changed, 28 insertions(+), 9 deletions(-) diff --git a/tests/TestSuite_vf_kernel.py b/tests/TestSuite_vf_kernel.py index a4e1e52a..e685db65 100755 --- a/tests/TestSuite_vf_kernel.py +++ b/tests/TestSuite_vf_kernel.py @@ -315,7 +315,10 @@ class TestVfKernel(TestCase): self.verify("Link detected: yes" in out, "Wrong link status") # Unload VF1 kernel driver and expect no problem for VF0 - self.vm1_dut.send_expect("rmmod %svf" % self.kdriver, "#") + if self.kdriver == "i40e": + self.vm1_dut.send_expect("rmmod iavf", "#") + else: + self.vm1_dut.send_expect("rmmod %svf" % self.kdriver, "#") out = self.vm0_dut.send_expect("ethtool %s" % self.vm0_intf0, "#") self.verify("Link detected: yes" in out, "Wrong link status") vm0_vf0_mac = self.vm0_dut.ports_info[0]['port'].get_mac_addr() @@ -330,20 +333,29 @@ class TestVfKernel(TestCase): self.verify(self.verify_vm_tcpdump(self.vm0_dut, self.vm0_intf0, vm0_vf0_mac), "reset PF testpmd impact VF RX failure") - self.vm1_dut.send_expect("modprobe %svf" % self.kdriver, "#") + if self.kdriver == "i40e": + self.vm1_dut.send_expect("modprobe iavf", "#") + else: + self.vm1_dut.send_expect("modprobe %svf" % self.kdriver, "#") out = self.vm0_dut.send_expect("ethtool %s" % self.vm0_intf0, "#") self.verify("Link detected: yes" in out, "Wrong link status") vm0_vf0_mac = self.vm0_dut.ports_info[0]['port'].get_mac_addr() self.verify(self.verify_vm_tcpdump(self.vm0_dut, self.vm0_intf0, vm0_vf0_mac), "load VF1 kernel driver impact VF0") - self.vm1_dut.send_expect("rmmod %svf" % self.kdriver, "#") + if self.kdriver == "i40e": + self.vm1_dut.send_expect("rmmod iavf", "#") + else: + self.vm1_dut.send_expect("rmmod %svf" % self.kdriver, "#") out = self.vm0_dut.send_expect("ethtool %s" % self.vm0_intf0, "#") self.verify("Link detected: yes" in out, "Wrong link status") vm0_vf0_mac = self.vm0_dut.ports_info[0]['port'].get_mac_addr() self.verify(self.verify_vm_tcpdump(self.vm0_dut, self.vm0_intf0, vm0_vf0_mac), "Reset VF1 kernel driver impact VF0") - self.vm1_dut.send_expect("modprobe %svf" % self.kdriver, "#") + if self.kdriver == "i40e": + self.vm1_dut.send_expect("modprobe iavf", "#") + else: + self.vm1_dut.send_expect("modprobe %svf" % self.kdriver, "#") def test_address(self): """ @@ -1048,12 +1060,19 @@ class TestVfKernel(TestCase): Load kernel driver stress """ for i in range(100): - out = self.vm0_dut.send_expect("rmmod %svf" % self.kdriver, "#") - self.verify('error' not in out, + if self.kdriver == "i40e": + out = self.vm0_dut.send_expect("rmmod iavf", "#") + self.verify('error' not in out, + "stress error for rmmod iavf:%s" %out) + out = self.vm0_dut.send_expect("modprobe iavf", "#") + self.verify('error' not in out, "stress error for modprobe iavf:%s" %out) + else: + out = self.vm0_dut.send_expect("rmmod %svf" % self.kdriver, "#") + self.verify('error' not in out, "stress error for rmmod %svf:%s" % (self.kdriver, out)) - out = self.vm0_dut.send_expect("modprobe %svf" % self.kdriver, "#") - self.verify('error' not in out, "stress error for modprobe %svf:%s" % ( - self.kdriver, out)) + out = self.vm0_dut.send_expect("modprobe %svf" % self.kdriver, "#") + self.verify('error' not in out, "stress error for modprobe %svf:%s" % ( + self.kdriver, out)) def tear_down(self): """ -- 2.33.1 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V2 3/3] tests/vf_kernel: modify dts because of i40evf removed 2021-10-20 18:10 ` [dts] [PATCH V2 3/3] tests/vf_kernel: " Yan Xia @ 2021-10-21 10:21 ` Chen, LingliX 2021-10-22 6:29 ` Lin, Xueqin 1 sibling, 0 replies; 9+ messages in thread From: Chen, LingliX @ 2021-10-21 10:21 UTC (permalink / raw) To: dts [-- Attachment #1: Type: text/plain, Size: 454 bytes --] > -----Original Message----- > From: dts <dts-bounces@dpdk.org> On Behalf Of Yan Xia > Sent: Thursday, October 21, 2021 2:10 AM > To: dts@dpdk.org > Cc: Xia, YanX <yanx.xia@intel.com> > Subject: [dts] [PATCH V2 3/3] tests/vf_kernel: modify dts because of i40evf > removed > > modify dts because of i40evf removed > dpdk commit-id: c65f9258 > > Signed-off-by: Yan Xia <yanx.xia@intel.com> > --- Tested-by: Yan Xia <yanx.xia@intel.com> [-- Attachment #2: TestVfKernel.log --] [-- Type: application/octet-stream, Size: 399967 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V2 3/3] tests/vf_kernel: modify dts because of i40evf removed 2021-10-20 18:10 ` [dts] [PATCH V2 3/3] tests/vf_kernel: " Yan Xia 2021-10-21 10:21 ` Chen, LingliX @ 2021-10-22 6:29 ` Lin, Xueqin 2021-10-25 9:05 ` Tu, Lijuan 1 sibling, 1 reply; 9+ messages in thread From: Lin, Xueqin @ 2021-10-22 6:29 UTC (permalink / raw) To: Xia, YanX, dts; +Cc: Xia, YanX > -----Original Message----- > From: dts <dts-bounces@dpdk.org> On Behalf Of Yan Xia > Sent: Thursday, October 21, 2021 2:10 AM > To: dts@dpdk.org > Cc: Xia, YanX <yanx.xia@intel.com> > Subject: [dts] [PATCH V2 3/3] tests/vf_kernel: modify dts because of i40evf > removed > > modify dts because of i40evf removed > dpdk commit-id: c65f9258 > > Signed-off-by: Yan Xia <yanx.xia@intel.com> Acked-by: Xueqin Lin <xueqin.lin@intel.com> > --- > tests/TestSuite_vf_kernel.py | 37 +++++++++++++++++++++++++++--------- > 1 file changed, 28 insertions(+), 9 deletions(-) > > diff --git a/tests/TestSuite_vf_kernel.py b/tests/TestSuite_vf_kernel.py index > a4e1e52a..e685db65 100755 > --- a/tests/TestSuite_vf_kernel.py > +++ b/tests/TestSuite_vf_kernel.py > @@ -315,7 +315,10 @@ class TestVfKernel(TestCase): > self.verify("Link detected: yes" in out, "Wrong link status") > > # Unload VF1 kernel driver and expect no problem for VF0 > - self.vm1_dut.send_expect("rmmod %svf" % self.kdriver, "#") > + if self.kdriver == "i40e": > + self.vm1_dut.send_expect("rmmod iavf", "#") > + else: > + self.vm1_dut.send_expect("rmmod %svf" % self.kdriver, "#") > out = self.vm0_dut.send_expect("ethtool %s" % self.vm0_intf0, "#") > self.verify("Link detected: yes" in out, "Wrong link status") > vm0_vf0_mac = self.vm0_dut.ports_info[0]['port'].get_mac_addr() > @@ -330,20 +333,29 @@ class TestVfKernel(TestCase): > self.verify(self.verify_vm_tcpdump(self.vm0_dut, self.vm0_intf0, > vm0_vf0_mac), "reset PF testpmd impact VF RX > failure") > > - self.vm1_dut.send_expect("modprobe %svf" % self.kdriver, "#") > + if self.kdriver == "i40e": > + self.vm1_dut.send_expect("modprobe iavf", "#") > + else: > + self.vm1_dut.send_expect("modprobe %svf" % self.kdriver, > + "#") > out = self.vm0_dut.send_expect("ethtool %s" % self.vm0_intf0, "#") > self.verify("Link detected: yes" in out, "Wrong link status") > vm0_vf0_mac = self.vm0_dut.ports_info[0]['port'].get_mac_addr() > self.verify(self.verify_vm_tcpdump(self.vm0_dut, self.vm0_intf0, > vm0_vf0_mac), "load VF1 kernel driver impact VF0") > > - self.vm1_dut.send_expect("rmmod %svf" % self.kdriver, "#") > + if self.kdriver == "i40e": > + self.vm1_dut.send_expect("rmmod iavf", "#") > + else: > + self.vm1_dut.send_expect("rmmod %svf" % self.kdriver, "#") > out = self.vm0_dut.send_expect("ethtool %s" % self.vm0_intf0, "#") > self.verify("Link detected: yes" in out, "Wrong link status") > vm0_vf0_mac = self.vm0_dut.ports_info[0]['port'].get_mac_addr() > self.verify(self.verify_vm_tcpdump(self.vm0_dut, self.vm0_intf0, > vm0_vf0_mac), "Reset VF1 kernel driver impact VF0") > - self.vm1_dut.send_expect("modprobe %svf" % self.kdriver, "#") > + if self.kdriver == "i40e": > + self.vm1_dut.send_expect("modprobe iavf", "#") > + else: > + self.vm1_dut.send_expect("modprobe %svf" % self.kdriver, > + "#") > > def test_address(self): > """ > @@ -1048,12 +1060,19 @@ class TestVfKernel(TestCase): > Load kernel driver stress > """ > for i in range(100): > - out = self.vm0_dut.send_expect("rmmod %svf" % self.kdriver, "#") > - self.verify('error' not in out, > + if self.kdriver == "i40e": > + out = self.vm0_dut.send_expect("rmmod iavf", "#") > + self.verify('error' not in out, > + "stress error for rmmod iavf:%s" %out) > + out = self.vm0_dut.send_expect("modprobe iavf", "#") > + self.verify('error' not in out, "stress error for modprobe > iavf:%s" %out) > + else: > + out = self.vm0_dut.send_expect("rmmod %svf" % self.kdriver, "#") > + self.verify('error' not in out, > "stress error for rmmod %svf:%s" % (self.kdriver, out)) > - out = self.vm0_dut.send_expect("modprobe %svf" % self.kdriver, "#") > - self.verify('error' not in out, "stress error for modprobe %svf:%s" % ( > - self.kdriver, out)) > + out = self.vm0_dut.send_expect("modprobe %svf" % self.kdriver, "#") > + self.verify('error' not in out, "stress error for modprobe %svf:%s" % ( > + self.kdriver, out)) > > def tear_down(self): > """ > -- > 2.33.1 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V2 3/3] tests/vf_kernel: modify dts because of i40evf removed 2021-10-22 6:29 ` Lin, Xueqin @ 2021-10-25 9:05 ` Tu, Lijuan 0 siblings, 0 replies; 9+ messages in thread From: Tu, Lijuan @ 2021-10-25 9:05 UTC (permalink / raw) To: Lin, Xueqin, Xia, YanX, dts; +Cc: Xia, YanX > -----Original Message----- > From: dts <dts-bounces@dpdk.org> On Behalf Of Lin, Xueqin > Sent: 2021年10月22日 14:29 > To: Xia, YanX <yanx.xia@intel.com>; dts@dpdk.org > Cc: Xia, YanX <yanx.xia@intel.com> > Subject: Re: [dts] [PATCH V2 3/3] tests/vf_kernel: modify dts because of i40evf > removed > > > -----Original Message----- > > From: dts <dts-bounces@dpdk.org> On Behalf Of Yan Xia > > Sent: Thursday, October 21, 2021 2:10 AM > > To: dts@dpdk.org > > Cc: Xia, YanX <yanx.xia@intel.com> > > Subject: [dts] [PATCH V2 3/3] tests/vf_kernel: modify dts because of > > i40evf removed > > > > modify dts because of i40evf removed > > dpdk commit-id: c65f9258 > > > > Signed-off-by: Yan Xia <yanx.xia@intel.com> > Acked-by: Xueqin Lin <xueqin.lin@intel.com> Applied ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-10-25 9:06 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-10-20 18:10 [dts] [PATCH V2 0/3] *** modify dts because of i40evf removed *** Yan Xia 2021-10-20 18:10 ` [dts] [PATCH V2 1/3] framework: modify dts because of i40evf removed Yan Xia 2021-10-22 6:29 ` Lin, Xueqin 2021-10-20 18:10 ` [dts] [PATCH V2 2/3] test_plans: " Yan Xia 2021-10-22 6:31 ` Lin, Xueqin 2021-10-20 18:10 ` [dts] [PATCH V2 3/3] tests/vf_kernel: " Yan Xia 2021-10-21 10:21 ` Chen, LingliX 2021-10-22 6:29 ` Lin, Xueqin 2021-10-25 9:05 ` Tu, Lijuan
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).