On Mon, Apr 09, 2007 at 11:05:00PM -0400, Dave Jones wrote:
> This whole file is going away in .22, and we have a viable alternative in
> .21 (acpi-cpufreq), so I'm not overly worried about fixing this up
> given it only shows up in debug kernels, especially at this stage in -rc.
>
> (Yeah, it's a cop-out, but unless someone with more interest in this problem
> steps up, I've bigger fishes to fry).
One last try...
(I didn't think too long about this, so this might be equally busted,
but if so, see comment above).
Dave
diff --git a/arch/i386/kernel/cpu/cpufreq/speedstep-centrino.c b/arch/i386/kernel/cpu/cpufreq/speedstep-centrino.c
index f43b987..38e31ce 100644
--- a/arch/i386/kernel/cpu/cpufreq/speedstep-centrino.c
+++ b/arch/i386/kernel/cpu/cpufreq/speedstep-centrino.c
@@ -720,11 +720,13 @@ static int centrino_target (struct cpufreq_policy *policy,
cpu_set(j, set_mask);
set_cpus_allowed(current, set_mask);
+ preempt_disable();
if (unlikely(!cpu_isset(smp_processor_id(), set_mask))) {
dprintk("couldn't limit to CPUs in this domain\n");
retval = -EAGAIN;
if (first_cpu) {
/* We haven't started the transition yet. */
+ preempt_enable();
goto migrate_end;
}
break;
@@ -765,6 +767,7 @@ static int centrino_target (struct cpufreq_policy *policy,
break;
cpu_set(j, covered_cpus);
+ preempt_enable();
}
for_each_cpu_mask(k, online_policy_cpus) {
--
http://www.codemonkey.org.uk
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
