All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@osdl.org>
To: viro@parcelfarce.linux.theplanet.co.uk
Cc: pmarques@grupopie.com, mpm@selenic.com,
	linux-kernel@vger.kernel.org, bcasavan@sgi.com
Subject: Re: [PATCH] kallsyms data size reduction / lookup speedup
Date: Thu, 26 Aug 2004 02:59:04 -0700	[thread overview]
Message-ID: <20040826025904.02bf4c0e.akpm@osdl.org> (raw)
In-Reply-To: <20040825234345.GN21964@parcelfarce.linux.theplanet.co.uk>

viro@parcelfarce.linux.theplanet.co.uk wrote:
>
> On Thu, Aug 26, 2004 at 12:40:30AM +0100, Paulo Marques wrote:
>  > That is why I kept a big *If* in that sentence. I'm quite new to all
>  > this, and I'm still reading a lot of source code.
>  > 
>  > If the culprit is in fact seq_file, and seq_file can be improved in a
>  > way that works for everyone (not only kallsyms), then I also agree
>  > that is is the way to go. But hunting this down might prove that the
>  > problem is somewhere else. It is just too soon to draw conclusions.
> 
>  readprofile(1) ought to narrow it down with that kind of timing difference...



c014696c do_anonymous_page                             4   0.0133
c0133550 kallsyms_expand_symbol                       17   0.1545
c026b78f __copy_user_intel                            31   0.2039
c026a5ec vsnprintf                                    39   0.0283
c026a318 number                                       40   0.0552
c011d102 write_profile                                79   0.7182
c0131f39 is_exported                                3805  17.0628
c0104028 default_idle                               4254  86.8163
00000000 total                                      8325   0.0025

It's all in the O(n) is_exported().  Rusty's fault ;)

  reply	other threads:[~2004-08-26 10:07 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-08-25  4:04 [PATCH] kallsyms data size reduction / lookup speedup pmarques
2004-08-25 17:39 ` Matt Mackall
2004-08-25 18:46   ` Paulo Marques
2004-08-25 18:58     ` Matt Mackall
2004-08-25 19:09       ` Paulo Marques
2004-08-25 19:29         ` viro
2004-08-25 23:40           ` Paulo Marques
2004-08-25 23:43             ` viro
2004-08-26  9:59               ` Andrew Morton [this message]
2004-08-26 10:26                 ` Paulo Marques
2004-08-30 18:21                   ` Sam Ravnborg
2004-08-30 18:38                     ` Paulo Marques
2004-08-25 21:12         ` Matt Mackall
2004-08-25 23:50           ` Paulo Marques
2004-08-25 20:51 ` Sam Ravnborg
2004-08-25 21:25   ` Randy.Dunlap
2004-08-26  0:27   ` Paulo Marques
2004-08-26 11:01     ` Paulo Marques

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=20040826025904.02bf4c0e.akpm@osdl.org \
    --to=akpm@osdl.org \
    --cc=bcasavan@sgi.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mpm@selenic.com \
    --cc=pmarques@grupopie.com \
    --cc=viro@parcelfarce.linux.theplanet.co.uk \
    /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.