diff options
| author | Wang Sheng-Hui <shhuiw@gmail.com> | 2012-04-06 06:35:31 (GMT) |
|---|---|---|
| committer | Josef Bacik <josef@redhat.com> | 2012-05-11 14:56:38 (GMT) |
| commit | 39bab87ba6f4d8cce2b70c19e60233ad8030d7b4 (patch) | |
| tree | 000084281b76183533122fd2db482d3dc2376237 /fs/btrfs/extent_io.c | |
| parent | 1b303fc0545b4bfbb2b8a69eec89e6f077f69b56 (diff) | |
| download | linux-fsl-qoriq-39bab87ba6f4d8cce2b70c19e60233ad8030d7b4.tar.xz | |
Btrfs: fix btrfs_release_extent_buffer_page with the right usage of num_extent_pages
num_extent_pages returns the number of pages in the specific range, not
the index of the last page in the eb range.
btrfs_release_extent_buffer_page is called with start_idx set 0 in current
codes, so it's not a problem yet. But the logic is indeed wrong.
Fix it here.
Signed-off-by: Wang Sheng-Hui <shhuiw@gmail.com>
Diffstat (limited to 'fs/btrfs/extent_io.c')
| -rw-r--r-- | fs/btrfs/extent_io.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c index aeb98ce..455daec 100644 --- a/fs/btrfs/extent_io.c +++ b/fs/btrfs/extent_io.c @@ -3981,11 +3981,13 @@ static void btrfs_release_extent_buffer_page(struct extent_buffer *eb, unsigned long start_idx) { unsigned long index; + unsigned long num_pages; struct page *page; BUG_ON(extent_buffer_under_io(eb)); - index = num_extent_pages(eb->start, eb->len); + num_pages = num_extent_pages(eb->start, eb->len); + index = start_idx + num_pages; if (start_idx >= index) return; |