From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH] tmpfs: support SEEK_DATA and SEEK_HOLE (reprise) Date: Thu, 29 Nov 2012 16:18:21 -0800 Message-ID: <20121129161821.8103962c.akpm@linux-foundation.org> References: <20121129145924.9fb05982.akpm@linux-foundation.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: "Theodore Ts'o" , Zheng Liu , Jeff liu , Jim Meyering , Paul Eggert , Christoph Hellwig , Josef Bacik , Andi Kleen , Andreas Dilger , Dave Chinner , Marco Stornelli , Chris Mason , Sunil Mushran , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org To: Hugh Dickins Return-path: In-Reply-To: Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On Thu, 29 Nov 2012 15:29:15 -0800 (PST) Hugh Dickins wrote: > On Thu, 29 Nov 2012, Andrew Morton wrote: > > On Wed, 28 Nov 2012 17:22:03 -0800 (PST) > > Hugh Dickins wrote: > > > > > +/* > > > + * llseek SEEK_DATA or SEEK_HOLE through the radix_tree. > > > + */ > > > +static pgoff_t shmem_seek_hole_data(struct address_space *mapping, > > > + pgoff_t index, pgoff_t end, int origin) > > > > So I was starting at this wondering what on earth "origin" is and why > > it has the fishy-in-this-context type "int". > > > > There is a pretty well established convention that the lseek seek mode > > is called "whence". > > > > The below gets most of it. Too anal? > > No, not too anal: I'm all in favour of "whence", which is indeed > the name of that lseek argument - since mediaeval times I believe. Alas, the rest of us don't have personal memories from those days. > It's good to have words like that in the kernel source: while you're > in the mood, please see if you can find good homes for "whither" and > "thrice" and "widdershins". We use "thrice" quite a lot. And "whither" once coz alfa peeps cnat spel. No widdershins yet. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org