]> git.proxmox.com Git - mirror_ovs.git/commit
datapath: openvswitch: Optimize sample action for the clone use cases
authorAndy Zhou <azhou@ovn.org>
Thu, 6 Apr 2017 20:05:48 +0000 (13:05 -0700)
committerAndy Zhou <azhou@ovn.org>
Wed, 19 Apr 2017 19:59:46 +0000 (12:59 -0700)
commit7d9595e3516256e5839fb2ded5f8b787ca8e4dcb
tree016008f97790c9842671353bf76e1d619192b6c8
parent58afdef14e3e966cd82c0b1fe3ff7666f1923847
datapath: openvswitch: Optimize sample action for the clone use cases

Upstream commit:
    openvswitch: Optimize sample action for the clone use cases

    With the introduction of open flow 'clone' action, the OVS user space
    can now translate the 'clone' action into kernel datapath 'sample'
    action, with 100% probability, to ensure that the clone semantics,
    which is that the packet seen by the clone action is the same as the
    packet seen by the action after clone, is faithfully carried out
    in the datapath.

    While the sample action in the datpath has the matching semantics,
    its implementation is only optimized for its original use.
    Specifically, there are two limitation: First, there is a 3 level of
    nesting restriction, enforced at the flow downloading time. This
    limit turns out to be too restrictive for the 'clone' use case.
    Second, the implementation avoid recursive call only if the sample
    action list has a single userspace action.

    The main optimization implemented in this series removes the static
    nesting limit check, instead, implement the run time recursion limit
    check, and recursion avoidance similar to that of the 'recirc' action.
    This optimization solve both #1 and #2 issues above.

    One related optimization attempts to avoid copying flow key as
    long as the actions enclosed does not change the flow key. The
    detection is performed only once at the flow downloading time.

    Another related optimization is to rewrite the action list
    at flow downloading time in order to save the fast path from parsing
    the sample action list in its original form repeatedly.

Signed-off-by: Andy Zhou <azhou@ovn.org>
Acked-by: Pravin B Shelar <pshelar@ovn.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Upstream: 798c166173ff ("openvswitch: Optimize sample action for the clone use cases")
Signed-off-by: Andy Zhou <azhou@ovn.org>
Acked-by: Joe Stringer <joe@ovn.org>
datapath/actions.c
datapath/datapath.h
datapath/flow_netlink.c
datapath/linux/compat/include/linux/kernel.h
datapath/linux/compat/include/linux/openvswitch.h