diff options
| author |   Qu Wenruo <wqu@suse.com> | 2020-12-02 14:48:03 +0800 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2020-12-09 19:16:10 +0100 |
| commit | 4a3dc93843dd6ee17c68231d6a90c76231cb65fc (patch) | |
| tree | 03e45e4b704f8b4407936b8f9fe5acce487611e5 /fs/btrfs | |
| parent | btrfs: don't allow tree block to cross page boundary for subpage support (diff) | |
| download | linux-dev-4a3dc93843dd6ee17c68231d6a90c76231cb65fc.tar.xz linux-dev-4a3dc93843dd6ee17c68231d6a90c76231cb65fc.zip | |
btrfs: update num_extent_pages to support subpage sized extent buffer
For subpage sized extent buffer, we have ensured no extent buffer will
cross page boundary, thus we would only need one page for any extent
buffer.
Update function num_extent_pages to handle such case. Now
num_extent_pages() returns 1 for subpage sized extent buffer.
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: Qu Wenruo <wqu@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/extent_io.h | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/fs/btrfs/extent_io.h b/fs/btrfs/extent_io.h index 77f2211550e3..19221095c635 100644 --- a/fs/btrfs/extent_io.h +++ b/fs/btrfs/extent_io.h @@ -203,8 +203,14 @@ void btrfs_readahead_node_child(struct extent_buffer *node, int slot); static inline int num_extent_pages(const struct extent_buffer *eb) { - return (round_up(eb->start + eb->len, PAGE_SIZE) >> PAGE_SHIFT) - - (eb->start >> PAGE_SHIFT); + /* + * For sectorsize == PAGE_SIZE case, since nodesize is always aligned to + * sectorsize, it's just eb->len >> PAGE_SHIFT. + * + * For sectorsize < PAGE_SIZE case, we could have nodesize < PAGE_SIZE, + * thus have to ensure we get at least one page. + */ + return (eb->len >> PAGE_SHIFT) ?: 1; } static inline int extent_buffer_uptodate(const struct extent_buffer *eb) |