From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751073AbVH3Hdr (ORCPT ); Tue, 30 Aug 2005 03:33:47 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751214AbVH3Hdr (ORCPT ); Tue, 30 Aug 2005 03:33:47 -0400 Received: from gateway-1237.mvista.com ([12.44.186.158]:23036 "EHLO av.mvista.com") by vger.kernel.org with ESMTP id S1751073AbVH3Hdr (ORCPT ); Tue, 30 Aug 2005 03:33:47 -0400 Message-ID: <43140BC5.1090804@mvista.com> Date: Tue, 30 Aug 2005 00:33:25 -0700 From: George Anzinger Reply-To: george@mvista.com Organization: MontaVista Software User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050323 Fedora/1.7.6-1.3.2 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Tom Rini CC: akpm@osdl.org, linux-kernel@vger.kernel.org, ak@suse.de Subject: Re: [patch 1/3] x86_64: Add a notify_die() call to the "no context" part of do_page_fault() References: In-Reply-To: Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Tom Rini wrote: > CC: Andi Kleen > This adds a call to notify_die() in the "no context" portion of > do_page_fault() as someone on the chain might care and want to do a fixup. > > --- > > linux-2.6.13-trini/arch/x86_64/mm/fault.c | 4 ++++ > 1 files changed, 4 insertions(+) > > diff -puN arch/x86_64/mm/fault.c~x86_64-no_context_hook arch/x86_64/mm/fault.c > --- linux-2.6.13/arch/x86_64/mm/fault.c~x86_64-no_context_hook 2005-08-29 11:09:13.000000000 -0700 > +++ linux-2.6.13-trini/arch/x86_64/mm/fault.c 2005-08-29 11:09:13.000000000 -0700 > @@ -514,6 +514,10 @@ no_context: > if (is_errata93(regs, address)) > return; > > + if (notify_die(DIE_PAGE_FAULT, "no context", regs, error_code, 14, > + SIGSEGV) == NOTIFY_STOP) > + return; > + > /* > * Oops. The kernel tried to access some bad page. We'll have to > * terminate things with extreme prejudice. Please use a more descriptive text than "no context". This bit of info SHOULD be available to the gdb/kgdb user and should indicate why kgdb was entered. It thus should be something like "bad kernel address" or "illegal kernel address". > -- George Anzinger george@mvista.com HRT (High-res-timers): http://sourceforge.net/projects/high-res-timers/