diff options
| author | Thara Gopinath <thara.gopinath@linaro.org> | 2018-11-28 01:43:11 +0300 |
|---|---|---|
| committer | Zhang Rui <rui.zhang@intel.com> | 2018-11-30 11:57:51 +0300 |
| commit | 68000a0d983f539c95ebe5dccd4f29535c7ac0af (patch) | |
| tree | e0d573af5721a3ac9a10943bc09ce1a2a982ccef /drivers/thermal | |
| parent | 964f4843a455d2ffb199512b08be8d5f077c4cac (diff) | |
| download | linux-68000a0d983f539c95ebe5dccd4f29535c7ac0af.tar.xz | |
thermal: Fix locking in cooling device sysfs update cur_state
Sysfs interface to update cooling device cur_state does not
currently holding cooling device lock sometimes leading to
stale values in cur_state if getting updated simultanelously
from user space and thermal framework. Adding the proper locking
code fixes this issue.
Signed-off-by: Thara Gopinath <thara.gopinath@linaro.org>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
Diffstat (limited to 'drivers/thermal')
| -rw-r--r-- | drivers/thermal/thermal_sysfs.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/drivers/thermal/thermal_sysfs.c b/drivers/thermal/thermal_sysfs.c index 2241ceae7d7f..aa99edb4dff7 100644 --- a/drivers/thermal/thermal_sysfs.c +++ b/drivers/thermal/thermal_sysfs.c @@ -712,11 +712,14 @@ cur_state_store(struct device *dev, struct device_attribute *attr, if ((long)state < 0) return -EINVAL; + mutex_lock(&cdev->lock); + result = cdev->ops->set_cur_state(cdev, state); - if (result) - return result; - thermal_cooling_device_stats_update(cdev, state); - return count; + if (!result) + thermal_cooling_device_stats_update(cdev, state); + + mutex_unlock(&cdev->lock); + return result ? result : count; } static struct device_attribute |