[lkml]   [2003]   [Oct]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    SubjectReiserFS patch for updating ctimes of renamed files

    I ran into some trouble trying to do incremental backups with GNU tar
    (using --listed-incremental) where renaming a file in between backups would
    cause the file to disappear upon restoration. When investigating the issue
    I discovered that this doesn't happen on ext2, ext3, and tmpfs filesystems
    but only on ReiserFS filesystems. I also noticed that for example ext3
    updates the affected file's ctime upon rename whereas ReiserFS doesn't, so
    I'm thinking this causes tar to believe that the file existed before the
    first backup was taking under the new name, and as a result it doesn't back
    it up during the second backup. So I believe ReiserFS needs to update
    ctimes for renamed files in order for incremental GNU tar backups to work

    I made some changes to the reiserfs_rename function that I *think* should
    fix the problem. However, I don't know much about ReiserFS's internals, and
    I haven't been able to test them out to see if things work now since I can't
    afford to deal with potential FS corruption with my current Linux box.

    I included a patch below against the 2.4.22 kernel with my changes. Would
    somebody mind taking a look at this to see if I did things right here (and
    perhaps wouldn't mind testing it out either)? If it works then I (and I'm
    sure others who've experienced the same problem) would like to see the
    changes applied to the next 2.4.x (and 2.6.x?) release.

    Thanks a lot.


    --- fs/reiserfs/namei.c.orig Mon Aug 25 06:44:43 2003
    +++ fs/reiserfs/namei.c Sun Oct 12 00:39:05 2003
    @@ -1207,6 +1207,8 @@
    journal_mark_dirty (&th, old_dir->i_sb, old_de.de_bh);
    old_dir->i_ctime = old_dir->i_mtime = CURRENT_TIME;
    new_dir->i_ctime = new_dir->i_mtime = CURRENT_TIME;
    + old_inode->i_ctime = CURRENT_TIME;
    + reiserfs_update_sd (&th, old_inode);

    if (new_dentry_inode) {
    // adjust link number of the victim
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:49    [W:0.021 / U:4.008 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site