From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH v2 2/2] relatime: Allow making relatime the default behaviour Date: Sat, 29 Nov 2008 12:56:07 -0800 Message-ID: <20081129125607.55d70063.akpm@linux-foundation.org> References: <492DD035.5020705@oracle.com> <20081127150126.GA20941@srcf.ucam.org> <20081127150341.GB20941@srcf.ucam.org> <20081127170345.GA14991@infradead.org> <20081129002419.abfd2504.akpm@linux-foundation.org> <20081129130308.GX25548@parisc-linux.org> <20081129135742.GC11959@logfs.org> <20081129185645.GA20194@shareable.org> <20081129190245.GA21502@srcf.ucam.org> <20081129123220.67fd1e6a.akpm@linux-foundation.org> <20081129203856.GY25548@parisc-linux.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Matthew Garrett , Jamie Lokier , J?rn Engel , Christoph Hellwig , Randy Dunlap , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, mingo@redhat.com, val.henson@gmail.com To: Matthew Wilcox Return-path: Received: from smtp1.linux-foundation.org ([140.211.169.13]:56268 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752403AbYK2U4l (ORCPT ); Sat, 29 Nov 2008 15:56:41 -0500 In-Reply-To: <20081129203856.GY25548@parisc-linux.org> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Sat, 29 Nov 2008 13:38:57 -0700 Matthew Wilcox wrote: > On Sat, Nov 29, 2008 at 12:32:20PM -0800, Andrew Morton wrote: > > The standard, usual, expected way of modifying a filesystem's behaviour > > is via mount options. This is also quite flexible. > > > > Is there some extraordinary reason why the standard interface is not to > > be used here? > > Because it would have to be managed (and consulted) per ... what? > vfsmount? superblock? Per superblock, of course. > This is featuritis gone MAD. No it isn't - it's the expected and standard behaviour. We have all the kernel infrastructure and userspace tools in place for doing it this way. Modifying the behaviour of all filesystems with a single knob is a weird thing to do. MNT_RELATIME itself is already per-superblock.