Re: [PATCH rdma-rc 1/2] IB/core: Only enforce security for InfiniBand

[Jason Gunthorpe <jgg@ziepe.ca>]

On Tue, Nov 21, 2017 at 06:34:54PM +0200, Leon Romanovsky wrote:
> On Tue, Nov 21, 2017 at 09:04:42AM -0700, Jason Gunthorpe wrote:
> > On Tue, Nov 21, 2017 at 09:37:27AM -0600, Daniel Jurgens wrote:
> >
> > > The only warning that would make sense is if the mixed ports aren't
> > > all IB or RoCE. As you note, CX-3 can mix those two, we don't want to
> > > see warnings about that.
> >
> > I would really like to see cx3 be changed to not do that, then we
> > could finalize this issue upstream: All device ports must be the same
> > protocol.
> 
> I don't see the point of such artificial limitation, the users who
> brought CX-3 have option to work in mixed mode and IMHO it is not right
> to deprecate such ability just because it is hard for us to code for it.

I don't really think it is really too user visible.. Only the device
and port number change, but only if running in mixed mode.

It is not just 'hard for us' it is impossible to reconcile the
differences between ports when enforcing device level things.

This keeps coming up again and again..

Jason