[PATCH v2 05/10] block: abolish macro PAGE_SECTORS_SHIFT

[Zhen Lei]

The name of PAGE_SECTORS_SHIFT is quite hard to read.
1. use sectors_to_npage() to replace ">> PAGE_SECTORS_SHIFT"
2. use npage_to_sectors() to replace "<< PAGE_SECTORS_SHIFT"

Suggested-by: Matthew Wilcox <wi...@infradead.org>
Signed-off-by: Zhen Lei <thunder.leiz...@huawei.com>
---
 drivers/block/brd.c           | 6 ++----
 drivers/block/null_blk_main.c | 9 ++++-----
 2 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/drivers/block/brd.c b/drivers/block/brd.c
index 30df6daa9dbc..051c5a50497f 100644
--- a/drivers/block/brd.c
+++ b/drivers/block/brd.c
@@ -25,8 +25,6 @@
 
 #include <linux/uaccess.h>
 
-#define PAGE_SECTORS_SHIFT     (PAGE_SHIFT - SECTOR_SHIFT)
-
 /*
  * Each block ramdisk device has a radix_tree brd_pages of pages that stores
  * the pages containing the block device's contents. A brd page's ->index is
@@ -69,7 +67,7 @@ static struct page *brd_lookup_page(struct brd_device *brd, 
sector_t sector)
         * here, only deletes).
         */
        rcu_read_lock();
-       idx = sector >> PAGE_SECTORS_SHIFT; /* sector to page index */
+       idx = sectors_to_npage(sector);
        page = radix_tree_lookup(&brd->brd_pages, idx);
        rcu_read_unlock();
 
@@ -108,7 +106,7 @@ static struct page *brd_insert_page(struct brd_device *brd, 
sector_t sector)
        }
 
        spin_lock(&brd->brd_lock);
-       idx = sector >> PAGE_SECTORS_SHIFT;
+       idx = sectors_to_npage(sector);
        page->index = idx;
        if (radix_tree_insert(&brd->brd_pages, idx, page)) {
                __free_page(page);
diff --git a/drivers/block/null_blk_main.c b/drivers/block/null_blk_main.c
index 25048ff15858..81485f47dcf0 100644
--- a/drivers/block/null_blk_main.c
+++ b/drivers/block/null_blk_main.c
@@ -11,7 +11,6 @@
 #include <linux/init.h>
 #include "null_blk.h"
 
-#define PAGE_SECTORS_SHIFT     (PAGE_SHIFT - SECTOR_SHIFT)
 #define SECTOR_MASK            (PAGE_SECTORS - 1)
 
 #define FREE_BATCH             16
@@ -737,7 +736,7 @@ static void null_free_sector(struct nullb *nullb, sector_t 
sector,
        struct radix_tree_root *root;
 
        root = is_cache ? &nullb->dev->cache : &nullb->dev->data;
-       idx = sector >> PAGE_SECTORS_SHIFT;
+       idx = sectors_to_npage(sector);
        sector_bit = (sector & SECTOR_MASK);
 
        t_page = radix_tree_lookup(root, idx);
@@ -808,7 +807,7 @@ static struct nullb_page *__null_lookup_page(struct nullb 
*nullb,
        struct nullb_page *t_page;
        struct radix_tree_root *root;
 
-       idx = sector >> PAGE_SECTORS_SHIFT;
+       idx = sectors_to_npage(sector);
        sector_bit = (sector & SECTOR_MASK);
 
        root = is_cache ? &nullb->dev->cache : &nullb->dev->data;
@@ -855,7 +854,7 @@ static struct nullb_page *null_insert_page(struct nullb 
*nullb,
                goto out_freepage;
 
        spin_lock_irq(&nullb->lock);
-       idx = sector >> PAGE_SECTORS_SHIFT;
+       idx = sectors_to_npage(sector);
        t_page->page->index = idx;
        t_page = null_radix_tree_insert(nullb, idx, t_page, !ignore_cache);
        radix_tree_preload_end();
@@ -878,7 +877,7 @@ static int null_flush_cache_page(struct nullb *nullb, 
struct nullb_page *c_page)
 
        idx = c_page->page->index;
 
-       t_page = null_insert_page(nullb, idx << PAGE_SECTORS_SHIFT, true);
+       t_page = null_insert_page(nullb, npage_to_sectors(idx), true);
 
        __clear_bit(NULLB_PAGE_LOCK, c_page->bitmap);
        if (test_bit(NULLB_PAGE_FREE, c_page->bitmap)) {
-- 
2.26.0.106.g9fadedd