public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Detlev Zundel <dzu@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/4] Accurate boot time measurement
Date: Tue, 17 May 2011 10:20:39 +0200	[thread overview]
Message-ID: <m2vcx9viug.fsf@ohwell.denx.de> (raw)
In-Reply-To: <BANLkTi=wdddeKLjoBFRoNcqzXJN9uGnBSw@mail.gmail.com> (Simon Glass's message of "Mon, 16 May 2011 09:00:57 -0700")

Hi Simon and Wolfgang,

[...]

> In terms of all this discussion I can see your point :-) I did have
> expressions of interest from two people including one I thought was at your
> company, which I why I went to the effort to clean up and submit this. At
> that time I didn't realise it would be such a touchy subject.

I don't believe this topic to be touchy, it's just that Wolfgang trying
to keep the whole code base in shape has a healthy inertia before
introducing changes that may be difficult to keep consistent over the
multitude of SoCs that we support.

To throw in my personal view again, I still would like to see such an
infrastructure to get into U-Boot code.  I fully agree with Wolfgang
that practically the printfs and timing are a method already in place to
do measurements.  On the other hand my experience has shown that for
some reason or other this has never been widely used.  So effectively,
it wasn't too helpful for the project itself.  

So I still believe that if we _had_ an infrastructure like you propose,
we would get more people interested in _actually_ measuring and
improving the code base, which would certainly be a good thing.  

Of course we have systems that are very limited in their ressources, but
this is the reason why many features of U-Boot are opt-in features not
forcing any resource usage on such boards.  So if some of them are so
short on resources that they cannot use such a timing framework, then so
be it.  But as most of the probably are in the legacy code base, they
should not stop us from getting getting positive effects for the
currently important boards and architectures.

Cheers
  Detlev

-- 
A change in language can transform our appreciation of the cosmos
                       -- Benjamin Lee Whorf
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de

  parent reply	other threads:[~2011-05-17  8:20 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-13 20:51 [U-Boot] [PATCH 0/4] Accurate boot time measurement Simon Glass
2011-05-13 20:52 ` [U-Boot] [PATCH 1/4] Add microsecond " Simon Glass
2011-05-14 11:19   ` Mike Frysinger
2011-05-14 11:31   ` Mike Frysinger
2011-05-13 20:52 ` [U-Boot] [PATCH 2/4] Add bootstage progress report Simon Glass
2011-05-14 11:42   ` Mike Frysinger
2011-05-13 20:52 ` [U-Boot] [PATCH 3/4] Add bootstage calls in places appropriate for network booting Simon Glass
2011-05-14 11:44   ` Mike Frysinger
2011-05-13 20:52 ` [U-Boot] [PATCH 4/4] Add option to print boot stage report before starting kernel Simon Glass
2011-05-14 11:46   ` Mike Frysinger
2011-05-14 11:34 ` [U-Boot] [PATCH 0/4] Accurate boot time measurement Mike Frysinger
2011-05-14 17:32   ` Simon Glass
2011-05-15 10:03     ` Graeme Russ
2011-05-15 21:34       ` Simon Glass
2011-05-15 22:56         ` Graeme Russ
2011-05-16  5:56           ` Wolfgang Denk
2011-05-16  6:10             ` Graeme Russ
2011-05-15 11:53 ` Wolfgang Denk
2011-05-15 21:58   ` Simon Glass
2011-05-15 22:36     ` Graeme Russ
2011-05-16  5:55       ` Wolfgang Denk
2011-05-16  6:11         ` Graeme Russ
2011-05-16 11:18           ` Wolfgang Denk
2011-05-16  2:59     ` Mike Frysinger
2011-05-16  5:48     ` Wolfgang Denk
2011-05-16  6:23       ` Graeme Russ
2011-05-16 11:40         ` Wolfgang Denk
2011-05-16 18:25           ` Scott Wood
2011-05-16 23:09           ` Graeme Russ
2011-05-16 14:23         ` Premi, Sanjeev
2011-05-16 16:00       ` Simon Glass
2011-05-16 18:32         ` Wolfgang Denk
2011-05-16 21:35           ` Simon Glass
2011-05-17  8:20         ` Detlev Zundel [this message]
2011-05-17 17:00           ` Simon Glass
2011-05-17 22:57             ` Graeme Russ
2011-05-19  8:36             ` Detlev Zundel
2011-05-20  1:48               ` Simon Glass
2011-05-20  2:28                 ` Graeme Russ
2011-05-20  2:29                   ` Graeme Russ
2011-05-23 15:22                 ` Detlev Zundel
2011-05-24  2:55                   ` Mike Frysinger
2011-05-24  7:18                     ` Wolfgang Denk
2011-05-24 18:22                       ` Mike Frysinger
2011-05-24 19:24                         ` Wolfgang Denk

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=m2vcx9viug.fsf@ohwell.denx.de \
    --to=dzu@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