* Re: [PATCH 01/32] init.h: remove __cpuinit sections from the kernel
[not found] ` <1372102237-8757-2-git-send-email-paul.gortmaker@windriver.com>
@ 2013-06-24 19:51 ` Joe Perches
2013-06-24 22:10 ` Joe Perches
0 siblings, 1 reply; 2+ messages in thread
From: Joe Perches @ 2013-06-24 19:51 UTC (permalink / raw)
To: Paul Gortmaker; +Cc: linux-kernel, linux-embedded
On Mon, 2013-06-24 at 15:30 -0400, Paul Gortmaker wrote:
> The __cpuinit type of throwaway sections might have made sense
> some time ago when RAM was more constrained, but now the savings
> do not offset the cost and complications. For example, the fix in
> commit 5e427ec2d0 ("x86: Fix bit corruption at CPU resume time")
> is a good example of the nasty type of bugs that can be created
> with improper use of the various __init prefixes.
Hi Paul.
Could you please post the sizes of a generic kernel build
with and without this?
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH 01/32] init.h: remove __cpuinit sections from the kernel
2013-06-24 19:51 ` [PATCH 01/32] init.h: remove __cpuinit sections from the kernel Joe Perches
@ 2013-06-24 22:10 ` Joe Perches
0 siblings, 0 replies; 2+ messages in thread
From: Joe Perches @ 2013-06-24 22:10 UTC (permalink / raw)
To: Paul Gortmaker; +Cc: linux-kernel, linux-embedded
On Mon, 2013-06-24 at 12:51 -0700, Joe Perches wrote:
> On Mon, 2013-06-24 at 15:30 -0400, Paul Gortmaker wrote:
> > The __cpuinit type of throwaway sections might have made sense
> > some time ago when RAM was more constrained, but now the savings
> > do not offset the cost and complications. For example, the fix in
> > commit 5e427ec2d0 ("x86: Fix bit corruption at CPU resume time")
> > is a good example of the nasty type of bugs that can be created
> > with improper use of the various __init prefixes.
>
> Hi Paul.
>
> Could you please post the sizes of a generic kernel build
> with and without this?
Nevermind...
I just checked using readelf -S.
Seems sensible to me.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-06-24 22:10 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1372102237-8757-1-git-send-email-paul.gortmaker@windriver.com>
[not found] ` <1372102237-8757-2-git-send-email-paul.gortmaker@windriver.com>
2013-06-24 19:51 ` [PATCH 01/32] init.h: remove __cpuinit sections from the kernel Joe Perches
2013-06-24 22:10 ` Joe Perches
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).