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>
[PATCH-v3 0/6] add support for a lazytime mount option, Theodore Ts'o [PATCH-v3 2/6] vfs: add support for a lazytime mount option, Theodore Ts'o [PATCH-v3 1/6] fs: split update_time() into update_time() and write_time(), Theodore Ts'o Re: [PATCH-v3 1/6] fs: split update_time() into update_time() and write_time(), David Sterba <= [PATCH-v3 4/6] vfs: add lazytime tracepoints for better debugging, Theodore Ts'o [PATCH-v3 6/6] btrfs: add an is_readonly() so btrfs can use common code for update_time(), Theodore Ts'o Re: [PATCH-v3 6/6] btrfs: add an is_readonly() so btrfs can use common code for update_time(), David Sterba [PATCH-v3 3/6] vfs: don't let the dirty time inodes get more than a day stale, Theodore Ts'o Re: [PATCH-v3 3/6] vfs: don't let the dirty time inodes get more than a day stale, Rasmus Villemoes Re: [PATCH-v3 3/6] vfs: don't let the dirty time inodes get more than a day stale, Theodore Ts'o [PATCH-v3 5/6] ext4: add support for a lazytime mount option, Theodore Ts'o

<Prev in Thread]

Current Thread

[Next in Thread>

Previous by Date:	Re: [PATCH-v3 3/6] vfs: don't let the dirty time inodes get more than a day stale, Theodore Ts'o
Next by Date:	Re: [PATCH-v3 6/6] btrfs: add an is_readonly() so btrfs can use common code for update_time(), David Sterba
Previous by Thread:	[PATCH-v3 1/6] fs: split update_time() into update_time() and write_time(), Theodore Ts'o
Next by Thread:	[PATCH-v3 4/6] vfs: add lazytime tracepoints for better debugging, Theodore Ts'o
Indexes:	[Date] [Thread] [Top] [All Lists]

Previous by Date:

Re: [PATCH-v3 3/6] vfs: don't let the dirty time inodes get more than a day stale, Theodore Ts'o

Next by Date:

Re: [PATCH-v3 6/6] btrfs: add an is_readonly() so btrfs can use common code for update_time(), David Sterba

Previous by Thread:

[PATCH-v3 1/6] fs: split update_time() into update_time() and write_time(), Theodore Ts'o

Next by Thread:

[PATCH-v3 4/6] vfs: add lazytime tracepoints for better debugging, Theodore Ts'o

Indexes:

[Date] [Thread] [Top] [All Lists]

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