linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Bernd Schubert <bernd.schubert@fastmail.fm>
Cc: sfr@canb.auug.org.au, dhowells@redhat.com, nfsv4@linux-nfs.org,
	linux-kernel@vger.kernel.org, steved@redhat.com,
	linux-fsdevel@vger.kernel.org, rwheeler@redhat.com
Subject: Re: Pull request for FS-Cache, including NFS patches
Date: Thu, 18 Dec 2008 15:26:16 -0800	[thread overview]
Message-ID: <20081218152616.a24c013f.akpm@linux-foundation.org> (raw)
In-Reply-To: <200812190007.34581.bernd.schubert@fastmail.fm>

On Fri, 19 Dec 2008 00:07:33 +0100
Bernd Schubert <bernd.schubert@fastmail.fm> wrote:

> On Thursday 18 December 2008, Andrew Morton wrote:
> > On Thu, 18 Dec 2008 09:24:20 -0500
> >
> > Christoph Hellwig <hch@infradead.org> wrote:
> > > On Thu, Dec 18, 2008 at 10:44:18PM +1100, Stephen Rothwell wrote:
> > > > Added from today.
> > > >
> > > > Usual spiel: all patches in that branch must have been
> > > > 	posted to a relevant mailing list
> > > > 	reviewed
> > > > 	unit tested
> > > > 	destined for the next merge window (or the current release)
> > > > *before* they are included.
> > >
> > > I don't think we want fscache for .29 yet.  I'd rather let the
> > > credential code settle for one release, and have more time for actually
> > > reviewing it properly and have it 100% ready for .30.
> >
> > I don't believe that it has yet been convincingly demonstrated that we
> > want to merge it at all.
> >
> > It's a huuuuuuuuge lump of new code, so it really needs to provide
> > decent value.  Can we revisit this?  Yet again?  What do we get from
> > all this?
> >
> 
> I really don't understand why fs-cache is always rejected.

It's never been rejected.  For a long time it has been in a state where
we're looking for the data which would allow us to agree that its
benefits are worth its costs.  AFAIK that has never really been
convincingly demonstrated.  Nor has the converse case been
demonstrated, so it floats in limbo.

> Actually it is the 
> perfect solution for NFS booted systems - you have a big cluster of nodes and 
> in order to minimize administration overhead the nodes are booted over NFS 
> from one common chroot. With unionfs (preferred solution here is unionfs-fuse) 
> one then maintains files required to be differently by different clients.
> 
> Caching files on the local disk minimized the network access and boosts the 
> performance, so at least for this usage example fs-cache would be great.
> (Actually I have been thinking about to implement a caching branch into 
> unionfs-fuse, but if the kernel can do it on its own, it is also fine.)

Have you actually used cachefs/cachefiles for this?

> In the past David already posted many benchmarks and just a few weeks ago 
> again:
> 
> http://lkml.indiana.edu/hypermail/linux/kernel/0811.3/00584.html

OK, benchmarks are good.   But look:

 303 files changed, 21049 insertions(+), 3726 deletions(-)

it's an enormous hunk of code.  That will be in the kernel for ever and
ever, needing maintenance, adding additional burden to our effort to
evolve the kernel, etc.


Are any distros pushing for this?  Or shipping it?  If so, are they
able to weigh in and help us with this quite difficult decision?

  reply	other threads:[~2008-12-18 23:26 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-18  0:30 Pull request for FS-Cache, including NFS patches David Howells
2008-12-18 11:44 ` Stephen Rothwell
2008-12-18 14:24   ` Christoph Hellwig
2008-12-18 20:36     ` Andrew Morton
2008-12-18 23:07       ` Bernd Schubert
2008-12-18 23:26         ` Andrew Morton [this message]
2008-12-19  0:05           ` Stephen Rothwell
2008-12-29  3:45             ` Stephen Rothwell
2008-12-29  4:01               ` Andrew Morton
2008-12-29 14:30                 ` Trond Myklebust
2008-12-29 14:54                   ` Ric Wheeler
2008-12-29 23:05                   ` Muntz, Daniel
2008-12-30 18:44                     ` Trond Myklebust
2008-12-30 22:15                       ` Muntz, Daniel
2008-12-30 22:36                         ` Trond Myklebust
2008-12-30 23:00                           ` Muntz, Daniel
2008-12-30 23:17                             ` Trond Myklebust
2009-01-01  4:11                               ` Muntz, Daniel
2009-01-01  8:09                                 ` Arjan van de Ven
2009-01-01 18:40                                   ` Kyle Moffett
2008-12-31 11:15                             ` David Howells
2008-12-31  9:49                         ` Arjan van de Ven
2008-12-29  4:07               ` Andrew Morton
2008-12-29  5:26                 ` Stephen Rothwell
2008-12-29 15:01               ` David Howells
2008-12-29 15:04               ` David Howells
2008-12-29 14:26             ` David Howells
2008-12-19  2:27     ` David Howells
2008-12-19  2:44       ` Andrew Morton
2008-12-19  3:10         ` Ric Wheeler
2008-12-19 12:33         ` Trond Myklebust
2008-12-19 16:48           ` Gabor Gombas
2008-12-19 13:32         ` David Howells
2008-12-19  3:45       ` Muntz, Daniel
2008-12-19  4:09         ` J. Bruce Fields
2008-12-19 13:20         ` David Howells
2008-12-19 18:08           ` Muntz, Daniel
2008-12-19 18:24             ` David Howells
2008-12-19 19:53               ` Bryan Henderson
2008-12-20  1:20                 ` David Howells
2008-12-20  6:05                   ` Muntz, Daniel
2008-12-19 13:22         ` David Howells
2008-12-19 13:03       ` David Howells
  -- strict thread matches above, loose matches on Subject: below --
2008-12-20  6:06 Muntz, Daniel

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=20081218152616.a24c013f.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=bernd.schubert@fastmail.fm \
    --cc=dhowells@redhat.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nfsv4@linux-nfs.org \
    --cc=rwheeler@redhat.com \
    --cc=sfr@canb.auug.org.au \
    --cc=steved@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).