Commit 5f4b9fc5 authored Feb 22, 2013 by Naoya Horiguchi Committed by Linus Torvalds Feb 23, 2013

HWPOISON: change order of error_states[]'s elements



error_states[] has two separate states "unevictable LRU page" and
"mlocked LRU page", and the former one has the higher priority now.  But
because of that the latter one is rarely chosen because pages with
PageMlocked highly likely have PG_unevictable set.  On the other hand,
PG_unevictable without PageMlocked is common for ramfs or SHM_LOCKed
shared memory, so reversing the priority of these two states helps us
clearly distinguish them.

Signed-off-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: Chen Gong <gong.chen@linux.intel.com>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Wu Fengguang <fengguang.wu@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

parent 524fca1e

mm/memory-failure.c

+3 −3

Original line number	Diff line number	Diff line
		@@ -784,12 +784,12 @@ static struct page_state {
		{ sc\|dirty, sc\|dirty, "dirty swapcache", me_swapcache_dirty },
		{ sc\|dirty, sc, "clean swapcache", me_swapcache_clean },

		{ unevict\|dirty, unevict\|dirty, "dirty unevictable LRU", me_pagecache_dirty },
		{ unevict, unevict, "clean unevictable LRU", me_pagecache_clean },

		{ mlock\|dirty, mlock\|dirty, "dirty mlocked LRU", me_pagecache_dirty },
		{ mlock, mlock, "clean mlocked LRU", me_pagecache_clean },

		{ unevict\|dirty, unevict\|dirty, "dirty unevictable LRU", me_pagecache_dirty },
		{ unevict, unevict, "clean unevictable LRU", me_pagecache_clean },

		{ lru\|dirty, lru\|dirty, "dirty LRU", me_pagecache_dirty },
		{ lru\|dirty, lru, "clean LRU", me_pagecache_clean },

Admin message