From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay3.corp.sgi.com [198.149.34.15]) by oss.sgi.com (Postfix) with ESMTP id 962A97F51 for ; Mon, 24 Nov 2014 18:32:31 -0600 (CST) Received: from cuda.sgi.com (cuda2.sgi.com [192.48.176.25]) by relay3.corp.sgi.com (Postfix) with ESMTP id 14CE8AC002 for ; Mon, 24 Nov 2014 16:32:30 -0800 (PST) Received: from imap.thunk.org (imap.thunk.org [74.207.234.97]) by cuda.sgi.com with ESMTP id 7hV1tkmXU8zqKJmp (version=TLSv1 cipher=AES128-SHA bits=128 verify=NO) for ; Mon, 24 Nov 2014 16:32:29 -0800 (PST) Date: Mon, 24 Nov 2014 19:32:15 -0500 From: Theodore Ts'o Subject: Re: [PATCH-v2 0/5] add support for a lazytime mount option Message-ID: <20141125003215.GD31339@thunk.org> References: <1416675267-2191-1-git-send-email-tytso@mit.edu> <20141124090755.GA28534@infradead.org> <20141124115727.GA19918@thunk.org> <20141124221145.GB24003@fieldses.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20141124221145.GB24003@fieldses.org> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: "J. Bruce Fields" Cc: Christoph Hellwig , linux-fsdevel@vger.kernel.org, Ext4 Developers List , linux-btrfs@vger.kernel.org, xfs@oss.sgi.com On Mon, Nov 24, 2014 at 05:11:45PM -0500, J. Bruce Fields wrote: > On Mon, Nov 24, 2014 at 06:57:27AM -0500, Theodore Ts'o wrote: > > If we want to be paranoid, we handle i_version updates non-lazily; I > > can see arguments in favor of that. > > > > Ext4 only enables MS_I_VERSION if the user asks for it explicitly, so > > it wouldn't cause me any problems. However, xfs and btrfs enables it > > by default, so that means xfs and btrfs wouldn't see the benefits of > > lazytime (if you're going to have to push I_VERSION to disk, you might > > as well update the [acm]time while you're at it). I've always thought > > that we *should* do is to only enable it if nfsv4 is serving the file > > system, and not otherwise, though, which would also give us > > consistency across all the file systems. > > I guess you need to worry about the case where you shutdown nfsd, modify > a file, then restart nfsd--you don't want a client to miss the > modification in that case. Hmmm, is there a way we can determine whether the file system is being exported via nfsv4 is running, without using MS_I_VERSION? Maybe a new flag? We could just disable lazytime if nfsd is running but otherwise always increment i_version if lazytime is enabled. My main concern with i_version updates was not the in-memry update of i_version, but rather the unnecessary extra metadata write "tax" that would be inflicted on all users, including the many that aren't serving files via NFSv4. That way, if you shutdown the nfsv4 server, i_version would still be updated, but we wouldn't be forcing the writes to disk, but then when nfs v4 server is updated again, the i_version tax would be paid again. - Ted _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs