diff options
author | Xiaoguang Wang <wangxg.fnst@cn.fujitsu.com> | 2014-07-27 22:26:36 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2014-07-27 22:26:36 -0400 |
commit | b27b1535acc0e97b7435bccf205b3fe9c782afac (patch) | |
tree | 0bd7468d36828c1f7b28d7e60b7a4673e941d402 /fs | |
parent | 83447ccb4df6e6051027bedff2a8cfc6a52ebcf8 (diff) |
ext4: fix wrong size computation in ext4_mb_normalize_request()
As the member fe_len defined in struct ext4_free_extent is expressed as
number of clusters, the variable "size" computation is wrong, we need to
first translate fe_len to block number, then to bytes.
Signed-off-by: Xiaoguang Wang <wangxg.fnst@cn.fujitsu.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Reviewed-by: Lukas Czerner <lczerner@redhat.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/ext4/mballoc.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c index 18a16191249a..0e9466f9e767 100644 --- a/fs/ext4/mballoc.c +++ b/fs/ext4/mballoc.c | |||
@@ -3075,8 +3075,9 @@ ext4_mb_normalize_request(struct ext4_allocation_context *ac, | |||
3075 | (23 - bsbits)) << 23; | 3075 | (23 - bsbits)) << 23; |
3076 | size = 8 * 1024 * 1024; | 3076 | size = 8 * 1024 * 1024; |
3077 | } else { | 3077 | } else { |
3078 | start_off = (loff_t)ac->ac_o_ex.fe_logical << bsbits; | 3078 | start_off = (loff_t) ac->ac_o_ex.fe_logical << bsbits; |
3079 | size = ac->ac_o_ex.fe_len << bsbits; | 3079 | size = (loff_t) EXT4_C2B(EXT4_SB(ac->ac_sb), |
3080 | ac->ac_o_ex.fe_len) << bsbits; | ||
3080 | } | 3081 | } |
3081 | size = size >> bsbits; | 3082 | size = size >> bsbits; |
3082 | start = start_off >> bsbits; | 3083 | start = start_off >> bsbits; |