nfs: Can call nfs_clear_page_commit() instead

Tom Haynes · trondmypd · commit 487b9b8afde6 · 2015-02-13T17:47:26.000-05:00
Signed-off-by: Tom Haynes &lt;loghyr@primarydata.com&gt;
Signed-off-by: Trond Myklebust &lt;trond.myklebust@primarydata.com&gt;
diff --git a/fs/nfs/write.c b/fs/nfs/write.c
@@ -1600,11 +1600,8 @@ void nfs_retry_commit(struct list_head *page_list,
 		req = nfs_list_entry(page_list->next);
 		nfs_list_remove_request(req);
 		nfs_mark_request_commit(req, lseg, cinfo, ds_commit_idx);
-		if (!cinfo->dreq) {
-			dec_zone_page_state(req->wb_page, NR_UNSTABLE_NFS);
-			dec_bdi_stat(inode_to_bdi(page_file_mapping(req->wb_page)->host),
-				     BDI_RECLAIMABLE);
-		}
+		if (!cinfo->dreq)
+			nfs_clear_page_commit(req->wb_page);
 		nfs_unlock_and_release_request(req);
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -1600,11 +1600,8 @@ void nfs_retry_commit(struct list_head *page_list,`
`1600`	`1600`	`req = nfs_list_entry(page_list->next);`
`1601`	`1601`	`nfs_list_remove_request(req);`
`1602`	`1602`	`nfs_mark_request_commit(req, lseg, cinfo, ds_commit_idx);`
`1603`		`- if (!cinfo->dreq) {`
`1604`		`- dec_zone_page_state(req->wb_page, NR_UNSTABLE_NFS);`
`1605`		`- dec_bdi_stat(inode_to_bdi(page_file_mapping(req->wb_page)->host),`
`1606`		`- BDI_RECLAIMABLE);`
`1607`		`- }`
	`1603`	`+ if (!cinfo->dreq)`
	`1604`	`+ nfs_clear_page_commit(req->wb_page);`
`1608`	`1605`	`nfs_unlock_and_release_request(req);`
`1609`	`1606`	`}`
`1610`	`1607`	`}`