From: Alexander Zarochentsev From: Alexander Zarochentsev this patch addresses the following reiser4_drop_page problem : > drop_page() is a worry. Why _does_ reiser4 need to remove pages from > pagecache? That isn't a filesystem function. Signed-off-by: Alexander Zarochentsev Cc: Hans Reiser Signed-off-by: Andrew Morton --- fs/reiser4/page_cache.c | 8 +------- 1 files changed, 1 insertion(+), 7 deletions(-) diff -puN fs/reiser4/page_cache.c~reiser4-reiser4_drop_page-dont-call-remove_from_page_cache fs/reiser4/page_cache.c --- a/fs/reiser4/page_cache.c~reiser4-reiser4_drop_page-dont-call-remove_from_page_cache +++ a/fs/reiser4/page_cache.c @@ -261,7 +261,6 @@ void done_formatted_fake(struct super_bl sinfo = get_super_private_nocheck(super); if (sinfo->fake != NULL) { - assert("vs-1426", sinfo->fake->i_data.nrpages == 0); iput(sinfo->fake); sinfo->fake = NULL; } @@ -589,12 +588,7 @@ void drop_page(struct page *page) #if defined(PG_skipped) ClearPageSkipped(page); #endif - if (page->mapping != NULL) { - remove_from_page_cache(page); - unlock_page(page); - page_cache_release(page); - } else - unlock_page(page); + unlock_page(page); } /* this is called by truncate_jnodes_range which in its turn is always called _