From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755101AbZFDV3l (ORCPT ); Thu, 4 Jun 2009 17:29:41 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754657AbZFDV2Z (ORCPT ); Thu, 4 Jun 2009 17:28:25 -0400 Received: from one.firstfloor.org ([213.235.205.2]:40848 "EHLO one.firstfloor.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754653AbZFDV2Y (ORCPT ); Thu, 4 Jun 2009 17:28:24 -0400 From: Andi Kleen References: <200906041128.112757038@firstfloor.org> In-Reply-To: <200906041128.112757038@firstfloor.org> To: akpm@linux-foundation.org, npiggin@suse.de, linux-kernel@vger.kernel.org, linux-mm@kvack.org, fengguang.wu@intel.com Subject: [PATCH] [6/15] HWPOISON: Add various poison checks in mm/memory.c v2 Message-Id: <20090604212817.8E1F11D0290@basil.firstfloor.org> Date: Thu, 4 Jun 2009 23:28:17 +0200 (CEST) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Bail out early when hardware poisoned pages are found in page fault handling. Since they are poisoned they should not be mapped freshly into processes, because that would cause another (potentially deadly) machine check This is generally handled in the same way as OOM, just a different error code is returned to the architecture code. v2: Do a page unlock if needed (Fengguang Wu) Signed-off-by: Andi Kleen --- mm/memory.c | 6 ++++++ 1 file changed, 6 insertions(+) Index: linux/mm/memory.c =================================================================== --- linux.orig/mm/memory.c +++ linux/mm/memory.c @@ -2797,6 +2797,12 @@ static int __do_fault(struct mm_struct * if (unlikely(ret & (VM_FAULT_ERROR | VM_FAULT_NOPAGE))) return ret; + if (unlikely(PageHWPoison(vmf.page))) { + if (ret & VM_FAULT_LOCKED) + unlock_page(vmf.page); + return VM_FAULT_HWPOISON; + } + /* * For consistency in subsequent calls, make the faulted page always * locked.