Re: [PATCH] [13/16] HWPOISON: The high level memory error handler in the VM v3

Andi Kleen
2009-06-02T13:59:38Z

> > Ok you're right. That one is not needed. I will remove it.
>
> No! Please read the comment. In fact __remove_from_page_cache() has a
>
> BUG_ON(page_mapped(page));
>
> Or, at least correct that BUG_ON() line together.

Yes, but we already have them unmapped earlier and the poison check
in the page fault handler should prevent remapping.

So it really should not happen and if it happened we would deserve
the BUG.

-Andi

--
ak@linux.intel.com -- Speaking for myself only.