iommu/vt-d: Avoid GFP_ATOMIC where it is not needed
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Tue, 1 Dec 2020 01:31:49 +0000 (09:31 +0800)
committerWill Deacon <will@kernel.org>
Tue, 1 Dec 2020 15:02:20 +0000 (15:02 +0000)
There is no reason to use GFP_ATOMIC in a 'suspend' function.
Use GFP_KERNEL instead to give more opportunities to allocate the
requested memory.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/20201030182630.5154-1-christophe.jaillet@wanadoo.fr
Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
Link: https://lore.kernel.org/r/20201201013149.2466272-2-baolu.lu@linux.intel.com
Signed-off-by: Will Deacon <will@kernel.org>
drivers/iommu/intel/iommu.c

index e27eb6f..770c537 100644 (file)
@@ -3492,7 +3492,7 @@ static int iommu_suspend(void)
 
        for_each_active_iommu(iommu, drhd) {
                iommu->iommu_state = kcalloc(MAX_SR_DMAR_REGS, sizeof(u32),
-                                                GFP_ATOMIC);
+                                            GFP_KERNEL);
                if (!iommu->iommu_state)
                        goto nomem;
        }