Unionfs: minor cleanup in writepage
authorHugh Dickins <hugh@veritas.com>
Tue, 27 Nov 2007 00:53:53 +0000 (19:53 -0500)
committerErez Zadok <ezk@cs.sunysb.edu>
Tue, 27 Nov 2007 00:53:53 +0000 (19:53 -0500)
Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>
fs/unionfs/mmap.c

index 95c3fcb6156e0dbaa3d07ad085603aeee941739f..9fa5751b804ed0348722c07b880da3e046058440 100644 (file)
@@ -83,6 +83,7 @@ static int unionfs_writepage(struct page *page, struct writeback_control *wbc)
        copy_highpage(lower_page, page);
        flush_dcache_page(lower_page);
        SetPageUptodate(lower_page);
+       set_page_dirty(lower_page);
 
        /*
         * Call lower writepage (expects locked page).  However, if we are
@@ -93,13 +94,11 @@ static int unionfs_writepage(struct page *page, struct writeback_control *wbc)
         * success.
         */
        if (wbc->for_reclaim) {
-               set_page_dirty(lower_page);
                unlock_page(lower_page);
                goto out_release;
        }
 
        BUG_ON(!lower_mapping->a_ops->writepage);
-       set_page_dirty(lower_page);
        clear_page_dirty_for_io(lower_page); /* emulate VFS behavior */
        err = lower_mapping->a_ops->writepage(lower_page, wbc);
        if (err < 0)