On Tue, Jun 18, 2013 at 4:15 PM, Andy Zhou <az...@nicira.com> wrote:
> Add wildcarded flow support in kernel datapath.
>
> Wildcarded flow can improve OVS flow set up performance by avoid sending
> matching new flows to the user space program. The exact performance boost
> will largely dependent on wildcarded flow hit rate.
>
> In case all new flows hits wildcard flows, the flow set up rate is
> within 5% of that of linux bridge module.
>
> Pravin has made significant contributions to this patch. Including API
> clean ups and bug fixes.
>
> Co-authored-by: Pravin B Shelar <pshe...@nicira.com>
> Signed-off-by: Pravin B Shelar <pshe...@nicira.com>
> Signed-off-by: Andy Zhou <az...@nicira.com>

I made some incremental changes, can you take a look to see if they
seem reasonable to you?

Here are the main blocks:
 - Additional interface documentation.
 - Fix memory leak when installing a new flow if there is an
allocation failure for the mask.
 - Require Ethernet addresses to be present in the key, as before.
 - Make SNAP handling a little more integrated with the rest of the
parsing and validation code.
 - Make ovs_flow_free() and ovs_deferred_flow_free() have exactly the
same behavior, just with the addition of RCU, to avoid possible
confusion.
 - Return errors directly rather than through an intermediate variable
in ovs_flow_extract() and children.
 - Enforce an exact match for the outer EtherType for vlan packets,
similar to what we do for other protocols.
 - Miscellaneous style fixes.

Attachment: 0001-Cleanups.patch
Description: Binary data

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to