lkml.org 
[lkml]   [2007]   [Dec]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 1/2] remove __init modifier from header declaration
From
On Wed, Dec 19, 2007 at 11:12:36AM +0100, Ingo Molnar wrote:
> * Glauber de Oliveira Costa <gcosta@redhat.com> wrote:
>
> > This patch removes the __init modifier from an extern function
> > declaration in acpi.h.
> >
> > Besides not being strictly needed, it requires the inclusion of
> > linux/init.h, which is usually not even included directly, increasing
> > header mess by a lot.
>
> thanks, applied.
>
> btw., people have been talking about reducing the include file mess for
> nearly a decade now,

Some of us are actually doing it. ;-)

> but it didnt get that much better - at least not in
> include/asm-x86/*.h.

That's because hunting in include/linux/ is much more fruitful -- it
decreases amount of code after preprocessing for everyone. Below are
allyesconfig on x86_64 results for some kernels:

$ wc ../*.i
79018171 229433654 2049266884 ../2.6.18.i
86568111 250674099 2245115663 ../2.6.19.i
85187296 247470579 2221334183 ../2.6.20.i <===
88645422 242396928 2234855428 ../2.6.21.i
93897302 257990496 2377174442 ../2.6.22.i
98381373 268683402 2486321956 ../2.6.23.i

E.g. we can have ~1.07% decrease in cpp output despite ~126000 lines
added between .19 and .20.

> So include file dependency flattening patches would
> be more than welcome as well.

Yup! Provided they're compile-tested sufficiently well.

> (and unlike unification patches they have
> no expectation of being 100% perfect, so a natural ping-pong of fixes,
> until the changes are fully correct, would be natural.)

That way you'll never clean anything especially during merge window. ;-)


\
 
 \ /
  Last update: 2007-12-31 17:49    [W:0.070 / U:1.092 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site