From: Dave Chinner <david@fromorbit.com>
To: Cyrill Gorcunov <gorcunov@gmail.com>
Cc: Al Viro <viro@ZenIV.linux.org.uk>,
LINUXFS-ML <linux-fsdevel@vger.kernel.org>,
Andrew Morton <akpm@linux-foundation.org>,
Christoph Hellwig <hch@infradead.org>,
James Bottomley <jbottomley@parallels.com>,
xemul@openvz.org
Subject: Re: [patch 0/3] A few patches for dcache
Date: Fri, 29 Jul 2011 16:59:51 +1000 [thread overview]
Message-ID: <20110729065951.GE5404@dastard> (raw)
In-Reply-To: <20110729055918.GB15883@sun>
On Fri, Jul 29, 2011 at 09:59:18AM +0400, Cyrill Gorcunov wrote:
> On Fri, Jul 29, 2011 at 01:25:03PM +1000, Dave Chinner wrote:
> ...
> >
> > The VFS shrinker code is now already called on a per-sb basis. Each
> > sb has it's own shrinker context that deals with dentries, inodes
> > and anything a filesystem wants to have shrunk in the call. That
> > solves the original issue I had with your "limit the dentry cache
> > size" patch series in that it didn't shrink or limit the other VFS
> > caches that were the ones that were really consuming all your
> > memory...
>
> Thanks for comments, Dave! Still the read only lock without
> increasing sequence number might be useful, no? (patch 1)
I'll defer to Al on that one - the intricacies of the rename locking
are way over my head.
FWIW, the problems with the per-sb dcache LRU lock seem to be gone -
it's not causing my test machines to fall over with the current
Linus tree like it was during 2.6.39-rc and 3.0-rc kernels...
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
next prev parent reply other threads:[~2011-07-29 6:59 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-28 13:12 [patch 0/3] A few patches for dcache Cyrill Gorcunov
2011-07-28 13:12 ` [patch 1/3] vfs, dcache: Introduce lighten r/o rename_lock lockers Cyrill Gorcunov
2011-07-28 13:12 ` [patch 2/3] vfs, dcache: Factor out rename_lock locking Cyrill Gorcunov
2011-07-28 13:12 ` [patch 3/3] vfs: Make the rename_lock per-sb Cyrill Gorcunov
2011-07-29 3:25 ` [patch 0/3] A few patches for dcache Dave Chinner
2011-07-29 5:59 ` Cyrill Gorcunov
2011-07-29 6:59 ` Dave Chinner [this message]
2011-07-29 7:01 ` Cyrill Gorcunov
2011-07-29 7:24 ` Al Viro
2011-07-29 7:50 ` Cyrill Gorcunov
2011-08-15 7:42 ` Cyrill Gorcunov
2011-08-24 6:31 ` Pavel Emelyanov
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=20110729065951.GE5404@dastard \
--to=david@fromorbit.com \
--cc=akpm@linux-foundation.org \
--cc=gorcunov@gmail.com \
--cc=hch@infradead.org \
--cc=jbottomley@parallels.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=viro@ZenIV.linux.org.uk \
--cc=xemul@openvz.org \
/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.