test suite reviews and discussions
 help / color / mirror / Atom feed
* [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

* [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

* [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 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

* 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 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

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