On Sun, Apr 25, 2010 at 11:16:44AM +0200, Petter Reinholdtsen wrote:
> 
> Package: cpufrequtils
> Version:  006-2
> Tags: patch
> 
> I had a look at the powernowd-1.00/debian/cpufreq-detect.sh file in
> Ubuntu powernowd package version 1.00-1ubuntu5, which the loadcpufreq
> init.d script is based on, to see if there had been any updates.  This
> is the relevant patch.  Apparently Ubuntu now loads the
> speedstep-centrino module for Intel CPUs with the est flag, and only

odd, we never loaded speedstep-centrino on any est capable machine only
based on the kernel  version since (almost) forever, see:
http://git.kamineko.org/cgi-bin/gitweb.cgi?p=cpufrequtils.git;a=commitdiff;h=8c4880e717bf3fec4034ec2cfed346550eb1a213
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=502431
To be honest I think it should be the Ubuntu package that should get
code from Debian.

> load the e_powersaver module on VIA CPUs with the est flag (perhaps
> this is the solution for #566214?).

e_powersaver won't load anyway on non-est capable cpus, from
e_powersaver.c:
        if (!cpu_has(c, X86_FEATURE_EST))
                        return -ENODEV;

and really, looking at the help text from the kernel sources it looks
like acpi_cpufreq should be used instead:

        help
          This adds the CPUFreq driver for VIA C7 processors.  However, this 
driver
          does not have any safeguards to prevent operating the CPU out of spec
          and is thus considered dangerous.  Please use the regular ACPI cpufreq
          driver, enabled by CONFIG_X86_ACPI_CPUFREQ.

I would rather do that at this point.
-- 
mattia
:wq!



-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to