Re: [PATCH] nvme: Add 48-bit DMA address quirk

From: Christoph Hellwig
Date: Wed Feb 03 2021 - 06:16:38 EST


On Wed, Feb 03, 2021 at 12:12:31PM +0100, Filippo Sironi wrote:
> I don't disagree on the first part of your sentence, this is a big
> oversight.

But it is not what your commit log suggests.

> On the other hand, those controllers are out there and are in use by a lot
> of customers. We can keep relying on luck, hoping that customers don't run
> into troubles or we can merge a few lines of code :)

Your patch does not just quirk a few controllers out there, but all
current and future controllers with an Amazon vendor ID. We could
probably talk about quirking an existing vendor ID or two as long as
this doesn't happen for future hardware.