Re: [bloat] Measuring header file bloat effects on kernel buildperformance: a more than 2x slowdown ...

From: Ingo Molnar
Date: Mon May 23 2011 - 13:01:50 EST



* Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:

> On Mon, May 23, 2011 at 2:09 AM, Ingo Molnar <mingo@xxxxxxx> wrote:
> >
> > I've attached a totally hacky patch that removes all the big #include's from
> > kernel/pid.c and includes all structure and API definitions explicitly.
>
> Hmm.
>
> A less hacky patch might be to split up "sched.h" into multiple
> smaller things and at least get *part* of the way.
>

Yeah, absolutely - Peter already raised that a couple of days ago in a sched.h
discussion so it's on the radar.

The hacky patch was really just a throw-away attempt to see where we stand - we
only had vague impressions about the level of problems we have, now we know
some numbers.

> A lot of things want "struct task_struct" (and in some cases thread_info, but
> that's already split).
>
> Much fewer care about the signal stuff.
>
> And many things probably don't even need the task_struct definition, and
> might be perfectly happy with just function calls rather than having intimate
> knowledge of the structure layout and an inline function.

Yeah.

Thanks,

Ingo
--
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/