login
Header Space

 
 

Re: [rfc] the kernel workflow & trivial "global -> static" patches

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Ingo Molnar <mingo@...>
Cc: Adrian Bunk <bunk@...>, Peter Zijlstra <a.p.zijlstra@...>, <linux-kernel@...>, Andrew Morton <akpm@...>, Linus Torvalds <torvalds@...>, Sam Ravnborg <sam@...>, Alexander Viro <viro@...>, H. Peter Anvin <hpa@...>
Date: Monday, May 5, 2008 - 8:21 pm

Ingo Molnar <mingo@elte.hu> writes:


I don't think the code changes actually with current gcc for integer
code if you change something from global to static (unless it causes
gcc to inline the function, but then it might be well larger if you're
unlucky)

The only file size change you'll see will be from a smaller symbol
table in the vmlinux ELF file, but that is not even loaded at run time
or included into the bzImage (and the kallsyms table has statics too)

So if we follow that "smaller symbol table" is better model
should we make a rule that all globals be 8 characters only? Or perhaps 6?  @)
I'm sure that could be easily enforced by some tool... 

I could see some advantage from static in future compiler versions 
though from better optimization, but it's quite remote.

I agree with your point that it's not effective to spend human time
to generate such patches.

-Andi
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[2.6 patch] make sched_feat_{names,open} static, Adrian Bunk, (Mon May 5, 2:29 pm)
Re: [rfc] the kernel workflow & trivial "global -> st..., Andi Kleen, (Mon May 5, 8:21 pm)
speck-geostationary