Re: [PATCH] [6/16] HWPOISON: Add basic support for poisoned pages in fault handler v2

Hidehiro Kawai
2009-05-29T04:16:11Z

Andi Kleen wrote:

> - Add a new VM_FAULT_HWPOISON error code to handle_mm_fault. Right now
> architectures have to explicitely enable poison page support, so
> this is forward compatible to all architectures. They only need
> to add it when they enable poison page support.
> - Add poison page handling in swap in fault code
>
> v2: Add missing delayacct_clear_flag (Hidehiro Kawai)

[snip]

> goto out;
> }
> delayacct_set_flag(DELAYACCT_PF_SWAPIN);
> @@ -2484,6 +2492,10 @@
> /* Had to read the page from swap area: Major fault */
> ret = VM_FAULT_MAJOR;
> count_vm_event(PGMAJFAULT);
> + } else if (PageHWPoison(page)) {
> + ret = VM_FAULT_HWPOISON;
> + delayacct_set_flag(DELAYACCT_PF_SWAPIN);
> + goto out;

Is this delayacct_clear_flag()? :-p

Regards,
--
Hidehiro Kawai
Hitachi, Systems Development Laboratory
Linux Technology Center