The buddy page has already be handled in the very beginning. So remove redundant code. CC: Andi Kleen Signed-off-by: Wu Fengguang --- mm/memory-failure.c | 9 --------- 1 file changed, 9 deletions(-) --- linux-mm.orig/mm/memory-failure.c 2009-11-09 10:57:50.000000000 +0800 +++ linux-mm/mm/memory-failure.c 2009-11-09 10:59:26.000000000 +0800 @@ -379,14 +379,6 @@ static int me_unknown(struct page *p, un } /* - * Free memory - */ -static int me_free(struct page *p, unsigned long pfn) -{ - return DELAYED; -} - -/* * Clean (or cleaned) page cache page. */ static int me_pagecache_clean(struct page *p, unsigned long pfn) @@ -592,7 +584,6 @@ static struct page_state { int (*action)(struct page *p, unsigned long pfn); } error_states[] = { { reserved, reserved, "reserved kernel", me_ignore }, - { buddy, buddy, "free kernel", me_free }, /* * Could in theory check if slab page is free or if we can drop -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/