From: Chris Mason Subject: Re: [PATCH 1/4] fs: split update_time() into update_time() and write_time() Date: Fri, 21 Nov 2014 15:08:48 -0500 Message-ID: <1416600528.24312.10@mail.thefacebook.com> References: <1416599964-21892-1-git-send-email-tytso@mit.edu> <1416599964-21892-2-git-send-email-tytso@mit.edu> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Cc: , Ext4 Developers List , , , "Theodore Ts'o" To: "Theodore Ts'o" Return-path: In-Reply-To: <1416599964-21892-2-git-send-email-tytso@mit.edu> Sender: linux-btrfs-owner@vger.kernel.org List-Id: linux-ext4.vger.kernel.org On Fri, Nov 21, 2014 at 2:59 PM, Theodore Ts'o wrote: > In preparation for adding support for the lazytime mount option, we > need to be able to separate out the update_time() and write_time() > inode operations. Currently, only btrfs and xfs uses update_time(). > > We needed to preserve update_time() because btrfs wants to have a > special btrfs_root_readonly() check; otherwise we could drop the > update_time() inode operation entirely. No objections here, I'll give the queue a whirl. You can add my acked-by-or-Ill-fix-whatever-breaks I look forward to the patch that makes us all lazy by default. -chris