[ovs-dev] [vlans 5/5] Implement "native VLAN" feature.

Ben Pfaff blp at nicira.com
Wed Sep 21 21:37:57 UTC 2011


On Wed, Sep 21, 2011 at 01:17:16PM -0700, Ethan Jackson wrote:
> The comments in set_dst need reformating.  They have redundant blank lines in
> the beginining of the comment.  They need to capitolize sentences and use
> periods.  The ending "*/" needs to join with the last line if possible, etc.
> 
> +            802.1Q header.  A packet that egresses on a trunk port, it will
> port, it will => port will

Thanks, I applied those changes.

When I reran the tests, however, I found that we have regressions from
1.2.x in this area.

I also discovered that mirroring doesn't work with this new feature
properly.

I'm going to work on these problems, then I'll post the fix.



More information about the dev mailing list