Re: [PATCH 3/3] panic: Allow printing extra panic information on kdump

2021-12-26 Thread Guilherme G. Piccoli
On 26/12/2021 22:45, Dave Young wrote: > On 12/25/21 at 04:21pm, Guilherme G. Piccoli wrote: > [...] > Hi Guilherme, yes, I have the same concern. But there could be more > things like the panic_print in the future, it looks odd to have more > kernel cmdline params though. > Agreed! We're on

Re: [PATCH 3/3] panic: Allow printing extra panic information on kdump

2021-12-26 Thread Dave Young
On 12/25/21 at 04:21pm, Guilherme G. Piccoli wrote: > On 23/12/2021 22:35, Dave Young wrote: > > Hi Guilherme, > > [...] > > If only the doc update, I think it is fine to be another follup-up > > patch. > > > > About your 1st option in patch log, there is crash_kexec_post_notifiers > > kernel

Re: [PATCH 3/3] panic: Allow printing extra panic information on kdump

2021-12-25 Thread Guilherme G. Piccoli
On 23/12/2021 22:35, Dave Young wrote: > Hi Guilherme, > [...] > If only the doc update, I think it is fine to be another follup-up > patch. > > About your 1st option in patch log, there is crash_kexec_post_notifiers > kernel param which can be used to switch on panic notifiers before kdump >

Re: [PATCH 3/3] panic: Allow printing extra panic information on kdump

2021-12-23 Thread Dave Young
Hi Guilherme, On 12/22/21 at 09:34am, Guilherme G. Piccoli wrote: > On 22/12/2021 08:45, Dave Young wrote: > > Hi Guilherme, > > > > Thanks for you patch. Could you add kexec list for any following up > > patches? This could change kdump behavior so let's see if any comments > > from kexec

Re: [PATCH 3/3] panic: Allow printing extra panic information on kdump

2021-12-22 Thread Guilherme G. Piccoli
On 22/12/2021 08:45, Dave Young wrote: > Hi Guilherme, > > Thanks for you patch. Could you add kexec list for any following up > patches? This could change kdump behavior so let's see if any comments > from kexec list. > > Kudos for the lore+lei tool so that I can catch this by seeing this >

Re: [PATCH 3/3] panic: Allow printing extra panic information on kdump

2021-12-22 Thread Dave Young
Hi Guilherme, Thanks for you patch. Could you add kexec list for any following up patches? This could change kdump behavior so let's see if any comments from kexec list. Kudos for the lore+lei tool so that I can catch this by seeing this coming into Andrews tree :) On 11/09/21 at 05:28pm,