Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Pavel Machek
Hi! > > Hello Rafael, > > Thanks for all of the patches so far. :-) > > > Yet another patch in the suspend2 feature-set series. > > It sets cpufreq to maximum during the cycle. > > It is important to finish the process as quickly as possible (human > > engineer issue :) ) > > When running on b

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Alon Bar-Lev
On 7/21/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > I'm not sure it is good idea. We should either let cpu frequency > scaling do its job during suspend (it should just scale to the max > when it sees cpu load), or force it to _minimum_. Machines will > overheat on max cpufreq, and they may overl

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Till Maas
On Saturday 21 July 2007 11:50:01 Pavel Machek wrote: > I'm not sure it is good idea. We should either let cpu frequency > scaling do its job during suspend (it should just scale to the max > when it sees cpu load), or force it to _minimum_. Machines will > overheat on max cpufreq, and they may ov

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Alon Bar-Lev
On 7/21/07, Till Maas <[EMAIL PROTECTED]> wrote: > some battery life. But imho it is better to test it first, maybe the ondemand > govenor is even better. You may have a configuration in which ondemand can be in range of minimum to X, where X /proc/acpi/sony/brightness profile=On Demand Low [/Rule

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Holger Macht
On Sat 21. Jul - 01:24:51, Tim Dijkstra wrote: > On Fri, 20 Jul 2007 22:20:51 +0200 > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > > > Yet another patch in the suspend2 feature-set series. > > > It sets cpufreq to maximum during the cycle. > > I haven't looked at the patch, but this doesn't

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Alon Bar-Lev
On 7/21/07, Holger Macht <[EMAIL PROTECTED]> wrote: > This is already done in pm-utils. Upstream uses 'userspace' as governor, > which I personally think is wrong. openSUSE uses the 'performance' > governor and this only because of a bug in the kernel which caused the > system to hang on suspend AF

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Tim Dijkstra
On Sat, 21 Jul 2007 18:28:24 +0300 "Alon Bar-Lev" <[EMAIL PROTECTED]> wrote: > On 7/21/07, Holger Macht <[EMAIL PROTECTED]> wrote: > > This is already done in pm-utils. Upstream uses 'userspace' as governor, > > which I personally think is wrong. openSUSE uses the 'performance' > > governor and th

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Holger Macht
On Sat 21. Jul - 18:28:24, Alon Bar-Lev wrote: > On 7/21/07, Holger Macht <[EMAIL PROTECTED]> wrote: > > This is already done in pm-utils. Upstream uses 'userspace' as governor, > > which I personally think is wrong. openSUSE uses the 'performance' > > governor and this only because of a bug in the

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Alon Bar-Lev
On 7/21/07, Holger Macht <[EMAIL PROTECTED]> wrote: > Right. So if we start putting things like cpufreq in the suspend package, > we would have an argumentation to put everything which is in pm-utils, the > hibernate script etc. to it. And I don't think this is what we want. You can put vbetool, r

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Alon Bar-Lev
On 7/21/07, Alon Bar-Lev <[EMAIL PROTECTED]> wrote: > On 7/21/07, Holger Macht <[EMAIL PROTECTED]> wrote: > > Right. So if we start putting things like cpufreq in the suspend package, > > we would have an argumentation to put everything which is in pm-utils, the > > hibernate script etc. to it. And

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Tim Dijkstra
On Sat, 21 Jul 2007 23:08:30 +0300 "Alon Bar-Lev" <[EMAIL PROTECTED]> wrote: > On 7/21/07, Alon Bar-Lev <[EMAIL PROTECTED]> wrote: > > On 7/21/07, Holger Macht <[EMAIL PROTECTED]> wrote: > > > Right. So if we start putting things like cpufreq in the suspend package, > > > we would have an argument

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-21 Thread Alon Bar-Lev
On 7/21/07, Tim Dijkstra <[EMAIL PROTECTED]> wrote: > No. The vbetool code is linked in. It doesn't run as regular processes. > At the time we do those hacks, we are the only not-frozen part of userspace. > This is what makes it different from running it from a bunch of scripts. Are you sure? Can

[Suspend-devel] IBM ACPI suspend led is turned off during suspend

2007-07-21 Thread Alon Bar-Lev
Hello, IBM suspend led is turned on before suspend, but for some strange reason after kernel snapshot (before the actual page write) it is turned off, while it should stay blinking. You turn on the blink using: # echo 7 blink > /proc/acpi/ibm/led You turn off the blink using: # echo 7 off > /proc