linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Daniel Phillips <phillips@bonn-fries.net>
To: Alexander Viro <viro@math.psu.edu>
Cc: Jakob ?stergaard <jakob@unthought.net>,
	Alex Bligh - linux-kernel <linux-kernel@alex.org.uk>,
	John Levon <moz@compsoc.man.ac.uk>,
	linux-kernel@vger.kernel.org, Tim Jansen <tim@tjansen.de>
Subject: Re: PROPOSAL: dot-proc interface [was: /proc stuff]
Date: Mon, 5 Nov 2001 01:10:19 +0100	[thread overview]
Message-ID: <20011105000933Z17055-18972+28@humbolt.nl.linux.org> (raw)
In-Reply-To: <Pine.GSO.4.21.0111041841480.21449-100000@weyl.math.psu.edu>
In-Reply-To: <Pine.GSO.4.21.0111041841480.21449-100000@weyl.math.psu.edu>

On November 5, 2001 12:42 am, Alexander Viro wrote:
> On Sun, 4 Nov 2001, Daniel Phillips wrote:
> 
> > Doing 'top -d .1' eats 18% of a 1GHz cpu, which is abominable.  A kernel
> > profile courtesy of sgi's kernprof shows that scanning pages does not move
> > the needle, whereas sprintf does.  Notice that the biggest chunk of time
> 
> Huh?  Scanning pages is statm_pgd_range().  I'd say that it takes
> seriously more than vsnprintf() - look at your own results.

Yes, true, 2.6 seconds for the statm_pgd_range vs 1.2 for sprintf.  Still, 
sprintf is definitely burning cycles, pretty much the whole 1.2 seconds would 
be recovered with a binary interface.

Now look at the total time we spend in the kernel: 10.4 seconds, 4 times the 
page scanning overhead.  This is really wasteful.

For top does it really matter?  (yes, think slow computer)  What happens when 
proc stabilizes and applications start relying on it heavily as a kernel 
interface?  If we're still turning in this kind of stunningly poor 
performance, it won't be nice.

It's not that it doesn't work, it's just that it isn't the best.

--
Daniel

  reply	other threads:[~2001-11-05  0:10 UTC|newest]

Thread overview: 118+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20011104214229Z17052-23341+37@humbolt.nl.linux.org>
2001-11-04 23:42 ` PROPOSAL: dot-proc interface [was: /proc stuff] Alexander Viro
2001-11-05  0:10   ` Daniel Phillips [this message]
2001-11-04 23:06 Craig Thrall
2001-11-04 23:39 ` Jakob Østergaard
     [not found] <Pine.LNX.4.33.0111041141100.14150-100000@penguin.transmeta.com>
2001-11-04 19:53 ` Daniel Phillips
  -- strict thread matches above, loose matches on Subject: below --
