On Sun, Aug 31, 2008 at 7:51 PM, Paul E. McKenney
<paulmck@linux.vnet.ibm.com> wrote:
quoted text
>> > @@ -751,6 +749,7 @@ int __cpu_disable(void)
>> >        remove_siblinginfo(cpu);
>> >        cpu_clear(cpu, cpu_online_map);

Here ^^^

quoted text
>> >        fixup_irqs();
>> > +       cpu_clear(cpu, cpu_online_map);
>> >        local_flush_tlb_all();
>> >        cpu_clear(cpu, cpu_callin_map);
>> >        return 0;
>>
>> Hm. Maybe my eyes are going bad. But isn't that now clearing the cpu
>> both before and after calling fixup_irqs()?
>
> I don't believe so...  I removed the earlier one and added the later
> one.  Or is there another one hidden somewhere that I missed?

See marker above. It's not removed, is it?


Vegard

-- 
"The animistic metaphor of the bug that maliciously sneaked in while
the programmer was not looking is intellectually dishonest as it
disguises that the error is the programmer's own creation."
	-- E. W. Dijkstra, EWD1036
--
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/