diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2009-06-12 07:25:37 -0400 |
---|---|---|
committer | Dave Jones <davej@redhat.com> | 2009-06-15 11:49:43 -0400 |
commit | 1ff6e97f1d993dff2f9b6f4a9173687370660232 (patch) | |
tree | 19cadec61f22a61a0e4c2d338ccd8672710aed17 /arch/m32r/platforms/opsput/setup.c | |
parent | e3f996c26ff6c4c084aaaa64dce6e54d31f517be (diff) |
[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 'arch/m32r/platforms/opsput/setup.c')
0 files changed, 0 insertions, 0 deletions