xfs
[Top] [All Lists]

Re: [PATCH-v3 1/6] fs: split update_time() into update_time() and write_

To: Theodore Ts'o <tytso@xxxxxxx>
Subject: Re: [PATCH-v3 1/6] fs: split update_time() into update_time() and write_time()
From: David Sterba <dsterba@xxxxxxx>
Date: Tue, 25 Nov 2014 17:00:18 +0100
Cc: Linux Filesystem Development List <linux-fsdevel@xxxxxxxxxxxxxxx>, Ext4 Developers List <linux-ext4@xxxxxxxxxxxxxxx>, Linux btrfs Developers List <linux-btrfs@xxxxxxxxxxxxxxx>, XFS Developers <xfs@xxxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1416893674-419-2-git-send-email-tytso@xxxxxxx>
Mail-followup-to: dsterba@xxxxxxx, Theodore Ts'o <tytso@xxxxxxx>, Linux Filesystem Development List <linux-fsdevel@xxxxxxxxxxxxxxx>, Ext4 Developers List <linux-ext4@xxxxxxxxxxxxxxx>, Linux btrfs Developers List <linux-btrfs@xxxxxxxxxxxxxxx>, XFS Developers <xfs@xxxxxxxxxxx>
References: <1416893674-419-1-git-send-email-tytso@xxxxxxx> <1416893674-419-2-git-send-email-tytso@xxxxxxx>
Reply-to: dsterba@xxxxxxx
User-agent: Mutt/1.5.23.1-rc1 (2014-03-12)
On Tue, Nov 25, 2014 at 12:34:29AM -0500, 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.
> 
> Signed-off-by: Theodore Ts'o <tytso@xxxxxxx>
> Cc: xfs@xxxxxxxxxxx
> Cc: linux-btrfs@xxxxxxxxxxxxxxx

For the btrfs changes

Acked-by: David Sterba <dsterba@xxxxxxx>

<Prev in Thread] Current Thread [Next in Thread>