> -----Original Message----- > From: Alexander Duyck [mailto:[email protected]] > Sent: Wednesday, November 08, 2017 3:04 PM > To: Keller, Jacob E <[email protected]> > Cc: Brandeburg, Jesse <[email protected]>; [email protected]; > [email protected] > Subject: Re: [Intel-wired-lan] [jkirsher/next-queue PATCH 2/5] fm10k: Fix VLAN > configuration for macvlan offload > > On Wed, Nov 8, 2017 at 2:05 PM, Keller, Jacob E > <[email protected]> wrote: > >> -----Original Message----- > >> From: Intel-wired-lan [mailto:[email protected]] On Behalf > Of > >> Jesse Brandeburg > >> Sent: Friday, November 03, 2017 10:06 AM > >> To: Alexander Duyck <[email protected]> > >> Cc: [email protected]; [email protected] > >> Subject: Re: [Intel-wired-lan] [jkirsher/next-queue PATCH 2/5] fm10k: Fix > VLAN > >> configuration for macvlan offload > >> > >> On Thu, 2 Nov 2017 16:33:45 -0700 > >> Alexander Duyck <[email protected]> wrote: > >> > >> > From: Alexander Duyck <[email protected]> > >> > > >> > The fm10k driver didn't work correctly when macvlan offload was enabled. > >> > Specifically what would occur is that we would see no unicast packets > >> > being > >> > received. This was traced down to us not correctly configuring the > >> > default > >> > VLAN ID for the port and defaulting to 0. > >> > > >> > To correct this we either use the default ID provided by the switch or > >> > simply use 1. With that we are able to pass and receive traffic without > >> > any > >> > issues. > >> > >> Reviewed-by: Jesse Brandeburg <[email protected]> > >> > > > > Hi, > > > > I think this isn't quite right, since we recently made changes to the fm10k > > code > to stop assuming VLAN 1 in these cases. I believe it should just pass the > default_vid > > > > Thanks, > > Jake > > > > I kind of figured that might be the case. We can probably drop this > patch for now and if you want you can work this from our end as the > out-of-tree code doesn't make the upstream and I would imagine you > guys are planning to upstream that patch at some point. > > - Alex
I think the patch should just be changed from using hw->mac.default_vid ? : 1 to just using hw->mac.default_vid directly. Otherwise, I think the patch is necessary. Thanks, Jake
