[ovs-dev] [PATCH v3 0/3] Add dpif support for ct_clear action

[Eric Garver]

On Sat, Nov 18, 2017 at 07:41:43PM -0500, Eric Garver wrote:
> This series adds dpif support for OVS_ACTION_ATTR_CT_CLEAR. Previously the
> ct_clear action was a userspace only operation, but it has use cases in the
> dpif as well. Namely changing a packet's tuple after a ct() lookup has
> occurred.
> 
> Kernel support has already be accepted upstream:
>   b8226962b1c4 ("openvswitch: add ct_clear action").
> 
> travis-ci: https://travis-ci.org/erig0/ovs/builds/290487503
> 
> v3:
>     - add NEWS item
>     - add comment describing OVS_ACTION_ATTR_CT_CLEAR
>     - update parse_odp_action() and add ct_clear to odp test case
> v2:
>     - Specify port for nc in test (patch 3)
> 
> Eric Garver (3):
>   dpif: Add support for OVS_ACTION_ATTR_CT_CLEAR
>   system-common-macros: Check for ct_clear action in datapath
>   system-traffic: Add conntrack floating IP test
> 
>  NEWS                                              |  3 +
>  datapath/linux/compat/include/linux/openvswitch.h |  2 +
>  lib/conntrack.c                                   | 10 ++++
>  lib/conntrack.h                                   |  1 +
>  lib/dpif-netdev.c                                 |  1 +
>  lib/dpif.c                                        |  1 +
>  lib/odp-execute.c                                 |  7 +++
>  lib/odp-util.c                                    | 11 ++++
>  lib/ofp-actions.c                                 |  1 +
>  ofproto/ofproto-dpif-ipfix.c                      |  1 +
>  ofproto/ofproto-dpif-sflow.c                      |  1 +
>  ofproto/ofproto-dpif-xlate.c                      | 14 ++++-
>  ofproto/ofproto-dpif.c                            | 32 ++++++++++
>  ofproto/ofproto-dpif.h                            |  5 +-
>  tests/odp.at                                      |  1 +
>  tests/system-common-macros.at                     |  4 ++
>  tests/system-traffic.at                           | 73 +++++++++++++++++++++++
>  17 files changed, 166 insertions(+), 2 deletions(-)
> 
> -- 
> 2.12.0

Hi Ben, Justin,

This series has been ACKd for awhile. Can it be applied?
I can quickly rebase if needed.

Thanks.
Eric.