f2fs: clean up val{>>,<<}F2FS_BLKSIZE_BITS

Use F2FS_BYTES_TO_BLK(bytes) and F2FS_BLK_TO_BYTES(blk) for cleanup

Signed-off-by: Zhiguo Niu <zhiguo.niu@unisoc.com>
Reviewed-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>

diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
index bdd96329dddd4974231be2ae3a9538aa2305fdc5..f3d22b8ef2ffa56d529136829160c669f21b387c 100644 (file)
--- a/fs/f2fs/checkpoint.c
+++ b/fs/f2fs/checkpoint.c
@@ -1551,7 +1551,7 @@ static int do_checkpoint(struct f2fs_sb_info *sbi, struct cp_control *cpc)
                blk = start_blk + BLKS_PER_SEG(sbi) - nm_i->nat_bits_blocks;
                for (i = 0; i < nm_i->nat_bits_blocks; i++)
                        f2fs_update_meta_page(sbi, nm_i->nat_bits +
-                                       (i << F2FS_BLKSIZE_BITS), blk + i);
+                                       F2FS_BLK_TO_BYTES(i), blk + i);
        }
 
        /* write out checkpoint buffer at block 0 */

diff --git a/fs/f2fs/debug.c b/fs/f2fs/debug.c
index 8b0e1e71b667448afce3cb7b094fc2bc4644a892..546b8ba91261300c3b7517066d7ed25b08d17b90 100644 (file)
--- a/fs/f2fs/debug.c
+++ b/fs/f2fs/debug.c
@@ -275,7 +275,7 @@ static void update_mem_info(struct f2fs_sb_info *sbi)
        /* build nm */
        si->base_mem += sizeof(struct f2fs_nm_info);
        si->base_mem += __bitmap_size(sbi, NAT_BITMAP);
-       si->base_mem += (NM_I(sbi)->nat_bits_blocks << F2FS_BLKSIZE_BITS);
+       si->base_mem += F2FS_BLK_TO_BYTES(NM_I(sbi)->nat_bits_blocks);
        si->base_mem += NM_I(sbi)->nat_blocks *
                                f2fs_bitmap_size(NAT_ENTRY_PER_BLOCK);
        si->base_mem += NM_I(sbi)->nat_blocks / 8;

diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
index a2d466ca25799e8f1a9bed4964e3eeec7672aa61..bddcb2cd945aa03898f08997561d16e3f1144240 100644 (file)
--- a/fs/f2fs/file.c
+++ b/fs/f2fs/file.c
@@ -2979,9 +2979,9 @@ static int f2fs_move_file_range(struct file *file_in, loff_t pos_in,
        }
 
        f2fs_lock_op(sbi);
-       ret = __exchange_data_block(src, dst, pos_in >> F2FS_BLKSIZE_BITS,
-                               pos_out >> F2FS_BLKSIZE_BITS,
-                               len >> F2FS_BLKSIZE_BITS, false);
+       ret = __exchange_data_block(src, dst, F2FS_BYTES_TO_BLK(pos_in),
+                               F2FS_BYTES_TO_BLK(pos_out),
+                               F2FS_BYTES_TO_BLK(len), false);
 
        if (!ret) {
                if (dst_max_i_size)

diff --git a/fs/f2fs/node.c b/fs/f2fs/node.c
index 2fe182f7359954a174d7d0b619029616cf53c2bf..9e7a6e21f30c715bcf94ead564733fd73f0a43ef 100644 (file)
--- a/fs/f2fs/node.c
+++ b/fs/f2fs/node.c
@@ -3166,7 +3166,7 @@ static int __get_nat_bitmaps(struct f2fs_sb_info *sbi)
 
        nm_i->nat_bits_blocks = F2FS_BLK_ALIGN((nat_bits_bytes << 1) + 8);
        nm_i->nat_bits = f2fs_kvzalloc(sbi,
-                       nm_i->nat_bits_blocks << F2FS_BLKSIZE_BITS, GFP_KERNEL);
+                       F2FS_BLK_TO_BYTES(nm_i->nat_bits_blocks), GFP_KERNEL);
        if (!nm_i->nat_bits)
                return -ENOMEM;
 
@@ -3185,7 +3185,7 @@ static int __get_nat_bitmaps(struct f2fs_sb_info *sbi)
                if (IS_ERR(page))
                        return PTR_ERR(page);
 
-               memcpy(nm_i->nat_bits + (i << F2FS_BLKSIZE_BITS),
+               memcpy(nm_i->nat_bits + F2FS_BLK_TO_BYTES(i),
                                        page_address(page), F2FS_BLKSIZE);
                f2fs_put_page(page, 1);
        }

diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index 4d2f3bb8e418b8bf4d50e6391b1857eed1ee26e1..977f038f355784872ba7e48543f5eca52d814b7b 100644 (file)
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -3317,7 +3317,7 @@ loff_t max_file_blocks(struct inode *inode)
         * fit within U32_MAX + 1 data units.
         */
 
-       result = min(result, (((loff_t)U32_MAX + 1) * 4096) >> F2FS_BLKSIZE_BITS);
+       result = min(result, F2FS_BYTES_TO_BLK(((loff_t)U32_MAX + 1) * 4096));
 
        return result;
 }

diff --git a/include/linux/f2fs_fs.h b/include/linux/f2fs_fs.h
index 01bee2b289c2a7f766b7f68b9a0d839bcb4cb04b..f17f89a259e206a73e668249eace5198672e511c 100644 (file)
--- a/include/linux/f2fs_fs.h
+++ b/include/linux/f2fs_fs.h
@@ -19,7 +19,6 @@
 #define F2FS_BLKSIZE_BITS              PAGE_SHIFT /* bits for F2FS_BLKSIZE */
 #define F2FS_MAX_EXTENSION             64      /* # of extension entries */
 #define F2FS_EXTENSION_LEN             8       /* max size of extension */
-#define F2FS_BLK_ALIGN(x)      (((x) + F2FS_BLKSIZE - 1) >> F2FS_BLKSIZE_BITS)
 
 #define NULL_ADDR              ((block_t)0)    /* used as block_t addresses */
 #define NEW_ADDR               ((block_t)-1)   /* used as block_t addresses */
@@ -28,6 +27,7 @@
 #define F2FS_BYTES_TO_BLK(bytes)       ((bytes) >> F2FS_BLKSIZE_BITS)
 #define F2FS_BLK_TO_BYTES(blk)         ((blk) << F2FS_BLKSIZE_BITS)
 #define F2FS_BLK_END_BYTES(blk)                (F2FS_BLK_TO_BYTES(blk + 1) - 1)
+#define F2FS_BLK_ALIGN(x)                      (F2FS_BYTES_TO_BLK((x) + F2FS_BLKSIZE - 1))
 
 /* 0, 1(node nid), 2(meta nid) are reserved node id */
 #define F2FS_RESERVED_NODE_NUM         3