diff options
author | Akira Fujita <a-fujita@rs.jp.nec.com> | 2009-09-05 22:11:55 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2009-09-05 22:11:55 -0400 |
commit | daea696dbac0e33af3cfe304efbfb8d74e0effe6 (patch) | |
tree | 35b64870a9377d1a89d2891084623bca8dd13e70 /fs | |
parent | 70d5d3dcea47c16058d2b093c29e07fdf61b56ad (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.c | 3 |
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 | } |