Thread (175 messages) 175 messages, 28 authors, 2022-05-24

Re: [PATCH 21/30] panic: Introduce the panic pre-reboot notifier list

From: Petr Mladek <pmladek@suse.com>
Date: 2022-05-17 14:13:52
Also in: kexec, linux-alpha, linux-edac, linux-hyperv, linux-leds, linux-mips, linux-remoteproc, linux-s390, linux-tegra, linux-um, linuxppc-dev, lkml, netdev, rcu, sparclinux, xen-devel

On Mon 2022-05-16 13:33:51, Guilherme G. Piccoli wrote:
On 16/05/2022 13:18, Luck, Tony wrote:
quoted
quoted
[...]
Would it be possible to have some global "kdump is configured + enabled" flag?

Then notifiers could make an informed choice on whether to deep dive to
get all the possible details (when there is no kdump) or just skim the high
level stuff (to maximize chance of getting a successful kdump).

-Tony
Good idea Tony! What if I wire a kexec_crash_loaded() in the notifier?
I like this idea.

One small problem is that kexec_crash_loaded() has valid result
only under kexec_mutex. On the other hand, it should stay true
once loaded so that the small race window should be innocent.
With that, are you/Petr/Dinh OK in moving it for the info list?
Sounds good to me.

Best Regards,
Petr
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help