Messages in this thread Patch in this message | | | From | Wanpeng Li <> | Subject | [PATCH v2 3/3] mm/hwpoison: fix return value of madvise_hwpoison | Date | Tue, 27 Aug 2013 10:39:31 +0800 |
| |
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: Wanpeng Li <liwanp@linux.vnet.ibm.com> --- mm/madvise.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mm/madvise.c b/mm/madvise.c index a20764c..19b71e4 100644 --- a/mm/madvise.c +++ b/mm/madvise.c @@ -359,7 +359,7 @@ static int madvise_hwpoison(int bhv, unsigned long start, unsigned long end) page_to_pfn(p), start); ret = soft_offline_page(p, MF_COUNT_INCREASED); if (ret) - break; + return ret; continue; } pr_info("Injecting memory failure for page %#lx at %#lx\n", -- 1.8.1.2
| |