aboutsummaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorAkira Fujita <a-fujita@rs.jp.nec.com>2009-09-05 22:11:55 -0400
committerTheodore Ts'o <tytso@mit.edu>2009-09-05 22:11:55 -0400
commitdaea696dbac0e33af3cfe304efbfb8d74e0effe6 (patch)
tree35b64870a9377d1a89d2891084623bca8dd13e70 /fs
parent70d5d3dcea47c16058d2b093c29e07fdf61b56ad (diff)
ext4: Remove unneeded BUG_ON() in ext4_move_extents()
The ext4_move_extents() functions checks with BUG_ON() whether the exchanged blocks count accords with request blocks count. But, if the target range (orig_start + len) includes sparse block(s), 'moved_len' (exchanged blocks count) does not agree with 'len' (request blocks count), since sparse block is not counted in 'moved_len'. This causes us to hit the BUG_ON(), even though the function succeeded. Signed-off-by: Akira Fujita <a-fujita@rs.jp.nec.com> Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs')
-rw-r--r--fs/ext4/move_extent.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c
index c593eb2b193a..c8c66b167cd7 100644
--- a/fs/ext4/move_extent.c
+++ b/fs/ext4/move_extent.c
@@ -1322,8 +1322,5 @@ out2:
1322 if (ret) 1322 if (ret)
1323 return ret; 1323 return ret;
1324 1324
1325 /* All of the specified blocks must be exchanged in succeed */
1326 BUG_ON(*moved_len != len);
1327
1328 return 0; 1325 return 0;
1329} 1326}