ACPI thermal procfs I/F removed from kernel 2.6.37

Bug #670511 reported by André Ventura
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
GNOME Sensors Applet
Unknown
Unknown
sensors-applet (Ubuntu)
New
Undecided
Unassigned

Bug Description

Binary package hint: sensors-applet

From the 2.6.36 changelog (http://www.kernel.org/pub/linux/kernel/v2.6/ChangeLog-2.6.36):

Mark the ACPI thermal procfs I/F deprecated, because /sys/class/thermal/ is already available and has been working for years w/o any problem.
The ACPI thermal procfs I/F will be removed in 2.6.37.
---

sensors-applet 2.2.7-2ubuntu1

Tags: patch
Revision history for this message
Alex Murray (alexmurray) wrote :

Looks like the acpi plugin will need to be ported to use the thermal zone stuff from sysfs - http://lwn.net/Articles/268958/

Thanks for reporting this, unfortunately I don't have a machine which supports acpi thermal zones so it will be hard to do the port myself and ensure it is correct, but I will see what I can do - patches are welcome :)

Revision history for this message
André Ventura (afv) wrote :

Well, I made this patch, when I reported the bug, but it didn't work, so I didn't upload. Here it goes, nevertheless. :)

Revision history for this message
Hernando Torque (htorque) wrote :

Can't work because '/sys/class/thermal' contains only symlinks, which 'lib/sensors-applet-plugin.c' doesn't care about.

Here's another patch. Works for me, unfortunately the sensors are now labeled "thermal_zone*" which is pretty useless as I don't know which of the six thermal zones I should keep a look at.

Revision history for this message
Hernando Torque (htorque) wrote :
Revision history for this message
Alex Murray (alexmurray) wrote :

The latest lm-sensors (aka libsensors) apparently support reading the acpi thermal zone temperatures so we can probably just remove the acpi plugin from sensors-applet to fix this - can anyone confirm if this is the case? i.e. if the libsensors plugin shows these temperatures?

Revision history for this message
Alex Murray (alexmurray) wrote :

These will probably just show up as tempX under libsensors.

tags: added: patch
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.