[PATCH 10/30] alpha: Clean-up the panic notifier code
Petr Mladek
pmladek at suse.com
Tue May 10 07:16:06 PDT 2022
On Mon 2022-05-09 11:13:17, Guilherme G. Piccoli wrote:
> On 27/04/2022 19:49, Guilherme G. Piccoli wrote:
> > The alpha panic notifier has some code issues, not following
> > the conventions of other notifiers. Also, it might halt the
> > machine but still it is set to run as early as possible, which
> > doesn't seem to be a good idea.
Yeah, it is pretty strange behavior.
I looked into the history. This notifier was added into the alpha code
in 2.4.0-test2pre2. In this historic code, the default panic() code
either rebooted after a timeout or ended in a infinite loop. There
was not crasdump at that times.
The notifier allowed to change the behavior. There were 3 notifiers:
+ mips and mips64 ended with blinking in panic()
+ alpha did __halt() in this srm case
They both still do this. I guess that it is some historic behavior
that people using these architectures are used to.
Anyway, it makes sense to do this as the last notifier after
dumping other information.
Reviewed-by: Petr Mladek <pmladek at suse.com>
Best Regards,
Petr
More information about the linux-um
mailing list