Hi Chris, Chris Wright <[email protected]> wrote @ Tue, 26 Jun 2012 20:15:55 +0200:
> This looks buggy to me: > > drivers/iommu/tegra-smmu.c::smmu_iommu_domain_init() > > spin_lock_irqsave(&tmp->lock, flags); <-- lock held, interrupts off > alloc_pdir(as = tmp) > devm_kzalloc(..., GFP_KERNEL); <-- potentially sleeping allocation > ... > alloc_page(..., GFP_KERNEL | __GFP_DMA); <-- same here Right. I totally agree. Thank you for pointing this out. The fix patch follows. _______________________________________________ iommu mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/iommu
