For __process_pages_contig() and process_one_page(), to handle subpage
we only need to pass bytenr in and call subpage helpers to handle
dirty/error/writeback status.

Signed-off-by: Qu Wenruo <w...@suse.com>
---
 fs/btrfs/extent_io.c | 24 ++++++++++++++++--------
 1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c
index 94f8b3ffe6a7..f40e229960d7 100644
--- a/fs/btrfs/extent_io.c
+++ b/fs/btrfs/extent_io.c
@@ -1813,10 +1813,16 @@ bool btrfs_find_delalloc_range(struct extent_io_tree 
*tree, u64 *start,
  * Return -EGAIN if the we need to try again.
  * (For PAGE_LOCK case but got dirty page or page not belong to mapping)
  */
-static int process_one_page(struct address_space *mapping,
+static int process_one_page(struct btrfs_fs_info *fs_info,
+                           struct address_space *mapping,
                            struct page *page, struct page *locked_page,
-                           unsigned long page_ops)
+                           unsigned long page_ops, u64 start, u64 end)
 {
+       u32 len;
+
+       ASSERT(end + 1 - start != 0 && end + 1 - start < U32_MAX);
+       len = end + 1 - start;
+
        if (page_ops & PAGE_SET_ORDERED)
                SetPageOrdered(page);
 
@@ -1824,13 +1830,13 @@ static int process_one_page(struct address_space 
*mapping,
                return 1;
 
        if (page_ops & PAGE_SET_ERROR)
-               SetPageError(page);
+               btrfs_page_clamp_set_error(fs_info, page, start, len);
        if (page_ops & PAGE_START_WRITEBACK) {
-               clear_page_dirty_for_io(page);
-               set_page_writeback(page);
+               btrfs_page_clamp_clear_dirty(fs_info, page, start, len);
+               btrfs_page_clamp_set_writeback(fs_info, page, start, len);
        }
        if (page_ops & PAGE_END_WRITEBACK)
-               end_page_writeback(page);
+               btrfs_page_clamp_clear_writeback(fs_info, page, start, len);
        if (page_ops & PAGE_LOCK) {
                lock_page(page);
                if (!PageDirty(page) || page->mapping != mapping) {
@@ -1848,6 +1854,7 @@ static int __process_pages_contig(struct address_space 
*mapping,
                                  u64 start, u64 end, unsigned long page_ops,
                                  u64 *processed_end)
 {
+       struct btrfs_fs_info *fs_info = btrfs_sb(mapping->host->i_sb);
        pgoff_t start_index = start >> PAGE_SHIFT;
        pgoff_t end_index = end >> PAGE_SHIFT;
        pgoff_t index = start_index;
@@ -1884,8 +1891,9 @@ static int __process_pages_contig(struct address_space 
*mapping,
                for (i = 0; i < found_pages; i++) {
                        int process_ret;
 
-                       process_ret = process_one_page(mapping, pages[i],
-                                       locked_page, page_ops);
+                       process_ret = process_one_page(fs_info, mapping,
+                                       pages[i], locked_page, page_ops,
+                                       start, end);
                        if (process_ret < 0) {
                                for (; i < found_pages; i++)
                                        put_page(pages[i]);
-- 
2.31.1

Reply via email to