* [PATCH] kernel 2.6.5-mm1 : laptop-mode @ 2004-04-05 13:50 Angelo Dell'Aera 2004-04-05 21:52 ` Andrew Morton 0 siblings, 1 reply; 3+ messages in thread From: Angelo Dell'Aera @ 2004-04-05 13:50 UTC (permalink / raw) To: Linux-Kernel; +Cc: Andrew Morton After upgrading to 2.6.5-mm1 I noticed the script laptop_mode failed to initiliaze laptop mode. It is due to the new position of the sysctl laptop_mode under /proc. This is an update to the documentation (and the script). Please apply. Regards. -- Angelo Dell'Aera 'buffer' Antifork Research, Inc. http://buffer.antifork.org --- linux-2.6.5-mm1/Documentation/laptop-mode.txt.old 2004-04-05 15:39:25.000000000 +0200 +++ linux-2.6.5-mm1/Documentation/laptop-mode.txt 2004-04-05 15:47:33.000000000 +0200 @@ -80,7 +80,7 @@ The details ----------- -Laptop-mode is controlled by the flag /proc/sys/vm/laptop_mode. When this +Laptop-mode is controlled by the flag /proc/sys/fs/laptop_mode. When this flag is set, any physical disk read operation (that might have caused the hard disk to spin up) causes Linux to flush all dirty blocks. The result of this is that after a disk has spun down, it will not be spun up anymore @@ -321,12 +321,12 @@ # like the rest of the external world. Unfortunately this cannot be automated. :( XFS_HZ=1000 -if [ ! -e /proc/sys/vm/laptop_mode ]; then +if [ ! -e /proc/sys/fs/laptop_mode ]; then echo "Kernel is not patched with laptop_mode patch." exit 1 fi -if [ ! -w /proc/sys/vm/laptop_mode ]; then +if [ ! -w /proc/sys/fs/laptop_mode ]; then echo "You do not have enough privileges to enable laptop_mode." exit 1 fi @@ -355,11 +355,11 @@ case "$KLEVEL" in "2.4") - echo "1" > /proc/sys/vm/laptop_mode + echo "1" > /proc/sys/fs/laptop_mode echo "30 500 0 0 $AGE $AGE 60 20 0" > /proc/sys/vm/bdflush ;; "2.6") - echo "5" > /proc/sys/vm/laptop_mode + echo "5" > /proc/sys/fs/laptop_mode echo "$AGE" > /proc/sys/vm/dirty_writeback_centisecs echo "$AGE" > /proc/sys/vm/dirty_expire_centisecs echo "$DIRTY_RATIO" > /proc/sys/vm/dirty_ratio @@ -389,7 +389,7 @@ U_AGE=$((100*$DEF_UPDATE)) B_AGE=$((100*$DEF_AGE)) echo -n "Stopping laptop_mode" - echo "0" > /proc/sys/vm/laptop_mode + echo "0" > /proc/sys/fs/laptop_mode if [ -f /proc/sys/fs/xfs/age_buffer ] && [ ! -f /proc/sys/fs/xfs/lm_age_buffer ] ; then # These need to be restored though, if there are no lm_*. echo "$(($XFS_HZ*$DEF_XFS_AGE_BUFFER))" > /proc/sys/fs/xfs/age_buffer ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] kernel 2.6.5-mm1 : laptop-mode 2004-04-05 13:50 [PATCH] kernel 2.6.5-mm1 : laptop-mode Angelo Dell'Aera @ 2004-04-05 21:52 ` Andrew Morton 2004-04-05 22:33 ` Angelo Dell'Aera 0 siblings, 1 reply; 3+ messages in thread From: Andrew Morton @ 2004-04-05 21:52 UTC (permalink / raw) To: Angelo Dell'Aera; +Cc: linux-kernel "Angelo Dell'Aera" <buffer@antifork.org> wrote: > > After upgrading to 2.6.5-mm1 I noticed the script laptop_mode > failed to initiliaze laptop mode. It is due to the new position > of the sysctl laptop_mode under /proc. This is an update to the > documentation (and the script). Please apply. > ... > > -Laptop-mode is controlled by the flag /proc/sys/vm/laptop_mode. When this > +Laptop-mode is controlled by the flag /proc/sys/fs/laptop_mode. When this erk. No, that was not intended. Looks like `patch' decided to move some code around for me. I'll fix that up, thanks. laptop_mode shall remain in /proc/sys/vm/ ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] kernel 2.6.5-mm1 : laptop-mode 2004-04-05 21:52 ` Andrew Morton @ 2004-04-05 22:33 ` Angelo Dell'Aera 0 siblings, 0 replies; 3+ messages in thread From: Angelo Dell'Aera @ 2004-04-05 22:33 UTC (permalink / raw) To: Andrew Morton; +Cc: Linux-Kernel On Mon, 5 Apr 2004 14:52:09 -0700 Andrew Morton <akpm@osdl.org> wrote: >"Angelo Dell'Aera" <buffer@antifork.org> wrote: >> >> After upgrading to 2.6.5-mm1 I noticed the script laptop_mode >> failed to initiliaze laptop mode. It is due to the new position >> of the sysctl laptop_mode under /proc. This is an update to the >> documentation (and the script). Please apply. >> ... >> >> -Laptop-mode is controlled by the flag /proc/sys/vm/laptop_mode. When this >> +Laptop-mode is controlled by the flag /proc/sys/fs/laptop_mode. When this > >erk. No, that was not intended. Looks like `patch' decided to move some code around >for me. I'll fix that up, thanks. laptop_mode shall remain in /proc/sys/vm/ OK. Attached a patch which addresses this matter. Regards. -- Angelo Dell'Aera 'buffer' Antifork Research, Inc. http://buffer.antifork.org --- linux-2.6.5-mm1/kernel/sysctl.c.old 2004-04-06 00:26:15.000000000 +0200 +++ linux-2.6.5-mm1/kernel/sysctl.c 2004-04-06 00:28:29.000000000 +0200 @@ -744,6 +744,26 @@ .mode = 0644, .proc_handler = &proc_dointvec }, + { + .ctl_name = VM_LAPTOP_MODE, + .procname = "laptop_mode", + .data = &laptop_mode, + .maxlen = sizeof(laptop_mode), + .mode = 0644, + .proc_handler = &proc_dointvec, + .strategy = &sysctl_intvec, + .extra1 = &zero, + }, + { + .ctl_name = VM_BLOCK_DUMP, + .procname = "block_dump", + .data = &block_dump, + .maxlen = sizeof(block_dump), + .mode = 0644, + .proc_handler = &proc_dointvec, + .strategy = &sysctl_intvec, + .extra1 = &zero, + }, { .ctl_name = 0 } }; @@ -854,26 +874,6 @@ .mode = 0644, .proc_handler = &proc_dointvec, }, - { - .ctl_name = VM_LAPTOP_MODE, - .procname = "laptop_mode", - .data = &laptop_mode, - .maxlen = sizeof(laptop_mode), - .mode = 0644, - .proc_handler = &proc_dointvec, - .strategy = &sysctl_intvec, - .extra1 = &zero, - }, - { - .ctl_name = VM_BLOCK_DUMP, - .procname = "block_dump", - .data = &block_dump, - .maxlen = sizeof(block_dump), - .mode = 0644, - .proc_handler = &proc_dointvec, - .strategy = &sysctl_intvec, - .extra1 = &zero, - }, { .ctl_name = 0 } }; ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2004-04-05 22:36 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2004-04-05 13:50 [PATCH] kernel 2.6.5-mm1 : laptop-mode Angelo Dell'Aera 2004-04-05 21:52 ` Andrew Morton 2004-04-05 22:33 ` Angelo Dell'Aera
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox