[RFC PATCH v2 14/15] mm: Add alloc-free handshake to trigger memory region compaction

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



We need a way to decide when to trigger the worker threads to perform
region evacuation/compaction. So the strategy used is as follows:

Alloc path of page allocator:
----------------------------

This accurately tracks the allocations and detects the first allocation
in a new region and notes down that region number. Performing compaction
rightaway is not going to be helpful because we need free pages in the
lower regions to be able to do that. And the page allocator allocated in
this region precisely because there was no memory available in lower regions.
So the alloc path just notes down the freshly used region's id.

Free path of page allocator:
---------------------------

When we enter this path, we know that some memory is being freed. Here we
check if the alloc path had noted down any region for compaction. If so,
we trigger the worker function that tries to compact that memory.

Also, we avoid any locking/synchronization overhead over this worker
function in the alloc/free path, by attaching appropriate semantics to the
available status flags etc, such that we won't need any special locking
around them.

Signed-off-by: Srivatsa S. Bhat <srivatsa.bhat@xxxxxxxxxxxxxxxxxx>
---

 mm/page_alloc.c |   61 +++++++++++++++++++++++++++++++++++++++++++++++++++----
 1 file changed, 57 insertions(+), 4 deletions(-)

diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index db7b892..675a435 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -631,6 +631,7 @@ static void add_to_freelist(struct page *page, struct free_list *free_list,
 	struct list_head *prev_region_list, *lru;
 	struct mem_region_list *region;
 	int region_id, prev_region_id;
+	struct mem_power_ctrl *mpc;
 
 	lru = &page->lru;
 	region_id = page_zone_region_id(page);
@@ -639,6 +640,17 @@ static void add_to_freelist(struct page *page, struct free_list *free_list,
 	region->nr_free++;
 	region->zone_region->nr_free += 1 << order;
 
+	/*
+	 * If the alloc path detected the usage of a new region, now is
+	 * the time to complete the handshake and queue a worker
+	 * to try compaction on that region.
+	 */
+	mpc = &page_zone(page)->mem_power_ctrl;
+	if (!is_mem_pwr_work_in_progress(mpc) && mpc->region) {
+		set_mem_pwr_work_in_progress(mpc);
+		queue_work(system_unbound_wq, &mpc->work);
+	}
+
 	if (region->page_block) {
 		list_add_tail(lru, region->page_block);
 		return;
@@ -696,7 +708,9 @@ static void rmqueue_del_from_freelist(struct page *page,
 {
 	struct list_head *lru = &page->lru;
 	struct mem_region_list *mr_list;
-	int region_id;
+	struct zone_mem_region *zmr;
+	struct mem_power_ctrl *mpc;
+	int region_id, mt;
 
 #ifdef CONFIG_DEBUG_PAGEALLOC
 	WARN((free_list->list.next != lru),
@@ -706,8 +720,27 @@ static void rmqueue_del_from_freelist(struct page *page,
 	list_del(lru);
 
 	/* Fastpath */
+	region_id = free_list->next_region - free_list->mr_list;
 	mr_list = free_list->next_region;
-	mr_list->zone_region->nr_free -= 1 << order;
+	zmr = mr_list->zone_region;
+	if (region_id != 0 && (zmr->nr_free == zmr->present_pages)) {
+		/*
+		 * This is the first alloc in this memory region. So try
+		 * compacting this region in the near future. Don't bother
+		 * if this is an unmovable/non-reclaimable allocation.
+		 * Also don't try compacting region 0 because its pointless.
+		 */
+		mt = get_freepage_migratetype(page);
+		if (is_migrate_cma(mt) || mt == MIGRATE_MOVABLE ||
+						mt == MIGRATE_RECLAIMABLE) {
+
+			mpc = &page_zone(page)->mem_power_ctrl;
+			if (!is_mem_pwr_work_in_progress(mpc))
+				mpc->region = zmr;
+		}
+	}
+
+	zmr->nr_free -= 1 << order;
 
 	if (--(mr_list->nr_free)) {
 
@@ -723,7 +756,6 @@ static void rmqueue_del_from_freelist(struct page *page,
 	 * in this freelist.
 	 */
 	free_list->next_region->page_block = NULL;
-	region_id = free_list->next_region - free_list->mr_list;
 	clear_region_bit(region_id, free_list);
 
 	/* Set 'next_region' to the new first region in the freelist. */
@@ -736,7 +768,9 @@ static void del_from_freelist(struct page *page, struct free_list *free_list,
 {
 	struct list_head *prev_page_lru, *lru, *p;
 	struct mem_region_list *region;
-	int region_id;
+	struct zone_mem_region *zmr;
+	struct mem_power_ctrl *mpc;
+	int region_id, mt;
 
 	lru = &page->lru;
 
@@ -746,6 +780,25 @@ static void del_from_freelist(struct page *page, struct free_list *free_list,
 
 	region_id = page_zone_region_id(page);
 	region = &free_list->mr_list[region_id];
+
+	zmr = region->zone_region;
+	if (region_id != 0 && (zmr->nr_free == zmr->present_pages)) {
+		/*
+		 * This is the first alloc in this memory region. So try
+		 * compacting this region in the near future. Don't bother
+		 * if this is an unmovable/non-reclaimable allocation.
+		 * Also don't try compacting region 0 because its pointless.
+		 */
+		mt = get_freepage_migratetype(page);
+		if (is_migrate_cma(mt) || mt == MIGRATE_MOVABLE ||
+						mt == MIGRATE_RECLAIMABLE) {
+
+			mpc = &page_zone(page)->mem_power_ctrl;
+			if (!is_mem_pwr_work_in_progress(mpc))
+				mpc->region = zmr;
+		}
+	}
+
 	region->nr_free--;
 	region->zone_region->nr_free -= 1 << order;
 

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]