Bug 169054 - pm-utils is missing a laptop-mode type script
pm-utils is missing a laptop-mode type script
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: pm-utils (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Phil Knirsch
: FutureFeature
Depends On:
Blocks: FC5Target
  Show dependency treegraph
 
Reported: 2005-09-22 12:32 EDT by Richard Hughes
Modified: 2015-03-04 20:15 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2006-02-24 07:24:54 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
Example file that could be used in pm-utils (1.62 KB, text/plain)
2005-09-22 12:32 EDT, Richard Hughes
no flags Details

  None (edit)
Description Richard Hughes 2005-09-22 12:32:29 EDT
Description of problem:
pm-utils has hibernate and suspend scripts that work really well in a fedora
core rawhide system.
It does not have a script to change the various vm arguments like the script
laptop-tools does, e.g.

echo $DISK_IDLE_SECS > /proc/sys/vm/laptop_mode

Version-Release number of selected component (if applicable):
pm-utils-0.04-1

Additional info:
HAL will call this script for SetLowPower() dbus methods that is invoked on
*only* laptops.
Comment 1 Richard Hughes 2005-09-22 12:32:30 EDT
Created attachment 119149 [details]
Example file that could be used in pm-utils
Comment 2 Richard Hughes 2005-09-28 12:52:47 EDT
Should this be a blocker for FC5?
Comment 3 Bill Nottingham 2005-09-28 13:03:09 EDT
It's on the target list for FC5.
Comment 4 Richard Hughes 2006-02-11 10:05:20 EST
Any news? FC5t3 will be out soon, and it's simply a case of adding one file to a
package... This single file makes laptop-mode tools obsolete, and also means
gnome-power-manager doesn't spit out errors from hal about no LowPowerMode script.
Hal currently looks for pm-powersave, so if the new script is called that name,
then it will just work with HAL. If you change the name to something like
pm-laptopmode, then I can trivially patch HAL - it's up to you.
SuSE already support laptop-mode by default in the powersaved package.
Comment 5 David Zeuthen 2006-02-12 12:29:57 EST
Bill, Phil, could we put this in? It makes a lot of sense to include it and it
seems like pretty low risk.
Comment 6 David Zeuthen 2006-02-23 16:44:01 EST
Ping? Can we do this please? It seems like extremely low risk since a number of
our competitors is already doing this.
Comment 7 Phil Knirsch 2006-02-24 03:41:13 EST
Yeah, will put it in today.

Read ya, Phil
Comment 8 Richard Hughes 2006-02-24 12:09:46 EST
What's the script called, then I can send a patch for HAL. Thanks for doing this.
Comment 9 David Zeuthen 2006-02-24 12:49:27 EST
I've already fix hal upstream. The hal-0.5.7-1 Fedora package will include this
patch.

--- hal-system-power-set-power-save	2005/11/29 17:47:54	1.4
+++ hal-system-power-set-power-save	2006/02/24 16:47:11	1.5
@@ -25,10 +25,10 @@
 elif [ -f /etc/redhat-release ] || [ -f /etc/fedora-release ] ; then
 	if [ -x "/usr/sbin/pm-powersave" ] ; then
 		if [ $value = "true" ]; then
-			/usr/sbin/pm-powersave on
+			/usr/sbin/pm-powersave true
 			RET=$?
 		elif [ $value = "false" ]; then
-			/usr/sbin/pm-powersave off
+			/usr/sbin/pm-powersave false
 			RET=$?
 		fi
 	else
Comment 10 Richard Hughes 2006-02-24 13:34:07 EST
Good man, thanks for doing this.

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