Re: [PATCH 0/2] nvme: Add kernel API for admin command

Martin K. Petersen
2019-09-19T00:40:38Z

Christoph,

> On Wed, Sep 18, 2019 at 11:08:07AM -0600, Keith Busch wrote:
>> And yes, that bouncing is really nasty, but it's really only needed for
>> PRP, so maybe let's just not ignore that transfer mode and support
>> extended metadata iff the controller supports SGLs. We just need a
>> special SGL setup routine to weave the data and metadata.
>
> Well, what is the point? If people really want to use metadata they
> should just buy a drive supporting the separate metadata pointer. In
> fact I haven't had to deal with a drive that only supports interleaved
> metadata so far given how awkward that is to deal with.

Yep. There's a reason we did DIX...

--
Martin K. Petersen Oracle Linux Engineering