![]() | |||||||||||||
Messages in this thread Patch in this message |
Traditionally, Unix filesystems updated the inode ctime of the file being rename()'d. While SuS does not strictly require this it does permit it, and a number of programs (including dump and some commercial backup tools) count on it. People on linux-kernel may remember the Reiserfs people discovering and patching this rename() issue and the discussion that resulted. After 2.2.12, it was noticed that Linux ext2 didn't do this, and it was duly patched in 2.2.13. The ext3 people picked up the change at that time, down to the comment, and you will find the comment there in ext3 to this day (2.4 and 2.6 both). Unfortunately, however, the change was not pushed into 2.3, and so 2.4 and now 2.6 have a ext2 rename() implementation that does not update inode ctime. Here is a 2.4 patch to fix this; it's been tested and it works. I believe a similar if not identical patch would work on 2.6, but I don't have a 2.6 system to test it with. (It's possible that the use of 'mark_inode_dirty' is no longer necessary, but I don't know enough about ext2 internals to be sure.) ===== fs/ext2/namei.c 1.3 vs edited ===== --- 1.3/fs/ext2/namei.c Tue Feb 5 02:41:03 2002 +++ edited/fs/ext2/namei.c Wed Dec 10 17:13:30 2003 @@ -313,6 +313,13 @@ ext2_inc_count(new_dir); } + /* + * Like most other Unix systems, set the ctime for inodes on a + * rename. + */ + old_inode->i_ctime = CURRENT_TIME; + mark_inode_dirty(old_inode); + ext2_delete_entry (old_de, old_page); ext2_dec_count(old_inode); - cks - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | ||||||||||||
| Last update: 2005-03-22 12:59 [from the cache] ©2003-2008 | |||||||||||||