Re: [PATCH v8 13/13] nvmet: Optionally use PCI P2P memory

Logan Gunthorpe
2018-10-01T23:43:29Z

On 01/10/18 04:23 PM, Sagi Grimberg wrote:
>> I did not realize the namespace would be available at this time. I guess
>> I can give this a try, but it's going to be a fairly big change from
>> what's presented here... Though, I agree it'll probably be an
>> improvement.
>
> Thanks, if it turns out to create to much of a churn, we could defer
> that to a later stage, but we can at least document it.

Yeah, it's going to create a bunch of churn, but it's probably worth
doing before merging because I think it will remove a bunch of
complexity (ie. the need for the whole p2p client infrastructure because
we now only need to worry about only one namespace at a time, instead of
needing to find a p2p device that works with all namespaces at once).

I'll try to get a v9 with this change published in the next day or two.

Logan