Re: [01/22] sched: Provide scheduler_ipi() callback in response to smp_send_reschedule()

Milton Miller
2011-03-15T04:00:22Z

On Fri, 11 Mar 2011 16:27:27 +0100, Peter Zijlstra wrote:
>
> On Fri, 2011-03-11 at 09:07 -0600, Milton Miller wrote:
> > > case PPC_MSG_RESCHEDULE:
> > > /* we notice need_resched on exit */
> > > + smp_reschedule_ipi()
> >
> > This comment should also be removed, as it was documenting the empty
> > action.
>
> But its still true, TIF_NEED_RESCHED isn't going away and we still
> notice that on the interrupt return path.

Just because it is true does not mean it is useful. Why is this site
different than the twenty odd other call sites that it needs to document
the behaivors requried upon exiting the kernel from interrupt deep in this
interrupt handling call path? Why is it not placed after the call, where
it would talk about what happens next instead of what will happen later?
For that matter, what does on exit mean?

I went ahead and sent this when I saw the comments questioning the
sparc 32 part of the patch and thought you might respin. But if this is
already committed then I will submit a followup patch for consideration
after this is merged.

milton