On Thu, Feb 04, 2016 at 04:10:10PM -0800, Jarno Rajahalme wrote:
> From: Ethan Jackson <et...@nicira.com>
>
> There are certain use cases (such as bond rebalancing) where a
> datapath flow's actions may change, while it's wildcard pattern
> remains the same. Before this patch, revalidators would note the
> change, delete the flow, and wait for the handlers to install an
> updated version. This is inefficient, as many packets could get
> punted to userspace before the new flow is finally installed.
>
> To improve the situation, this patch implements in place modification
> of datapath flows. If the revalidators detect the only change to a
> given ukey is its actions, instead of deleting it, it does a put with
> the MODIFY flag set.
>
> This patch is a backport of commit 43b2f13 to branch-2.3.
> This patch also squashes in commit c56eba3b7ab0 ("ofproto-dpif-upcall: Don't
> delete modified ukeys.").
>
> Signed-off-by: Ethan J. Jackson <et...@nicira.com>
> Signed-off-by: Jarno Rajahalme <ja...@ovn.org>
I reviewed this from a buffer management point of view. I'll leave the
other technicalities to Joe.
Acked-by: Ben Pfaff <b...@ovn.org>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
