From: Daniel Phillips <phillips@arcor.de>
To: Rik van Riel <riel@conectiva.com.br>
Cc: Andrew Morton <akpm@zip.com.au>,
Linus Torvalds <torvalds@transmeta.com>,
lkml <linux-kernel@vger.kernel.org>
Subject: Re: [patch 1/13] minimal rmap
Date: Wed, 17 Jul 2002 21:55:41 +0200 [thread overview]
Message-ID: <E17Uuti-0004PT-00@starship> (raw)
In-Reply-To: <Pine.LNX.4.44L.0207171639480.12241-100000@imladris.surriel.com>
On Wednesday 17 July 2002 21:41, Rik van Riel wrote:
> On Wed, 17 Jul 2002, Daniel Phillips wrote:
> > On Wednesday 17 July 2002 21:31, Rik van Riel wrote:
> > > On Wed, 17 Jul 2002, Daniel Phillips wrote:
> > > > On Wednesday 17 July 2002 07:29, Andrew Morton wrote:
> > > > > 11: The nightly updatedb run is still evicting everything.
> > > >
> > > > That is not a problem with rmap per se, it's a result of not properly
> > > > handling streaming IO.
> > >
> > > Umm, updatedb isn't exactly streaming...
> >
> > You're right, it's not exactly, it's hitting every directory entry on
> ^^^^^^^^^^^^^^^
> > the system, hopefully just once. Let's not call it streaming, let's
> > call it... err... use-once? ;-)
>
> Nope. If it hits every directory entry once, it'll hit every
> page with directory or inode information multiple times, causing
> that page to enter the active list and push out process pages.
>
> This is exactly what we want to prevent.
Yes, it could get challenging, but it's very desirable to do the
optimization somehow. It's even more desirable to stay focussed on
the immediate issues, imho.
Food for thought: readdir actually picks up the entries 4K-worth at a
time, however that's randomly aligned with respect to directory block
boundaries, meaning that most blocks get two hits, which could well be
the problem.
Inode table blocks present a more difficult problem. It's really hard
to know when to evict them, and just arbitrarily holding them to a
certain percentage of memory isn't nice either.
I think Andrew also mentioned he sees a lot of buffers sitting around
in the morning. His strip-buffers-immediately hack would kill that
one dead.
--
Daniel
next prev parent reply other threads:[~2002-07-17 19:51 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-07-17 5:29 [patch 1/13] minimal rmap Andrew Morton
2002-07-17 8:24 ` Russell King
2002-07-17 12:10 ` Rik van Riel
2002-07-17 12:21 ` Rik van Riel
2002-07-17 17:57 ` Daniel Phillips
2002-07-17 18:11 ` Robert Love
2002-07-17 18:26 ` Daniel Phillips
2002-07-17 20:37 ` Rik van Riel
2002-07-17 20:42 ` Robert Love
2002-07-17 20:46 ` Daniel Phillips
2002-07-17 20:48 ` Robert Love
2002-07-17 19:31 ` Rik van Riel
2002-07-17 19:38 ` Daniel Phillips
2002-07-17 19:41 ` Rik van Riel
2002-07-17 19:55 ` Daniel Phillips [this message]
2002-07-17 20:04 ` Rik van Riel
2002-07-17 20:13 ` Daniel Phillips
2002-07-17 20:25 ` Rik van Riel
2002-07-17 20:25 ` Rik van Riel
2002-07-17 20:36 ` Daniel Phillips
2002-07-17 20:39 ` Rik van Riel
2002-07-17 20:50 ` Dave Jones
2002-07-17 20:53 ` Robert Love
2002-07-22 2:40 ` Andrew Morton
2002-07-17 19:42 ` Robert Love
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=E17Uuti-0004PT-00@starship \
--to=phillips@arcor.de \
--cc=akpm@zip.com.au \
--cc=linux-kernel@vger.kernel.org \
--cc=riel@conectiva.com.br \
--cc=torvalds@transmeta.com \
/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