From: Christoph Lameter <clameter@sgi.com>

On Sat, 30 Sep 2006, Andrew Morton wrote:

> BUILD_BUG_ON()?

Good idea. We may want to take all of these patches out if Andy can come
up with an easy modification to the macros that avoids ZONEID_PGSHIFT to
unintentionally become 0.

Signed-off-by: Christoph Lameter <clameter@sgi.com>
Cc: Andy Whitcroft <apw@shadowen.org>
Cc: Dave Hansen <haveblue@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
---

 include/linux/mm.h |    2 +-
 1 files changed, 1 insertion(+), 1 deletion(-)

diff -puN include/linux/mm.h~get-rid-of-zone_table-fix-4 include/linux/mm.h
--- a/include/linux/mm.h~get-rid-of-zone_table-fix-4
+++ a/include/linux/mm.h
@@ -445,7 +445,7 @@ static inline enum zone_type page_zonenu
  */
 static inline int page_zone_id(struct page *page)
 {
-	BUG_ON(ZONEID_PGSHIFT == 0 && ZONEID_MASK);
+	BUILD_BUG_ON(ZONEID_PGSHIFT == 0 && ZONEID_MASK);
 	return (page->flags >> ZONEID_PGSHIFT) & ZONEID_MASK;
 }
 
_