All of lore.kernel.org
 help / color / mirror / Atom feed
From: Haggai Eran <haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
To: Achiad Shochat <achiad-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Tom Talpey <tom-CLs1Zie5N5HQT0dZR+AlfA@public.gmane.org>,
	Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Matan Barak <matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Jason Gunthorpe
	<jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>,
	Or Gerlitz <ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Somnath Kotur
	<Somnath.Kotur-idTK6quXuVS1Z/+hSey0Gg@public.gmane.org>
Subject: Re: [PATCH for-next 00/10] Add RoCE support to the mlx5 driver
Date: Wed, 26 Aug 2015 08:31:35 +0300	[thread overview]
Message-ID: <55DD4F37.4020306@mellanox.com> (raw)
In-Reply-To: <55DC685A.8060402-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>

On 25/08/2015 16:06, Achiad Shochat wrote:
> On 8/25/2015 3:32 PM, Tom Talpey wrote:
>> But I don't understand how it will work per-GID. What if the target
>> node is RoCEv2 and on another subnet? How will it discover the remote's
>> capability and establish the right protocol? How does the initiator
>> select the protocol, if there is a choice?
>>
>> Another way of asking this question is, why is all this stuff in the
>> driver, at the bottom of the stack? I think it should be in the
>> rdmacm layer.
>>
>> Tom.
>>
> 
> Selection of the RoCE version to be used per connection and discovering
> the RoCE versions supported by the target is up to the application.
> 
> It is not done by the driver of course, neither by the kernel stack.
> 
> The driver only exposes add/del_gid callbacks, used by the ib_core to
> configure the device GID table entries (including RoCE version per
> entry) based on netdev IPs configured in the system.

Let me add that when using RDMA CM, it is in charge of choosing the gid
index for a connection, and needs to be aware of the different
protocols. The patchset that includes the core changes for RoCE v2 [1]
allows configuring the preferred protocol for each port. There is also a
heuristic that selects RoCE v2 (which is routable) when the address
resolution detects more than one hop to the destination.

Regards,
Haggai

[1] http://www.spinics.net/lists/linux-rdma/msg28120.html

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

      parent reply	other threads:[~2015-08-26  5:31 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-20 16:46 [PATCH for-next 00/10] Add RoCE support to the mlx5 driver Achiad Shochat
     [not found] ` <1440089189-3361-1-git-send-email-achiad-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-08-20 16:46   ` [PATCH for-next 01/10] IB/mlx5: Support IB device's callback for getting the link layer Achiad Shochat
2015-08-20 16:46   ` [PATCH for-next 02/10] IB/mlx5: Support IB device's callback for getting its netdev Achiad Shochat
     [not found]     ` <1440089189-3361-3-git-send-email-achiad-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-11 13:32       ` Matan Barak
2015-08-20 16:46   ` [PATCH for-next 03/10] net/mlx5_core: Break down the vport mac address query function Achiad Shochat
2015-08-20 16:46   ` [PATCH for-next 04/10] net/mlx5_core: Introduce access functions to enable/disable RoCE Achiad Shochat
2015-08-20 16:46   ` [PATCH for-next 05/10] net/mlx5_core: Introduce access functions to query vport RoCE fields Achiad Shochat
2015-08-20 16:46   ` [PATCH for-next 06/10] IB/mlx5: Extend query_device/port to support RoCE Achiad Shochat
2015-08-20 16:46   ` [PATCH for-next 07/10] IB/mlx5: Set network_hdr_type upon RoCE responder completion Achiad Shochat
2015-08-20 16:46   ` [PATCH for-next 08/10] IB/mlx5: Support IB device's callbacks for adding/deleting GIDs Achiad Shochat
2015-08-20 16:46   ` [PATCH for-next 09/10] IB/mlx5: Add RoCE fields to Address Vector Achiad Shochat
2015-08-20 16:46   ` [PATCH for-next 10/10] IB/mlx5: Support RoCE Achiad Shochat
2015-08-24 20:40   ` [PATCH for-next 00/10] Add RoCE support to the mlx5 driver Tom Talpey
     [not found]     ` <55DB8152.6080402-CLs1Zie5N5HQT0dZR+AlfA@public.gmane.org>
2015-08-25  8:35       ` Achiad Shochat
     [not found]         ` <55DC28C2.4020705-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-08-25 12:32           ` Tom Talpey
     [not found]             ` <55DC6040.5080907-CLs1Zie5N5HQT0dZR+AlfA@public.gmane.org>
2015-08-25 13:06               ` Achiad Shochat
     [not found]                 ` <55DC685A.8060402-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-08-26  5:31                   ` Haggai Eran [this message]

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=55DD4F37.4020306@mellanox.com \
    --to=haggaie-vpraknaxozvwk0htik3j/w@public.gmane.org \
    --cc=Somnath.Kotur-idTK6quXuVS1Z/+hSey0Gg@public.gmane.org \
    --cc=achiad-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=tom-CLs1Zie5N5HQT0dZR+AlfA@public.gmane.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 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.