On Tue, Aug 23, 2011 at 12:54:19AM +0200, Philippe Jung wrote:
> Le 22/08/2011 22:27, Ben Pfaff a ?crit :
>
> > 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.
>
> Please find enclosed the new version of the patch to add native VLAN feature.
> Compared to previous version, major changes are:
> - fixed the previously mentionned bug;
> - after performing more tests, I found few mistakes in code logic. they are
> now fixed;
> - took into account your remarks (PVID to native, coding style, enums ...). I
> hope I fixed everything :-)
Hi Philippe. Just dropping you a note to let you know that I haven't
forgotten your patch. I spent some time writing a separate commit
that fixes the bug that you reported and posted it in this thread,
along with some unit tests for VLANs:
http://openvswitch.org/pipermail/dev/2011-August/010779.html
I'll get back to your patch soon.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
