diff options
author | Aaron Lu <aaron.lu@intel.com> | 2014-05-21 16:33:27 +0800 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-07-17 16:21:03 -0700 |
commit | c37ea82d4c02f7d3bdce6a43f61af6baf9528377 (patch) | |
tree | 6eb6ad201d63a5bbeb90e719bc61cdc15fad3aed /drivers/hwmon | |
parent | ec10ade4090d1be88d35b0c3432c710d24f6323a (diff) | |
download | lwn-c37ea82d4c02f7d3bdce6a43f61af6baf9528377.tar.gz lwn-c37ea82d4c02f7d3bdce6a43f61af6baf9528377.zip |
thermal: hwmon: Make the check for critical temp valid consistent
commit e8db5d6736a712a3e2280c0e31f4b301d85172d8 upstream.
On 05/21/2014 04:22 PM, Aaron Lu wrote:
> On 05/21/2014 01:57 PM, Kui Zhang wrote:
>> Hello,
>>
>> I get following error when rmmod thermal.
>>
>> rmmod thermal
>> Killed
While dealing with this problem, I found another problem that also
results in a kernel crash on thermal module removal:
From: Aaron Lu <aaron.lu@intel.com>
Date: Wed, 21 May 2014 16:05:38 +0800
Subject: thermal: hwmon: Make the check for critical temp valid consistent
We used the tz->ops->get_crit_temp && !tz->ops->get_crit_temp(tz, temp)
to decide if we need to create the temp_crit attribute file but we just
check if tz->ops->get_crit_temp exists to decide if we need to remove
that attribute file. Some ACPI thermal zone doesn't have a valid critical
trip point and that would result in removing a non-existent device file
on thermal module unload.
Signed-off-by: Aaron Lu <aaron.lu@intel.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/hwmon')
0 files changed, 0 insertions, 0 deletions