[ovs-dev] [PATCH 1/1] vswitchd: PVID (Port VLAN ID) tagging feature

Ben Pfaff blp at nicira.com
Mon Aug 22 20:27:05 UTC 2011


On Mon, Aug 22, 2011 at 10:17:03PM +0200, Philippe Jung wrote:
> In original ofproto-dpif.c, line 1058, I read
> 
> 1058     /* Get trunked VLANs. */
> 1059     trunks = s->vlan == -1 ? NULL : s->trunks;
> 
> Is there a reason to explain that when the port is trunk (vlan=-1),
> trunks is set to nul and when port is access (vlan<>-1) trunks is set to
> s->trunk ? I would have inverted the test case.

I think you're right.  I guess nobody (except you) tests trunking a
limited set of VLANs.  We should fix this, thanks for the bug report.



More information about the dev mailing list