Add Inspiron (i8k) fan speed support

Bug #700204 reported by swalker
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
byobu
Fix Released
Low
Unassigned
byobu (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Currently, the fan speed script doesn't print any values for my Dell Inspiron.

Related branches

Revision history for this message
swalker (sdwalker) wrote :
Revision history for this message
swalker (sdwalker) wrote :

+ if [ -r /proc/i8k ]; then
+ for i in $(awk '{ print $7, $8 }' /proc/i8k); do
+ echo "$i"
+ done
+ else
 ...
+ fi

would probably be wanted for the detail case as well.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Thanks for the patch. I fixed it up a little bit. I'd appreciate any testing feedback, when you get a chance.

Committed revision 1286.

Changed in byobu:
status: New → Fix Committed
Changed in byobu (Ubuntu):
importance: Undecided → Low
status: New → Fix Committed
Changed in byobu:
importance: Undecided → Low
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package byobu - 3.26-0ubuntu1

---------------
byobu (3.26-0ubuntu1) natty; urgency=low

  [ Dustin Kirkland ]
  * usr/bin/byobu-reconnect-sockets, usr/lib/byobu/date,
    usr/share/byobu/keybindings/f-keys,
    usr/share/byobu/keybindings/screen-escape-keys:
    - add a ctrl-f5 keybinding for reconnecting sockets
  * usr/share/byobu/profiles/common: explicitly set deflogin on,
    LP: #525552

  [ swalker <email address hidden> ]
  * A set of fixes for LP: #700911
  * usr/bin/byobu-status, usr/bin/byobu-status-detail:
    - improve support for byobu-status-detail on systems without vim
  * usr/lib/byobu/mail: test for mail dir, avoids errors
  * usr/lib/byobu/reboot_required: test for file existence, avoids errors

  [ Dustin Kirkland + swalker <email address hidden> ]
  * usr/lib/byobu/wifi_quality: avoid potential divide by zero error
  * usr/lib/byobu/fan_speed: add support for /proc/i8k for some other
    systems' fan speeds (e.g. Dell Inspiron), LP: #700204
 -- Dustin Kirkland <email address hidden> Sun, 23 Jan 2011 16:28:45 -0600

Changed in byobu (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
swalker (sdwalker) wrote :

The detail case is broken:
$ /usr/lib/byobu/fan_speed --detail
/sys/class/hwmon/hwmon0:

$ sh -x /usr/lib/byobu/fan_speed --detail
+ PKG=byobu
+ color
+ DIR=/sys/class/hwmon
+ [ --detail = --detail ]
+ echo /sys/class/hwmon/hwmon0:
/sys/class/hwmon/hwmon0:
+ cat /sys/class/hwmon//sys/class/hwmon/hwmon0/*
+ echo /proc/i8k:
/proc/i8k:
+ cat /sys/class/hwmon//proc/i8k/*
+ exit 0

The normal usage is working though.

Revision history for this message
Dustin Kirkland  (kirkland) wrote : Re: [Bug 700204] Re: Add Inspiron (i8k) fan speed support

Okay, thanks.

I'm just going to no-op the --detail section of fan_speed. There's
nothing very interesting happening there right now anyway.

Changed in byobu:
status: Fix Committed → Fix Released
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.