Re: [PATCH v2 8/8] habanalabs: enable 64-bit DMA mask in POWER9

From: Oded Gabbay
Date: Sun Jun 16 2019 - 07:33:56 EST


On Sun, Jun 16, 2019 at 12:55 PM Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
>
> On Sat, Jun 15, 2019 at 03:12:36PM +0300, Oded Gabbay wrote:
> > So after the dust has settled a bit, do you think it is reasonable to
> > add this patch upstream ?
>
> I'm not Greg, but the answer is a very clear no. drivers have abslutely
> no business adding these hacks.

So the alternative is that my device won't work on POWER9. Does that make sense?
What is the reason for this logic?
I'm not adding code that will be used by other drivers/users.
I'm just doing a special configuration to my device's H/W and I
condition it upon the PCI device ID of my parent PCI device.
What is the harm in that?

Thanks,
Oded