All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nick Piggin <nickpiggin@yahoo.com.au>
To: Andrew Morton <akpm@osdl.org>
Cc: "Chen, Kenneth W" <kenneth.w.chen@intel.com>,
	linux-kernel@vger.kernel.org, Ingo Molnar <mingo@elte.hu>
Subject: Re: re-inline sched functions
Date: Sat, 12 Mar 2005 00:08:49 +1100	[thread overview]
Message-ID: <42319861.7000805@yahoo.com.au> (raw)
In-Reply-To: <20050310163056.64878c24.akpm@osdl.org>

Andrew Morton wrote:

>"Chen, Kenneth W" <kenneth.w.chen@intel.com> wrote:
>
>>This could be part of the unknown 2% performance regression with
>>db transaction processing benchmark.
>>
>>The four functions in the following patch use to be inline.  They
>>are un-inlined since 2.6.7.
>>
>>We measured that by re-inline them back on 2.6.9, it improves performance
>>for db transaction processing benchmark, +0.2% (on real hardware :-)
>>
>>

Can you also inline requeue_task? No performance gain expected, but
it is just a simple wrapper around a list function.

>>The cost is certainly larger kernel size, cost 928 bytes on x86, and
>>2728 bytes on ia64.  But certainly worth the money for enterprise
>>customer since they improve performance on enterprise workload.
>>
>
>Less that 1k on x86 versus >2k on ia64.  No wonder those things have such
>big caches ;)
>
>
>>...
>>Possible we can introduce them back?
>>
>
>OK by me.
>
>

What happens if you leave task_timeslice out of line? It isn't exactly
huge, but it is called from a handful of places.



  reply	other threads:[~2005-03-11  2:10 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-11  0:24 re-inline sched functions Chen, Kenneth W
2005-03-11  0:30 ` Andrew Morton
2005-03-11 13:08   ` Nick Piggin [this message]
2005-03-11  9:31 ` Ingo Molnar
2005-03-11 18:39   ` Chen, Kenneth W
  -- strict thread matches above, loose matches on Subject: below --
2005-03-24 21:16 Chen, Kenneth W
2005-03-24 22:22 ` Ingo Molnar

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=42319861.7000805@yahoo.com.au \
    --to=nickpiggin@yahoo.com.au \
    --cc=akpm@osdl.org \
    --cc=kenneth.w.chen@intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.