From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support
Date: Wed, 21 Nov 2012 17:14:17 +0100 [thread overview]
Message-ID: <20121121161417.5ED232003CF@gemini.denx.de> (raw)
In-Reply-To: <20121121150332.GC28899@gmail.com>
Dear Lee Jones,
In message <20121121150332.GC28899@gmail.com> you wrote:
>
> > Neither ATAGS not the device tree are intended nor designed for
> > passing logfile information. Yes, you can use them like that, and it
> > will actually work.
>
> ATAGs were exactly designed for this type of thing. To pass small
> data structures though to the kernel. In our case, our trace-points
> are held in a small data structure. They're not logs.
You appear to have a specific definition of log data in mind. It must
be different to mine.
Also, you contradict yourself - here you write "pass small data
structures", earlier you wrote about "lots of trace-points", which
sounds as if the total amount of data would be not exactly small -
actually so big that yu are afraid of annoying users with it.
Anyway. This doesn't take us further.
> They're not poor augments if the data stored isn't log messages,
> which these aren't. If anything I would say that ramming them in as
> textual kernel messages, then parsing the log text using a userspace
> tool was an abuse of the system. If we create them as data in the
> bootloader, then pass them to the kernel as data, then process them
> as data, _that_ would be the correct mechanism.
Well, I could pooint out here a number of pretty basic design
decisions made earlier in a number of pretty important and successful
software projects, like the fact that a large number of internet
protocols are based on plain text implementations. Or how useful it
is if you can just to a post-mortem dump of the log buffer and
actually _read_ the entries, without need to special tools.
I think I should stop here, though. It appears it makes little sense
trying to discuss alternative approaches when you have already fixed
your mind about the one and only "correct" way to do this.
To summarize: NAK.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
No journaling file system can recover your data if the disk dies.
- Steve Rago in <D4Cw1p.L9E@plc.com>
next prev parent reply other threads:[~2012-11-21 16:14 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-20 14:33 [U-Boot] [PATCH 0/8] Adding boottime support Lee Jones
2012-11-20 14:33 ` [U-Boot] [PATCH 1/8] u8500: Correct unnecessary mathematical roll-over Lee Jones
2012-11-20 18:14 ` Wolfgang Denk
2012-11-21 10:02 ` Lee Jones
2012-11-21 13:51 ` Wolfgang Denk
2012-11-21 14:54 ` Lee Jones
2012-11-20 14:33 ` [U-Boot] [PATCH 2/8] u8500: Add utimer support Lee Jones
2012-11-20 14:33 ` [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support Lee Jones
2012-11-20 18:20 ` Wolfgang Denk
2012-11-21 9:50 ` Lee Jones
2012-11-21 13:44 ` Wolfgang Denk
2012-11-21 15:03 ` Lee Jones
2012-11-21 16:14 ` Wolfgang Denk [this message]
2012-11-21 17:26 ` Lee Jones
2012-11-21 19:04 ` Wolfgang Denk
2012-11-26 6:08 ` Simon Glass
2012-11-26 9:00 ` Lee Jones
2012-11-26 19:57 ` Simon Glass
2012-11-27 8:55 ` Lee Jones
2012-11-27 13:46 ` Wolfgang Denk
2012-11-27 14:28 ` Lee Jones
2012-11-20 14:33 ` [U-Boot] [PATCH 4/8] boottime: Apply some key boottime tags into common code Lee Jones
2012-11-20 18:22 ` Wolfgang Denk
2012-11-21 9:36 ` Lee Jones
2012-11-21 13:40 ` Wolfgang Denk
2012-11-21 15:07 ` Lee Jones
2012-11-20 14:33 ` [U-Boot] [PATCH 5/8] arm: Add boottime support for the ARM architecture Lee Jones
2012-11-20 15:11 ` Otavio Salvador
2012-11-20 15:52 ` Lee Jones
2012-11-20 18:24 ` Wolfgang Denk
2012-11-21 9:17 ` Lee Jones
2012-11-21 9:30 ` Wolfgang Denk
2012-11-21 10:13 ` Lee Jones
2012-11-21 13:58 ` Wolfgang Denk
2012-11-21 14:39 ` Lee Jones
2012-11-21 16:05 ` Wolfgang Denk
2012-11-21 17:48 ` Lee Jones
2012-11-21 19:18 ` Wolfgang Denk
2012-11-22 10:14 ` Lee Jones
2012-11-22 13:04 ` Wolfgang Denk
2012-11-22 16:08 ` Lee Jones
2012-11-22 17:40 ` Wolfgang Denk
2012-11-23 10:08 ` Lee Jones
2012-11-20 14:33 ` [U-Boot] [PATCH 6/8] arm: Add some boottime tags into prime booting locations Lee Jones
2012-11-20 14:33 ` [U-Boot] [PATCH 7/8] href: Enable boottime functionality Lee Jones
2012-11-20 14:33 ` [U-Boot] [PATCH 8/8] snowball: " Lee Jones
2012-11-20 18:08 ` [U-Boot] [PATCH 0/8] Adding boottime support Wolfgang Denk
2012-11-21 10:03 ` Lee Jones
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=20121121161417.5ED232003CF@gemini.denx.de \
--to=wd@denx.de \
--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