Commit:     d0e671a932cb9c653b27393cec26aec012a8d97e
Parent:     fba2591bf4e418b6c3f9f8794c9dd8fe40ae7bd9
Author:     Dave Kleikamp <[EMAIL PROTECTED]>
AuthorDate: Wed Dec 20 15:55:35 2006 -0600
Committer:  Linus Torvalds <[EMAIL PROTECTED]>
CommitDate: Thu Dec 21 09:24:03 2006 -0800

    [PATCH] Fix JFS after clear_page_dirty() removal
    This patch removes some questionable code that attempted to make a
    no-longer-used page easier to reclaim.
    Calling metapage_writepage against such a page will not result in any
    I/O being performed, so removing this code shouldn't be a big deal.
    [ It's likely that we could have just replaced the "clear_page_dirty()"
      call with a call to "cancel_dirty_page()" instead, but in the
      meantime this is cleaner and simpler anyway, so unless there is some
      overriding reason (and Dave implies there isn't) I'll just use this
      patch as-is.                      - Linus ]
    Signed-off-by: Dave Kleikamp <[EMAIL PROTECTED]>
    Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>
 fs/jfs/jfs_metapage.c |   15 +--------------
 1 files changed, 1 insertions(+), 14 deletions(-)

diff --git a/fs/jfs/jfs_metapage.c b/fs/jfs/jfs_metapage.c
index b1a1c72..ceaf03b 100644
--- a/fs/jfs/jfs_metapage.c
+++ b/fs/jfs/jfs_metapage.c
@@ -764,22 +764,9 @@ void release_metapage(struct metapage * mp)
        } else if (mp->lsn)     /* discard_metapage doesn't remove it */
-#if MPS_PER_PAGE == 1
-       /*
-        * If we know this is the only thing in the page, we can throw
-        * the page out of the page cache.  If pages are larger, we
-        * don't want to do this.
-        */
-       /* Retest mp->count since we may have released page lock */
-       if (test_bit(META_discard, &mp->flag) && !mp->count) {
-               clear_page_dirty(page);
-               ClearPageUptodate(page);
-       }
        /* Try to keep metapages from using up too much memory */
        drop_metapage(page, mp);
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at

Reply via email to