Date: Mon, 2 Jan 2006 10:21:37 -0500
From: Kurt Wall <kwall@kurtwerks.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: linux-kernel@vger.kernel.org
Subject: Re: Arjan's noinline Patch
Message-ID: <20060102152137.GI5213@kurtwerks.com>
Mail-Followup-To: Ingo Molnar <mingo@elte.hu>, linux-kernel@vger.kernel.org
References: <20060101155710.GA5213@kurtwerks.com> <20060102110404.GA10996@elte.hu>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20060102110404.GA10996@elte.hu>
User-Agent: Mutt/1.4.2.1i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1507
Lines: 37

On Mon, Jan 02, 2006 at 12:04:04PM +0100, Ingo Molnar took 0 lines to write:
> 
> * Kurt Wall <kwallnator@gmail.com> wrote:
> 
> > After applying Arjan's noline patch (http://www.fenrus.org/noinlin), I 
> > see almost a 10% reduction in the size of .text (against 2.6.15-rc6) 
> > with no apparent errors and no reduction in functionality:
> 
> just to make sure: this was with -Os _also_ turned on, right? So what 
> you measured was the effect of Arjan's patch plus -Os, combined, 
> correct?

Yes. And it was a faulty approach. I've posted new results with the 
effects of Arjan's patch. CONFIG_CC_OPTIMIZE_FOR_SIZE was enabled for
all kernels.

> if yes you should measure the two effects in isolation, like the vmlinux 
> numbers i posted. Every patch applied and every change to .config 
> options must be measured separately, to get valid results. This doesnt 
> invalidate your raw vmlinux measurements - you simply need to add a 
> "Arjan's patch applied but no -Os turned on" number - but it does 
> invalidate your conclusion quoted above.

Quite so. I'm working on the "no -Os" case now.

Kurt
-- 
Alden's Laws:
	(1) Giving away baby clothes and furniture is the major cause
	    of pregnancy.
	(2) Always be backlit.
	(3) Sit down whenever possible.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/