On Mon, Sep 23, 2013 at 02:18:27PM -0700, Paul E. McKenney wrote:

Passes light testing, but I am confused about why the change to
resched_task() when only resched_cpu() is invoked elsewhere in the patch.
Enlightenment?

quoted

quoted

-#ifdef CONFIG_SMP
 void resched_task(struct task_struct *p)
 {
 	int cpu;

-	assert_raw_spin_locked(&task_rq(p)->lock);
+	lockdep_assert_held(&task_rq(p)->lock);

 	if (test_tsk_need_resched(p))
 		return;

That one? Fly by fixup I suppose.. the lockdep check is cheaper as it
compiles away for !lockdep kernels, also its more correct.