Re: [lm-sensors] coretemp.0 folder contents changed

From: Srinivas Pandruvada
Date: Tue May 06 2014 - 11:16:45 EST


On 05/06/2014 04:55 AM, Jean Delvare wrote:
Hi Guenter, Srinivas,

On Mon, 5 May 2014 10:32:31 -0700, Guenter Roeck wrote:
On Mon, May 05, 2014 at 10:13:11AM -0700, Srinivas Pandruvada wrote:
for kernel : 3.15.rc3 .

Is there any change in the coretemp? Previously we used to see,
tempx data (like temp2_input, temp2_max etc.)
/sys/devices/platform/coretemp.0/.
That isn't where you are supposed to look for hwmon attributes.
Actually I used to recommend looking there when people were not using
libsensors (e.g. for pwmconfig / fancontrol or custom scripts.) This
path had the great merit of being stable across reboots, while hwmon
class device numbers are not (or at least they aren't guaranteed to be.)
I maintain thermal daemon, which was using the path. So once Ubuntu upgrade kernel,
this will break. So I have to make sure that corresponding user space change is submitted.
I don't want to depend on too many libraries as it also runs on many embedded platforms
where many library pre-builts don't exists.
(...)
To give you the background, hwmon attributes are in the process of
being moved from the parent device to the hwmon device, or from
/sys/class/hwmon/hwmonX/device/ to /sys/class/hwmon/hwmonX/,
as part of an effort to streamline the code and make it more
consistent and maintainable.
I am just realizing that we are also losing the stability of hardware
device based paths with that move :( I suppose I shouldn't have
bothered adding support for this to fancontrol.

Don't get me wrong, I still believe this is the right move, but I fear
that the question of persistent hwmon device names will resurface every
now and then again. libsensors offers a solution but 1* it lacks
support for pwm attributes and 2* it doesn't help with shell or perl
scripts such as pwmconfig and fancontrol.
Also when using Android like platform with limited library support,
we have to make sure that libsensor exists.


Thanks,
Srinivas

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