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
