Re: [PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-23 Thread Tom Rini
On Mon, Mar 23, 2020 at 09:37:12AM -0600, Simon Glass wrote: > Hi, > > On Fri, 20 Mar 2020 at 05:30, Wolfgang Denk wrote: > > > > Dear Frédéric, > > > > In message <9b0b5776-cb47-dfb2-ab77-36c89dc64...@collabora.com> you wrote: > > > > > > But, when debugging kernel crashes occurring either

Re: [PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-23 Thread Simon Glass
Hi, On Fri, 20 Mar 2020 at 05:30, Wolfgang Denk wrote: > > Dear Frédéric, > > In message <9b0b5776-cb47-dfb2-ab77-36c89dc64...@collabora.com> you wrote: > > > > But, when debugging kernel crashes occurring either early in boot or > > hard crashes later, it can be helpful to be able to retrieve

Re: [PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-20 Thread Wolfgang Denk
Dear Frédéric, In message <9b0b5776-cb47-dfb2-ab77-36c89dc64...@collabora.com> you wrote: > > But, when debugging kernel crashes occurring either early in boot or > hard crashes later, it can be helpful to be able to retrieve logs using > the U-Boot session. Full agreement, and we had such a

Re: [PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-20 Thread Frédéric Danis
Dear Wolfgang, On 20/03/2020 11:35, Wolfgang Denk wrote: Dear Frédéric, In message <03674974-2a2c-3804-ce02-4f3d38ff0...@collabora.com> you wrote: I understand what Linus has is one-way, only focussing on crash dump, i. e. it does not allow to pass information from U-Boot to Linux? Yes,

Re: [PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-20 Thread Wolfgang Denk
Dear Frédéric, In message <03674974-2a2c-3804-ce02-4f3d38ff0...@collabora.com> you wrote: > > > I understand what Linus has is one-way, only focussing on crash > > dump, i. e. it does not allow to pass information from U-Boot to Linux? > > Yes, currently it is not possible to pass information

Re: [PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-20 Thread Frédéric Danis
Hi Wolfgang, Heinrich, Heiko, On 20/03/2020 09:08, Wolfgang Denk wrote: Dear Heinrich, In message you wrote: To me this sounds very much like what you are adding here (plus a few features more). Does it make sense to unify such code? It seems you are relating to

Re: [PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-20 Thread Heiko Schocher
Hello Wolfgang, Heinrich, Am 20.03.2020 um 09:08 schrieb Wolfgang Denk: Dear Heinrich, In message you wrote: To me this sounds very much like what you are adding here (plus a few features more). Does it make sense to unify such code? It seems you are relating to

Re: [PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-20 Thread Wolfgang Denk
Dear Heinrich, In message you wrote: > > > To me this sounds very much like what you are adding here (plus a > > few features more). Does it make sense to unify such code? > > It seems you are relating to > https://lore.kernel.org/lkml/844oyrqvvb@sauna.l.org/t/ No, I'm not. I was talking

Re: [PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-19 Thread Heinrich Schuchardt
On 3/19/20 9:37 PM, Wolfgang Denk wrote: Dear Frédéric, In message <20200319175737.10166-1-frederic.da...@collabora.com> you wrote: This serie of patches adds a new pstore command allowing to display or save ramoops logs (oops, panic, console, ftrace and user) generated by a previous kernel

Re: [PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-19 Thread Wolfgang Denk
Dear Frédéric, In message <20200319175737.10166-1-frederic.da...@collabora.com> you wrote: > This serie of patches adds a new pstore command allowing to display or save > ramoops logs (oops, panic, console, ftrace and user) generated by a previous > kernel crash. > PStore parameters can be set in

[PATCH v3 0/3] Add command to display or save Linux PStore dumps

2020-03-19 Thread Frédéric Danis
This serie of patches adds a new pstore command allowing to display or save ramoops logs (oops, panic, console, ftrace and user) generated by a previous kernel crash. PStore parameters can be set in U-Boot configuration file, or at run-time using "pstore set" command. For kernel using Device Tree,