From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail138.messagelabs.com (mail138.messagelabs.com [216.82.249.35]) by kanga.kvack.org (Postfix) with ESMTP id 7A8DE6B007B for ; Wed, 15 Sep 2010 08:27:58 -0400 (EDT) From: Mel Gorman Subject: [PATCH 3/8] vmscan: Synchronous lumpy reclaim should not call congestion_wait() Date: Wed, 15 Sep 2010 13:27:46 +0100 Message-Id: <1284553671-31574-4-git-send-email-mel@csn.ul.ie> In-Reply-To: <1284553671-31574-1-git-send-email-mel@csn.ul.ie> References: <1284553671-31574-1-git-send-email-mel@csn.ul.ie> Sender: owner-linux-mm@kvack.org To: Andrew Morton Cc: linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, Linux Kernel List , Johannes Weiner , Minchan Kim , Wu Fengguang , KAMEZAWA Hiroyuki , KOSAKI Motohiro , Mel Gorman List-ID: From: KOSAKI Motohiro congestion_wait() mean "waiting queue congestion is cleared". However, synchronous lumpy reclaim does not need this congestion_wait() as shrink_page_list(PAGEOUT_IO_SYNC) uses wait_on_page_writeback() and it provides the necessary waiting. Signed-off-by: KOSAKI Motohiro Signed-off-by: Mel Gorman Reviewed-by: Minchan Kim Reviewed-by: Johannes Weiner Reviewed-by: Wu Fengguang Reviewed-by: KAMEZAWA Hiroyuki --- mm/vmscan.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/mm/vmscan.c b/mm/vmscan.c index 652650f..e8b5224 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -1341,8 +1341,6 @@ shrink_inactive_list(unsigned long nr_to_scan, struct zone *zone, /* Check if we should syncronously wait for writeback */ if (should_reclaim_stall(nr_taken, nr_reclaimed, priority, sc)) { - congestion_wait(BLK_RW_ASYNC, HZ/10); - /* * The attempt at page out may have made some * of the pages active, mark them inactive again. -- 1.7.1 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org