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

Ethan Jackson ethan at nicira.com
Mon Feb 7 21:52:34 UTC 2011


How about control_cb as in shove it up to the control path (bridge).
Or perhaps special_cb as in handle special packets.

Ethan

On Mon, Feb 7, 2011 at 1:32 PM, Ben Pfaff <blp at nicira.com> wrote:
> 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!
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
>




More information about the dev mailing list