Return-Path: Received: from cantor2.suse.de ([195.135.220.15]:34166 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932724Ab0EYRDa (ORCPT ); Tue, 25 May 2010 13:03:30 -0400 Date: Tue, 25 May 2010 10:00:03 -0700 From: Greg KH To: Hugh Dickins Cc: Alan Cox , Tharindu Rukshan Bamunuarachchi , linux-mm@kvack.org, linux-nfs@vger.kernel.org, linux-kernel@vger.kernel.org, stable@kernel.org Subject: Re: TMPFS over NFSv4 Message-ID: <20100525170003.GD9154@suse.de> References: <20100524110903.72524853@lxorguk.ukuu.org.uk> Content-Type: text/plain; charset=us-ascii In-Reply-To: Sender: linux-nfs-owner@vger.kernel.org List-ID: MIME-Version: 1.0 On Mon, May 24, 2010 at 04:46:24PM -0700, Hugh Dickins wrote: > Hi Greg, > > On Mon, 24 May 2010, Alan Cox wrote: > > On Mon, 24 May 2010 02:57:30 -0700 > > Hugh Dickins wrote: > > > On Mon, May 24, 2010 at 2:26 AM, Tharindu Rukshan Bamunuarachchi > > > wrote: > > > > thankx a lot Hugh ... I will try this out ... (bit harder patch > > > > already patched SLES kernel :-p ) .... > > > > > > If patch conflicts are a problem, you really only need to put in the > > > two-liner patch to mm/mmap.c: Alan was seeking perfection in > > > the rest of the patch, but you can get away without it. > > > > > > > > > > > BTW, what does Alan means by "strict overcommit" ? > > > > > > Ah, that phrase, yes, it's a nonsense, but many of us do say it by mistake. > > > Alan meant to say "strict no-overcommit". > > > > No I always meant to say 'strict overcommit'. It avoids excess negatives > > and "no noovercommit" discussions. > > > > I guess 'strict overcommit control' would have been clearer 8) > > > > Alan > > I see we've just missed 2.6.27.47-rc1, but if there's to be an -rc2, > please include Alan's 2.6.28 oops fix below: which Tharindu appears > to be needing - just now discussed on linux-mm and linux-nfs. > Failing that, please queue it up for 2.6.27.48. There is now going to be a -rc2 due to other problems, so I'll go queue this one up as well. > Or if you'd prefer a smaller patch for -stable, then just the mm/mmap.c > part of it should suffice: I think it's fair to say that the rest of the > patch was more precautionary - as Alan describes, for catching other bugs, > so good for an ongoing development tree, but not necessarily in -stable. > (However, Alan may disagree - I've already misrepresented him once here!) The original is best, it makes more sense. thanks, greg k-h