[dts] [PATCH V1] update fortville vlan filte case

Liu, Yong yong.liu at intel.com
Wed Aug 24 10:48:55 CEST 2016


Huilong, that's fine. Please send V2 patch with these comments.

> -----Original Message-----
> From: Xu, HuilongX
> Sent: Tuesday, August 23, 2016 3:53 PM
> To: Liu, Yong; Tu, LijuanX A; dts at dpdk.org
> Cc: Tu, LijuanX A
> Subject: RE: [dts] [PATCH V1] update fortville vlan filte case
> 
> Hi yong,
> I am sorry, this patch from me.
> FVL Nic the vlan filter and promisc use same switch. So when we test vlan
> filter must disable promisc.
> 
> > -----Original Message-----
> > From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of Liu, Yong
> > Sent: Tuesday, August 23, 2016 3:44 PM
> > To: Tu, LijuanX A; dts at dpdk.org
> > Cc: Tu, LijuanX A
> > Subject: Re: [dts] [PATCH V1] update fortville vlan filte case
> >
> > Lijuan, please add more information in the git log.
> > We couldn't get the point that why need to disable promisc mode
> > mandatory.
> >
> >
> > > -----Original Message-----
> > > From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of Lijuan Tu
> > > Sent: Tuesday, August 23, 2016 2:19 PM
> > > To: dts at dpdk.org
> > > Cc: Tu, LijuanX A
> > > Subject: [dts] [PATCH V1] update fortville vlan filte case
> > >
> > >     FVL nic must disabel promisc for vlan filter.
> > >
> > > Signed-off-by: Lijuan Tu <lijuanx.a.tu at intel.com>
> > > ---
> > >  tests/TestSuite_vlan.py | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/tests/TestSuite_vlan.py b/tests/TestSuite_vlan.py
> > > index 70a9813..43b025d 100644
> > > --- a/tests/TestSuite_vlan.py
> > > +++ b/tests/TestSuite_vlan.py
> > > @@ -77,6 +77,7 @@ class TestVlan(TestCase):
> > >
> > >          if self.nic in ["fortville_eagle", "fortville_spirit",
> > > "fortville_spirit_single", "fortpark_TLV"]:
> > >              self.dut.send_expect("vlan set filter on %s" %
> > dutRxPortId,
> > > "testpmd> ")
> > > +            self.dut.send_expect("set promisc all off",  "testpmd> ")
> > >
> > >          self.dut.send_expect("vlan set strip off %s" % dutRxPortId,
> > > "testpmd> ")
> > >          self.verify('Set mac packet forwarding mode' in out, "set fwd
> > > rxonly error")
> > > --
> > > 1.9.3



More information about the dts mailing list