Btrfs: Make sure pages are dirty before doing delalloc for them
authorChris Mason <chris.mason@oracle.com>
Mon, 10 Nov 2008 12:31:30 +0000 (07:31 -0500)
committerChris Mason <chris.mason@oracle.com>
Mon, 10 Nov 2008 12:31:30 +0000 (07:31 -0500)
This adds a PageDirty check to the writeback path that locks pages
for delalloc.  If a page wasn't dirty at this point, it is in the
process of being truncated away.

Signed-off-by: Chris Mason <chris.mason@oracle.com>
fs/btrfs/extent_io.c

index ad75a9c..69ea096 100644 (file)
@@ -1205,7 +1205,8 @@ static noinline int lock_delalloc_pages(struct inode *inode,
                         */
                        if (pages[i] != locked_page) {
                                lock_page(pages[i]);
-                               if (pages[i]->mapping != inode->i_mapping) {
+                               if (!PageDirty(pages[i]) ||
+                                   pages[i]->mapping != inode->i_mapping) {
                                        ret = -EAGAIN;
                                        unlock_page(pages[i]);
                                        page_cache_release(pages[i]);