From: "H. Peter Anvin" <hpa@zytor.com>
To: Arjan van de Ven <arjan@infradead.org>
Cc: Jan Kara <jack@suse.cz>,
linux-kernel@vger.kernel.org, linux-ext4@vger.kernel.org
Subject: Re: [RFC] [PATCH 3/3] Recursive mtime for ext3
Date: Tue, 06 Nov 2007 10:04:47 -0800 [thread overview]
Message-ID: <4730ACBF.6050606@zytor.com> (raw)
In-Reply-To: <20071106094056.3352a574@laptopd505.fenrus.org>
Arjan van de Ven wrote:
> On Tue, 6 Nov 2007 18:19:45 +0100
> Jan Kara <jack@suse.cz> wrote:
>
>> Implement recursive mtime (rtime) feature for ext3. The feature works
>> as follows: In each directory we keep a flag EXT3_RTIME_FL
>> (modifiable by a user) whether rtime should be updated. In case a
>> directory or a file in it is modified and when the flag is set,
>> directory's rtime is updated, the flag is cleared, and we move to the
>> parent. If the flag is set there, we clear it, update rtime and
>> continue upwards upto the root of the filesystem. In case a regular
>> file or symlink is modified, we pick arbitrary of its parents
>> (actually the one that happens to be at the head of i_dentry list)
>> and start the rtime update algorith there.
>
> Ok since mtime (and rtime) are part of the inode and not the dentry...
> how do you deal with hardlinks? And with cases of files that have been
> unlinked? (ok the later is a wash obviously other than not crashing)
>
There is only one possible answer... he only updates the directory path
that was used to touch the particular file involved. Thus, the
semantics gets grotty not just in the presence of hard links, but also
in the presence of bind- and other non-root mounts.
-hpa
next prev parent reply other threads:[~2007-11-06 18:08 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-06 17:15 [RFC] [PATCH 0/3] Recursive mtime for ext3 Jan Kara
2007-11-06 17:18 ` [RFC] [PATCH 1/3] " Jan Kara
2007-11-06 17:19 ` [RFC] [PATCH 2/3] " Jan Kara
2007-11-06 17:19 ` [RFC] [PATCH 3/3] " Jan Kara
2007-11-06 17:40 ` Arjan van de Ven
2007-11-06 18:04 ` H. Peter Anvin [this message]
2007-11-07 11:51 ` Jan Kara
2007-11-06 18:01 ` Al Viro
2007-11-07 14:54 ` Jan Kara
2007-11-06 19:40 ` Theodore Tso
2007-11-07 14:36 ` Jan Kara
2007-11-08 0:20 ` Theodore Tso
2007-11-08 10:56 ` Jan Kara
2007-11-08 14:37 ` Theodore Tso
2007-11-08 15:28 ` Jan Kara
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4730ACBF.6050606@zytor.com \
--to=hpa@zytor.com \
--cc=arjan@infradead.org \
--cc=jack@suse.cz \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).