]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
ext4: Automatically allocate delay allocated blocks on rename
authorTheodore Ts'o <tytso@mit.edu>
Tue, 2 Jun 2009 12:09:24 +0000 (08:09 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 12 Jun 2009 03:01:48 +0000 (20:01 -0700)
(cherry picked from commit 8750c6d5fcbd3342b3d908d157f81d345c5325a7)

When renaming a file such that a link to another inode is overwritten,
force any delay allocated blocks that to be allocated so that if the
filesystem is mounted with data=ordered, the data blocks will be
pushed out to disk along with the journal commit.  Many application
programs expect this, so we do this to avoid zero length files if the
system crashes unexpectedly.

Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
fs/ext4/namei.c

index e2028b3b5a81481c93839cccc1bdfbe01add9f52..6a71680c0e26b677286f8af1b2345e1f91d4bcbf 100644 (file)
@@ -2314,7 +2314,7 @@ static int ext4_rename (struct inode * old_dir, struct dentry *old_dentry,
        struct inode * old_inode, * new_inode;
        struct buffer_head * old_bh, * new_bh, * dir_bh;
        struct ext4_dir_entry_2 * old_de, * new_de;
-       int retval;
+       int retval, force_da_alloc = 0;
 
        old_bh = new_bh = dir_bh = NULL;
 
@@ -2452,6 +2452,7 @@ static int ext4_rename (struct inode * old_dir, struct dentry *old_dentry,
                ext4_mark_inode_dirty(handle, new_inode);
                if (!new_inode->i_nlink)
                        ext4_orphan_add(handle, new_inode);
+               force_da_alloc = 1;
        }
        retval = 0;
 
@@ -2460,6 +2461,8 @@ end_rename:
        brelse (old_bh);
        brelse (new_bh);
        ext4_journal_stop(handle);
+       if (retval == 0 && force_da_alloc)
+               ext4_alloc_da_blocks(old_inode);
        return retval;
 }