From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [PATCH v3 0/3] Add command to display or save Linux PStore dumps
Date: Mon, 23 Mar 2020 11:46:19 -0400 [thread overview]
Message-ID: <20200323154619.GE5793@bill-the-cat> (raw)
In-Reply-To: <CAPnjgZ3vgUNAk4N_eAAbWJbvHZ4W1Ry-5nnM=60ceVLu82sVyQ@mail.gmail.com>
On Mon, Mar 23, 2020 at 09:37:12AM -0600, Simon Glass wrote:
> Hi,
>
> On Fri, 20 Mar 2020 at 05:30, Wolfgang Denk <wd@denx.de> wrote:
> >
> > Dear Fr?d?ric,
> >
> > In message <9b0b5776-cb47-dfb2-ab77-36c89dc64359@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 feature working and in mainline
> > already 17+ years ago.
> >
> > It's sad that people rather reinvent the wheel instead of using
> > existing functionality (or reviving it, if it has been broken over
> > time).
>
> Indeed.
>
> Let's make sure there are tests added to this new feature (which looks
> extremely useful to me), so it stays working.
I'll just chime in to say that I have now reached the point in my career
where I've both been told "you're re-inventing the wheel (again)" and
now seen the wheel be re-re...-re-invented.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200323/1b2573d1/attachment.sig>
prev parent reply other threads:[~2020-03-23 15:46 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-19 17:57 [PATCH v3 0/3] Add command to display or save Linux PStore dumps Frédéric Danis
2020-03-19 17:57 ` [PATCH v3 1/3] cmd: " Frédéric Danis
2020-03-19 17:57 ` [PATCH v3 2/3] test: Add PStore command tests Frédéric Danis
2020-03-19 20:39 ` Wolfgang Denk
2020-03-19 17:57 ` [PATCH v3 3/3] cmd: Fixup DT to pass PStore Ramoops parameters Frédéric Danis
2020-03-19 19:14 ` Heinrich Schuchardt
2020-03-19 20:37 ` [PATCH v3 0/3] Add command to display or save Linux PStore dumps Wolfgang Denk
2020-03-19 22:30 ` Heinrich Schuchardt
2020-03-20 8:08 ` Wolfgang Denk
2020-03-20 8:57 ` Heiko Schocher
2020-03-20 10:13 ` Frédéric Danis
2020-03-20 10:35 ` Wolfgang Denk
2020-03-20 11:19 ` Frédéric Danis
2020-03-20 11:29 ` Wolfgang Denk
2020-03-23 15:37 ` Simon Glass
2020-03-23 15:46 ` Tom Rini [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200323154619.GE5793@bill-the-cat \
--to=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox