[ovs-dev] [PATCH] dpif-provider: Get rid of redundant operations.

Joe Stringer joestringer at nicira.com
Thu Aug 7 02:25:42 UTC 2014


On 16 July 2014 11:09, Ben Pfaff <blp at nicira.com> wrote:

> The dpif provider 'operate' call duplicates all of the features available
> from the 'flow_put', 'flow_del', and 'execute' calls, yielding redundant
> code in providers that support both mechanisms.  This change drops the
> latter calls in favor of making every dpif provider support 'operate'.
> The result is code that is overall less duplicative.
>
> It might make sense to do the same with flow_get but so far 'operate'
> doesn't support flow_get.
>
> Signed-off-by: Ben Pfaff <blp at nicira.com>


Sorry for the delay on this. It looks good, but it doesn't apply against
master.



More information about the dev mailing list