From: Christoph Lameter Remove two unnecessary PageSwapCache checks. The page refcount is raised and therefore page migration cannot occur in both functions. Signed-off-by: Christoph Lameter Cc: Hugh Dickins Signed-off-by: Andrew Morton --- mm/shmem.c | 8 -------- mm/swapfile.c | 6 ------ 2 files changed, 14 deletions(-) diff -puN mm/shmem.c~migration-remove-unnecessary-pageswapcache-checks mm/shmem.c --- devel/mm/shmem.c~migration-remove-unnecessary-pageswapcache-checks 2006-04-22 01:39:27.000000000 -0700 +++ devel-akpm/mm/shmem.c 2006-04-22 01:39:27.000000000 -0700 @@ -1081,14 +1081,6 @@ repeat: page_cache_release(swappage); goto repeat; } - if (!PageSwapCache(swappage)) { - /* Page migration has occured */ - shmem_swp_unmap(entry); - spin_unlock(&info->lock); - unlock_page(swappage); - page_cache_release(swappage); - goto repeat; - } if (PageWriteback(swappage)) { shmem_swp_unmap(entry); spin_unlock(&info->lock); diff -puN mm/swapfile.c~migration-remove-unnecessary-pageswapcache-checks mm/swapfile.c --- devel/mm/swapfile.c~migration-remove-unnecessary-pageswapcache-checks 2006-04-22 01:39:27.000000000 -0700 +++ devel-akpm/mm/swapfile.c 2006-04-22 01:39:27.000000000 -0700 @@ -751,12 +751,6 @@ again: wait_on_page_locked(page); wait_on_page_writeback(page); lock_page(page); - if (!PageSwapCache(page)) { - /* Page migration has occured */ - unlock_page(page); - page_cache_release(page); - goto again; - } wait_on_page_writeback(page); /* _