btrfs: pass only eb to num_extent_pages

Almost all callers pass the start and len as 2 arguments but this is not
necessary, all the information is provided by the eb. By reordering the
calls to num_extent_pages, we don't need the local variables with
start/len.

Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c
index b3e4571..fd34c74 100644 (file)
--- a/fs/btrfs/extent_io.c
+++ b/fs/btrfs/extent_io.c
@@ -2059,7 +2059,7 @@ int repair_eb_io_failure(struct btrfs_fs_info *fs_info,
                         struct extent_buffer *eb, int mirror_num)
 {
        u64 start = eb->start;
-       unsigned long i, num_pages = num_extent_pages(eb->start, eb->len);
+       unsigned long i, num_pages = num_extent_pages(eb);
        int ret = 0;
 
        if (sb_rdonly(fs_info->sb))
@@ -3588,7 +3588,7 @@ lock_extent_buffer_for_io(struct extent_buffer *eb,
        if (!ret)
                return ret;
 
-       num_pages = num_extent_pages(eb->start, eb->len);
+       num_pages = num_extent_pages(eb);
        for (i = 0; i < num_pages; i++) {
                struct page *p = eb->pages[i];
 
@@ -3718,7 +3718,7 @@ static noinline_for_stack int write_one_eb(struct extent_buffer *eb,
        int ret = 0;
 
        clear_bit(EXTENT_BUFFER_WRITE_ERR, &eb->bflags);
-       num_pages = num_extent_pages(eb->start, eb->len);
+       num_pages = num_extent_pages(eb);
        atomic_set(&eb->io_pages, num_pages);
 
        /* set btree blocks beyond nritems with 0 to avoid stale content. */
@@ -4653,7 +4653,7 @@ static void btrfs_release_extent_buffer_page(struct extent_buffer *eb)
 
        BUG_ON(extent_buffer_under_io(eb));
 
-       index = num_extent_pages(eb->start, eb->len);
+       index = num_extent_pages(eb);
        if (index == 0)
                return;
 
@@ -4746,7 +4746,7 @@ struct extent_buffer *btrfs_clone_extent_buffer(struct extent_buffer *src)
        unsigned long i;
        struct page *p;
        struct extent_buffer *new;
-       unsigned long num_pages = num_extent_pages(src->start, src->len);
+       unsigned long num_pages = num_extent_pages(src);
 
        new = __alloc_extent_buffer(src->fs_info, src->start, src->len);
        if (new == NULL)
@@ -4778,12 +4778,11 @@ struct extent_buffer *__alloc_dummy_extent_buffer(struct btrfs_fs_info *fs_info,
        unsigned long num_pages;
        unsigned long i;
 
-       num_pages = num_extent_pages(start, len);
-
        eb = __alloc_extent_buffer(fs_info, start, len);
        if (!eb)
                return NULL;
 
+       num_pages = num_extent_pages(eb);
        for (i = 0; i < num_pages; i++) {
                eb->pages[i] = alloc_page(GFP_NOFS);
                if (!eb->pages[i])
@@ -4847,7 +4846,7 @@ static void mark_extent_buffer_accessed(struct extent_buffer *eb,
 
        check_buffer_tree_ref(eb);
 
-       num_pages = num_extent_pages(eb->start, eb->len);
+       num_pages = num_extent_pages(eb);
        for (i = 0; i < num_pages; i++) {
                struct page *p = eb->pages[i];
 
@@ -4944,7 +4943,7 @@ struct extent_buffer *alloc_extent_buffer(struct btrfs_fs_info *fs_info,
                                          u64 start)
 {
        unsigned long len = fs_info->nodesize;
-       unsigned long num_pages = num_extent_pages(start, len);
+       unsigned long num_pages;
        unsigned long i;
        unsigned long index = start >> PAGE_SHIFT;
        struct extent_buffer *eb;
@@ -4967,6 +4966,7 @@ struct extent_buffer *alloc_extent_buffer(struct btrfs_fs_info *fs_info,
        if (!eb)
                return ERR_PTR(-ENOMEM);
 
+       num_pages = num_extent_pages(eb);
        for (i = 0; i < num_pages; i++, index++) {
                p = find_or_create_page(mapping, index, GFP_NOFS|__GFP_NOFAIL);
                if (!p) {
@@ -5163,7 +5163,7 @@ void clear_extent_buffer_dirty(struct extent_buffer *eb)
        unsigned long num_pages;
        struct page *page;
 
-       num_pages = num_extent_pages(eb->start, eb->len);
+       num_pages = num_extent_pages(eb);
 
        for (i = 0; i < num_pages; i++) {
                page = eb->pages[i];
@@ -5197,7 +5197,7 @@ int set_extent_buffer_dirty(struct extent_buffer *eb)
 
        was_dirty = test_and_set_bit(EXTENT_BUFFER_DIRTY, &eb->bflags);
 
-       num_pages = num_extent_pages(eb->start, eb->len);
+       num_pages = num_extent_pages(eb);
        WARN_ON(atomic_read(&eb->refs) == 0);
        WARN_ON(!test_bit(EXTENT_BUFFER_TREE_REF, &eb->bflags));
 
@@ -5213,7 +5213,7 @@ void clear_extent_buffer_uptodate(struct extent_buffer *eb)
        unsigned long num_pages;
 
        clear_bit(EXTENT_BUFFER_UPTODATE, &eb->bflags);
-       num_pages = num_extent_pages(eb->start, eb->len);
+       num_pages = num_extent_pages(eb);
        for (i = 0; i < num_pages; i++) {
                page = eb->pages[i];
                if (page)
@@ -5228,7 +5228,7 @@ void set_extent_buffer_uptodate(struct extent_buffer *eb)
        unsigned long num_pages;
 
        set_bit(EXTENT_BUFFER_UPTODATE, &eb->bflags);
-       num_pages = num_extent_pages(eb->start, eb->len);
+       num_pages = num_extent_pages(eb);
        for (i = 0; i < num_pages; i++) {
                page = eb->pages[i];
                SetPageUptodate(page);
@@ -5252,7 +5252,7 @@ int read_extent_buffer_pages(struct extent_io_tree *tree,
        if (test_bit(EXTENT_BUFFER_UPTODATE, &eb->bflags))
                return 0;
 
-       num_pages = num_extent_pages(eb->start, eb->len);
+       num_pages = num_extent_pages(eb);
        for (i = 0; i < num_pages; i++) {
                page = eb->pages[i];
                if (wait == WAIT_NONE) {
@@ -5580,7 +5580,7 @@ void copy_extent_buffer_full(struct extent_buffer *dst,
 
        ASSERT(dst->len == src->len);
 
-       num_pages = num_extent_pages(dst->start, dst->len);
+       num_pages = num_extent_pages(dst);
        for (i = 0; i < num_pages; i++)
                copy_page(page_address(dst->pages[i]),
                                page_address(src->pages[i]));

diff --git a/fs/btrfs/extent_io.h b/fs/btrfs/extent_io.h
index 0bfd4ae..d8382a4 100644 (file)
--- a/fs/btrfs/extent_io.h
+++ b/fs/btrfs/extent_io.h
@@ -440,10 +440,10 @@ int read_extent_buffer_pages(struct extent_io_tree *tree,
                             int mirror_num);
 void wait_on_extent_buffer_writeback(struct extent_buffer *eb);
 
-static inline unsigned long num_extent_pages(u64 start, u64 len)
+static inline unsigned long num_extent_pages(const struct extent_buffer *eb)
 {
-       return ((start + len + PAGE_SIZE - 1) >> PAGE_SHIFT) -
-               (start >> PAGE_SHIFT);
+       return ((eb->start + eb->len + PAGE_SIZE - 1) >> PAGE_SHIFT) -
+               (eb->start >> PAGE_SHIFT);
 }
 
 static inline void extent_buffer_get(struct extent_buffer *eb)