Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: "Guilherme G. Piccoli" <gpiccoli@xxxxxxxxxx>
Subject: Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
From: Borislav Petkov <bp@xxxxxxxxx>
Date: Thu, 18 Aug 2022 00:00:58 +0200
Cc: pmladek@xxxxxxxx, Dinh Nguyen <dinguyen@xxxxxxxxxx>, Tony Luck <tony.luck@xxxxxxxxx>, akpm@xxxxxxxxxxxxxxxxxxxx, bhe@xxxxxxxxxx, kexec@xxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-hyperv@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, x86@xxxxxxxxxx, kernel-dev@xxxxxxxxxx, kernel@xxxxxxxxxxxx, halves@xxxxxxxxxxxxx, fabiomirmar@xxxxxxxxx, alejandro.j.jimenez@xxxxxxxxxx, andriy.shevchenko@xxxxxxxxxxxxxxx, arnd@xxxxxxxx, corbet@xxxxxxx, d.hatayama@xxxxxxxxxxxxxx, dave.hansen@xxxxxxxxxxxxxxx, dyoung@xxxxxxxxxx, feng.tang@xxxxxxxxx, gregkh@xxxxxxxxxxxxxxxxxxx, mikelley@xxxxxxxxxxxxx, hidehiro.kawai.ez@xxxxxxxxxxx, jgross@xxxxxxxx, john.ogness@xxxxxxxxxxxxx, keescook@xxxxxxxxxxxx, luto@xxxxxxxxxx, mhiramat@xxxxxxxxxx, mingo@xxxxxxxxxx, paulmck@xxxxxxxxxx, peterz@xxxxxxxxxxxxx, rostedt@xxxxxxxxxxx, senozhatsky@xxxxxxxxxxxx, stern@xxxxxxxxxxxxxxxxxxx, tglx@xxxxxxxxxxxxx, vgoyal@xxxxxxxxxx, vkuznets@xxxxxxxxxx, will@xxxxxxxxxx, linux-edac@xxxxxxxxxxxxxxx
In-reply-to: <1ee275b3-97e8-4c2b-be88-d50898d17e23@igalia.com>
References: <20220719195325.402745-1-gpiccoli@igalia.com> <20220719195325.402745-11-gpiccoli@igalia.com> <Yv0mCY04heUXsGiC@zn.tnic> <46137c67-25b4-6657-33b7-cffdc7afc0d7@igalia.com> <Yv1C0Y25u2IB7PCs@zn.tnic> <7f016d7f-a546-a45d-c65c-bc35269b4faa@igalia.com> <Yv1XVRmTXHLhOkER@zn.tnic> <c0250075-ec87-189f-52c5-e0520325a015@igalia.com> <Yv1hn2ayPoyKBAj8@zn.tnic> <1ee275b3-97e8-4c2b-be88-d50898d17e23@igalia.com>

On Wed, Aug 17, 2022 at 06:56:11PM -0300, Guilherme G. Piccoli wrote:
> But do you agree that currently, in case of a kdump, that information
> *is not collected*, with our without my patch?

If for some reason that panic notifier does not get run before kdump,
then that's a bug and that driver should not use a panic notifier to log
hw errors in the first place.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

_______________________________________________
kexec mailing list
kexec@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/kexec

Follow-Ups:
- Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
  - From: Guilherme G. Piccoli

References:
- [PATCH v2 00/13] The panic notifiers refactor strikes back - fixes/clean-ups
  - From: Guilherme G. Piccoli
- [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
  - From: Guilherme G. Piccoli
- Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
  - From: Borislav Petkov
- Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
  - From: Guilherme G. Piccoli
- Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
  - From: Borislav Petkov
- Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
  - From: Guilherme G. Piccoli
- Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
  - From: Borislav Petkov
- Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
  - From: Guilherme G. Piccoli
- Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
  - From: Borislav Petkov
- Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
  - From: Guilherme G. Piccoli

Prev by Date: Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
Next by Date: Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
Previous by thread: Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
Next by thread: Re: [PATCH v2 10/13] EDAC/altera: Skip the panic notifier if kdump is loaded
Index(es):
- Date
- Thread

[Index of Archives] [LM Sensors] [Linux Sound] [ALSA Users] [ALSA Devel] [Linux Audio Users] [Linux Media] [Kernel] [Gimp] [Yosemite News] [Linux Media]