linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] mm: compaction: push isolate search base of compact control one pfn ahead
@ 2011-11-24 12:38 Hillf Danton
  2011-11-24 14:03 ` Andrea Arcangeli
  0 siblings, 1 reply; 2+ messages in thread
From: Hillf Danton @ 2011-11-24 12:38 UTC (permalink / raw)
  To: Mel Gorman; +Cc: Andrea Arcangeli, Andrew Morton, linux-mm, LKML

After isolated the current pfn will no longer be scanned and isolated if the
next round is necessary, so push the isolate_migratepages search base of the
given compact_control one step ahead.

Signed-off-by: Hillf Danton <dhillf@gmail.com>
---

--- a/mm/compaction.c	Thu Nov 24 20:23:28 2011
+++ b/mm/compaction.c	Thu Nov 24 20:32:41 2011
@@ -365,8 +365,10 @@ static isolate_migrate_t isolate_migrate
 		nr_isolated++;

 		/* Avoid isolating too much */
-		if (cc->nr_migratepages == COMPACT_CLUSTER_MAX)
+		if (cc->nr_migratepages == COMPACT_CLUSTER_MAX) {
+			++low_pfn;
 			break;
+		}
 	}

 	acct_isolated(zone, cc);

--
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/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] mm: compaction: push isolate search base of compact control one pfn ahead
  2011-11-24 12:38 [PATCH] mm: compaction: push isolate search base of compact control one pfn ahead Hillf Danton
@ 2011-11-24 14:03 ` Andrea Arcangeli
  0 siblings, 0 replies; 2+ messages in thread
From: Andrea Arcangeli @ 2011-11-24 14:03 UTC (permalink / raw)
  To: Hillf Danton; +Cc: Mel Gorman, Andrew Morton, linux-mm, LKML

On Thu, Nov 24, 2011 at 08:38:14PM +0800, Hillf Danton wrote:
> After isolated the current pfn will no longer be scanned and isolated if the
> next round is necessary, so push the isolate_migratepages search base of the
> given compact_control one step ahead.

Reviewed-by: Andrea Arcangeli <aarcange@redhat.com>

Actually I did this change in my tree already while playing with
compaction last few days so I didn't actually need to review.

--
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/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-11-24 14:03 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-11-24 12:38 [PATCH] mm: compaction: push isolate search base of compact control one pfn ahead Hillf Danton
2011-11-24 14:03 ` Andrea Arcangeli

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).