iommu/amd: Drop the lock while allocating new irq remap table
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>
Thu, 22 Mar 2018 15:22:40 +0000 (16:22 +0100)
committerJoerg Roedel <jroedel@suse.de>
Thu, 29 Mar 2018 08:38:16 +0000 (10:38 +0200)
commit993ca6e063a69a0c65ca42ed449b6bc1b3844151
tree9a503357e65658e3128f643031fa33d3ae360796
parent2fcc1e8ac4a8514c64f946178fc36c2e30e56a41
iommu/amd: Drop the lock while allocating new irq remap table

The irq_remap_table is allocated while the iommu_table_lock is held with
interrupts disabled.
>From looking at the call sites, all callers are in the early device
initialisation (apic_bsp_setup(), pci_enable_device(),
pci_enable_msi()) so make sense to drop the lock which also enables
interrupts and try to allocate that memory with GFP_KERNEL instead
GFP_ATOMIC.

Since during the allocation the iommu_table_lock is dropped, we need to
recheck if table exists after the lock has been reacquired. I *think*
that it is impossible that the "devid" entry appears in irq_lookup_table
while the lock is dropped since the same device can only be probed once.
However I check for both cases, just to be sure.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Joerg Roedel <jroedel@suse.de>
drivers/iommu/amd_iommu.c