Hi Joerg, Joerg Roedel <[email protected]> wrote @ Mon, 2 Jul 2012 11:57:47 +0200:
> On Wed, Jun 27, 2012 at 12:54:01PM +0300, Hiroshi DOYU wrote: > > allo_pdir() is called in smmu_iommu_domain_init() with spin_lock > > held. memory allocations in it have to be atomic/unsleepable. > > > > Signed-off-by: Hiroshi DOYU <[email protected]> > > Reported-by: Chris Wright <[email protected]> > > --- > > drivers/iommu/tegra-smmu.c | 4 ++-- > > 1 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/iommu/tegra-smmu.c b/drivers/iommu/tegra-smmu.c > > index ecd6790..3f3d09d 100644 > > --- a/drivers/iommu/tegra-smmu.c > > +++ b/drivers/iommu/tegra-smmu.c > > @@ -550,13 +550,13 @@ static int alloc_pdir(struct smmu_as *as) > > return 0; > > > > as->pte_count = devm_kzalloc(smmu->dev, > > - sizeof(as->pte_count[0]) * SMMU_PDIR_COUNT, GFP_KERNEL); > > + sizeof(as->pte_count[0]) * SMMU_PDIR_COUNT, GFP_ATOMIC); > > if (!as->pte_count) { > > dev_err(smmu->dev, > > "failed to allocate smmu_device PTE cunters\n"); > > return -ENOMEM; > > } > > - as->pdir_page = alloc_page(GFP_KERNEL | __GFP_DMA); > > + as->pdir_page = alloc_page(GFP_ATOMIC | __GFP_DMA); > > if (!as->pdir_page) { > > dev_err(smmu->dev, > > "failed to allocate smmu_device page directory\n"); > > Okay, I queued this one for v3.5. It is small enough so that it counts > as a fix. The other two patches are 3.6 stuff. The other 2 patehs are the replacement of this one. They are exclusive. Both of them are basically for the same fix. _______________________________________________ iommu mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/iommu
