Re: [PATCH v2] printk: Add a short description string to kmsg_dump()

2024-07-17 Thread Jocelyn Falempe
On 02/07/2024 14:26, Jocelyn Falempe wrote: kmsg_dump doesn't forward the panic reason string to the kmsg_dumper callback. This patch adds a new struct kmsg_dump_detail, that will hold the reason and description, and pass it to the dump() callback. To avoid updating all kmsg_dump() cal

Re: [PATCH v2] printk: Add a short description string to kmsg_dump()

2024-07-12 Thread Jocelyn Falempe
On 12/07/2024 15:34, Kees Cook wrote: On July 12, 2024 2:59:30 AM PDT, Jocelyn Falempe wrote: Gentle ping, I need reviews from powerpc, usermod linux, mtd, pstore and hyperv, to be able to push it in the drm-misc tree. Oops, I thought I'd Acked already! Acked-by: Kees Cook And,

Re: [PATCH v2] printk: Add a short description string to kmsg_dump()

2024-07-12 Thread Jocelyn Falempe
On 02/07/2024 14:26, Jocelyn Falempe wrote: kmsg_dump doesn't forward the panic reason string to the kmsg_dumper callback. This patch adds a new struct kmsg_dump_detail, that will hold the reason and description, and pass it to the dump() callback. To avoid updating all kmsg_dump() cal

Re: [PATCH v2] printk: Add a short description string to kmsg_dump()

2024-07-03 Thread Jocelyn Falempe
On 03/07/2024 18:27, Kees Cook wrote: On Wed, Jul 03, 2024 at 10:22:11AM +0200, Petr Mladek wrote: On Wed 2024-07-03 09:57:26, Jocelyn Falempe wrote: On 02/07/2024 22:29, Kees Cook wrote: On Tue, Jul 02, 2024 at 02:26:04PM +0200, Jocelyn Falempe wrote: kmsg_dump doesn't forwar

Re: [PATCH v2] printk: Add a short description string to kmsg_dump()

2024-07-03 Thread Jocelyn Falempe
On 02/07/2024 22:29, Kees Cook wrote: On Tue, Jul 02, 2024 at 02:26:04PM +0200, Jocelyn Falempe wrote: kmsg_dump doesn't forward the panic reason string to the kmsg_dumper callback. This patch adds a new struct kmsg_dump_detail, that will hold the reason and description, and pass it t

[PATCH v2] printk: Add a short description string to kmsg_dump()

2024-07-02 Thread Jocelyn Falempe
_detail to hold the reason and description pointer, for more flexibility if we want to add other parameters. (Kees Cook) * Fix powerpc/nvram_64 build, as I didn't update the forward declaration of oops_to_nvram() Signed-off-by: Jocelyn Falempe --- arch/powerpc/kernel/nvram_64.c

Re: [PATCH] printk: Add a short description string to kmsg_dump()

2024-06-28 Thread Jocelyn Falempe
On 26/06/2024 18:26, Kees Cook wrote: On Tue, Jun 25, 2024 at 02:39:29PM +0200, Jocelyn Falempe wrote: kmsg_dump doesn't forward the panic reason string to the kmsg_dumper callback. This patch adds a new parameter "const char *desc" to the kmsg_dumper dump() callback, and upd

Re: [PATCH] printk: Add a short description string to kmsg_dump()

2024-06-27 Thread Jocelyn Falempe
On 26/06/2024 10:00, Petr Mladek wrote: On Tue 2024-06-25 14:39:29, Jocelyn Falempe wrote: kmsg_dump doesn't forward the panic reason string to the kmsg_dumper callback. This patch adds a new parameter "const char *desc" to the kmsg_dumper dump() callback, and update all d

[PATCH] printk: Add a short description string to kmsg_dump()

2024-06-26 Thread Jocelyn Falempe
ion and a macro for backward compatibility. I've written this for drm_panic, but it can be useful for other kmsg_dumper. It allows to see the panic reason, like "sysrq triggered crash" or "VFS: Unable to mount root fs on " on the drm panic screen. Signed-off-by: J