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

From: Andi Kleen
Date: Tue Jun 02 2009 - 09:59:42 EST


> > 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@xxxxxxxxxxxxxxx -- Speaking for myself only.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/