On Tue, Jul 14, 2015 at 04:01:04PM -0500, Steve Wise wrote: > > Right, a local_dma_lkey is not an rkey, and iwarp requires the > > rkey for the read destination MR. Further that rkey needs > > REMOTE_WRITE. > > BTW: What use is an IB rkey with no REMOTE_ flags set? Can it be > used somehow differently than the associated lkey? Don't think so. Sagi? This looks like it is just an artifact of the sloppy API that treats a rkey MR and lkey MR as the same thing - they are clearly not, and we should start talking about APIs that return lkeys or rkeys, never both (and enforcing that lkey and rkey MRS have the right ACCESS flags). Having looked at this for a bit now, I am of the view that it is very hard to use a MR as both rkey and lkey without creating some kind of security problem. At least every place in current ULPs that does this is a security problem :) So the API should prevent it, IMHO. local_dma_lkey is an excellent step, and if Sagi's MR unification patch is careful to have a lkey/rkey API entry point for the two usages we can maybe nuke this problem for good... Jason -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
