[ovs-dev] [PATCH 0/4] Native tunnel: Update neigh entries in tnl termination.

Paolo Valerio pvalerio at redhat.com
Tue Nov 2 17:12:06 UTC 2021


- Dropped the RFC tag
- Included Flavio's suggestions
- Squashed patch #5 into #2 and #3

The series is composed of the following patches:

#1: Expires is modified in different contexts (revalidator, pmd-rx, bfd-tx).
    It's probably not very racy for many reasons, but it seems a good idea
    to use an atomic here (especially with the introduction of #3).

#2: Introduces the possibility to set and read the entries timeout.
    It's useful for debugging purpose. Includes a self-test.

#3: Fixes a use case in which the tnl neigh entry never
    gets updated (further details in the commit message).
    Includes a self-test.

#4: While working on #2 and #3 I noticed unstable timeouts (more 
    visible the ageing time is low) during nd snoop. 
    It turned out to be a problem of revalidation. Additional
    details can be found in the commit description.

The use case fixed in #3 could be fixed relaxing the check in
tnl_arp_snoop(), allowing to snoop ARP requests as well, but such
approach could add unneeded entries in the cache.

Paolo Valerio (4):
      Native tunnel: Read/write expires atomically.
      Native tunnel: Add tnl/neigh/ageing command.
      Native tunnel: Do not refresh the entry while revalidating.
      Tunnel: Snoop ingress packets and update neigh cache if needed.


 NEWS                            |   3 +
 lib/tnl-neigh-cache.c           | 133 +++++++++++++++++++++++++-------
 lib/tnl-neigh-cache.h           |   4 +-
 ofproto/ofproto-dpif-xlate.c    |  17 +++-
 ofproto/ofproto-tnl-unixctl.man |   9 +++
 tests/tunnel-push-pop-ipv6.at   |  66 ++++++++++++++++
 tests/tunnel-push-pop.at        |  65 ++++++++++++++++
 7 files changed, 268 insertions(+), 29 deletions(-)



More information about the dev mailing list