NFS: Add nfs_set_page_dirty()
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Tue, 5 Dec 2006 05:35:40 +0000 (00:35 -0500)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Wed, 6 Dec 2006 15:46:38 +0000 (10:46 -0500)
We will want to allow nfs_writepage() to distinguish between pages that
have been marked as dirty by the VM, and those that have been marked as
dirty by nfs_updatepage().
In the former case, the entire page will want to be written out, and so any
requests that were pending need to be flushed out first.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/file.c
fs/nfs/write.c
include/linux/nfs_fs.h
include/linux/nfs_page.h

index 143a19037ce8c0b3ba301f5f135975f95b912d4c..c2fe3bd83ab17bfa746be49e7406e9d3cab33ca9 100644 (file)
@@ -331,7 +331,7 @@ static int nfs_release_page(struct page *page, gfp_t gfp)
 const struct address_space_operations nfs_file_aops = {
        .readpage = nfs_readpage,
        .readpages = nfs_readpages,
-       .set_page_dirty = __set_page_dirty_nobuffers,
+       .set_page_dirty = nfs_set_page_dirty,
        .writepage = nfs_writepage,
        .writepages = nfs_writepages,
        .prepare_write = nfs_prepare_write,
index f0720b544b1260b1991f2adbe050936c5f945ca5..266fea71cfc01bcd1e79894acb9ab4205e31a47a 100644 (file)
@@ -251,16 +251,23 @@ int nfs_writepage(struct page *page, struct writeback_control *wbc)
 {
        struct nfs_open_context *ctx;
        struct inode *inode = page->mapping->host;
+       struct nfs_page *req;
        unsigned offset;
-       int err;
+       int err = 0;
 
        nfs_inc_stats(inode, NFSIOS_VFSWRITEPAGE);
        nfs_add_stats(inode, NFSIOS_WRITEPAGES, 1);
 
-       /* Ensure we've flushed out any previous writes */
-       nfs_wb_page_priority(inode, page, wb_priority(wbc));
+       req = nfs_page_find_request(page);
+       if (req != NULL) {
+               int flushme = test_bit(PG_NEED_FLUSH, &req->wb_flags);
+               nfs_release_request(req);
+               if (!flushme)
+                       goto out;
+               /* Ensure we've flushed out the invalid write */
+               nfs_wb_page_priority(inode, page, wb_priority(wbc));
+       }
 
-       err = 0;
        offset = nfs_page_length(page);
        if (!offset)
                goto out;
@@ -655,7 +662,7 @@ int nfs_flush_incompatible(struct file *file, struct page *page)
 {
        struct nfs_open_context *ctx = (struct nfs_open_context *)file->private_data;
        struct nfs_page *req;
-       int             status = 0;
+       int do_flush, status;
        /*
         * Look for a request corresponding to this page. If there
         * is one, and it belongs to another file, we flush it out
@@ -664,15 +671,18 @@ int nfs_flush_incompatible(struct file *file, struct page *page)
         * Also do the same if we find a request from an existing
         * dropped page.
         */
-       req = nfs_page_find_request(page);
-       if (req != NULL) {
-               int do_flush = req->wb_page != page || req->wb_context != ctx;
-
+       do {
+               req = nfs_page_find_request(page);
+               if (req == NULL)
+                       return 0;
+               do_flush = req->wb_page != page || req->wb_context != ctx
+                       || test_bit(PG_NEED_FLUSH, &req->wb_flags);
                nfs_release_request(req);
-               if (do_flush)
-                       status = nfs_wb_page(page->mapping->host, page);
-       }
-       return (status < 0) ? status : 0;
+               if (!do_flush)
+                       return 0;
+               status = nfs_wb_page(page->mapping->host, page);
+       } while (status == 0);
+       return status;
 }
 
 /*
@@ -1437,6 +1447,19 @@ int nfs_wb_page(struct inode *inode, struct page* page)
        return nfs_wb_page_priority(inode, page, 0);
 }
 
+int nfs_set_page_dirty(struct page *page)
+{
+       struct nfs_page *req;
+
+       req = nfs_page_find_request(page);
+       if (req != NULL) {
+               /* Mark any existing write requests for flushing */
+               set_bit(PG_NEED_FLUSH, &req->wb_flags);
+               nfs_release_request(req);
+       }
+       return __set_page_dirty_nobuffers(page);
+}
+
 
 int __init nfs_init_writepagecache(void)
 {
index f2ec9be1e22f48dc1e71a0b20d04bcdb8ccec8c4..1b38c2085a5304ca34e12d61eecbf17b6ef98ce3 100644 (file)
@@ -427,6 +427,7 @@ extern int  nfs_flush_incompatible(struct file *file, struct page *page);
 extern int  nfs_updatepage(struct file *, struct page *, unsigned int, unsigned int);
 extern int nfs_writeback_done(struct rpc_task *, struct nfs_write_data *);
 extern void nfs_writedata_release(void *);
+extern int nfs_set_page_dirty(struct page *);
 
 /*
  * Try to write back everything synchronously (but check the
index 38aa15fea6383c020970a28aaec5db32e53e4247..d111be6391409a194602a99723842ff47c2c564b 100644 (file)
@@ -30,6 +30,7 @@
 #define PG_BUSY                        0
 #define PG_NEED_COMMIT         1
 #define PG_NEED_RESCHED                2
+#define PG_NEED_FLUSH          3
 
 struct nfs_inode;
 struct nfs_page {