On Thu, 23 Jan 2014 09:26:30 +0100 Peter Zijlstra <pet...@infradead.org> wrote:
> On Thu, Jan 23, 2014 at 12:11:04PM -0500, Dongsheng Yang wrote: > > On 01/22/2014 11:00 PM, Steven Rostedt wrote: > > > > > >Bah, I just noticed that TASK_NICE is in kernel/sched/sched.h not > > >include/linux/sched.h > > > > > >Peter, is there a reason that task_nice() is not a static inline in > > >sched.h and have these macros there too? They only reference fields in > > >task_struct that are already defined there. I don't see why they need > > >to be private to kernel/sched. > > > > Agree. These macros are useful to other modules out of kernel/sched. > > But they are private to kernel/sched currently. > > And the floodgates open.. _why_ would a module care about nice values? > That's sounds just so full of wrong. As Dongsheng already said, it's not for modules, but for other parts of the core kernel. It's not like modules or other parts can't just reimplement those macros. All the fields are already public in linux/sched.h. In fact, the reason for this discussion is to get rid of an open coded implementation in the tracing facility. -- Steve -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/