Re: RFC: remove __read_mostly

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Adrian Bunk <bunk@...>
Cc: Andi Kleen <andi@...>, <linux-kernel@...>, <linux-arch@...>
Date: Friday, December 14, 2007 - 12:16 pm

On Thursday 13 December 2007, Adrian Bunk wrote:

Just to make sure everyone interprets this correctly:

The file size in the first example suggests a 8552 byte
(0.02%) size improvement for removing __read_mostly.

The size output shows a -3935 byte (0.01%) size penalty
instead, much smaller because data that was moved out to
the .data.read_mostly section from .bss now takes space
in the binary but won't consume more RAM.

Since 'size' does not take any sections except text, data and
bss into account, its output is more often than not also
misleading, but at least it shows that the footprint is likely
to get larger without __read_mostly rather than smaller.

	Arnd <><
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
RFC: remove __read_mostly, Adrian Bunk, (Thu Dec 13, 6:20 pm)
Re: RFC: remove __read_mostly, Matt Mackall, (Fri Dec 14, 11:24 am)
Re: RFC: remove __read_mostly, Eric Dumazet, (Fri Dec 14, 11:38 am)
Re: RFC: remove __read_mostly, Matt Mackall, (Fri Dec 14, 11:42 am)
Re: RFC: remove __read_mostly, Kyle McMartin, (Thu Dec 13, 7:54 pm)
Re: RFC: remove __read_mostly, Andi Kleen, (Thu Dec 13, 8:33 pm)
Re: RFC: remove __read_mostly, Eric Dumazet, (Thu Dec 13, 6:48 pm)
Re: RFC: remove __read_mostly, Adrian Bunk, (Thu Dec 13, 7:00 pm)
Re: RFC: remove __read_mostly, David Miller, (Thu Dec 13, 6:32 pm)
Re: RFC: remove __read_mostly, Harvey Harrison, (Thu Dec 13, 6:44 pm)
Re: RFC: remove __read_mostly, Andi Kleen, (Thu Dec 13, 7:06 pm)
Re: RFC: remove __read_mostly, Andi Kleen, (Thu Dec 13, 6:29 pm)
Re: RFC: remove __read_mostly, Adrian Bunk, (Thu Dec 13, 6:41 pm)
Re: RFC: remove __read_mostly, Arnd Bergmann, (Fri Dec 14, 12:16 pm)
Re: RFC: remove __read_mostly, Eric Dumazet, (Fri Dec 14, 12:31 pm)