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 ;)
next prev parent 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 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome