Re: [PATCH 0/7] Permit filesystem local caching and NFS superblocksharing [try #13]

From: Andrew Morton
Date: Wed Aug 30 2006 - 16:52:30 EST


On Wed, 30 Aug 2006 21:37:18 +0100
David Howells <dhowells@xxxxxxxxxx> wrote:

> Andrew Morton <akpm@xxxxxxxx> wrote:
>
> > > These patches add local caching for network filesystems such as NFS and AFS.
> >
> > <fercrissake>
> >
> > Not interested. Please go learn quilt, send incremental patches.
>
> What's quilt able to do that StGIT can't? AFAICT from quilt's manpage, it
> can't mail incremental patches, so how does it help anyway?
>

It was just a suggestion. Please:

- test the patches which are presently in -mm. I don't even know if they
work, and we prefer to send Linus working stuff.

- Send fine-grained incremental patches. It's OK to do complete
replacement patchsets when the code is new, but this stuff is supposed to
be stabilised.

It took me quite a lot of time to extract the incremental patches out
of try#12 and I don't want to do it again, plus it's just another step in
which errors can be introduced.

Why incremental patches?

- So we can see what changed and don't have to re-review the whole thing

- So the recipient doesn't have to re-fix the same pile of rejects each time.

- So fixes which came in via other sources don't get lost.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/