Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/shaggy/jfs-2.6

* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/shaggy/jfs-2.6:
  JFS: Fix multiple errors in metapage_releasepage

1 files changed, 5 insertions, 15 deletions

diff --git a/fs/jfs/jfs_metapage.c b/fs/jfs/jfs_metapage.c
index f28696f235c4..2b220dd6b4e7 100644
--- a/fs/jfs/jfs_metapage.c
+++ b/fs/jfs/jfs_metapage.c
@@ -542,7 +542,7 @@ add_failed:
 static int metapage_releasepage(struct page *page, gfp_t gfp_mask)
 {
         struct metapage *mp;
-        int busy = 0;
+        int ret = 1;
         unsigned int offset;
 
         for (offset = 0; offset < PAGE_CACHE_SIZE; offset += PSIZE) {
@@ -552,30 +552,20 @@ static int metapage_releasepage(struct page *page, gfp_t gfp_mask)
                         continue;
 
                 jfs_info("metapage_releasepage: mp = 0x%p", mp);
-                if (mp->count || mp->nohomeok) {
+                if (mp->count || mp->nohomeok ||
+                    test_bit(META_dirty, &mp->flag)) {
                         jfs_info("count = %ld, nohomeok = %d", mp->count,
                                  mp->nohomeok);
-                        busy = 1;
+                        ret = 0;
                         continue;
                 }
-                wait_on_page_writeback(page);
-                //WARN_ON(test_bit(META_dirty, &mp->flag));
-                if (test_bit(META_dirty, &mp->flag)) {
-                        dump_mem("dirty mp in metapage_releasepage", mp,
-                                 sizeof(struct metapage));
-                        dump_mem("page", page, sizeof(struct page));
-                        dump_stack();
-                }
                 if (mp->lsn)
                         remove_from_logsync(mp);
                 remove_metapage(page, mp);
                 INCREMENT(mpStat.pagefree);
                 free_metapage(mp);
         }
-        if (busy)
-                return -1;
-
-        return 0;
+        return ret;
 }
 
 static void metapage_invalidatepage(struct page *page, unsigned long offset)