Home » Mailing list archives » linux-kernel » 2007 » April » 20

[PATCH -mm] fix undefined symbol if CONFIG_PAGE_GROUP_BY_MOBILITY not set

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Pierre Peiffer <Pierre.Peiffer@...>
To: Mel Gorman <mel@...>, Andy Whitcroft <apw@...>
Cc: LKML <linux-kernel@...>
Subject: [PATCH -mm] fix undefined symbol if CONFIG_PAGE_GROUP_BY_MOBILITY not set
Date: Friday, April 20, 2007 - 4:07 am

Hi,

This is a fix against the patch
do-not-group-pages-by-mobility-type-on-low-memory-systems.patch (include in -mm
tree):
The error "page_group_by_mobility_disabled undefinied" occured if
CONFIG_PAGE_GROUP_BY_MOBILITY is not set.

Signed-off-by: Pierre Peiffer <pierre.peiffer@bull.net>

---
 mm/page_alloc.c |    5 +++++
 1 file changed, 5 insertions(+)

Index: b/mm/page_alloc.c
===================================================================
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -2297,6 +2297,7 @@ void __meminit build_all_zonelists(void)
 	}
 	vm_total_pages = nr_free_pagecache_pages();
 
+#ifdef CONFIG_PAGE_GROUP_BY_MOBILITY
 	/*
 	 * Disable grouping by mobility if the number of pages in the
 	 * system is too low to allow the mechanism to work. It would be
@@ -2313,6 +2314,10 @@ void __meminit build_all_zonelists(void)
 			num_online_nodes(),
 			page_group_by_mobility_disabled ? "off" : "on",
 			vm_total_pages);
+#else
+	printk("Built %i zonelists.  Total pages: %ld\n",
+	       num_online_nodes(), vm_total_pages);
+#endif
 }
 
 /*


-- 
Pierre
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH -mm] fix undefined symbol if CONFIG_PAGE_GROUP_BY_MOB..., Pierre Peiffer, (Fri Apr 20, 4:07 am)
Re: [PATCH -mm] fix undefined symbol if CONFIG_PAGE_GROUP_BY..., Mel Gorman, (Fri Apr 20, 4:42 am)
Re: [PATCH -mm] fix undefined symbol if CONFIG_PAGE_GROUP_BY..., Pierre Peiffer, (Fri Apr 20, 4:46 am)