f2fs: fix overflow when calculating utilization on 32-bit
authorChangman Lee <cm224.lee@samsung.com>
Mon, 25 Feb 2013 08:38:02 +0000 (17:38 +0900)
committerJaegeuk Kim <jaegeuk.kim@samsung.com>
Fri, 8 Mar 2013 01:02:56 +0000 (10:02 +0900)
Use div_u64 to fix overflow when calculating utilization.
*long int* is 4-bytes on 32-bit so (user blocks * 100) might be
overflow if disk size is over e.g. 512GB.

Signed-off-by: Changman Lee <cm224.lee@samsung.com>
Reviewed-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
fs/f2fs/segment.h

index 552dadb..e399bd4 100644 (file)
@@ -464,8 +464,7 @@ static inline bool has_not_enough_free_secs(struct f2fs_sb_info *sbi, int freed)
 
 static inline int utilization(struct f2fs_sb_info *sbi)
 {
-       return (long int)valid_user_blocks(sbi) * 100 /
-                       (long int)sbi->user_block_count;
+       return div_u64(valid_user_blocks(sbi) * 100, sbi->user_block_count);
 }
 
 /*