Re: [dts] [PATCH V1] tests/vf daemon : open vf filter

["Liu, Yong" <yong.liu@intel.com>]

Meijuan,
Since vlan filter setting is the prerequisite for vlan strip and filter, why only vlan_strip case added this setting?
Is it also needed for test_vlan_filter case?

Thanks,
Marvin

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of zhao,meijuan
> Sent: Wednesday, May 23, 2018 2:42 PM
> To: dts@dpdk.org
> Cc: Zhao, MeijuanX <meijuanx.zhao@intel.com>
> Subject: [dts] [PATCH V1] tests/vf daemon : open vf filter
> 
> before set rx_vlan add command, need enable vf filter,
> before v18.02, vf filter default is enable
> 
> Signed-off-by: zhao,meijuan <meijuanx.zhao@intel.com>
> ---
>  tests/TestSuite_vf_daemon.py | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/TestSuite_vf_daemon.py b/tests/TestSuite_vf_daemon.py
> index b1ad522..fddfb98 100644
> --- a/tests/TestSuite_vf_daemon.py
> +++ b/tests/TestSuite_vf_daemon.py
> @@ -622,6 +622,7 @@ class Testvf_daemon(TestCase):
>          self.vm0_testpmd.execute_cmd('set verbose 1')
>          self.vm0_testpmd.execute_cmd('start')
>          for rx_vlan in rx_vlans:
> +            self.vm0_testpmd.execute_cmd('vlan set filter on 0')
>              self.vm0_testpmd.execute_cmd('rx_vlan add %s 0' % rx_vlan)
>              self.dut_testpmd.execute_cmd('set vf vlan stripq 0 0 off')
>              time.sleep(3)
> @@ -632,7 +633,7 @@ class Testvf_daemon(TestCase):
>              time.sleep(3)
>              out = self.send_and_vlanstrip(self.vf0_mac,rx_vlan)
>              self.verify(rx_vlan not in out, "Failed to disable strip
> vlan!!!")
> -
> +        self.vm0_testpmd.execute_cmd('vlan set filter off 0')
> 
>      def test_vlan_filter(self):
>          """
> --
> 1.9.3