nfs: Remove writepage
authorMatthew Wilcox (Oracle) <willy@infradead.org>
Fri, 15 Dec 2023 20:47:07 +0000 (20:47 +0000)
committerAnna Schumaker <Anna.Schumaker@Netapp.com>
Thu, 4 Jan 2024 15:47:56 +0000 (10:47 -0500)
NFS already has writepages and migrate_folio, so it does not need to
implement writepage.  The writepage operation is deprecated as it leads
to worse performance under high memory pressure due to folios being
written out in LRU order rather than sequentially within a file.

Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
fs/nfs/file.c
fs/nfs/write.c
include/linux/nfs_fs.h

index 3f9768810427df609c74cf5a077e6748cc5019ce..e437de2a5ad490b475e12c922f0af2c3b9cb13c7 100644 (file)
@@ -558,7 +558,6 @@ const struct address_space_operations nfs_file_aops = {
        .read_folio = nfs_read_folio,
        .readahead = nfs_readahead,
        .dirty_folio = filemap_dirty_folio,
-       .writepage = nfs_writepage,
        .writepages = nfs_writepages,
        .write_begin = nfs_write_begin,
        .write_end = nfs_write_end,
index b664caea8b4e6704bc0b46e8388487999697b80b..0b4fdad511d924a5e397d8576cad3c5142fe191f 100644 (file)
@@ -680,17 +680,6 @@ static int nfs_writepage_locked(struct folio *folio,
        return err;
 }
 
-int nfs_writepage(struct page *page, struct writeback_control *wbc)
-{
-       struct folio *folio = page_folio(page);
-       int ret;
-
-       ret = nfs_writepage_locked(folio, wbc);
-       if (ret != AOP_WRITEPAGE_ACTIVATE)
-               unlock_page(page);
-       return ret;
-}
-
 static int nfs_writepages_callback(struct folio *folio,
                                   struct writeback_control *wbc, void *data)
 {
index 279262057a92582f81514e9d08bcfd57c57a19a0..f5ce7b1011461385968723826f4052f3f492a4af 100644 (file)
@@ -595,7 +595,6 @@ extern void nfs_complete_unlink(struct dentry *dentry, struct inode *);
  * linux/fs/nfs/write.c
  */
 extern int  nfs_congestion_kb;
-extern int  nfs_writepage(struct page *page, struct writeback_control *wbc);
 extern int  nfs_writepages(struct address_space *, struct writeback_control *);
 extern int  nfs_flush_incompatible(struct file *file, struct folio *folio);
 extern int  nfs_update_folio(struct file *file, struct folio *folio,