[ovs-dev] [PATCH] vswitchd: Process special packets more aggressively.

Ben Pfaff blp at nicira.com
Mon Feb 7 21:32:24 UTC 2011


On Fri, Feb 04, 2011 at 03:02:37PM -0800, Ethan Jackson wrote:
> Before this patch, special packets such as LACP and CFM messages
> were only processed if they had NORMAL open flow actions.  With
> this patch these messages are always processed unless originated in
> ofproto_send_packet().

It looks fine, but is "admissible" the right term?  We already have a
use of that term through the function is_admissible() in bridge.c.  This
commit introduces a new and (I think) quite different meaning of that
word.  I'd rather use a different word or phrase to avoid confusion.  

Otherwise I'm happy with this.  Thank you!




More information about the dev mailing list