Commit 5241060e authored by Trond Myklebust's avatar Trond Myklebust Committed by Anna Schumaker

NFS: Convert the function nfs_wb_page() to use folios

Convert to use the folio functions, but pass the struct page to
nfs_writepage_locked() for now.
Signed-off-by: default avatarTrond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: default avatarAnna Schumaker <Anna.Schumaker@Netapp.com>
parent ab75bff1
...@@ -2069,13 +2069,18 @@ int nfs_wb_folio_cancel(struct inode *inode, struct folio *folio) ...@@ -2069,13 +2069,18 @@ int nfs_wb_folio_cancel(struct inode *inode, struct folio *folio)
return ret; return ret;
} }
/* /**
* Write back all requests on one page - we do this before reading it. * nfs_wb_folio - Write back all requests on one page
* @inode: pointer to page
* @folio: pointer to folio
*
* Assumes that the folio has been locked by the caller, and will
* not unlock it.
*/ */
int nfs_wb_page(struct inode *inode, struct page *page) int nfs_wb_folio(struct inode *inode, struct folio *folio)
{ {
loff_t range_start = page_file_offset(page); loff_t range_start = folio_file_pos(folio);
loff_t range_end = range_start + (loff_t)(PAGE_SIZE - 1); loff_t range_end = range_start + (loff_t)folio_size(folio) - 1;
struct writeback_control wbc = { struct writeback_control wbc = {
.sync_mode = WB_SYNC_ALL, .sync_mode = WB_SYNC_ALL,
.nr_to_write = 0, .nr_to_write = 0,
...@@ -2087,15 +2092,15 @@ int nfs_wb_page(struct inode *inode, struct page *page) ...@@ -2087,15 +2092,15 @@ int nfs_wb_page(struct inode *inode, struct page *page)
trace_nfs_writeback_page_enter(inode); trace_nfs_writeback_page_enter(inode);
for (;;) { for (;;) {
wait_on_page_writeback(page); folio_wait_writeback(folio);
if (clear_page_dirty_for_io(page)) { if (folio_clear_dirty_for_io(folio)) {
ret = nfs_writepage_locked(page, &wbc); ret = nfs_writepage_locked(&folio->page, &wbc);
if (ret < 0) if (ret < 0)
goto out_error; goto out_error;
continue; continue;
} }
ret = 0; ret = 0;
if (!PagePrivate(page)) if (!folio_test_private(folio))
break; break;
ret = nfs_commit_inode(inode, FLUSH_SYNC); ret = nfs_commit_inode(inode, FLUSH_SYNC);
if (ret < 0) if (ret < 0)
...@@ -2106,17 +2111,9 @@ int nfs_wb_page(struct inode *inode, struct page *page) ...@@ -2106,17 +2111,9 @@ int nfs_wb_page(struct inode *inode, struct page *page)
return ret; return ret;
} }
/** int nfs_wb_page(struct inode *inode, struct page *page)
* nfs_wb_folio - Write back all requests on one page
* @inode: pointer to page
* @folio: pointer to folio
*
* Assumes that the folio has been locked by the caller, and will
* not unlock it.
*/
int nfs_wb_folio(struct inode *inode, struct folio *folio)
{ {
return nfs_wb_page(inode, &folio->page); return nfs_wb_folio(inode, page_folio(page));
} }
#ifdef CONFIG_MIGRATION #ifdef CONFIG_MIGRATION
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment