public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jason Baron <jbaron@redhat.com>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: linux-kernel@vger.kernel.org, Ingo Molnar <mingo@elte.hu>,
	Andrew Morton <akpm@linux-foundation.org>,
	"H. Peter Anvin" <hpa@zytor.com>
Subject: Re: [PATCH 5/7] x86/jump labels: Use etiher 5 byte or 2 byte jumps
Date: Mon, 12 Mar 2012 14:03:51 -0400	[thread overview]
Message-ID: <20120312180350.GC2607@redhat.com> (raw)
In-Reply-To: <1331573227.25686.650.camel@gandalf.stny.rr.com>

[-- Attachment #1: Type: text/plain, Size: 1103 bytes --]

On Mon, Mar 12, 2012 at 01:27:07PM -0400, Steven Rostedt wrote:
> Here's my numbers from the latest changes. They are not as big, perhaps
> I had debug enabled too. I took a debian 3.0 config, enabled jump labels
> and some tracing, did a make localyesconfig (something broke it as the
> config still had modules that I had to manual convert to =y). Here's the
> results:
> 
> 
> gcc 4.6.0
> 
>    text	   data	    bss	    dec	    hex	filename
> 14355064	1810272	5722112	21887448	14df9d8	vmlinux
> 14354490	1810400	5722112	21887002	14df81a	vmlinux-patched
> 
> It's only a 574 byte savings. That's not much. I wish I had the config I
> used the first time. I may have had debug on that gave it a bigger
> impact.
> 
> Attached is the config:
> 
> -- Steve
> 

At least you got a decrease! Using your .config I got the same 574 byte
savings:

14310698        1810160 5722112 21842970        14d4c1a vmlinux-orig
14310124        1810288 5722112 21842524        14d4a5c vmlinux-new

I'm attaching my .config which is basically a defconfig with a few extra
configs for my h/w + kvm + ext4.

Thanks,

-Jason

[-- Attachment #2: jasons-config.bz2 --]
[-- Type: application/x-bzip2, Size: 16491 bytes --]

  reply	other threads:[~2012-03-12 18:03 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-08 22:17 [PATCH 0/7] [GIT PULL] jump-labels: Implement 2 and 5 byte jumps Steven Rostedt
2012-03-08 22:17 ` [PATCH 1/7] x86/jump-label: Use best default nops for inital jump label calls Steven Rostedt
2012-03-08 22:17 ` [PATCH 2/7] x86/jump-label: Do not bother updating nops if they are correct Steven Rostedt
2012-03-08 22:17 ` [PATCH 3/7] x86/jump-label: Add safety checks to jump label conversions Steven Rostedt
2012-03-08 22:17 ` [PATCH 4/7] jump labels: Add infrastructure to update jump labels at compile time Steven Rostedt
2012-03-08 22:17 ` [PATCH 5/7] x86/jump labels: Use etiher 5 byte or 2 byte jumps Steven Rostedt
2012-03-12 16:17   ` Jason Baron
2012-03-12 16:29     ` Steven Rostedt
2012-03-12 17:27     ` Steven Rostedt
2012-03-12 18:03       ` Jason Baron [this message]
2012-03-08 22:17 ` [PATCH 6/7] x86/jump lables: Show where and what was wrong on errors Steven Rostedt
2012-03-08 22:17 ` [PATCH 7/7] x86/jump labels: Handle initialization of enabled nops Steven Rostedt

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=20120312180350.GC2607@redhat.com \
    --to=jbaron@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=rostedt@goodmis.org \
    /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