On Fri, 14 Dec 2007, Andi Kleen wrote:
> Miao Xie <[EMAIL PROTECTED]> writes:
>
> > Hi everyone,
> > I think we need a check in the function add_timer_on() to avoid adding a
> > timer into the timer list of an offline cpu.
> >
> > This patch fixes this problem in add_timer_on().
>
> No it doesn't -- it will just crash if a caller does that. If it's a real
> problem then you either need to fix the callers or change add_timer_on()
> to transparently put it onto another CPU.
>
> I don't think it's a real problem though because CPU offlining does
> a stop machine and that should take care of most such races.
No. CPU offlining does not prevent code to try inserting a timer on a
offlined CPU. If there is a real problem with this we need to fix the
calling code. We should not insert it somewhere else magically.
Thanks,
tglx
--
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/
