public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Vivek Goyal <vgoyal@redhat.com>
To: jmerkey@wolfmountaingroup.com
Cc: Jay Lan <jlan@sgi.com>,
	Christoph Lameter <cl@linux-foundation.org>,
	Stefan Richter <stefanr@s5r6.in-berlin.de>,
	Nick Piggin <nickpiggin@yahoo.com.au>,
	Andi Kleen <andi@firstfloor.org>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Josh Boyer <jwboyer@gmail.com>,
	linux-kernel@vger.kernel.org,
	Takenori Nagano <t-nagano@ah.jp.nec.com>,
	Bernhard Walle <bwalle@suse.de>, Keith Owens <kaos@ocs.com.au>
Subject: Re: [ANNOUNCE] Merkey's Kernel Debugger
Date: Thu, 7 Aug 2008 21:26:23 -0400	[thread overview]
Message-ID: <20080808012623.GA3911@redhat.com> (raw)
In-Reply-To: <2232.69.2.248.210.1218137662.squirrel@webmail.wolfmountaingroup.com>

On Thu, Aug 07, 2008 at 01:34:22PM -0600, jmerkey@wolfmountaingroup.com wrote:
> > Christoph Lameter wrote:
> >> Stefan Richter wrote:
> >>> Christoph Lameter wrote:
> >>>> The competing implementation is kdb not kgdb. kgdb is just a stub for
> >>>> remote
> >>>> debugging using gdb. kdb is an in-kernel debugger like the one
> >>>> proposed here.
> >>> Is there work underway to get kdb merged?  (I'm just asking because I
> >>> don't know; I personally don't need kdb nor mdb.)
> >>
> >> KDB still exists in patches but the merge effort was given up when Linus
> >> stated that he did not want a kernel debugger. No problem to start merge
> >> attempts again AFAICT. Jay?
> >
> > To merge KDB or any other RAS tools, you need to deal with kdump. Kdump
> > hijack panic() before the die calling chain. For KDB or a RAS tool to
> > work, an infrastructure such as the "add new notifier function" by
> > Takenori Nagano should be in place.
> >
> > His last attempt fell short, in my opinion, was partly due to his
> > "[PATCH 3/3] Move crash_kexec() into  panic_notifier" did not do what it
> > meant to do: to fit kexec/kdump into the new infrastructure. That is
> > not fatal; it can be fixed to make it right. If community is interested
> > in getting a kernel debugger to the kernel, we can continue Takenori's
> > work. Once the infrastructure is accepted, then merging KDB or any other
> > kernel debugger will make sense.
> >
> > Regards,
> >  - jay
> >
> >
> >>
> 
> As I look through entry_32.S and traps_32.c I do not see where kdump is
> hooking the notify_die handler which would intercept calls to a debugger.
> 
> Where does kdump hook this path?
> 

kdump uses crash_kexec() call for hooking. It hooks in panic(), die_nmi()
and die().

Thanks
Vivek

  reply	other threads:[~2008-08-08  1:27 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-03 17:22 [ANNOUNCE] Merkey's Kernel Debugger jmerkey
     [not found] ` <17494.166.70.238.46.1217784156.squirrel@webmail.wolfmountaingroup.com >
2008-08-03 19:36   ` jmerkey
2008-08-03 20:00     ` Rene Herman
2008-08-04  0:14     ` Josh Boyer
2008-08-04  2:19       ` jmerkey
2008-08-04 13:41         ` Stefan Richter
2008-08-04 14:33           ` jmerkey
2008-08-05  9:41             ` Geert Uytterhoeven
2008-08-05 15:02               ` jmerkey
2008-08-05 15:33                 ` Nick Piggin
2008-08-05 15:19                   ` jmerkey
2008-08-05 15:45                     ` Nick Piggin
2008-08-05 15:32                       ` jmerkey
2008-08-05 16:38                         ` Nick Piggin
2008-08-05 16:45                           ` jmerkey
2008-08-06 19:47                           ` Rene Herman
2008-08-05 16:04                     ` Chris Friesen
2008-08-05 16:39                       ` jmerkey
2008-08-07 20:43                         ` Daniel Barkalow
2008-08-07 21:02                           ` jmerkey
2008-08-07 21:04                             ` jmerkey
2008-08-05 17:21                     ` Paul Mundt
2008-08-05 17:10                       ` jmerkey
2008-08-06  3:08                   ` Andi Kleen
2008-08-06  5:50                     ` Nick Piggin
2008-08-07 17:45                       ` Christoph Lameter
2008-08-07 17:53                         ` jmerkey
2008-08-07 18:08                         ` Stefan Richter
2008-08-07 19:10                           ` Christoph Lameter
2008-08-07 19:47                             ` Jay Lan
2008-08-07 19:34                               ` jmerkey
2008-08-08  1:26                                 ` Vivek Goyal [this message]
2008-08-07 20:06                               ` Andi Kleen
2008-08-07 20:07                                 ` Bernhard Walle
2008-08-07 20:09                                   ` Andi Kleen
2008-08-07 20:11                                     ` Bernhard Walle
2008-08-07 22:28                                 ` Keith Owens
2008-08-08  1:15                                   ` Vivek Goyal
2008-08-08  2:29                                     ` Andi Kleen
2008-08-08 12:08                                       ` Cliff Wickman
2008-08-08 12:20                                         ` Andi Kleen
2008-08-08 13:19                                           ` jmerkey
2008-08-08 15:06                                           ` Cliff Wickman
2008-08-08 13:29                                       ` Vivek Goyal
2008-08-08 14:50                                         ` Cliff Wickman
2008-08-08 16:57                                           ` Jay Lan
2008-08-11 12:56                                           ` Vivek Goyal
2008-08-08 18:03                                         ` Andi Kleen
2008-08-11 13:02                                           ` Vivek Goyal
2008-08-11 13:11                                             ` jmerkey
2008-08-11 13:50                                               ` Andi Kleen
2008-08-11 16:16                                                 ` jmerkey
2008-08-18  9:12                                             ` Takenori Nagano
2008-08-08  8:40                         ` Nick Piggin
2008-08-06 13:11                     ` Bill Davidsen
2008-08-06 13:37                       ` Stefan Richter
2008-08-06 13:54                       ` Olivier Galibert
2008-08-06 13:45                         ` jmerkey
2008-08-06 14:16                       ` Nick Piggin
2008-08-06 17:21                     ` Jason Wessel
2008-08-06 18:57                       ` Andi Kleen
2008-08-07 12:45                         ` jmerkey
2008-08-07 15:17                           ` Peter Zijlstra
2008-08-07 16:07                           ` Andi Kleen
2008-08-07 15:52                             ` jmerkey
2008-08-07 17:04                               ` Stefan Richter
2008-08-08  0:28                           ` Stefan Richter
2008-08-11 10:36 ` jidong xiao

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080808012623.GA3911@redhat.com \
    --to=vgoyal@redhat.com \
    --cc=andi@firstfloor.org \
    --cc=bwalle@suse.de \
    --cc=cl@linux-foundation.org \
    --cc=geert@linux-m68k.org \
    --cc=jlan@sgi.com \
    --cc=jmerkey@wolfmountaingroup.com \
    --cc=jwboyer@gmail.com \
    --cc=kaos@ocs.com.au \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nickpiggin@yahoo.com.au \
    --cc=stefanr@s5r6.in-berlin.de \
    --cc=t-nagano@ah.jp.nec.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox