From: Antonio Quartulli <antonio@openvpn.net>
To: netdev@vger.kernel.org
Cc: Antonio Quartulli <antonio@openvpn.net>,
Sabrina Dubroca <sd@queasysnail.net>,
Ralf Lici <ralf@mandelbit.com>, Jakub Kicinski <kuba@kernel.org>,
Paolo Abeni <pabeni@redhat.com>,
Andrew Lunn <andrew+netdev@lunn.ch>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>
Subject: [PATCH net-next 0/7] pull request: ovpn 2026-10-07
Date: Wed, 7 Oct 2026 15:30:09 +0200 [thread overview]
Message-ID: <20261007133018.1451958-1-antonio@openvpn.net> (raw)
Hi all!
Here is v3 of the ovpn batch for net-next, rebased on current net-next.
It is mostly cleanup. The one functional change is that PEER_DEL_NTF now
carries the same peer object returned by PEER_GET: a disconnecting
peer's final counters were not reachable before, because
ovpn_peer_remove() unlinks the peer before emitting the notification.
The selftests get TCP_NODELAY on ovpn-cli's TCP sockets and proper
synchronization against the server-side key setup. Ralf is added to
MAINTAINERS as ovpn reviewer.
Changes since v2:
* 5/7: the wait on the server-side key setup was unbounded, and the
subshell only reported the status of the last new_key. Bound the wait
and report a failure from any peer.
* 6/7: the notification filter used to drop every message carrying a
remote-ipv6, which also dropped the IPv6 float events. Those are not
deterministic and are not in the fixtures, so letting them through
made the notification check fail at random. Drop them again, this time
only for peer-float-ntf.
Both were reported by sashiko on v2. The second one was making roughly
15% of the selftest runs fail, so it is worth having before this lands.
With it, 33 consecutive script runs are clean.
Please pull or let me know of any issue!
Thanks a lot,
Antonio
The following changes since commit 6e23f90f2b8c3ea3bf904f86f3ed06bf2844b4bc:
Merge branch 'net-consolidate-devmem-net_iov-freelist-and-dma-handling' (2026-09-29 15:41:35 +0200)
are available in the Git repository at:
https://github.com/OpenVPN/ovpn-net-next.git tags/ovpn-net-next-20261007
for you to fetch changes up to e724d2ebb9519ce89ceadcd136ca6f2b9e404a32:
MAINTAINERS: ovpn: add Ralf Lici as reviewer (2026-10-06 02:24:31 +0200)
----------------------------------------------------------------
Included changes:
* add Ralf Lici as ovpn reviewer
* include the peer object in the PEER_DEL notification
* wait for the server-side key setup before generating TCP traffic in
selftests
* enable TCP_NODELAY on TCP sockets in selftests
* drop redundant peer NULL checks in the crypto post functions
* remove the unused work field from struct ovpn_socket
* fix the documented return value of ovpn_bind_from_sockaddr()
----------------------------------------------------------------
Antonio Quartulli (2):
selftests: ovpn: wait for the TCP server key setup before traffic
MAINTAINERS: ovpn: add Ralf Lici as reviewer
Karl Mehltretter (1):
ovpn: Fix the return value in ovpn_bind_from_sockaddr() kernel-doc
Marco Baffo (2):
ovpn: remove redundant peer NULL checks in crypto post functions
selftests: ovpn: enable TCP_NODELAY on TCP sockets
Ralf Lici (2):
ovpn: remove unused work field from struct ovpn_socket
ovpn: send peer object along with PEER_DEL_NTF
MAINTAINERS | 1 +
drivers/net/ovpn/bind.c | 2 +-
drivers/net/ovpn/io.c | 6 +-
drivers/net/ovpn/netlink.c | 60 +++++++-----
drivers/net/ovpn/socket.h | 2 -
tools/testing/selftests/net/ovpn/common.sh | 108 +++++++++++++++++----
.../selftests/net/ovpn/json/peer0-float.json | 12 +--
.../selftests/net/ovpn/json/peer0-symm-float.json | 10 +-
.../selftests/net/ovpn/json/peer0-symm.json | 7 +-
tools/testing/selftests/net/ovpn/json/peer0.json | 12 +--
.../selftests/net/ovpn/json/peer1-symm.json | 2 +-
tools/testing/selftests/net/ovpn/json/peer1.json | 2 +-
.../selftests/net/ovpn/json/peer2-symm.json | 2 +-
tools/testing/selftests/net/ovpn/json/peer2.json | 2 +-
.../selftests/net/ovpn/json/peer3-symm.json | 2 +-
tools/testing/selftests/net/ovpn/json/peer3.json | 2 +-
.../selftests/net/ovpn/json/peer4-symm.json | 2 +-
tools/testing/selftests/net/ovpn/json/peer4.json | 2 +-
.../selftests/net/ovpn/json/peer5-symm.json | 2 +-
tools/testing/selftests/net/ovpn/json/peer5.json | 2 +-
.../selftests/net/ovpn/json/peer6-symm.json | 2 +-
tools/testing/selftests/net/ovpn/json/peer6.json | 2 +-
tools/testing/selftests/net/ovpn/ovpn-cli.c | 23 +++++
.../selftests/net/ovpn/test-close-socket.sh | 2 +
tools/testing/selftests/net/ovpn/test.sh | 56 +++++------
25 files changed, 225 insertions(+), 100 deletions(-)
mode change 120000 => 100644 tools/testing/selftests/net/ovpn/json/peer0-symm-float.json
mode change 120000 => 100644 tools/testing/selftests/net/ovpn/json/peer0-symm.json
next reply other threads:[~2026-10-07 13:30 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-07 13:30 Antonio Quartulli [this message]
2026-10-07 13:30 ` [PATCH net-next 1/7] ovpn: Fix the return value in ovpn_bind_from_sockaddr() kernel-doc Antonio Quartulli
2026-10-07 13:30 ` [PATCH net-next 2/7] ovpn: remove unused work field from struct ovpn_socket Antonio Quartulli
2026-10-07 13:30 ` [PATCH net-next 3/7] ovpn: remove redundant peer NULL checks in crypto post functions Antonio Quartulli
2026-10-07 13:30 ` [PATCH net-next 4/7] selftests: ovpn: enable TCP_NODELAY on TCP sockets Antonio Quartulli
2026-10-08 13:32 ` netdev-bot+sashiko
2026-10-08 13:59 ` Neal Cardwell
2026-10-07 13:30 ` [PATCH net-next 5/7] selftests: ovpn: wait for the TCP server key setup before traffic Antonio Quartulli
2026-10-08 13:32 ` netdev-bot+sashiko
2026-10-07 13:30 ` [PATCH net-next 6/7] ovpn: send peer object along with PEER_DEL_NTF Antonio Quartulli
2026-10-08 13:32 ` netdev-bot+sashiko
2026-10-07 13:30 ` [PATCH net-next 7/7] MAINTAINERS: ovpn: add Ralf Lici as reviewer Antonio Quartulli
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20261007133018.1451958-1-antonio@openvpn.net \
--to=antonio@openvpn.net \
--cc=andrew+netdev@lunn.ch \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=kuba@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=ralf@mandelbit.com \
--cc=sd@queasysnail.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox