diff options
author | Wu Fengguang <fengguang.wu@intel.com> | 2009-12-16 06:19:58 -0500 |
---|---|---|
committer | Andi Kleen <ak@linux.intel.com> | 2009-12-16 06:19:58 -0500 |
commit | 95d01fc664b9476e0d18e3d745bb209a42a33588 (patch) | |
tree | 0b6a46b0f2e13fb169e64ebf0b270a52c666e7f6 | |
parent | dc2a1cbf7d862e9d0abea1d1b4c8712dfbb5a398 (diff) |
HWPOISON: remove the free buddy page handler
The buddy page has already be handled in the very beginning.
So remove redundant code.
Signed-off-by: Wu Fengguang <fengguang.wu@intel.com>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
-rw-r--r-- | mm/memory-failure.c | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/mm/memory-failure.c b/mm/memory-failure.c index 9a285f8cdbe1..676ab394200e 100644 --- a/mm/memory-failure.c +++ b/mm/memory-failure.c | |||
@@ -401,14 +401,6 @@ static int me_unknown(struct page *p, unsigned long pfn) | |||
401 | } | 401 | } |
402 | 402 | ||
403 | /* | 403 | /* |
404 | * Free memory | ||
405 | */ | ||
406 | static int me_free(struct page *p, unsigned long pfn) | ||
407 | { | ||
408 | return DELAYED; | ||
409 | } | ||
410 | |||
411 | /* | ||
412 | * Clean (or cleaned) page cache page. | 404 | * Clean (or cleaned) page cache page. |
413 | */ | 405 | */ |
414 | static int me_pagecache_clean(struct page *p, unsigned long pfn) | 406 | static int me_pagecache_clean(struct page *p, unsigned long pfn) |
@@ -604,7 +596,6 @@ static int me_huge_page(struct page *p, unsigned long pfn) | |||
604 | #define tail (1UL << PG_tail) | 596 | #define tail (1UL << PG_tail) |
605 | #define compound (1UL << PG_compound) | 597 | #define compound (1UL << PG_compound) |
606 | #define slab (1UL << PG_slab) | 598 | #define slab (1UL << PG_slab) |
607 | #define buddy (1UL << PG_buddy) | ||
608 | #define reserved (1UL << PG_reserved) | 599 | #define reserved (1UL << PG_reserved) |
609 | 600 | ||
610 | static struct page_state { | 601 | static struct page_state { |
@@ -614,7 +605,10 @@ static struct page_state { | |||
614 | int (*action)(struct page *p, unsigned long pfn); | 605 | int (*action)(struct page *p, unsigned long pfn); |
615 | } error_states[] = { | 606 | } error_states[] = { |
616 | { reserved, reserved, "reserved kernel", me_ignore }, | 607 | { reserved, reserved, "reserved kernel", me_ignore }, |
617 | { buddy, buddy, "free kernel", me_free }, | 608 | /* |
609 | * free pages are specially detected outside this table: | ||
610 | * PG_buddy pages only make a small fraction of all free pages. | ||
611 | */ | ||
618 | 612 | ||
619 | /* | 613 | /* |
620 | * Could in theory check if slab page is free or if we can drop | 614 | * Could in theory check if slab page is free or if we can drop |