On Tue, Apr 22 2008, Peter Zijlstra wrote: > On Tue, 2008-04-22 at 20:50 +0200, Jens Axboe wrote: > > > +int smp_call_function_single(int cpu, void (*func) (void *info), void *info, > > + int retry, int wait) > > +{ > > + unsigned long flags; > > + /* prevent preemption and reschedule on another processor */ > > + int me = get_cpu(); > > + > > + /* Can deadlock when called with interrupts disabled */ > > + WARN_ON(wait && irqs_disabled()); > > With this fallback to wait the above condition isn't sufficient. What deadlock are you concerned with here? Would making cfd_fallback per-cpu make you feel better? -- Jens Axboe -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html