2001-11-01 10:32 [PATCH] 2.5 PROPOSAL: Replacement for current /proc of shit Rusty Russell
2001-11-04 12:30 ` Tim Jansen
2001-11-04 13:36   ` Daniel Kobras
2001-11-04 14:13     ` Tim Jansen
2001-11-04 15:33       ` PROPOSAL: dot-proc interface [was: /proc stuff] Jakob Østergaard
2001-11-04 16:05         ` Gábor Lénárt
2001-11-04 16:31         ` Daniel Phillips
2001-11-04 17:30           ` Jakob Østergaard
2001-11-04 16:45         ` Tim Jansen
2001-11-04 17:28           ` Daniel Phillips
2001-11-04 17:41             ` Jakob Østergaard
2001-11-04 17:54               ` SpaceWalker
2001-11-04 20:45                 ` Albert D. Cahalan
2001-11-04 17:59               ` John Levon
2001-11-04 18:31                 ` Jakob Østergaard
2001-11-04 18:40                   ` Alexander Viro
2001-11-04 19:04                     ` Jakob Østergaard
2001-11-04 19:24                       ` Alex Bligh - linux-kernel
2001-11-04 19:45                         ` Jakob Østergaard
2001-11-04 19:52                           ` Alexander Viro
2001-11-04 20:06                             ` Jakob Østergaard
2001-11-04 22:01                             ` Daniel Phillips
2001-11-04 21:12                           ` Albert D. Cahalan
2001-11-04 21:20                             ` Jakob Østergaard
2001-11-04 21:42                               ` Tim Jansen
2001-11-04 22:13                               ` Albert D. Cahalan
2001-11-05 11:23                                 ` Martin Dalecki
2001-11-05 15:58                                   ` Alexander Viro
2001-11-05 18:30                                     ` Martin Dalecki
2001-11-05 23:00                                       ` Albert D. Cahalan
2001-11-06 13:47                                         ` Martin Dalecki
2001-11-06 17:13                                         ` Gerhard Mack
2001-11-05 16:38                                 ` Stephen Satchell
2001-11-05 18:39                                   ` Martin Dalecki
2001-11-05 18:28                                     ` Ben Greear
2001-11-05 18:40                                       ` Rik van Riel
2001-11-05 21:03                                         ` Tim Jansen
2001-11-05 21:58                                           ` Ben Greear
2001-11-05 22:51                                             ` Tim Jansen
2001-11-05 22:59                                               ` Erik Andersen
2001-11-05 23:35                                                 ` Tim Jansen
2001-11-05 23:41                                                   ` Alexander Viro
2001-11-06 13:49                                                     ` Martin Dalecki
2001-11-06 19:49                                                 ` dank
2001-11-06 22:22                                                   ` Erik Andersen
2001-11-06 22:47                                                     ` dank
2001-11-06 23:11                                                       ` Erik Andersen
2001-11-06 23:39                                                       ` Ricky Beam
2001-11-07 12:45                                                     ` Remco Post
2001-11-07  1:06                                                   ` George Greer
2001-11-05 19:58                                 ` Jonathan Lundell
2001-11-05 21:43                                 ` Stephen Satchell
2001-11-06  5:22                                   ` Ragnar Hojland Espinosa
2001-11-04 21:22                             ` Alex Bligh - linux-kernel
2001-11-05  4:03                           ` Stuart Young
2001-11-05  4:05                             ` Alexander Viro
2001-11-05  4:55                             ` Stuart Young
2001-11-05 16:32                               ` SpaceWalker
2001-11-06  6:46                                 ` Jakob Østergaard
2001-11-04 19:29                       ` Alexander Viro
2001-11-04 19:50                         ` Jakob Østergaard
2001-11-04 20:01                           ` Alexander Viro
2001-11-04 20:09                             ` Jakob Østergaard
2001-11-06  7:23                           ` Kai Henningsen
2001-11-06 14:00                             ` Jakob Østergaard
2001-11-04 18:27               ` Tim Jansen
2001-11-04 18:35                 ` Alexander Viro
2001-11-04 18:39                 ` Jakob Østergaard
2001-11-07  1:20                 ` Pavel Machek
2001-11-07 21:14                   ` Rik van Riel
2000-01-01  0:13                     ` Pavel Machek
2001-11-04 18:20             ` Tim Jansen
2001-11-04 18:30               ` Alexander Viro
2001-11-04 18:52                 ` Jakob Østergaard
2001-11-04 19:18                   ` Daniel Phillips
2001-11-04 21:41                 ` Albert D. Cahalan
2001-11-05 11:06                 ` Martin Dalecki
2001-11-05 10:28                   ` Daniel Phillips
2001-11-05 22:46                     ` Albert D. Cahalan
2001-11-06  0:54                       ` Daniel Phillips
2001-11-06  1:11                       ` Stephen Satchell
2001-11-04 18:46               ` Jakob Østergaard
2001-11-04 19:07             ` Linus Torvalds
2001-11-04 19:20               ` Jakob Østergaard
2001-11-04 19:32                 ` Dave Jones
2001-11-04 19:52                   ` Jakob Østergaard
2001-11-04 20:06                     ` Alexander Viro
2001-11-04 20:11                       ` Jakob Østergaard
2001-11-11 10:06                     ` Kai Henningsen
2001-11-11 19:43                       ` Jakob Østergaard
2001-11-12 13:43                         ` Pascal Schmidt
2001-11-13 12:09                           ` Jakob Østergaard
2001-11-13 14:41                         ` Riley Williams
2001-11-04 22:09               ` Luigi Genoni
2001-11-04 17:48           ` Jakob Østergaard
2001-11-04 18:02             ` John Levon
2001-11-04 18:34             ` Tim Jansen
2001-11-04 18:59               ` Jakob Østergaard
2001-11-04 19:19                 ` Tim Jansen
2001-11-04 19:24                   ` Jakob Østergaard
2001-11-04 19:41                     ` Tim Jansen
2001-11-04 19:55                       ` Jakob Østergaard
2001-11-04 20:13                         ` Tim Jansen
2001-11-04 20:11                           ` Jakob Østergaard
2001-11-04 20:47                             ` Alex Bligh - linux-kernel
2001-11-04 21:02                               ` Jakob Østergaard
2001-11-04 22:53                         ` Stephen Satchell
2001-11-05 11:04         ` zmwillow
2001-11-05 13:41         ` Petr Baudis
2001-11-05 20:49           ` Tim Jansen
2001-11-05 22:01             ` Ben Greear
     [not found]             ` <20011105223413.U11619@pasky.ji.cz>
     [not found]               ` <160rly-1tl3XUC@fmrl05.sul.t-online.com>
2001-11-05 22:07                 ` Petr Baudis
2001-11-06  7:25           ` Jakob Østergaard
2001-11-06  8:21             ` Petr Baudis
2001-11-06  8:34               ` Alexander Viro
2001-11-06 13:43                 ` Jakob Østergaard
2001-11-06 17:01                 ` Petr Baudis

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=20011105000933Z17055-18972+28@humbolt.nl.linux.org \
    --to=phillips@bonn-fries.net \
    --cc=jakob@unthought.net \
    --cc=linux-kernel@alex.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=moz@compsoc.man.ac.uk \
    --cc=tim@tjansen.de \
    --cc=viro@math.psu.edu \
    /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;
as well as URLs for NNTP newsgroup(s).