btrfs: open code folio_index() in btree_clear_folio_dirty_tag()
authorKairui Song <kasong@tencent.com>
Wed, 30 Apr 2025 18:10:48 +0000 (02:10 +0800)
committerDavid Sterba <dsterba@suse.com>
Fri, 2 May 2025 11:20:56 +0000 (13:20 +0200)
The folio_index() helper is only needed for mixed usage of page cache
and swap cache, for pure page cache usage, the caller can just use
folio->index instead.

It can't be a swap cache folio here.  Swap mapping may only call into fs
through 'swap_rw' but btrfs does not use that method for swap.

Reviewed-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Reviewed-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: Kairui Song <kasong@tencent.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/extent_io.c

index 8515c31..13bdd60 100644 (file)
@@ -3508,8 +3508,8 @@ static void btree_clear_folio_dirty_tag(struct folio *folio)
        ASSERT(folio_test_locked(folio));
        xa_lock_irq(&folio->mapping->i_pages);
        if (!folio_test_dirty(folio))
-               __xa_clear_mark(&folio->mapping->i_pages,
-                               folio_index(folio), PAGECACHE_TAG_DIRTY);
+               __xa_clear_mark(&folio->mapping->i_pages, folio->index,
+                               PAGECACHE_TAG_DIRTY);
        xa_unlock_irq(&folio->mapping->i_pages);
 }