Re: [PATCH] wait: include linux/sched.h

[Peter Zijlstra]

On Tue, 2011-05-10 at 12:36 +0200, Ingo Molnar wrote:
> 
> Obviously sched.h would include a lot of basic types to begin with, because it 
> is a fundamental 'container' type (struct task_struct) sitting at a top, 
> highlevel node of the type hierarchy - so the initial step you suggest would 
> indeed get us most of the benefits. 

I very well understand the whole header mess, and in particular last
time I looked at the sched.h vs wait.h thing simply removing the non
sched.c related things was enough to resolve this particular problem.

I really don't like the foo_type.h headers much and think we should only
use them as absolute last resort solutions.

I might have mis-read your email, but to me your:

  "To fix these super-headers like sched.h and to make wait.h
self-sufficient the right and cleanest approach would be to split data
types and primitive accessor functions.."

Seems to suggest to begin with that split, instead of starting with sane
cleanups like moving the signal and process (groups of tasks) bits out
into their own headers.



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/