]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
orangefs: Convert orangefs_write_end() to use a folio
authorMatthew Wilcox (Oracle) <willy@infradead.org>
Wed, 10 Jul 2024 21:06:19 +0000 (17:06 -0400)
committerChristian Brauner <brauner@kernel.org>
Wed, 7 Aug 2024 09:32:01 +0000 (11:32 +0200)
Convert the passed page to a folio and operate on that.
Replaces five calls to compound_head() with one.

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/orangefs/inode.c

index fdb9b65db1de9ffd1e3e9beef4169e6cd2167d9b..6595417f62b189be46e97b3e52fdcc5b6b431149 100644 (file)
@@ -367,7 +367,8 @@ okay:
 static int orangefs_write_end(struct file *file, struct address_space *mapping,
     loff_t pos, unsigned len, unsigned copied, struct page *page, void *fsdata)
 {
-       struct inode *inode = page->mapping->host;
+       struct folio *folio = page_folio(page);
+       struct inode *inode = folio->mapping->host;
        loff_t last_pos = pos + copied;
 
        /*
@@ -377,23 +378,23 @@ static int orangefs_write_end(struct file *file, struct address_space *mapping,
        if (last_pos > inode->i_size)
                i_size_write(inode, last_pos);
 
-       /* zero the stale part of the page if we did a short copy */
-       if (!PageUptodate(page)) {
+       /* zero the stale part of the folio if we did a short copy */
+       if (!folio_test_uptodate(folio)) {
                unsigned from = pos & (PAGE_SIZE - 1);
                if (copied < len) {
-                       zero_user(page, from + copied, len - copied);
+                       folio_zero_range(folio, from + copied, len - copied);
                }
                /* Set fully written pages uptodate. */
-               if (pos == page_offset(page) &&
+               if (pos == folio_pos(folio) &&
                    (len == PAGE_SIZE || pos + len == inode->i_size)) {
-                       zero_user_segment(page, from + copied, PAGE_SIZE);
-                       SetPageUptodate(page);
+                       folio_zero_segment(folio, from + copied, PAGE_SIZE);
+                       folio_mark_uptodate(folio);
                }
        }
 
-       set_page_dirty(page);
-       unlock_page(page);
-       put_page(page);
+       folio_mark_dirty(folio);
+       folio_unlock(folio);
+       folio_put(folio);
 
        mark_inode_dirty_sync(file_inode(file));
        return copied;