On Tue, 4 Oct 2016, tip-bot for Ingo Molnar wrote: > The original change widens a preempt-off section, to avoid a seemingly unsafe > smp_processor_id() use. > > During review I overlooked two facts: > > - The code to calls a non-trivial function callback: > > ht->park(td->cpu); > > ... which might (and does occasionally) sleep, triggering the warning. > > - More importantly, as pointed out by Peter Zijlstra, using > smp_processor_id() in that context is safe, if it's done from > a kernel thread that is pinned to a single CPU - which is the > case here. > > So revert to the original code that enables preemption sooner. > > Reported-by: kernel test robot <xiaolong...@intel.com> > Acked-by: Peter Zijlstra <pet...@infradead.org>
Acked-by: Thomas Gleixner <t...@linutronix.de>
