diff options
author | Namhyung Kim <namhyung@gmail.com> | 2011-03-22 19:32:45 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-03-22 20:44:02 -0400 |
commit | 1d16871d8c96deadc5f9753b6b096074f2cbcbe1 (patch) | |
tree | 9f3f0c5a60e7a5bf4bfa9f2bd4c9492db36cecfe /mm/page_alloc.c | |
parent | e64a782fec684c29a8204c51b3cb554dce588592 (diff) |
mm: batch-free pcp list if possible
free_pcppages_bulk() frees pages from pcp lists in a round-robin fashion
by keeping batch_free counter. But it doesn't need to spin if there is
only one non-empty list. This can be checked by batch_free ==
MIGRATE_PCPTYPES.
[akpm@linux-foundation.org: fix comment]
Signed-off-by: Namhyung Kim <namhyung@gmail.com>
Acked-by: Johannes Weiner <hannes@cmpxchg.org>
Cc: Mel Gorman <mel@csn.ul.ie>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/page_alloc.c')
-rw-r--r-- | mm/page_alloc.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/mm/page_alloc.c b/mm/page_alloc.c index 36a168e383b5..426056aff12a 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c | |||
@@ -614,6 +614,10 @@ static void free_pcppages_bulk(struct zone *zone, int count, | |||
614 | list = &pcp->lists[migratetype]; | 614 | list = &pcp->lists[migratetype]; |
615 | } while (list_empty(list)); | 615 | } while (list_empty(list)); |
616 | 616 | ||
617 | /* This is the only non-empty list. Free them all. */ | ||
618 | if (batch_free == MIGRATE_PCPTYPES) | ||
619 | batch_free = to_free; | ||
620 | |||
617 | do { | 621 | do { |
618 | page = list_entry(list->prev, struct page, lru); | 622 | page = list_entry(list->prev, struct page, lru); |
619 | /* must delete as __free_one_page list manipulates */ | 623 | /* must delete as __free_one_page list manipulates */ |