Commit dd73e85f authored by Dean Nelson's avatar Dean Nelson Committed by Linus Torvalds

HWPOISON: convert pr_debug()s to pr_info()s

Commit fb46e735 ("HWPOISON: Convert pr_debugs to pr_info) authored
by Andi Kleen converted a number of pr_debug()s to pr_info()s.

About the same time additional code with pr_debug()s was added by two
other commits 8c6c2ecb ("HWPOSION, hugetlb: recover from free hugepage
error when !MF_COUNT_INCREASED") and d950b958 ("HWPOISON, hugetlb:
soft offlining for hugepage").  And these pr_debug()s failed to get
converted to pr_info()s.

This patch converts them as well.  And does some minor related whitespace
cleanup.
Signed-off-by: default avatarDean Nelson <dnelson@redhat.com>
Reviewed-by: default avatarAndi Kleen <ak@linux.intel.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 72a2ebd8
...@@ -1310,7 +1310,7 @@ int unpoison_memory(unsigned long pfn) ...@@ -1310,7 +1310,7 @@ int unpoison_memory(unsigned long pfn)
* to the end. * to the end.
*/ */
if (PageHuge(page)) { if (PageHuge(page)) {
pr_debug("MCE: Memory failure is now running on free hugepage %#lx\n", pfn); pr_info("MCE: Memory failure is now running on free hugepage %#lx\n", pfn);
return 0; return 0;
} }
if (TestClearPageHWPoison(p)) if (TestClearPageHWPoison(p))
...@@ -1419,7 +1419,7 @@ static int soft_offline_huge_page(struct page *page, int flags) ...@@ -1419,7 +1419,7 @@ static int soft_offline_huge_page(struct page *page, int flags)
if (PageHWPoison(hpage)) { if (PageHWPoison(hpage)) {
put_page(hpage); put_page(hpage);
pr_debug("soft offline: %#lx hugepage already poisoned\n", pfn); pr_info("soft offline: %#lx hugepage already poisoned\n", pfn);
return -EBUSY; return -EBUSY;
} }
...@@ -1433,8 +1433,8 @@ static int soft_offline_huge_page(struct page *page, int flags) ...@@ -1433,8 +1433,8 @@ static int soft_offline_huge_page(struct page *page, int flags)
list_for_each_entry_safe(page1, page2, &pagelist, lru) list_for_each_entry_safe(page1, page2, &pagelist, lru)
put_page(page1); put_page(page1);
pr_debug("soft offline: %#lx: migration failed %d, type %lx\n", pr_info("soft offline: %#lx: migration failed %d, type %lx\n",
pfn, ret, page->flags); pfn, ret, page->flags);
if (ret > 0) if (ret > 0)
ret = -EIO; ret = -EIO;
return ret; return ret;
...@@ -1505,7 +1505,7 @@ int soft_offline_page(struct page *page, int flags) ...@@ -1505,7 +1505,7 @@ int soft_offline_page(struct page *page, int flags)
} }
if (!PageLRU(page)) { if (!PageLRU(page)) {
pr_info("soft_offline: %#lx: unknown non LRU page type %lx\n", pr_info("soft_offline: %#lx: unknown non LRU page type %lx\n",
pfn, page->flags); pfn, page->flags);
return -EIO; return -EIO;
} }
...@@ -1566,7 +1566,7 @@ int soft_offline_page(struct page *page, int flags) ...@@ -1566,7 +1566,7 @@ int soft_offline_page(struct page *page, int flags)
} }
} else { } else {
pr_info("soft offline: %#lx: isolation failed: %d, page count %d, type %lx\n", pr_info("soft offline: %#lx: isolation failed: %d, page count %d, type %lx\n",
pfn, ret, page_count(page), page->flags); pfn, ret, page_count(page), page->flags);
} }
if (ret) if (ret)
return ret; return ret;
......
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