Messages in this thread | | | Subject | Re: [01/22] sched: Provide scheduler_ipi() callback in response to smp_send_reschedule() | From | Milton Miller <> | Date | Mon, 14 Mar 2011 21:59:18 -0600 |
| |
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
| |