Re: [PATCH] ib_core: Enable and expose force_mr module parameter

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 27, 2017 at 12:24:20PM +0300, Sagi Grimberg wrote:
>
> > > It will be "marked" as UAPI and we won't be able to change it,
> > > which IMHO bad for the debug. It also invites users to set it
> > > and in perfect world, we should test our HCAs with this flag
> > > on and off, because curious user can set it.
> >
> > We have loads of legacy debugging interfaces.
> >
> > And yes, ULP developers should be aware of it and test with
> > both settings.
> >
> > I'm not convinced any of this is a problem for this particular
> > setting. However, I guess the thing for you to do is propose a
> > patch.
>
> Are you guys seriously still on this? Its simply a trivial
> debug flag, whats all the fuss really about?

I hope that we are done.
https://patchwork.kernel.org/patch/9808615/

Thanks

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux