Re: [PATCH v4 7/9] IB/core: generic RDMA READ/WRITE API

On 03/10/2016 08:21 AM, Christoph Hellwig wrote:

I see the use case for enabling MRs for debugging, but I still don't
think the ULP is the right place.  I also don't see where you
cna do this currently: for srpt, the only driver converted in this
series there is no support for registrations at all currently,
and with the the series you can force it to register with a one
line code change in rdma_rw_use_mr.  For iSERt, which I plan to
convert for 4.7 (and which has some WIP code in git) MR support
exists, but is only enabled if the device supports the vendor
specific signature handover extension (aka if running on mlx5 hardware).
With the future conversion you'll still have a simply compile time
switch to always enabled it, and I'd be fine with adding a module
parameter to force it as well.

Hello Christoph,

Had you already noticed the ib_srp register_always kernel module parameter?

Thanks,

Bart.
--
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