Re: [PATCH] pci: Add support for multiple DMA aliases

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2016-01-20 at 11:46 -0600, Bjorn Helgaas wrote:
> 
> I don't really want to merge things that only exist to enable
> out-of-tree development, because (1) they're an extra maintenance
> burden for which we get risk without benefit, and (2) we can't see
> the out-of-tree code, so it's easy for people to make changes that
> accidentally break that code.
> 
> Looking at the patch again, I see that even without the export,
> there's no current benefit,

This is just a PCI quirk; I'm not sure it should be considered part of
the driver code at all. With this patch, even without a Linux driver,
we could correctly handle assignment to VM guests (which *might* have a
driver), and also theoretically we should be able to handle fault
storms and shoot the right device in the head if it was left in an odd
state and misbehaves (not that I've hooked that up yet).

So I'm not sure it makes sense to tie this patch to the existence of a
driver.
 
-- 
David Woodhouse                            Open Source Technology Centre
David.Woodhouse@xxxxxxxxx                              Intel Corporation

Attachment: smime.p7s
Description: S/MIME cryptographic signature


[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux