All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jakub Kicinski <kuba@kernel.org>
To: Saeed Mahameed <saeed@kernel.org>
Cc: Leon Romanovsky <leon@kernel.org>,
	Jason Gunthorpe <jgg@nvidia.com>,
	"David S. Miller" <davem@davemloft.net>,
	Paolo Abeni <pabeni@redhat.com>,
	Eric Dumazet <edumazet@google.com>,
	Saeed Mahameed <saeedm@nvidia.com>,
	linux-rdma@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: pull-request: mlx5-next 2023-01-24 V2
Date: Fri, 3 Feb 2023 17:57:39 -0800	[thread overview]
Message-ID: <20230203175739.1fef3a24@kernel.org> (raw)
In-Reply-To: <Y92rHsui8dmZclca@x130>

On Fri, 3 Feb 2023 16:47:26 -0800 Saeed Mahameed wrote:
> On 03 Feb 13:14, Jakub Kicinski wrote:
> >I believe Paolo is planning to look next week. No idea why the patch
> >got marked as Accepted 🤷️
> >
> >On Fri, 3 Feb 2023 12:05:56 -0800 Saeed Mahameed wrote:  
> >> I don't agree, RDMA isn't proprietary, and I wish not to go into this
> >> political discussion, as this series isn't the right place for that.  
> >
> >I don't think it's a political discussion. Or at least not in the sense
> >of hidden agendas because our agendas aren't hidden. I'm a maintainer
> >of an open source networking stack, you're working for a vendor who
> >wants to sell their own networking stack.
> 
> we don't own any networking stack.. yes we do work on multiple opesource
> fronts and projects, but how is that related to this patchset ? 
> For the sake of this patchset, this purely mlx5 device management, and
> yes for RoCE traffic, RoCE is RDMA spec and standard and an open source
> mainstream kernel stack.

My memory is that Leon proposed IPsec offload, I said "you're doing
this for RDMA", he said "no we will also need this for TC redirect",
I said "if you implement TC redirect that's a legit use of netdev APIs".

And now RDMA integration is coming, and no TC in sight.

I think it's reasonable for me to feel mislead.

> >I don't think we can expect Linus to take a hard stand on this, but
> >do not expect us to lend you our APIs and help you sell your product.
> >
> >Saying that RDMA/RoCE is not proprietary because there is a "standard"
> >is like saying that Windows is an open source operating system because
> >it supports POSIX.
> 
> Apples and oranges, really :) .. 
> 
> Sorry but I have to disagree, the difference here is that the spec
> is open and the stack is in the mainstream linux, and there are at least
> 10 active vendors currently contributing to rdma with open source driver
> and open source user space, and there is pure software RoCE
> implementation for the paranoid who don't trust hw vendors, oh and it uses
> netdev APIs, should that be also forbidden ??

I don't want to be having theoretical discussions.
In theory there could exist a fully open RoCE implementation which
inter-operates with all other implementations perfectly. Agreed.

> What you're really saying here is that no vendor is allowed to do any
> offload or acceleration ..

IDK where you got that form, and it's obviously counter factual.
If I was nacking all offloads, I've have nacked the "full" IPsec
offload and we wouldn't be having this conversation at all.

  reply	other threads:[~2023-02-04  1:57 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-26 23:08 pull-request: mlx5-next 2023-01-24 V2 Saeed Mahameed
2023-02-02  7:46 ` Leon Romanovsky
2023-02-02 17:13   ` Jakub Kicinski
2023-02-02 17:14     ` Jason Gunthorpe
2023-02-02 17:25       ` Jakub Kicinski
2023-02-02 17:44         ` Jason Gunthorpe
2023-02-02 17:54           ` Jakub Kicinski
2023-02-02 18:03             ` Leon Romanovsky
2023-02-02 18:15               ` Saeed Mahameed
2023-02-02 18:30                 ` Jakub Kicinski
2023-02-03 20:05                   ` Saeed Mahameed
2023-02-03 21:14                     ` Jakub Kicinski
2023-02-04  0:18                       ` Jason Gunthorpe
2023-02-04  1:45                         ` Jakub Kicinski
2023-02-06 14:58                           ` Jason Gunthorpe
2023-02-07  0:38                             ` Jakub Kicinski
2023-02-07 19:52                               ` Jason Gunthorpe
2023-02-07 22:03                                 ` Jakub Kicinski
2023-02-08  9:17                                   ` Leon Romanovsky
2023-02-08 16:13                                   ` Jason Gunthorpe
2023-02-08 23:19                                     ` Jakub Kicinski
2023-02-09  0:27                                       ` Jason Gunthorpe
2023-02-09  0:48                                         ` Jakub Kicinski
2023-02-09  0:59                                           ` Jason Gunthorpe
2023-02-09  1:16                                             ` Jakub Kicinski
2023-02-10 17:15                                               ` Jason Gunthorpe
2023-02-09  0:36                                       ` Saeed Mahameed
2023-02-09  0:52                                         ` Jakub Kicinski
2023-02-04  0:47                       ` Saeed Mahameed
2023-02-04  1:57                         ` Jakub Kicinski [this message]
2023-02-05 10:26                           ` Leon Romanovsky
2023-02-02 18:07       ` Leon Romanovsky
2023-02-03 20:14 ` Saeed Mahameed

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=20230203175739.1fef3a24@kernel.org \
    --to=kuba@kernel.org \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=jgg@nvidia.com \
    --cc=leon@kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=saeed@kernel.org \
    --cc=saeedm@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 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.