Messages in this thread |  | | | Subject | Re: [PATCH] wait: include linux/sched.h | | From | Peter Zijlstra <> | | Date | Tue, 10 May 2011 13:07:00 +0200 |
| |
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.
|  |