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

Ethan Jackson ethan at nicira.com
Fri Feb 4 19:18:31 UTC 2011


> 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

I thought about doing it this way briefly.  I could imagine in the
future having a use for being able to manipulate the actions list.
For example, if the flow's action was to output on a bonded port, it
could be replaced by an action to output on one of the slaves.  At any
rate, this kind of functionality would be easy to add later so I think
it is clearer for now to simplify the callback and have false
automatically clear the actions

> 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.

I will look into this.




More information about the dev mailing list