On Fri, Mar 02, 2018 at 07:55:48PM +0000, Robin Murphy wrote:
> On the other hand, HCD_LOCAL_MEM implies a per-device coherent pool. Since
> those bypass the arch-specific code, then depending on how unlikely we
> think it is for devices covered by a single driver to exist both with and
> without their own memory then it *might* also be reasonable to make the
> change below. Christoph, what do you reckon?
For the case where the coherent pool really is per-device this looks
technically ok. But it would also cover cases like arm nommu where
we have a single global nommu pool and we wouldn't warn for a driver
that frees with irqs disabled.
That being said I think allowing coherent frees from irqs disabled
context is a very bad idea to do in general, so if we can at all I'd
rather fix the driver than lifting any restrictions.
iommu mailing list