All of lore.kernel.org
 help / color / mirror / Atom feed
From: Allison Henderson <achender@kernel.org>
To: netdev@vger.kernel.org, linux-rdma@vger.kernel.org,
	pabeni@redhat.com, edumazet@google.com, kuba@kernel.org,
	horms@kernel.org
Cc: achender@kernel.org
Subject: [PATCH net-next v4 0/2] net/rds: restrict the rdma_cm ids to IB devices
Date: Sat, 26 Sep 2026 23:30:56 -0700	[thread overview]
Message-ID: <20260927063058.170273-1-achender@kernel.org> (raw)

Hi all,

This is v4 of the rdma_cm id restriction (v1 at [1], v2 at [2], v3 at
[3]).  Patch 1 is the restriction itself; patch 2 is a small follow-on
the v3 review turned up in the function patch 1 touches.

  Patch 1 restricts the listener, the per-connection id and the probe
  id to RDMA_NODE_IB_CA with rdma_restrict_node_type(), so the rdma_cm
  never installs the RDS listener on, or binds an RDS id to, a device
  RDS has no transport for.

  Patch 2 has rds_rdma_listen_init_common() print the port it actually
  bound: the IPv6 listener sits on RDS_CM_PORT, not RDS_PORT.

Changes since v3 [3]:
 - Fixes: dcdede0406d3 ("RDS: Drop stale iWARP RDMA transport") added,
   with a note that stable trees without rdma_restrict_node_type() need
   the separate handler fix rather than a backport of this one, and
   that the two patches are independent.
 - New patch 2 for the listener debug line.
 - Rebased onto current net-next.
 - The v3 notes linked the wrong message id for v2; corrected below.

Changes since v2 [2]:
 - The changelog no longer claims a handler-side rejection exists in
   this tree; it names the separate net fix for the handler and the
   a760e80e90f5 prerequisite.  The node_type half of the
   rds_ib_laddr_check_cm() test, dead once the id is restricted, is
   removed rather than described.

[1] https://lore.kernel.org/netdev/20260917074108.174262-1-achender@kernel.org/
[2] https://lore.kernel.org/netdev/20260919061042.250462-1-achender@kernel.org/
[3] https://lore.kernel.org/netdev/20260922084837.390414-1-achender@kernel.org/

Thank you,
Allison


Allison Henderson (2):
  net/rds: restrict the rdma_cm ids to IB devices
  net/rds: log the port a listener actually bound

 net/rds/ib.c             | 14 ++++++--------
 net/rds/ib_cm.c          | 12 ++++++++++++
 net/rds/rdma_transport.c | 13 ++++++++++++-
 3 files changed, 30 insertions(+), 9 deletions(-)

-- 
2.25.1


             reply	other threads:[~2026-09-27  6:30 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-27  6:30 Allison Henderson [this message]
2026-09-27  6:30 ` [PATCH net-next v4 1/2] net/rds: restrict the rdma_cm ids to IB devices Allison Henderson
2026-09-27  6:37   ` sashiko-bot
2026-10-01 12:32   ` Simon Horman
2026-09-27  6:30 ` [PATCH net-next v4 2/2] net/rds: log the port a listener actually bound Allison Henderson
2026-09-27  6:37   ` sashiko-bot
2026-10-01 12:32   ` Simon Horman
2026-10-02  0:20 ` [PATCH net-next v4 0/2] net/rds: restrict the rdma_cm ids to IB devices patchwork-bot+netdevbpf

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=20260927063058.170273-1-achender@kernel.org \
    --to=achender@kernel.org \
    --cc=edumazet@google.com \
    --cc=horms@kernel.org \
    --cc=kuba@kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.