On Tue, 6 Jan 2009, Linus Torvalds wrote: > Btw, I hate the name of the helper function for that. "task_is_current()"? > "current" means something totally different in the linux kernel: it means > that the task is _this_ task. > > So the only sensible implementation of "task_is_current(task)" is to just > make it return "task == current", but that's obviously not what the > function wants to do. > > So it should be renamed. Something like "task_is_oncpu()" or whatever. > > I realize that the scheduler internally has that whole "rq->curr" thing, > but that's an internal scheduler thing, and should not be confused with > the overall kernel model of "current".
I totally agree that we should change the name of that. I never thought about the confusion that it would cause to someone that is not working so heavy on the scheduler. Thanks for the perspective. Sometimes us scheduler guys have our heads too much up the scheduler ass that all we see is scheduler crap ;-) -- Steve -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
