summaryrefslogtreecommitdiffstats
path: root/fs/ext4/migrate.c
diff options
context:
space:
mode:
authorTheodore Ts'o <tytso@mit.edu>2014-09-01 14:37:09 -0400
committerTheodore Ts'o <tytso@mit.edu>2014-09-01 14:37:09 -0400
commitdfe5080939ea4686b3414b5d970a9b26733c57a4 (patch)
tree3a97b3bef0d585e4fdb445b040284e90cccf654d /fs/ext4/migrate.c
parent4f224b8b7be6856a3ceaf7f9d9c1860d467174ae (diff)
downloadlinux-stable-dfe5080939ea4686b3414b5d970a9b26733c57a4.tar.gz
linux-stable-dfe5080939ea4686b3414b5d970a9b26733c57a4.tar.bz2
linux-stable-dfe5080939ea4686b3414b5d970a9b26733c57a4.zip
ext4: drop EXT4_EX_NOFREE_ON_ERR from rest of extents handling code
Drop EXT4_EX_NOFREE_ON_ERR from ext4_ext_create_new_leaf(), ext4_split_extent(), ext4_convert_unwritten_extents_endio(). This requires fixing all of their callers to potentially ext4_ext_find_extent() to free the struct ext4_ext_path object in case of an error, and there are interlocking dependencies all the way up to ext4_ext_map_blocks(), ext4_swap_extents(), and ext4_ext_remove_space(). Once this is done, we can drop the EXT4_EX_NOFREE_ON_ERR flag since it is no longer necessary. Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/migrate.c')
-rw-r--r--fs/ext4/migrate.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ext4/migrate.c b/fs/ext4/migrate.c
index d3567f27bae7..aff7bdfdc461 100644
--- a/fs/ext4/migrate.c
+++ b/fs/ext4/migrate.c
@@ -81,7 +81,7 @@ static int finish_range(handle_t *handle, struct inode *inode,
goto err_out;
}
}
- retval = ext4_ext_insert_extent(handle, inode, path, &newext, 0);
+ retval = ext4_ext_insert_extent(handle, inode, &path, &newext, 0);
err_out:
up_write((&EXT4_I(inode)->i_data_sem));
if (path) {