public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andreas Dilger <adilger@turbolabs.com>
To: Beau Kuiper <kuib-kl@ljbc.wa.edu.au>
Cc: Chris Mason <mason@suse.com>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Significant performace improvements on reiserfs systems, kupdated bugfixes
Date: Thu, 20 Sep 2001 10:15:44 -0600	[thread overview]
Message-ID: <20010920101544.A14526@turbolinux.com> (raw)
In-Reply-To: <391950000.1000988162@tiny> <Pine.LNX.4.30.0109202305350.19677-100000@gamma.student.ljbc>
In-Reply-To: <Pine.LNX.4.30.0109202305350.19677-100000@gamma.student.ljbc>

On Sep 20, 2001  23:20 +0800, Beau Kuiper wrote:
> Patch 3 doesn't improve performace much (even in theory the number of
> dirty buffers being wrongly flushed is pretty low)

Actually, it _may_ even make performance worse (hard to say).  Consider
the case where the "young" dirty buffers are in the same area of the
disk as the "old" dirty buffers.  Once you are forced to write the "old"
buffers, you pretty much get to write the new buffers for free (low seek
overhead).  They _could_ be merged in the elevator code.

Sadly, it is hard to tell whether this is possible or not, because the
code to do these things live in different places.  Maybe we could have
an "optimistic" elevator merge, which only added "young" buffers if
they merged with other old buffers.

Cheers, Andreas
--
Andreas Dilger  \ "If a man ate a pound of pasta and a pound of antipasto,
                 \  would they cancel out, leaving him still hungry?"
http://www-mddsp.enel.ucalgary.ca/People/adilger/               -- Dogbert


  reply	other threads:[~2001-09-20 16:16 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-09-20  7:12 [PATCH] Significant performace improvements on reiserfs systems, kupdated bugfixes Beau Kuiper
2001-09-20 12:16 ` Chris Mason
2001-09-20 15:20   ` Beau Kuiper
2001-09-20 16:15     ` Andreas Dilger [this message]
2001-09-20 16:22       ` Beau Kuiper
2001-09-21 20:12         ` Lehmann 
2001-09-21 20:57           ` Rik van Riel
2001-09-21 22:50             ` Lehmann 
2001-09-21 13:26 ` Matthias Andree
2001-09-22 18:38   ` Beau Kuiper
2001-09-22 19:32     ` Matthias Andree
2001-09-23 15:24       ` Chris Mason
2001-09-24 14:37   ` Stephen C. Tweedie
  -- strict thread matches above, loose matches on Subject: below --
2001-09-20 17:23 Chris Mason

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=20010920101544.A14526@turbolinux.com \
    --to=adilger@turbolabs.com \
    --cc=kuib-kl@ljbc.wa.edu.au \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mason@suse.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