Commit 8302423b authored by Wanpeng Li's avatar Wanpeng Li Committed by Linus Torvalds

mm/madvise.c: fix return value of madvise_hwpoison()

The return value outside for loop is always zero which means
madvise_hwpoison return success, however, this is not truth for
soft_offline_page w/ failure return value.
Signed-off-by: default avatarWanpeng Li <liwanp@linux.vnet.ibm.com>
Reviewed-by: default avatarNaoya Horiguchi <n-horiguchi@ah.jp.nec.com>
Cc: Andi Kleen <andi@firstfloor.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 3ba5eebc
...@@ -361,7 +361,7 @@ static int madvise_hwpoison(int bhv, unsigned long start, unsigned long end) ...@@ -361,7 +361,7 @@ static int madvise_hwpoison(int bhv, unsigned long start, unsigned long end)
page_to_pfn(p), start); page_to_pfn(p), start);
ret = soft_offline_page(p, MF_COUNT_INCREASED); ret = soft_offline_page(p, MF_COUNT_INCREASED);
if (ret) if (ret)
break; return ret;
continue; continue;
} }
pr_info("Injecting memory failure for page %#lx at %#lx\n", pr_info("Injecting memory failure for page %#lx at %#lx\n",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment