On Thu, Feb 03, 2011 at 07:24:48PM -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().
I don't see a problem with the code, but I think that it could use better documentation. The skip_actions_cb member, in my opinion, deserves a bigger comment explaining what the actions_cb is for and why one would want to skip it. I always find "negative" flags confusing, so I would consider renaming skip_actions_cb as call_actions_cb and inverting the meaning. The actions_cb only takes the actions and nf_output_iface parameters so that it can set them to constants if and only if it returns false. It might be better to drop those parameters and have the caller take care of those tasks if it returns false. It just occurred to me that we still probably don't get some of these packets up in the bridge if there is no flow at all. I guess we could still pass packets that miss in the flow table up to the actions hook somehow? Especially if we drop those "extra" parameters. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
