lkml.org 
[lkml]   [2008]   [Oct]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [pull request] getting rid of __cpuinit
On Fri, Oct 24, 2008 at 02:20:56PM -0700, Arjan van de Ven wrote:
> Hi Linus,
>
> this is one of those pull requests for which the best timing is likely at the
> end of the merge window due to touching a large number of files in a trivial way.
> In fact, 6 of the 8 patches were done via regexps:
>
> find -name "*.c" | xargs sed -i "s/^__cpuinit //g"
> find arch/ -name "*.h" | xargs sed -i "s/ __cpuinit//g"
> find -name "*.c" | xargs sed -i "s/ __cpuinit$//g"
> find -name "*.c" | xargs sed -i "s/ __cpuinitdata//g"
> find -name "*.c" | xargs sed -i "s/ __cpuexit / /g"
> find -name "*.c" | xargs sed -i "s/ __cpuinit / /g"
>
> (I suppose asking for a "git sed" is too much to do...)
>
> The point of this series is to get rid of __cpuinit. (I'll be working on other __XXXXinit after this)
> __cpuinit is a NOOP for CONFIG_HOTPLUG_CPU, which on for just about anybody in practice
> (in fact I had to disable 6 common options to even be ABLE to turn it off for size testing)
> and all distros also ship with it enabled.
>
> Yet it's a rather error prone construct since the rules of this sort of __XXXXinit thing are
> rather complex and convoluted.. better to just get rid of it entirely.
>
> As for the code size effect, after I spent a bunch of time cursing and trying to find out
> how to disable HOTPLUG_CPU, I managed to get an x86-64 build that actually shows how much space
> we ACTUALLY save with all the __cpuinit's in the tree... it's less than 2Kb. That is, if you manage
> to turn it actually off.
>
> text data bss dec hex filename
> 4618096 1358684 848588 6825368 682598 vmlinux.before
> 4619965 1356908 848716 6825589 682675 vmlinux.after

The interesting number here is the size of the cpuinit/cpuexit sections.
cpuinit sections are discarded after init and cpuexit sections are discarded
if you do not use modules.

Can you share these numbers with us?

Sam


\
 
 \ /
  Last update: 2008-10-25 10:01    [W:0.597 / U:0.052 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site