Your message dated Thu, 10 Jul 2014 13:03:48 +0000 with message-id <[email protected]> and subject line Bug#659260: fixed in pm-utils 1.4.1-15 has caused the Debian Bug report #659260, regarding incompatible with /etc/sysctl.conf and not obvious to be marked as done.
This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the Bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what this message is talking about, this may indicate a serious mail system misconfiguration somewhere. Please contact [email protected] immediately.) -- 659260: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=659260 Debian Bug Tracking System Contact [email protected] with problems
--- Begin Message ---Package: pm-utils Version: 1.4.1-9 Severity: normal Tags: patch === Problem and consideration === I was trying to set slow updates for my SSD on my laptop even under AC power ==== # ssd_ac_mode default (for testing) vm.laptop_mode = 5 vm.dirty_ratio = 60 vm.background_ratio = 40 vm.dirty_writeback_centisecs = 60000 ==== It did not work. Reading manpage of pm-powersave did not give me any hints. Reading pm-utils source and few experiments lead me to realize that the hard-coded numbers embedded in the laptop_mode as "write_values 0 10 5 500" was the place to edit such configuration. This is absurd. This may be somewhat of regression of parallel init script. This script was executed after /etc/syscyrl.conf now and since there were no state file this hard-coded value was used when booted under AC. /etc/syscyrl.conf was used for AC when booted under battery and plugged. === Solution 1: simple fix === In laptop_mode, change laptop_mode_ac() function as follows. === laptop_mode_ac() { # disable laptop mode, set vm parameters back to sane defaults if state_exists laptop_mode_default; then write_values $(restorestate laptop_mode_default) fi echo "Laptop mode disabled." } === If you wish to change via /etc/syscyrl.conf for AC setting, it works. === Solution 2: robust fix with added feature === Since above hard-coded segment may have been introduced for some other interference, I think simple solution of providing legible and fixed code setting both AC and battery mode should be safer and more stable for /usr/lib/pm-utils/power.d/laptop-mode. This code has additional benefit of using slow battery mode for SSD on laptop as default. If you think this is too much for default configuration, please consider to add this to /usr/share/pm-utils/examples/laptop_mode ====== EXPERIMENTAL RESULT ===== = Normal system = == Normal - start on AC == • boot on AC • hard-coded AC setting in laptop_mode of pm-utils (no state file) • unplug • hard-coded BATT setting in laptop_mode of pm-utils (state file with hard-coded AC setting in laptop_mode of pm-utils) • plug • hard-coded AC setting in laptop_mode of pm-utils == Normal - start on BATT == • boot on BATT • hard-coded BATT setting in laptop_mode of pm-utils (state file with sysctrl.conf) • plug • setting in /etc/sysctrl.conf content (state file with sysctrl.conf) • unplug • hard-coded BATT setting in laptop_mode of pm-utils (state file with sysctrl.conf) = Disable hard-coded AC - solution 1 applied = = Disable hard-coded AC - start on AC = • boot on AC • setting in /etc/sysctrl.conf content (no state file) • unplug • hard-coded BATT setting in laptop_mode of pm-utils (state file with sysctrl.conf) • plug • setting in /etc/sysctrl.conf content (state file with sysctrl.conf) = Disable hard-coded AC - start on BATT = • boot on BATT • hard-coded BATT setting in laptop_mode of pm-utils (state file with sysctrl.conf) • plug • setting in /etc/sysctrl.conf content (state file with sysctrl.conf) • unplug -- System Information: Debian Release: wheezy/sid APT prefers unstable APT policy: (500, 'unstable') Architecture: amd64 (x86_64) Kernel: Linux 3.2.0-1-amd64 (SMP w/8 CPU cores) Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Versions of packages pm-utils depends on: ii powermgmt-base 1.31 Versions of packages pm-utils recommends: ii hdparm 9.32-1 ii kbd 1.15.3-7 ii procps 1:3.3.2-3 ii vbetool 1.1-2 Versions of packages pm-utils suggests: ii cpufrequtils 007-2 ii ethtool 1:3.1-1 ii radeontool 1.6.2-1.1 ii wireless-tools 30~pre9-8 -- no debconf information
--- End Message ---
--- Begin Message ---Source: pm-utils Source-Version: 1.4.1-15 We believe that the bug you reported is fixed in the latest version of pm-utils, which is due to be installed in the Debian FTP archive. A summary of the changes between this version and the previous one is attached. Thank you for reporting the bug, which will now be closed. If you have further comments please address them to [email protected], and the maintainer will reopen the bug report if appropriate. Debian distribution maintenance software pp. Martin Pitt <[email protected]> (supplier of updated pm-utils package) (This message was generated automatically at their request; if you believe that there is a problem with it please contact the archive administrators by mailing [email protected]) -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 Format: 1.8 Date: Thu, 10 Jul 2014 14:58:31 +0200 Source: pm-utils Binary: pm-utils Architecture: source all Version: 1.4.1-15 Distribution: unstable Urgency: medium Maintainer: Tim Dijkstra <[email protected]> Changed-By: Martin Pitt <[email protected]> Description: pm-utils - utilities and scripts for power management Closes: 659260 Changes: pm-utils (1.4.1-15) unstable; urgency=medium . * laptop-mode: If there is no state file (such as at boot), don't overwrite /proc/sys/vm/ values with fixed kernel defaults, simply do nothing. This allows sysctl.conf to set these values. Thanks to Osamu Aoki! (Closes: #659260) * Add 17-fix-wireless-hook.patch: Fix typo in wireless hook to make enabled check actually work. Thanks to Whoopie! (LP: #1299975) Checksums-Sha1: 8fa5da8f7997892f6da8c49f1389575d2fd4fee3 1923 pm-utils_1.4.1-15.dsc 856c311edc16a41909fabf1962788f36e8ee6eb3 26748 pm-utils_1.4.1-15.debian.tar.xz 036b84bbd55c41c5043c1f860a3c095d651b2c23 120538 pm-utils_1.4.1-15_all.deb Checksums-Sha256: ff9419f88c5d9e887b5ab59281ab855b65ccdd81de5747afd31bf6918a4821f0 1923 pm-utils_1.4.1-15.dsc cd408d1af824a1fcf5dcac65e66c9adae7cf1e3a48c87aaf888dc228b1dfdfc1 26748 pm-utils_1.4.1-15.debian.tar.xz 2f915aebc15b1e816f4d0a7d81795b07bbf241d03ecd00acc73a4c0867e09219 120538 pm-utils_1.4.1-15_all.deb Files: 81797870b698b665ed1ca4b89ed4f269 120538 admin optional pm-utils_1.4.1-15_all.deb 1fcb586c8cbec4acbce36eb41d051e58 1923 admin optional pm-utils_1.4.1-15.dsc ffd8a055a7433deae9355be85cf54a84 26748 admin optional pm-utils_1.4.1-15.debian.tar.xz -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBCAAGBQJTvo6CAAoJENFO8V2v4RNHyGMQAMliC4kUyJETCb1hm+BxNlDx q4DZJgXa5Z8KQatriN/D1gxhYyRbpEaH01wBcv6nP81h7wwMFApzrATje6SNIy6g dFRVbE1zzydXePBeohYOSO/EEILb5D8egF4o85l8tFpTsNqt2liizMv1CTgBkPyb kbrSHPvtYJPIPUC9aQl6U37pxWKR4G3NVELJao7rKlpt0XpSY6ICXQBQhvYxt6fb SCLFV9jY5gtAW4Xdq4ThC6pHnOZhjkQJtK2L5yT47tYlQPGoJ1TnXtqim58NTUKj C7C1KZplLubZLbEVax0tmerBqR6DOOIfU7EcqG8PMAxliqbfjcLRwdUo9vvWErLg NJxCGXudYVc4HQWyYMSCzvqjtikeqBS4z72CQlI9oiVcbAzElXDzPsoI2uOwuJZ+ dpubeWRS1U0tldaTCPmE441VlDX+wXvi7YKXRq14zdZt3UWMSRu0dyEAfutHjzbQ 0wF50JaMgxo4ZdCkUoobuOK4FLVbDnU8VraFMJtDlE+4s9//hJh0upGbTI7R7nHC quEzRr5zuCR+fhTKkH29yR/MU9Fexuqb9qftuEwCId81HHL8mBZ+lktcJ1KD1ftq ZEa0PbpC17UeBISpkV6WPTQ+HPKp0ti+YykVotjClYbZ5mXh7D06i9tugUA2DkYv YOWwrt9bZgullET4Ducv =98FZ -----END PGP SIGNATURE-----
--- End Message ---
