All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Zarochentsev <zam@namesys.com>
To: Hans Reiser <reiser@namesys.com>, Elena Gryaznova <grev@namesys.com>
Cc: reiserfs-list@namesys.com, Jeff Mahoney <jeffm@suse.com>
Subject: Re: [PATCH] reiserfs: on-demand bitmap loading (testing only)
Date: Fri, 8 Jul 2005 09:35:28 +0400	[thread overview]
Message-ID: <200507080935.28944.zam@namesys.com> (raw)
In-Reply-To: <42CC5F1E.6060505@namesys.com>

On Thursday 07 July 2005 02:45, Hans Reiser wrote:
> Jeff, are you sure that you need this code to exist?  Here are the
> problems I see:
>
>     for the average case, it is suboptimal.  The seeks to the bitmaps
> are far more expensive than the averaged cost of keeping them in ram.

It would be good to see the benchmark results which your opinion is based on.

I know that on-demand bitmap loading in reiser4 was under suspicion 
several times but I don't remember any benchmark results which showed a
slowdown regarding that.  

>
>     for 16TB filesystems, they will have plenty of budget for ram
>
>     it complicates code if it has to worry about such things as not
> enough clean memory for holding bitmaps, etc.
>
>     It is more appropriate to write this kind of code for the
> development branch which is V4.  This kind of code is likely to have
> hard to test and hit bugs.
>

Lena, may you check how the bitmap on-demand loading patch affects, say,
mongo results for reiserfs?

Thanks, 
Alex.


  parent reply	other threads:[~2005-07-08  5:35 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-07-06 21:22 [PATCH] reiserfs: on-demand bitmap loading (testing only) Jeff Mahoney
2005-07-06 22:45 ` Hans Reiser
2005-07-07 16:04   ` Jeff Mahoney
2005-07-07 17:59     ` studdugie
2005-07-07 22:37       ` Pierre Etchemaïté
2005-07-07 23:15         ` David Masover
2005-07-08  4:52           ` Jeffrey Mahoney
2005-07-08  5:52             ` Stefan Traby
2005-07-08 16:55             ` David Masover
2005-07-08 21:41           ` Jeff Mahoney
2005-07-09  7:46             ` Hans Reiser
2005-07-08  5:35   ` Alexander Zarochentsev [this message]
2005-07-08  6:04     ` Stefan Traby

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=200507080935.28944.zam@namesys.com \
    --to=zam@namesys.com \
    --cc=grev@namesys.com \
    --cc=jeffm@suse.com \
    --cc=reiser@namesys.com \
    --cc=reiserfs-list@namesys.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 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.