public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Kurt Wall <kwall@kurtwerks.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: linux-kernel@vger.kernel.org
Subject: Re: Arjan's noinline Patch
Date: Mon, 2 Jan 2006 10:21:37 -0500	[thread overview]
Message-ID: <20060102152137.GI5213@kurtwerks.com> (raw)
In-Reply-To: <20060102110404.GA10996@elte.hu>

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.

      reply	other threads:[~2006-01-02 15:18 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-01 15:57 Arjan's noinline Patch Kurt Wall
2006-01-01 16:12 ` Arjan van de Ven
2006-01-02  0:54   ` Kurt Wall
2006-01-02  3:43 ` Kurt Wall
2006-01-02 10:03   ` Matan Peled
2006-01-02 15:14     ` Kurt Wall
2006-01-02 16:24       ` Matan Peled
2006-01-02 16:30         ` Matan Peled
2006-01-03  1:20         ` Kurt Wall
2006-01-02 16:26       ` Kurt Wall
2006-01-02 18:51     ` Jan Engelhardt
2006-01-02 20:27       ` Matan Peled
2006-01-03 19:49         ` Jan Engelhardt
2006-01-02 22:06       ` Adrian Bunk
2006-01-02 11:04 ` Ingo Molnar
2006-01-02 15:21   ` Kurt Wall [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060102152137.GI5213@kurtwerks.com \
    --to=kwall@kurtwerks.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox