diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2009-06-12 13:25:37 +0200 |
---|---|---|
committer | Dave Jones <davej@redhat.com> | 2009-06-15 17:49:43 +0200 |
commit | 1ff6e97f1d993dff2f9b6f4a9173687370660232 (patch) | |
tree | 19cadec61f22a61a0e4c2d338ccd8672710aed17 /crypto | |
parent | [CPUFREQ] cpumask: avoid cpumask games in arch/x86/kernel/cpu/cpufreq/speedst... (diff) | |
download | linux-1ff6e97f1d993dff2f9b6f4a9173687370660232.tar.xz linux-1ff6e97f1d993dff2f9b6f4a9173687370660232.zip |
[CPUFREQ] cpumask: avoid playing with cpus_allowed in powernow-k8.c
cpumask: avoid playing with cpus_allowed in powernow-k8.c
It's generally a very bad idea to mug some process's cpumask: it could
legitimately and reasonably be changed by root, which could break us
(if done before our code) or them (if we restore the wrong value).
I did not replace powernowk8_target; it needs fixing, but it grabs a
mutex (so no smp_call_function_single here) but Mark points out it can
be called multiple times per second, so work_on_cpu is too heavy.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
To: cpufreq@vger.kernel.org
Acked-by: Mark Langsdorf <mark.langsdorf@amd.com>
Tested-by: Mark Langsdorf <mark.langsdorf@amd.com>
Signed-off-by: Dave Jones <davej@redhat.com>
Diffstat (limited to 'crypto')
0 files changed, 0 insertions, 0 deletions