* Keith Owens <kaos@xxxxxxx> [2007-06-01 02:37]:
> Takenori Nagano (on Thu, 31 May 2007 16:37:59 +0900) wrote:
> >Hi,
> >
> >kdb has some problem to use with kdump.
> >This patch fixes some of them.
> >
> >1) We can't use kdb when machine panicked.
> >
> >crash_kexec() is called before notifier_call_chain(&panic_notifier_chain)=
> >.
> >This patch makes KDB_ENTER() is called before crash_kexec().
>
> Both KDB and crash_kexec should be using the panic_notifier_chain, with
> KDB having a higher priority than crash_exec. The whole point of
> notifier chains is to handle cases like this, so we should not be
> adding more code to the panic routine.
That's true. But the problem is: KDB is not mainline while kdump is.
So, if mainline doesn't accept changes for some reason, the changes
must be included in KDB patches. :-(
> >2) We can't take a kdump when KDB_FLAG is set CATASTROPHIC.
> >
> >kdb_do_dump() does not support kdump.
> >This patch makes machine_kexec() is called from kdb_do_dump().
>
> Ugly. All the code for selecting which dump to take (lkcd, kexec,
> anything else) should be in a common kernel routine that anybody can
> call. It should not be just in KDB.
Well, but if KDB is *disabled*, there should be a way that KDB just
doesn't do anything. It would help if CONFIG_KDB_CONTINUE_CATASTROPHIC
would be a *runtime* setting, not a *compile time* setting.
Thanks,
Bernhard
---------------------------
Use http://oss.sgi.com/ecartis to modify your settings or to unsubscribe.
|