Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ec0304da authored by Mel Gorman's avatar Mel Gorman Committed by Vinayak Menon
Browse files

mm: vmscan: treat pages marked for immediate reclaim as zone congestion 



Currently a zone will only be marked congested if the underlying BDI is
congested but if dirty pages are spread across zones it is possible that
an individual zone is full of dirty pages without being congested.  The
impact is that zone gets scanned very quickly potentially reclaiming
really clean pages.  This patch treats pages marked for immediate
reclaim as congested for the purposes of marking a zone ZONE_CONGESTED
and stalling in wait_iff_congested.

Signed-off-by: default avatarMel Gorman <mgorman@suse.de>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: Michal Hocko <mhocko@suse.cz>
Cc: Rik van Riel <riel@redhat.com>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Jiri Slaby <jslaby@suse.cz>
Cc: Valdis Kletnieks <Valdis.Kletnieks@vt.edu>
Cc: Zlatko Calusic <zcalusic@bitsync.net>
Cc: dormando <dormando@rydia.net>
Cc: Trond Myklebust <trond.myklebust@fys.uio.no>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
Git-commit: d04e8acd03e5c3421ef18e3da7bc88d56179ca42
Git-repo: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git


Change-Id: I321615bb32c4efe5889df9ce6482c825d7a816e6
Signed-off-by: default avatarVinayak Menon <vinmenon@codeaurora.org>
parent f2d250b8

mm/vmscan.c

+8 −2
Original line number Diff line number Diff line
@@ -849,9 +849,15 @@ static unsigned long shrink_page_list(struct list_head *page_list, 
		if (dirty && !writeback)

			nr_unqueued_dirty++;



		/* Treat this page as congested if underlying BDI is */

		/*

		 * Treat this page as congested if the underlying BDI is or if

		 * pages are cycling through the LRU so quickly that the

		 * pages marked for immediate reclaim are making it to the

		 * end of the LRU a second time.

		 */

		mapping = page_mapping(page);

		if (mapping && bdi_write_congested(mapping->backing_dev_info))

		if ((mapping && bdi_write_congested(mapping->backing_dev_info)) ||

		    (writeback && PageReclaim(page)))

			nr_congested++;



		/*