Message-ID: <419152D8.9030303@mvista.com>
Date: Tue, 09 Nov 2004 15:29:28 -0800
From: George Anzinger <george@mvista.com>
Reply-To: george@mvista.com
Organization: MontaVista Software
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4.2) Gecko/20040308
MIME-Version: 1.0
To: Andrew Morton <akpm@osdl.org>
CC: dhowells@redhat.com, torvalds@osdl.org, davidm@snapgear.com,
       linux-kernel@vger.kernel.org, uclinux-dev@uclinux.org
Subject: Re: [PATCH] Additional kgdb hooks
References: <200411081432.iA8EWf0c023426@warthog.cambridge.redhat.com> <20041108144433.079f0f7c.akpm@osdl.org>
In-Reply-To: <20041108144433.079f0f7c.akpm@osdl.org>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 952
Lines: 25

Andrew Morton wrote:
> dhowells@redhat.com wrote:
> 
>>The attached patch adds a couple of extra hooks by which kgdb or an equivalent
>>gdbstub can catch bad_page() and panic() invocations.
> 
> 
> Tom is valiantly flogging his way through a generic KGDB implementation.  I
> think it would be better to push ahead with that and to not put into
> generic code hooks which are specific to one arch's kgdb implementation.
> 
IMNSHO the trap should be in dump_stack().  That way it catches a bunch of 
things all at once.

Also, panic has a notify option that kgdb should use just like everybody else.

-- 
George Anzinger   george@mvista.com
High-res-timers:  http://sourceforge.net/projects/high-res-timers/

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/