[ovs-dev] [PATCH v7 0/4] conntrack: add all-zero SNAT
Paolo Valerio
pvalerio at redhat.com
Mon Jun 21 10:05:42 UTC 2021
Two more patches has been introduced.
1/4 is a prereq for the series because it fixes an issue that prevents
OVN to use all-zero snat due to the way it builds the pipeline.
The patch 2/4 has been introduced in v6 as a prereq of 3/4.
All the versions up to v6 were about patch 3/4.
{2,3}/4 are untouched, so the acks are kept.
v7: fixed the multiple ct(commit,[dst|src]) issue spotted by Dumitru
and adds the ct_get_features callback for userspace datapath
advertising the availability of all-zero snat.
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].
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.
v4: no code changes, just restored some removed new line.
v3: replaced NULL with all-zero in the commit message.
v2: enabled NULL SNAT self-test also for userspace.
Note for the maintainers:
{3,4}/4 depend on:
https://patchwork.ozlabs.org/project/openvswitch/patch/162331699885.2208579.16546865084041166731.stgit@ebuild/
Paolo Valerio (4):
conntrack: handle already natted packets
util.h: add token concatenation macro with argument expansion
conntrack: handle SNAT with all-zero IP address
dpif-netdev: add all-zero SNAT to the advertised features of ct
NEWS | 3 +
lib/cmap.h | 5 +-
lib/conntrack-private.h | 33 +++
lib/conntrack.c | 335 +++++++++++++++++++------------
lib/dpif-netdev.c | 12 +-
lib/ovs-actions.xml | 3 +-
lib/util.h | 7 +
tests/system-userspace-macros.at | 8 +-
8 files changed, 270 insertions(+), 136 deletions(-)
More information about the dev
mailing list