lkml.org 
[lkml]   [2022]   [May]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH 21/30] panic: Introduce the panic pre-reboot notifier list
    From
    On 17/05/2022 11:11, Petr Mladek wrote:
    > [...]
    >>> 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

    Perfect, I'll do that for V2 then =)

    Tony / Dinh - can I just *skip* this notifier *if kdump* is set or else
    we run the code as-is? Does that make sense to you?

    I'll postpone it to run almost in the end of info list (last position is
    for panic_print).

    Thanks,


    Guilherme

    \
     
     \ /
      Last update: 2022-05-17 18:47    [W:6.331 / U:1.096 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site