Bug 250013 - Sensors applet failed to read /proc/* after upgrade to 2.6.22-.1-33.fc7
Summary: Sensors applet failed to read /proc/* after upgrade to 2.6.22-.1-33.fc7
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: gnome-applet-sensors
Version: 7
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Hans de Goede
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-07-29 12:59 UTC by Edmond
Modified: 2007-11-30 22:12 UTC (History)
0 users

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2007-08-03 13:02:26 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
screenshot of error (44.02 KB, image/png)
2007-07-29 12:59 UTC, Edmond
no flags Details
ls -l from /proc/acpi (1.48 KB, text/plain)
2007-07-29 16:17 UTC, Edmond
no flags Details

Description Edmond 2007-07-29 12:59:01 UTC
Description of problem:

The GNOME sensors applet failed to read sensor info from /proc after kernel upgrade.

Version-Release number of selected component (if applicable):
kernel-2.6.22.1-33.fc7
lm_sensors-2.10.3-2.fc7
gnome-applet-sensors-1.8.1-1.fc7

How reproducible:
reproducible every time

Steps to Reproduce:
1. upgrade to 2.6.22 kernel via yum
2. login to GNOME and add gnome sensor applet
3.
  
Actual results:
see attached screen shot

Expected results:
no impact

Additional info:

Comment 1 Edmond 2007-07-29 12:59:01 UTC
Created attachment 160187 [details]
screenshot of error

Comment 2 Hans de Goede 2007-07-29 13:10:42 UTC
Its quite well possible that these files have moved, confusing the
sensors-applet, can you do:
ls -R /proc/acpi

And report the output here.

Also can you try adding the sensors applet to your panel a second time
(preferably to another panel if you have more then one) Then it will redetect
which sensors are there. Hopefully it will find the sensors in their new place,
then this can be fixed by simply removing the applet and then adding it again.
If the new applet doesn't find the sensors, then we will have to dig deeper.


Comment 3 Edmond 2007-07-29 16:16:07 UTC
Only CPU sensor left after uninstall+re-install....

Comment 4 Edmond 2007-07-29 16:17:38 UTC
Created attachment 160196 [details]
ls -l from /proc/acpi

Comment 5 Hans de Goede 2007-07-29 17:41:17 UTC
Hmm, no more acpi directory. I happen to have an 2.6.23 rc1 kenrel tree on my
disk, so I directly went to the ibm_acpi now thinkpad_acpi driver source, they
did some pretty major surgery on the interface to make it more standard (good),
but forgot to keep the old interface for backward compatibility (bad).

So this really is a kernel bug, but marking it as such isn't going to help you.
So instead I'll write libsensors support code for this, as the thinkpad_acpi
driver now uses the standard hwmon interface, which is supported in
gnome-applet-sensors through libsensors, but first libsensors must be thought
about the new thinkpad_acpi "chip"

Can you do an ls -R /sys/class/hwmon? If I'm right you will find your sensors
there, you can cat the _input files to get readings for now (until my patch is
done). Please let me know the output of ls -R /sys/class/hwmon, I will need that
to write libsensors support.

Also I will be going on vacation for 6 days starting tomorrow, so this will have
to wait till the next weekend, I hope to have something to test for you at the
end of the next weekend.


Comment 6 Edmond 2007-07-30 02:43:34 UTC
no... it seems not that simple... 8(

#ls -R /sys/class/hwmon*
ls: cannot access /sys/class/hwmon*: No such file or directory

%114[ehui@mva]/sys/class#ls
backlight  graphics     mem            printer       scsi_host     usb_host
bluetooth  i2c-adapter  misc           rtc           sound         vc
dma        ieee80211    net            scsi_device   tty           vtconsole
dmi        input        pci_bus        scsi_disk     usb_device
firmware   leds         pcmcia_socket  scsi_generic  usb_endpoint


Thanks for looking into this.... have a good vacation. get back to me later.

cheers,

Comment 7 Hans de Goede 2007-07-30 06:49:07 UTC
Ah,

The driver got renamed too, so its probably not getting loaded because of this. Try:
modprobe thinkpad_acpi

And after that do the ls of both /proc/acpi and /sys/class/hwmon again. Also try
removing and adding the applet again.

Really leaving for vacation now :)

Comment 8 Edmond 2007-07-30 12:14:22 UTC
indeed thinkpad_acpi is notloaded. after modprobe:

[root@mva ~]# ls /proc/acpi
ac_adapter  dsdt                 fadt  info            thermal_zone
battery     embedded_controller  fan   power_resource  video
button      event                ibm   processor       wakeup
[root@mva ~]# ls /sys/class/hwmon
hwmon0

see you when you are back.

Comment 9 Hans de Goede 2007-08-02 16:51:26 UTC
Hi back,

Could you do the following for me?:
ls -R /proc/acpi/ibm
ls -l /class/hwmon/hwmon0/device
cat /class/hwmon/hwmon0/device/name

And what happens if you remove and re-add the applet with the new driver loaded?


Comment 10 Edmond 2007-08-03 12:23:13 UTC
Please see log below:

[root@mva ~]# ls -R /proc/acpi/ibm
ls: cannot access /proc/acpi/ibm: No such file or directory
[root@mva ~]# ls -l /class/hwmon/hwmon0/device
ls: cannot access /class/hwmon/hwmon0/device: No such file or directory
[root@mva ~]# cat /class/hwmon/hwmon0/device/name
[root@mva ~]# 
[root@mva ~]# !modp
modprobe thinkpad_acpi
[root@mva ~]# ls -R /proc/acpi/ibm
/proc/acpi/ibm:
beep       brightness  driver  hotkey  light    video
bluetooth  cmos        fan     led     thermal  volume
[root@mva ~]# ls -l /class/hwmon/hwmon0/device
ls: cannot access /class/hwmon/hwmon0/device: No such file or directory
[root@mva ~]# cat /class/hwmon/hwmon0/device/name
cat: /class/hwmon/hwmon0/device/name: No such file or directory


When I do a modprobe thinkpad_api and re-add, all senor is backed..... so I
guess the problem is thinkpad_api somehow not automatically load?

Comment 11 Hans de Goede 2007-08-03 13:02:26 UTC
Ah, so it was just the module name change, and the new version does have
backward compatibility, good!

Please open a new bug against udev for the not autoloading, I'm closing this one.

I still want to implement libsensors support for the now also available hwmon
interface though. So could you still do the following for me? :

[root@mva ~]# ls -l /class/hwmon/hwmon0/device
ls: cannot access /class/hwmon/hwmon0/device: No such file or directory
[root@mva ~]# cat /class/hwmon/hwmon0/device/name
cat: /class/hwmon/hwmon0/device/name: No such file or directory

My bad that should have been:
ls -l /sys/class/hwmon/hwmon0/device
cat /sys/class/hwmon/hwmon0/device/name


Comment 12 Edmond 2007-08-06 14:36:09 UTC
k, will do.

Thanks for the help!

Comment 13 Edmond 2007-08-06 14:52:50 UTC
bug opened: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251015


Note You need to log in before you can comment on or make changes to this bug.