All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@infradead.org>
To: Mikulas Patocka <mpatocka@redhat.com>,
	dm-devel@redhat.com, Christoph Hellwig <hch@infradead.org>
Subject: Re: New dm-bufio with shrinker API
Date: Mon, 5 Sep 2011 11:07:15 -0400	[thread overview]
Message-ID: <20110905150714.GA2722@infradead.org> (raw)
In-Reply-To: <20110905144913.GB22386@ubuntu>

On Mon, Sep 05, 2011 at 03:49:14PM +0100, Joe Thornber wrote:
> I changed the test suite to reset the peak_allocated parameter before
> each test and record it at the end of each test.  It's very hard to
> say what is right and wrong when talking about cache sizes, since you
> always have to qualify anything by saying 'for this particular load'.
> However, I think bufio could be more aggressive about recycling cache
> entries.  With the old block manager the test suite ran nicely with
> less than 256k, from memory I think I started seeing slow down around
> 128k.  With bufio I'm seeing consistently larger cache sizes for the
> same performance.

IS there any reason you'll need a fixed size?  This is fairly similar in
concept to the XFS buffercache, which does perfectly well by allocation
memory as needed, and letting the shrinker reclaim buffers when under
memory pressure.

  reply	other threads:[~2011-09-05 15:07 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-02 21:34 New dm-bufio with shrinker API Mikulas Patocka
2011-09-05  9:04 ` Joe Thornber
2011-09-05 14:49   ` Joe Thornber
2011-09-05 15:07     ` Christoph Hellwig [this message]
2011-09-06  8:50       ` Joe Thornber
2011-09-06  9:53       ` Joe Thornber
2011-09-06 15:57       ` Mikulas Patocka
2011-09-06 16:08         ` Christoph Hellwig
2011-09-07 18:47           ` Mikulas Patocka
2011-09-05 16:01     ` Mikulas Patocka
2011-09-06  9:33       ` Joe Thornber
2011-09-06 16:08         ` Mikulas Patocka
2011-09-07  9:46           ` Joe Thornber
2011-09-07 18:39             ` Mikulas Patocka

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=20110905150714.GA2722@infradead.org \
    --to=hch@infradead.org \
    --cc=dm-devel@redhat.com \
    --cc=mpatocka@redhat.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.