linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Trond Myklebust <trond.myklebust@fys.uio.no>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>,
	Bernd Schubert <bernd.schubert@fastmail.fm>,
	nfsv4@linux-nfs.org, linux-kernel@vger.kernel.org,
	steved@redhat.com, dhowells@redhat.com,
	linux-next@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	rwheeler@redhat.com
Subject: Re: Pull request for FS-Cache, including NFS patches
Date: Mon, 29 Dec 2008 09:30:56 -0500	[thread overview]
Message-ID: <1230561056.7046.47.camel@heimdal.trondhjem.org> (raw)
In-Reply-To: <20081228200134.426bf203.akpm@linux-foundation.org>

On Sun, 2008-12-28 at 20:01 -0800, Andrew Morton wrote:
> On Mon, 29 Dec 2008 14:45:33 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> 
> > Hi David,
> > 
> > On Fri, 19 Dec 2008 11:05:39 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > >
> > > Given the ongoing discussions around FS-Cache, I have removed it from
> > > linux-next.  Please ask me to include it again (if sensible) once some
> > > decision has been reached about its future.
> > 
> > What was the result of discussions around FS-Cache?
> 
> There was none.
> 
> Dan Muntz's question:
> 
>   Solaris has had CacheFS since ~1995, HPUX had a port of it since
>   ~1997.  I'd be interested in evidence of even a small fraction of
>   Solaris and/or HPUX shops using CacheFS.  I am aware of customers who
>   thought it sounded like a good idea, but ended up ditching it for
>   various reasons (e.g., CacheFS just adds overhead if you almost
>   always hit your local mem cache).
> 
> was an very very good one.
> 
> Seems that instead of answering it, we've decided to investigate the
> fate of those who do not learn from history.

David has given you plenty of arguments for why it helps scale the
server (including specific workloads), has given you numbers validating
his claim, and has presented claims that Red Hat has customers using
cachefs in RHEL-5.
The arguments I've seen against it, have so far been:

     1. Solaris couldn't sell their implementation
     2. It's too big
     3. It's intrusive

Argument (1) has so far appeared to be pure FUD. In order to discuss the
lessons of history, you need to first do the work of analysing and
understanding it first. I really don't see how it is relevant to Linux
whether or not the Solaris and HPUX cachefs implementations worked out
unless you can demonstrate that that their experience shows some fatal
flaw in the arguments and numbers that David presented, and that his
customers are deluded.
If you want examples of permanent caches that clearly do help servers
scale, then look no further than the on-disk caches used in almost all
http browser implemantations. Alternatively, as David mentioned, there
are the on-disk caches used by AFS/DFS/coda.

(2) may be valid, but I have yet to see specifics for where you'd like
to see the cachefs code slimmed down. Did I miss them?

(3) was certainly true 3 years ago, when the code was first presented
for review, and so we did a review and critique then. The NFS specific
changes have improved greatly as a result, and as far as I know, the
security folks are happy too. If you're not happy with the parts that
affect the memory management code then, again, it would be useful to see
specifics that what you want changed.

If there is still controversy concerning this, then I can temporarily
remove cachefs from the nfs linux-next branch, but I'm definitely
keeping it in the linux-mm branch until someone gives me a reason for
why it shouldn't be merged in its current state.

Trond

  reply	other threads:[~2008-12-29 14:30 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
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 [this message]
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=1230561056.7046.47.camel@heimdal.trondhjem.org \
    --to=trond.myklebust@fys.uio.no \
    --cc=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=linux-next@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).