Thread (6 messages) 6 messages, 3 authors, 2016-06-14

Re: [PATCH 02/21] fs: ext4: Use current_fs_time() for inode timestamps

From: Linus Torvalds <torvalds@linux-foundation.org>
Date: 2016-06-09 18:55:25
Also in: linux-fsdevel, lkml

On Thu, Jun 9, 2016 at 11:45 AM, Linus Torvalds
[off-list ref] wrote:
All existing users and all the ones in this patch (and the others too,
although I didn't go through them very carefully) really would prefer
just passing in the inode directly, rather than the superblock.
Actually, there seems to be one exception to that "all existing
users", and that one exception (btrfs transacation time) really seems
to be broken. Exactly because it's *not* setting an inode time, it
shouldn't have used current_fs_time() to begin with, because it is
just setting an internal filesystem timestamp.

So not making the argument inode-related seems to actually encourage
people to misuse this function.

                Linus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help