From: Ilya Maximets <i.maximets@ovn.org>
To: netdev@vger.kernel.org
Cc: "David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Simon Horman <horms@kernel.org>,
Donald Hunter <donald.hunter@gmail.com>,
Shuah Khan <shuah@kernel.org>,
Adrian Moreno <amorenoz@redhat.com>, Jiri Benc <jbenc@redhat.com>,
Nicolas Dichtel <nicolas.dichtel@6wind.com>,
linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org,
Matteo Perin <matteo.perin@canonical.com>,
Ilya Maximets <i.maximets@ovn.org>
Subject: [PATCH net 2/5] selftests: net: add a test case for cross-namespace peer netns
Date: Fri, 15 May 2026 22:19:21 +0200 [thread overview]
Message-ID: <20260515201937.2813983-3-i.maximets@ovn.org> (raw)
In-Reply-To: <20260515201937.2813983-1-i.maximets@ovn.org>
The test makes requests with RTM_GETLINK and TARGET_NETNS and verifies
that reported LINK_NSID is correct and only reported when it is needed
from the querier's perspective.
Assisted-by: OpenCode:claude-opus-4.6
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
---
tools/testing/selftests/net/link_netns.py | 49 ++++++++++++++++++++++-
1 file changed, 47 insertions(+), 2 deletions(-)
diff --git a/tools/testing/selftests/net/link_netns.py b/tools/testing/selftests/net/link_netns.py
index aab043c59d695..2aae422d3f8a6 100755
--- a/tools/testing/selftests/net/link_netns.py
+++ b/tools/testing/selftests/net/link_netns.py
@@ -3,13 +3,14 @@
import time
-from lib.py import ksft_run, ksft_exit, ksft_true
+from lib.py import ksft_run, ksft_exit, ksft_eq, ksft_true
from lib.py import ip
from lib.py import NetNS, NetNSEnter
from lib.py import RtnlFamily
LINK_NETNSID = 100
+LINK_NETNSID2 = 200
def test_event() -> None:
@@ -132,8 +133,52 @@ def test_peer_net() -> None:
ip(f"link del foo", ns=tgt_net)
+def test_peer_net_cross_ns() -> None:
+ """Cross-namespace RTM_GETLINK queries using target-netnsid.
+ IFLA_LINK_NETNSID should report the link peer's namespace from the
+ querier's perspective. Absent means the peer is in the querier's
+ own namespace. Must not create self-referential nsid mappings."""
+
+ with NetNS() as ns1, NetNS() as ns2, NetNS() as ns3:
+ net1, net2, net3 = str(ns1), str(ns2), str(ns3)
+
+ ip(f"netns set {net2} {LINK_NETNSID}", ns=net1)
+ ip(f"netns set {net3} {LINK_NETNSID2}", ns=net1)
+
+ with NetNSEnter(net1):
+ rtnl = RtnlFamily()
+
+ cases = [
+ # "dev netns", "peer netns", "query nsid", expected link-netnsid.
+ (net2, net1, LINK_NETNSID, None),
+ (net2, net2, LINK_NETNSID, LINK_NETNSID),
+ (net2, net3, LINK_NETNSID, LINK_NETNSID2),
+ ]
+
+ for dev_ns, peer_ns, query_nsid, exp in cases:
+ ip(f"link add foo netns {dev_ns} type veth"
+ f" peer name bar netns {peer_ns}")
+
+ resp = rtnl.getlink({"target-netnsid": query_nsid,
+ "ifname": "foo"})
+ ksft_eq(resp.get("link-netnsid"), exp,
+ f"link-netnsid mismatch for dev={dev_ns} peer={peer_ns}")
+
+ ip("link del foo", ns=dev_ns)
+
+ # Verify no extra nsid was created by the queries.
+ nsids = ip("netns list-id", ns=net1, json=True)
+ ksft_eq(len(nsids), 2,
+ f"unexpected nsid mappings after cross-ns queries: {nsids}")
+
+
def main() -> None:
- ksft_run([test_event, test_link_net, test_peer_net])
+ ksft_run([
+ test_event,
+ test_link_net,
+ test_peer_net,
+ test_peer_net_cross_ns,
+ ])
ksft_exit()
--
2.53.0
next prev parent reply other threads:[~2026-05-15 20:19 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-15 20:19 [PATCH net 0/5] netlink: fixes for cross-namespace nsid reporting Ilya Maximets
2026-05-15 20:19 ` [PATCH net 1/5] net: rtnetlink: fix link nsid reported when the link is local Ilya Maximets
2026-05-15 20:19 ` Ilya Maximets [this message]
2026-05-15 20:19 ` [PATCH net 3/5] net: netlink: don't set nsid on local notifications Ilya Maximets
2026-05-15 20:19 ` [PATCH net 4/5] tools: ynl: support listening on all nsids Ilya Maximets
2026-05-15 20:19 ` [PATCH net 5/5] selftests: net: add a test case for nsid in all nsid notifications Ilya Maximets
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=20260515201937.2813983-3-i.maximets@ovn.org \
--to=i.maximets@ovn.org \
--cc=amorenoz@redhat.com \
--cc=davem@davemloft.net \
--cc=donald.hunter@gmail.com \
--cc=edumazet@google.com \
--cc=horms@kernel.org \
--cc=jbenc@redhat.com \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=matteo.perin@canonical.com \
--cc=netdev@vger.kernel.org \
--cc=nicolas.dichtel@6wind.com \
--cc=pabeni@redhat.com \
--cc=shuah@kernel.org \
/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