linux-kernel - Re: [lm-sensors] [REGRESSION, ABI] Re: LMSENSORS: 2.6.26-rc, enabling ACPI Termal Zone support costs sensors

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <485F7DB0.2030302@keyaccess.nl>
Date:	Mon, 23 Jun 2008 12:40:48 +0200
From:	Rene Herman <rene.herman@...access.nl>
To:	Zhang Rui <rui.zhang@...el.com>
CC:	Hans de Goede <j.w.r.degoede@....nl>, linux-acpi@...r.kernel.org,
	"Mark M. Hoffman" <mhoffman@...htlink.com>,
	Linux Kernel <linux-kernel@...r.kernel.org>,
	lm-sensors@...sensors.org, Jean Delvare <khali@...ux-fr.org>
Subject: Re: [lm-sensors] [REGRESSION, ABI] Re:  LMSENSORS: 2.6.26-rc,	enabling
 ACPI Termal Zone support costs sensors

On 23-06-08 03:44, Zhang Rui wrote:

> On Mon, 2008-06-23 at 05:58 +0800, Rene Herman wrote:
>> On 22-06-08 20:25, Rene Herman wrote:
>>
>>> Another way to solve it _could_ be to just make up a device link if
>>> something generic is available so that sensors doesn't trip over it
>>> in the first place but I don't know if that's a good option. You
>>> might.
>> This also works for me and, if correct, is ofcourse better than the
>> CONFIG option. Wants a comment from the thermal_zone side (for which
>> Zhang Rui seems the correct CC?) though.
> Hi, Rene and Hans,
> 
> Thank you for your efforts on this issue and sorry for the late
> response, I did not check my email during the whole weekend.
> 
> About the hwmon ABI, after the device symbol link is created, are there
> any other ABIs required in the device node?

Doesn't seem to. libsensors-2.10.6 is happy with this at least.

> If no, this patch seems to work, although it might break if the first
> registered ACPI thermal zone device is unregistered, which ONLY happens
> theoretically.

Mmm. Because more thermal zones may share one hwmon interface I gather?
Do you feel this is an okay minimal fix for 2.6.26 or is there something 
else trivial enough available? We're late in the -rc stage...

>> From: Rene Herman <rene.herman@...il.com>
>> Date: Sun, 22 Jun 2008 23:45:04 +0200
>> Subject: [PATCH] thermal: link the virtual/thermal/thermal_zoneN device for hwmon
>>
>> 2.6.26-rc gained a hwmon interface to the ACPI Thermal Zone
>> driver which unfortunately breaks lm-sensors 2 userspace and
>> renders all other (subsequent) hwmon sensors inoperable also.
>>
>> Many systems, current slackware systems among them, are still
>> using lm-sensors 2 and would be affected.
>>
>> The problem is userspace tripping over an absent "device"
>> link in the ACPI Thermal Zone sysfs /sys/class/hwmon/hwmonN
>> directory. This just adds the virtual "thermal/thermal_zoneN"
>> device as a device backlink which satisfies lm-sensors again.
>>
>> Tested and looks fairly obvious, but this wants comment from
>> a thermal_zone person, to confirm/deny that this is a proper
>> device pointer to use here for one.
>>
>> Signed-off-by: Rene Herman <rene.herman@...il.com>
>> CC: Hans de Goede <j.w.r.degoede@....nl>
>> CC: Zhang Rui <rui.zhang@...el.com>
>> CC: Mark M. Hoffman <mhoffman@...htlink.com>
>> CC: Jean Delvare <khali@...ux-fr.org>
>> CC: linux-acpi@...r.kernel.org
>> CC: lm-sensors@...sensors.org
>> CC; linux-kernel@...r.kernal.org
>> ---
>>  drivers/thermal/thermal_sys.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/thermal/thermal_sys.c b/drivers/thermal/thermal_sys.c
>> index 6098787..c21e03c 100644
>> --- a/drivers/thermal/thermal_sys.c
>> +++ b/drivers/thermal/thermal_sys.c
>> @@ -357,7 +357,7 @@ thermal_add_hwmon_sysfs(struct thermal_zone_device *tz)
>>  
>>  	INIT_LIST_HEAD(&hwmon->tz_list);
>>  	strlcpy(hwmon->type, tz->type, THERMAL_NAME_LENGTH);
>> -	hwmon->device = hwmon_device_register(NULL);
>> +	hwmon->device = hwmon_device_register(&tz->device);
>>  	if (IS_ERR(hwmon->device)) {
>>  		result = PTR_ERR(hwmon->device);
>>  		goto free_mem;
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/