On 13/04/17 10:59 PM, Christoph Hellwig wrote:

On Thu, Apr 13, 2017 at 04:05:15PM -0600, Logan Gunthorpe wrote:

quoted

This is a straight forward conversion in two places. Should kmap fail,
the code will return an INVALD_DATA error in the completion.

It really should be using nvmet_copy_from_sgl to make things safer,
as we don't want to rely on any particular SG list layout.  In fact
I'm pretty sure I did the conversion at some point, but it must never
have made it upstream.

Ha, I did the conversion too a couple times for my RFC series. I can
change this patch to do that. Or maybe I'll just send a patch for that
separately seeing it doesn't depend on anything and is pretty simple. I
can do that next week.

Thanks,

Logan