Re: [patch] add kdump_after_notifier

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Eric W. Biederman
Date: Wednesday, August 1, 2007 - 3:00 am

Takenori Nagano <t-nagano@ah.jp.nec.com> writes:


Possibly.



I respectfully disagree with his opinion, as using notifier chains
assumes more of the kernel works.  Although following it's argument
to it's logical conclusion we should call crash_kexec as the very
first thing inside of panic.  Given how much state something like
bust_spinlocks messes up that might not be a bad idea.

It does make adding an alternative debug mechanism in there difficult.
Does anyone know if this also affects kgdb?


Hmm.  Tricky.  These appear to be two code bases with a completely different
philosophy on what errors are being avoided.

The kexec on panic assumption is that the kernel is broken and we better not
touch it something horrible has gone wrong.  And this is the reason why
kexec on panic is replacing lkcd.  Because the strong assumption results
in more errors getting captured with less likely hood of messing up your
system.

The kdb assumption appears to be that the kernel is mostly ok, and that there
are just some specific thing that is wrong.

The easiest way I can think to resolve this is for kdb to simply set
a break point at the entry point of panic() when it initializes.  Then
it wouldn't even need to be on the panic_list.  That approach would probably
even give better debug information because you would not have the effects
of bust_spinlocks to undo.

Is there some reason why kdb doesn't want to hook panic with a some
kind of break point?

Eric
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch] add kdump_after_notifier, Takenori Nagano, (Thu Jul 19, 5:15 am)
Re: [patch] add kdump_after_notifier, Bernhard Walle, (Thu Jul 26, 7:07 am)
Re: [patch] add kdump_after_notifier, Vivek Goyal, (Thu Jul 26, 8:32 am)
Re: [patch] add kdump_after_notifier, Bernhard Walle, (Thu Jul 26, 8:34 am)
Re: [patch] add kdump_after_notifier, Vivek Goyal, (Thu Jul 26, 8:44 am)
Re: [patch] add kdump_after_notifier, Bernhard Walle, (Thu Jul 26, 8:47 am)
Re: [patch] add kdump_after_notifier, Vivek Goyal, (Thu Jul 26, 8:54 am)
Re: [patch] add kdump_after_notifier, Bernhard Walle, (Thu Jul 26, 9:14 am)
Re: [patch] add kdump_after_notifier, Bernhard Walle, (Thu Jul 26, 9:21 am)
Re: [patch] add kdump_after_notifier, Takenori Nagano, (Thu Jul 26, 4:28 pm)
Re: [patch] add kdump_after_notifier, Vivek Goyal, (Mon Jul 30, 2:16 am)
Re: [patch] add kdump_after_notifier, Eric W. Biederman, (Mon Jul 30, 6:42 am)
Re: [patch] add kdump_after_notifier, Takenori Nagano, (Mon Jul 30, 10:55 pm)
Re: [patch] add kdump_after_notifier, Eric W. Biederman, (Mon Jul 30, 11:53 pm)
Re: [patch] add kdump_after_notifier, Takenori Nagano, (Wed Aug 1, 2:26 am)
Re: [patch] add kdump_after_notifier, Eric W. Biederman, (Wed Aug 1, 3:00 am)
Re: [patch] add kdump_after_notifier, Takenori Nagano, (Thu Aug 2, 1:11 am)
Re: [patch] add kdump_after_notifier, Vivek Goyal, (Thu Aug 2, 4:28 am)
Re: [patch] add kdump_after_notifier , Keith Owens, (Thu Aug 2, 9:05 pm)
Re: [patch] add kdump_after_notifier, Andrew Morton, (Thu Aug 2, 11:25 pm)
Re: [patch] add kdump_after_notifier , Keith Owens, (Thu Aug 2, 11:34 pm)
Re: [patch] add kdump_after_notifier, Eric W. Biederman, (Fri Aug 3, 12:10 am)
Re: [patch] add kdump_after_notifier, Andrew Morton, (Fri Aug 3, 12:37 am)
Re: [patch] add kdump_after_notifier, Vivek Goyal, (Sun Aug 5, 4:07 am)
Re: [patch] add kdump_after_notifier, Takenori Nagano, (Tue Aug 14, 1:34 am)
Re: [patch] add kdump_after_notifier, Bernhard Walle, (Tue Aug 14, 1:37 am)
Re: [patch] add kdump_after_notifier, Takenori Nagano, (Tue Aug 14, 1:48 am)
Re: [patch] add kdump_after_notifier, Bernhard Walle, (Tue Aug 14, 1:53 am)
Re: [patch] add kdump_after_notifier, Vivek Goyal, (Tue Aug 14, 6:24 am)
Re: [patch] add kdump_after_notifier, Takenori Nagano, (Thu Aug 16, 2:26 am)
Re: [patch] add kdump_after_notifier, Bernhard Walle, (Thu Aug 16, 2:45 am)
Re: [patch] add kdump_after_notifier, Vivek Goyal, (Fri Aug 17, 3:56 am)
Re: [patch] add kdump_after_notifier, Takenori Nagano, (Tue Aug 21, 12:45 am)
Re: [patch] add kdump_after_notifier, Jay Lan, (Tue Aug 21, 6:18 am)
Re: [patch] add kdump_after_notifier, Bernhard Walle, (Tue Aug 21, 6:21 am)
Re: [patch] add kdump_after_notifier, Vivek Goyal, (Wed Aug 22, 8:52 pm)
Re: [patch] add kdump_after_notifier, Vivek Goyal, (Wed Aug 22, 8:56 pm)
Re: [patch] add kdump_after_notifier, Jay Lan, (Thu Aug 23, 10:34 am)