gfs2: Remove __gfs2_writepage()
authorMatthew Wilcox (Oracle) <willy@infradead.org>
Fri, 19 Jul 2024 17:51:02 +0000 (18:51 +0100)
committerAndreas Gruenbacher <agruenba@redhat.com>
Mon, 2 Sep 2024 12:46:29 +0000 (14:46 +0200)
Call aops->writepages() instead of using write_cache_pages() to call
aops->writepage.  Change the handling of -ENODATA to not set the
persistent error on the block device.

Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
fs/gfs2/log.c

index 6ee6013fb8253679f937e7e1faa9c5aecf3e4c5a..f9c5089783d24c605e1e7d16777be663b4e26172 100644 (file)
@@ -80,15 +80,6 @@ void gfs2_remove_from_ail(struct gfs2_bufdata *bd)
        brelse(bd->bd_bh);
 }
 
-static int __gfs2_writepage(struct folio *folio, struct writeback_control *wbc,
-                      void *data)
-{
-       struct address_space *mapping = data;
-       int ret = mapping->a_ops->writepage(&folio->page, wbc);
-       mapping_set_error(mapping, ret);
-       return ret;
-}
-
 /**
  * gfs2_ail1_start_one - Start I/O on a transaction
  * @sdp: The superblock
@@ -140,7 +131,7 @@ __acquires(&sdp->sd_ail_lock)
                if (!mapping)
                        continue;
                spin_unlock(&sdp->sd_ail_lock);
-               ret = write_cache_pages(mapping, wbc, __gfs2_writepage, mapping);
+               ret = mapping->a_ops->writepages(mapping, wbc);
                if (need_resched()) {
                        blk_finish_plug(plug);
                        cond_resched();
@@ -149,6 +140,7 @@ __acquires(&sdp->sd_ail_lock)
                spin_lock(&sdp->sd_ail_lock);
                if (ret == -ENODATA) /* if a jdata write into a new hole */
                        ret = 0; /* ignore it */
+               mapping_set_error(mapping, ret);
                if (ret || wbc->nr_to_write <= 0)
                        break;
                return -EBUSY;