[V1] tests/iavf_flexible_descriptor: add private flag for vf vlan

Message ID 20210222182620.7817-1-qinx.sun@intel.com (mailing list archive)
State Superseded
Headers
Series [V1] tests/iavf_flexible_descriptor: add private flag for vf vlan |

Commit Message

Sun, QinX Feb. 22, 2021, 6:26 p.m. UTC
  Signed-off-by: sunqin <qinx.sun@intel.com>
---
 tests/TestSuite_iavf_flexible_descriptor.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
  

Comments

Sun, QinX Feb. 22, 2021, 10:02 a.m. UTC | #1
Tested-by: Sun, QinX <qinx.sun@intel.com>
 
Regards,
Sun Qin

> -----Original Message-----
> From: Sun, QinX <qinx.sun@intel.com>
> Sent: Tuesday, February 23, 2021 2:26 AM
> To: dts@dpdk.org
> Cc: Sun, QinX <qinx.sun@intel.com>
> Subject: [dts] [PATCH V1] tests/iavf_flexible_descriptor: add private flag for vf vlan
  
Zhao, HaiyangX Feb. 22, 2021, 10:16 a.m. UTC | #2
Acked-by: Haiyang Zhao <haiyangx.zhao@intel.com>

Best Regards,
Zhao Haiyang

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Sun, QinX
> Sent: Monday, February 22, 2021 18:03
> To: dts@dpdk.org
> Subject: Re: [dts] [PATCH V1] tests/iavf_flexible_descriptor: add private flag
> for vf vlan
> 
> Tested-by: Sun, QinX <qinx.sun@intel.com>
> 
> Regards,
> Sun Qin
> 
> > -----Original Message-----
> > From: Sun, QinX <qinx.sun@intel.com>
> > Sent: Tuesday, February 23, 2021 2:26 AM
> > To: dts@dpdk.org
> > Cc: Sun, QinX <qinx.sun@intel.com>
> > Subject: [dts] [PATCH V1] tests/iavf_flexible_descriptor: add private flag
> for vf vlan
  
Tu, Lijuan March 2, 2021, 1:28 a.m. UTC | #3
> Signed-off-by: sunqin <qinx.sun@intel.com>
> ---
>  tests/TestSuite_iavf_flexible_descriptor.py | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/TestSuite_iavf_flexible_descriptor.py
> b/tests/TestSuite_iavf_flexible_descriptor.py
> index 69f38bf6..58dd850a 100644
> --- a/tests/TestSuite_iavf_flexible_descriptor.py
> +++ b/tests/TestSuite_iavf_flexible_descriptor.py
> @@ -79,10 +79,11 @@ class TestIavfFlexibleDescriptor(TestCase,
> FlexibleRxdBase):
>              netdev.bind_driver(driver=self.kdriver)
>          # set vf assign method and vf driver
>          vf_driver = 'vfio-pci'
> +        pf0_intf = self.dut.ports_info[self.dut_ports[dut_index]]['intf']
> +        self.dut.send_expect('ethtool --set-priv-flags {} vf-vlan-prune-disable
> on'.format(pf0_intf),'#')

Why add this setting ?

>          # generate 2 VFs on PF
>          self.dut.generate_sriov_vfs_by_port(
>              used_dut_port, 1, driver=self.kdriver)
> -        pf0_intf = self.dut.ports_info[self.dut_ports[dut_index]]['intf']
>          vf_mac = "00:11:22:33:44:55"
>          self.dut.send_expect(
>              'ip link set {} vf 0 mac {}'.format(pf0_intf, vf_mac), '#')
> --
> 2.17.1
  

Patch

diff --git a/tests/TestSuite_iavf_flexible_descriptor.py b/tests/TestSuite_iavf_flexible_descriptor.py
index 69f38bf6..58dd850a 100644
--- a/tests/TestSuite_iavf_flexible_descriptor.py
+++ b/tests/TestSuite_iavf_flexible_descriptor.py
@@ -79,10 +79,11 @@  class TestIavfFlexibleDescriptor(TestCase, FlexibleRxdBase):
             netdev.bind_driver(driver=self.kdriver)
         # set vf assign method and vf driver
         vf_driver = 'vfio-pci'
+        pf0_intf = self.dut.ports_info[self.dut_ports[dut_index]]['intf']
+        self.dut.send_expect('ethtool --set-priv-flags {} vf-vlan-prune-disable on'.format(pf0_intf),'#')
         # generate 2 VFs on PF
         self.dut.generate_sriov_vfs_by_port(
             used_dut_port, 1, driver=self.kdriver)
-        pf0_intf = self.dut.ports_info[self.dut_ports[dut_index]]['intf']
         vf_mac = "00:11:22:33:44:55"
         self.dut.send_expect(
             'ip link set {} vf 0 mac {}'.format(pf0_intf, vf_mac), '#')