]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
reiserfs: Convert grab_tail_page() to use a folio
authorMatthew Wilcox (Oracle) <willy@infradead.org>
Mon, 8 Jul 2024 15:06:01 +0000 (11:06 -0400)
committerChristian Brauner <brauner@kernel.org>
Wed, 7 Aug 2024 09:31:58 +0000 (11:31 +0200)
Removes a call to grab_cache_page() and a few hidden calls to
compound_head().

Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Christian Brauner <brauner@kernel.org>
fs/reiserfs/inode.c

index 9b43a81a6488ca2b0d8d5fd81832a518dd8c31db..2d558b2012eaa0fe5e250d7060f483fb4507a81a 100644 (file)
@@ -2178,7 +2178,7 @@ static int grab_tail_page(struct inode *inode,
        unsigned long offset = (inode->i_size) & (PAGE_SIZE - 1);
        struct buffer_head *bh;
        struct buffer_head *head;
-       struct page *page;
+       struct folio *folio;
        int error;
 
        /*
@@ -2190,20 +2190,20 @@ static int grab_tail_page(struct inode *inode,
        if ((offset & (blocksize - 1)) == 0) {
                return -ENOENT;
        }
-       page = grab_cache_page(inode->i_mapping, index);
-       error = -ENOMEM;
-       if (!page) {
-               goto out;
-       }
+       folio = __filemap_get_folio(inode->i_mapping, index,
+                       FGP_LOCK | FGP_ACCESSED | FGP_CREAT,
+                       mapping_gfp_mask(inode->i_mapping));
+       if (IS_ERR(folio))
+               return PTR_ERR(folio);
        /* start within the page of the last block in the file */
        start = (offset / blocksize) * blocksize;
 
-       error = __block_write_begin(page, start, offset - start,
+       error = __block_write_begin(&folio->page, start, offset - start,
                                    reiserfs_get_block_create_0);
        if (error)
                goto unlock;
 
-       head = page_buffers(page);
+       head = folio_buffers(folio);
        bh = head;
        do {
                if (pos >= start) {
@@ -2226,14 +2226,13 @@ static int grab_tail_page(struct inode *inode,
                goto unlock;
        }
        *bh_result = bh;
-       *page_result = page;
+       *page_result = &folio->page;
 
-out:
        return error;
 
 unlock:
-       unlock_page(page);
-       put_page(page);
+       folio_unlock(folio);
+       folio_put(folio);
        return error;
 }