Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753726AbYFWBoU (ORCPT ); Sun, 22 Jun 2008 21:44:20 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752354AbYFWBoD (ORCPT ); Sun, 22 Jun 2008 21:44:03 -0400 Received: from mga11.intel.com ([192.55.52.93]:44848 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751686AbYFWBoA (ORCPT ); Sun, 22 Jun 2008 21:44:00 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.27,687,1204531200"; d="scan'208";a="580762676" Subject: Re: [lm-sensors] [REGRESSION, ABI] Re: LMSENSORS: 2.6.26-rc, enabling ACPI Termal Zone support costs sensors From: Zhang Rui To: Rene Herman Cc: Hans de Goede , linux-acpi@vger.kernel.org, "Mark M. Hoffman" , Linux Kernel , lm-sensors@lm-sensors.org, Jean Delvare In-Reply-To: <485ECB23.4010305@keyaccess.nl> References: <485DA11C.7050906@keyaccess.nl> <485DFF35.6080008@hhs.nl> <485E505F.8010306@keyaccess.nl> <485E61DE.6020202@hhs.nl> <485E6F12.4010307@keyaccess.nl> <485E94ED.10602@hhs.nl> <485E992B.8010101@keyaccess.nl> <485ECB23.4010305@keyaccess.nl> Content-Type: text/plain Date: Mon, 23 Jun 2008 09:44:42 +0800 Message-Id: <1214185482.3001.45.camel@rzhang-dt.sh.intel.com> Mime-Version: 1.0 X-Mailer: Evolution 2.12.3 (2.12.3-4.fc8) Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3309 Lines: 82 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? 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. thanks, rui > > plain text document attachment > (0001-thermal-link-the-virtual-thermal-thermal_zoneN-devi.patch), > "0001-thermal-link-the-virtual-thermal-thermal_zoneN-devi.patch" > From b51a5f4105f6b5ff52a2819a96dabe2ca3116128 Mon Sep 17 00:00:00 2001 > From: Rene Herman > 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 > CC: Hans de Goede > CC: Zhang Rui > CC: Mark M. Hoffman > CC: Jean Delvare > CC: linux-acpi@vger.kernel.org > CC: lm-sensors@lm-sensors.org > CC; linux-kernel@vger.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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/