evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

iommu/arm-smmu: lock clock enabling and reference counting

Browse source 
Ensure that clock enabling and reference counting is done atomically
to avoid any potential race conditions.

An example of a potential race condition is that while thread one is
enabling the clocks thread two could enter and then exit the clock
enabling function early because of reference counting. This could
lead to thread two attempting to access registers before the clocks
are enabled.

Have removed the regulator reference because enabling the regulators
involves the use of a mutex so spin locks cannot be used to protect
the reference count. Also the use of a regulator reference count is
of limited benefit since there is only one regulator to enable.

Change-Id: I7215bbf9157907fde24c94841e347370769423c8
Signed-off-by: Liam Mark <lmark@codeaurora.org>
This commit is contained in:
Liam Mark 2015-06-19 10:14:31 -07:00 • committed by David Keitel
parent e723f7b4a1
commit 057dffa6b9
1 changed files with 16 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
drivers/iommu/arm-smmu.c
View file

@ -388,8 +388,8 @@ struct arm_smmu_device {
unsigned int num_impl_def_attach_registers;
struct mutex atos_lock;
atomic_t clock_refs_count;
atomic_t regulator_refs_count;
unsigned int clock_refs_count;
spinlock_t clock_refs_lock;
};
struct arm_smmu_cfg {
@ -709,9 +709,6 @@ static int arm_smmu_disable_regulators(struct arm_smmu_device *smmu)
if (!smmu->gdsc)
return 0;
if (atomic_dec_return(&smmu->regulator_refs_count) > 0)
return 0;
arm_smmu_unprepare_clocks(smmu);
return regulator_disable(smmu->gdsc);
}
@ -723,9 +720,6 @@ static int arm_smmu_enable_regulators(struct arm_smmu_device *smmu)
if (!smmu->gdsc)
return 0;
if (atomic_inc_return(&smmu->regulator_refs_count) > 1)
return 0;
ret = regulator_enable(smmu->gdsc);
if (ret)
return ret;
@ -760,9 +754,13 @@ static void arm_smmu_disable_clocks(struct arm_smmu_device *smmu)
static int arm_smmu_enable_clocks_atomic(struct arm_smmu_device *smmu)
{
int i, ret = 0;
unsigned long flags;
if (atomic_inc_return(&smmu->clock_refs_count) > 1)
spin_lock_irqsave(&smmu->clock_refs_lock, flags);
if (smmu->clock_refs_count++ > 0) {
spin_unlock_irqrestore(&smmu->clock_refs_lock, flags);
return 0;
}
for (i = 0; i < smmu->num_clocks; ++i) {
ret = clk_enable(smmu->clocks[i]);
@ -773,6 +771,7 @@ static int arm_smmu_enable_clocks_atomic(struct arm_smmu_device *smmu)
break;
}
}
spin_unlock_irqrestore(&smmu->clock_refs_lock, flags);
return ret;
}
@ -780,11 +779,17 @@ static int arm_smmu_enable_clocks_atomic(struct arm_smmu_device *smmu)
static void arm_smmu_disable_clocks_atomic(struct arm_smmu_device *smmu)
{
int i;
if (atomic_dec_return(&smmu->clock_refs_count) > 0)
unsigned long flags;
spin_lock_irqsave(&smmu->clock_refs_lock, flags);
if (smmu->clock_refs_count-- > 1) {
spin_unlock_irqrestore(&smmu->clock_refs_lock, flags);
return;
}
for (i = 0; i < smmu->num_clocks; ++i)
clk_disable(smmu->clocks[i]);
spin_unlock_irqrestore(&smmu->clock_refs_lock, flags);
}
/* Wait for any pending TLB invalidations to complete */
@ -2418,6 +2423,7 @@ static int arm_smmu_device_dt_probe(struct platform_device *pdev)
smmu->dev = dev;
mutex_init(&smmu->attach_lock);
mutex_init(&smmu->atos_lock);
spin_lock_init(&smmu->clock_refs_lock);
of_id = of_match_node(arm_smmu_of_match, dev->of_node);
if (!of_id)