Messages in this thread | | | Date | Fri, 21 Jan 2011 14:12:51 +0900 | From | Naoya Horiguchi <> | Subject | Re: [PATCH 3/3] Fix the wrong corrupted page information of hugetlb page. |
| |
Hi,
> @@ -991,18 +995,32 @@ int __memory_failure(unsigned long pfn, int trapno, int flags) > action_result(pfn, "free buddy", DELAYED); > return 0; > } else if (PageHuge(hpage)) { > - /* > - * Check "just unpoisoned", "filter hit", and > - * "race with other subpage." > - */ > lock_page_nosync(hpage); > - if (!PageHWPoison(p) > - || (hwpoison_filter(p) && TestClearPageHWPoison(p)) > - || (p != hpage && TestSetPageHWPoison(hpage))) { > - atomic_long_sub(nr_pages, &mce_bad_pages); > + /* Check "just unpoisoned". */ > + if (!PageHWPoison(hpage)) { ^^^^^ Isn't this "p" (as you changed in patch 2/3) ?
-- Naoya Horiguchi
| |