Linux RDMA and InfiniBand development
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgg@ziepe.ca>
To: Sean Hefty <shefty@nvidia.com>
Cc: Eric Davis <eric.davis@broadcom.com>,
	"sashiko-reviews@lists.linux.dev"
	<sashiko-reviews@lists.linux.dev>,
	"linux-rdma@vger.kernel.org" <linux-rdma@vger.kernel.org>,
	Leon Romanovsky <leon@kernel.org>
Subject: Re: [PATCH rdma-next 02/13] RDMA/core: add the UET GID types
Date: Thu, 15 Oct 2026 19:54:27 -0300	[thread overview]
Message-ID: <20261015225427.GY14213@ziepe.ca> (raw)
In-Reply-To: <DS6PR12MB534567DB240CF76594BE5B5D0DBD9C2@DS6PR12MB534567.namprd12.prod.outlook.com>

On Thu, Oct 15, 2026 at 09:14:22PM +0000, Sean Hefty wrote:
> > > > > > Could this cause incoming UET packets to be incorrectly mapped
> > > > > > to a
> > > > > > RoCEv2 GID index, breaking the UET transport response?
> > > > >
> > > > > On a port that runs both RoCE and UET the MAD traffic is RoCE, so
> > > > > mapping it to RoCEv2 is correct.
> > > >
> > > > You will need seperate ROCE and UET ib_devices with the current
> > > > state of affairs, we don't support multi-protocol today.
> > >
> > > UEC intentionally considered multi-protocol devices, with a vendor deciding
> > whether it would need to provide separate SW devices.  E.g. using UET for
> > MPI, but RoCE for storage.  Protocol selection is via GID, same as RoCEv1 vs
> > v2.
> > >
> > > What blocks multi-protocol support?
> > 
> > Principally I don't think we have the APIs to discover and link the GUID to
> > protocol to QP type appropriately..
> 
> I'm missing something regarding the GUID comment.
> 
> The QP protocol comes from GID index, which is discoverable via
> ibv_gid_entry::gid_type.  (UET QPs require the GID index on create.)

Hmm, that's not the normal case, but I see what you are thinking

> E.g. a dual RoCE + UET device would report node_type = UNSPECIFIED &
> transport_type = UNSPECIFIED.  GID entries indicate supported
> protocols.

Ah, that would need some opt in scheme to get there, but I supppose it
may not be so bad.

I thought there were other issues like PDs not being entirely
compatible, MRs, and so on?

Jason

  reply	other threads:[~2026-10-15 22:54 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-14 22:21 [PATCH rdma-next 00/13] RDMA/core: Ultra Ethernet transport verbs Eric Davis via B4 Relay
2026-10-14 22:21 ` [PATCH rdma-next 01/13] RDMA/core: widen the uverbs object key Eric Davis via B4 Relay
2026-10-14 22:29   ` sashiko-bot
2026-10-14 22:21 ` [PATCH rdma-next 02/13] RDMA/core: add the UET GID types Eric Davis via B4 Relay
2026-10-14 22:35   ` sashiko-bot
2026-10-15 15:57     ` Eric Davis
2026-10-15 17:33       ` Jason Gunthorpe
2026-10-15 18:57         ` Sean Hefty
2026-10-15 19:54           ` Jason Gunthorpe
2026-10-15 21:14             ` Sean Hefty
2026-10-15 22:54               ` Jason Gunthorpe [this message]
2026-10-15 23:43                 ` Sean Hefty
2026-10-14 22:21 ` [PATCH rdma-next 03/13] RDMA/core: report the reliable unconnected device capabilities Eric Davis via B4 Relay
2026-10-14 22:30   ` sashiko-bot
2026-10-14 22:21 ` [PATCH rdma-next 04/13] RDMA/core: add the reliable unconnected queue pair type Eric Davis via B4 Relay
2026-10-14 22:43   ` sashiko-bot
2026-10-15 16:07     ` Eric Davis
2026-10-15 16:09       ` Eric Davis
2026-10-15 17:30         ` Jason Gunthorpe
2026-10-15 18:35           ` Doug Ledford
2026-10-14 22:21 ` [PATCH rdma-next 05/13] RDMA/core: extended address handle attributes Eric Davis via B4 Relay
2026-10-14 22:50   ` sashiko-bot
2026-10-15 16:15     ` Eric Davis
2026-10-15 17:30       ` Jason Gunthorpe
2026-10-14 22:21 ` [PATCH rdma-next 06/13] RDMA/core: query the semantics a queue pair type guarantees Eric Davis via B4 Relay
2026-10-14 22:34   ` sashiko-bot
2026-10-15 16:18     ` Eric Davis
2026-10-14 22:21 ` [PATCH rdma-next 07/13] RDMA/core: job and job key objects Eric Davis via B4 Relay
2026-10-14 22:36   ` sashiko-bot
2026-10-14 22:21 ` [PATCH rdma-next 08/13] RDMA/uverbs: factor out the queue pair attribute copy Eric Davis via B4 Relay
2026-10-14 22:30   ` sashiko-bot
2026-10-14 22:21 ` [PATCH rdma-next 09/13] RDMA/core: create a queue pair ready to send Eric Davis via B4 Relay
2026-10-14 22:39   ` sashiko-bot
2026-10-14 22:21 ` [PATCH rdma-next 10/13] RDMA/core: queue pair attributes for unconnected transports Eric Davis via B4 Relay
2026-10-14 22:41   ` sashiko-bot
2026-10-15 16:21     ` Eric Davis
2026-10-14 22:21 ` [PATCH rdma-next 11/13] RDMA/core: extended memory registration Eric Davis via B4 Relay
2026-10-14 22:36   ` sashiko-bot
2026-10-14 22:21 ` [PATCH rdma-next 12/13] RDMA/core: 64-bit memory keys Eric Davis via B4 Relay
2026-10-14 22:32   ` sashiko-bot
2026-10-14 22:21 ` [PATCH rdma-next 13/13] RDMA/core: attach a memory region to a queue pair Eric Davis via B4 Relay
2026-10-14 22:37   ` sashiko-bot
2026-10-14 23:57 ` [PATCH rdma-next 00/13] RDMA/core: Ultra Ethernet transport verbs Doug Ledford
2026-10-15  0:08   ` Jason Gunthorpe
2026-10-15  1:02     ` Doug Ledford
2026-10-15 17:29       ` Jason Gunthorpe
2026-10-15 18:22         ` Doug Ledford
2026-10-15 16:27     ` Eric Davis

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=20261015225427.GY14213@ziepe.ca \
    --to=jgg@ziepe.ca \
    --cc=eric.davis@broadcom.com \
    --cc=leon@kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=sashiko-reviews@lists.linux.dev \
    --cc=shefty@nvidia.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox