diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2022-08-30 20:15:38 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2022-08-30 20:18:47 +0200 |
| commit | 72c976fc3a4b3ff264127f4ced7c06f35cede858 (patch) | |
| tree | f5a4c532d2218ce7429998a651207ccc7b9ca2fb | |
| parent | 02af1db05dcce05603eb51ba3e93c6f88d6ea287 (diff) | |
| download | linux-stable-72c976fc3a4b3ff264127f4ced7c06f35cede858.tar.gz linux-stable-72c976fc3a4b3ff264127f4ced7c06f35cede858.tar.bz2 linux-stable-72c976fc3a4b3ff264127f4ced7c06f35cede858.zip | |
thermal: gov_user_space: Do not lock thermal zone mutex
Commit 670a5e356cb6 ("thermal/core: Move the thermal zone lock out of
the governors") moved thermal zone locking away from governors, but it
forgot about the user space one which deadlocks now.
Fix this by removing the thermal zone locking from the user space
governor.
Fixes: 670a5e356cb6 ("thermal/core: Move the thermal zone lock out of the governors")
Tested-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
| -rw-r--r-- | drivers/thermal/gov_user_space.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/thermal/gov_user_space.c b/drivers/thermal/gov_user_space.c index a62a4e90bd3f..8bc1c22aaf03 100644 --- a/drivers/thermal/gov_user_space.c +++ b/drivers/thermal/gov_user_space.c @@ -34,7 +34,8 @@ static int notify_user_space(struct thermal_zone_device *tz, int trip) char *thermal_prop[5]; int i; - mutex_lock(&tz->lock); + lockdep_assert_held(&tz->lock); + thermal_prop[0] = kasprintf(GFP_KERNEL, "NAME=%s", tz->type); thermal_prop[1] = kasprintf(GFP_KERNEL, "TEMP=%d", tz->temperature); thermal_prop[2] = kasprintf(GFP_KERNEL, "TRIP=%d", trip); @@ -43,7 +44,7 @@ static int notify_user_space(struct thermal_zone_device *tz, int trip) kobject_uevent_env(&tz->device.kobj, KOBJ_CHANGE, thermal_prop); for (i = 0; i < 4; ++i) kfree(thermal_prop[i]); - mutex_unlock(&tz->lock); + return 0; } |