mbox series

[ovs-dev,v6,0/2] conntrack: add all-zero SNAT and fix DNAT potential collision

Message ID 161964677372.293712.9125314386461543424.stgit@fed.void
Headers show
Series conntrack: add all-zero SNAT and fix DNAT potential collision | expand

Message

Paolo Valerio April 28, 2021, 9:56 p.m. UTC 
The patch has become a series of two patches.
The first patch has been introduced in v6 as a prereq of the second.
All the previous versions were about 2/2.

v2: enabled NULL SNAT self-test also for userspace.
v3: replaced NULL with all-zero in the commit message.
v4: no code changes, just restored some removed new line.
v5: added an entry to NEWS, updated ovs-actions.xml removing
    the kernel only exception, improved the range handling in
    case the packet source port is out of the ephemeral range
    (for SNAT without port range and DNAT actions), expanded
    some comment.
v6: moved CURSOR_JOIN from cmap.h to util.h and renamed it as OVS_JOIN
    reworked a little FOR_EACH_PORT_IN_RANGE() so that two arguments have
    been removed (one has been masked and the macro uses patch 1/2 to
    generate a unique index name based on __COUNTER__).
    INIT_ATT() has become N_PORT_ATTEMPTS().
    Moved all macros from conntrack.h to conntrack-private.h
    Changed ovs-actions.xml and system-userspace-macros.at according to the
    new version of [1].

Note for the maintainers:
patch 2/2 depends on [1]

[1] https://patchwork.ozlabs.org/project/openvswitch/patch/161943952616.327630.4676425878426520994.stgit@ebuild/

Paolo Valerio (2):
      util.h: add token concatenation macro with argument expansion
      conntrack: handle SNAT with all-zero IP address

 NEWS                             |   3 +
 lib/cmap.h                       |   5 +-
 lib/conntrack-private.h          |  33 +++
 lib/conntrack.c                  | 335 +++++++++++++++++++------------
 lib/ovs-actions.xml              |   3 +-
 lib/util.h                       |   7 +
 tests/system-userspace-macros.at |   8 +-
 7 files changed, 259 insertions(+), 135 deletions(-)