* [PATCH] Bootgraph.pl bug fix
@ 2011-06-09 21:40 Andrew Murray
2011-06-10 13:00 ` Jiri Kosina
0 siblings, 1 reply; 4+ messages in thread
From: Andrew Murray @ 2011-06-09 21:40 UTC (permalink / raw)
To: trivial; +Cc: linux-embedded, linux-kernel, Andrew Murray, Andrew Murray
This patch removes the assumption of the bootgraph.pl script that the
timing information reported by PRINTK_TIME will contain at least one
entry with a time of less than 100 seconds.
Not all boards correctly reset the system timer and in many cases the
inital times reported by PRINTK_TIME is high. When this occurs the
bootchart.pl script fails to give any useful output.
This patch sets the $firsttime variable to the largest value expected
by PRINTK_TIME
Signed-off-by: Andrew Murray <amurray@mpc-data.co.uk>
---
scripts/bootgraph.pl | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/scripts/bootgraph.pl b/scripts/bootgraph.pl
index 12caa82..b78fca9 100644
--- a/scripts/bootgraph.pl
+++ b/scripts/bootgraph.pl
@@ -44,7 +44,7 @@ my %end;
my %type;
my $done = 0;
my $maxtime = 0;
-my $firsttime = 100;
+my $firsttime = 99999;
my $count = 0;
my %pids;
my %pidctr;
--
1.7.4.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] Bootgraph.pl bug fix
2011-06-09 21:40 [PATCH] Bootgraph.pl bug fix Andrew Murray
@ 2011-06-10 13:00 ` Jiri Kosina
2011-06-11 3:21 ` Arjan van de Ven
0 siblings, 1 reply; 4+ messages in thread
From: Jiri Kosina @ 2011-06-10 13:00 UTC (permalink / raw)
To: Andrew Murray
Cc: linux-embedded, linux-kernel, Andrew Murray, Arjan van de Ven
On Thu, 9 Jun 2011, Andrew Murray wrote:
> This patch removes the assumption of the bootgraph.pl script that the
> timing information reported by PRINTK_TIME will contain at least one
> entry with a time of less than 100 seconds.
>
> Not all boards correctly reset the system timer and in many cases the
> inital times reported by PRINTK_TIME is high. When this occurs the
> bootchart.pl script fails to give any useful output.
>
> This patch sets the $firsttime variable to the largest value expected
> by PRINTK_TIME
>
> Signed-off-by: Andrew Murray <amurray@mpc-data.co.uk>
> ---
> scripts/bootgraph.pl | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/scripts/bootgraph.pl b/scripts/bootgraph.pl
> index 12caa82..b78fca9 100644
> --- a/scripts/bootgraph.pl
> +++ b/scripts/bootgraph.pl
> @@ -44,7 +44,7 @@ my %end;
> my %type;
> my $done = 0;
> my $maxtime = 0;
> -my $firsttime = 100;
> +my $firsttime = 99999;
> my $count = 0;
> my %pids;
> my %pidctr;
Adding Arjan to CC. Arjan, if you are OK with the patch, I can take it
through my tree with your Ack.
Thanks,
--
Jiri Kosina
SUSE Labs
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Bootgraph.pl bug fix
2011-06-10 13:00 ` Jiri Kosina
@ 2011-06-11 3:21 ` Arjan van de Ven
2011-06-12 22:05 ` Jiri Kosina
0 siblings, 1 reply; 4+ messages in thread
From: Arjan van de Ven @ 2011-06-11 3:21 UTC (permalink / raw)
To: Jiri Kosina; +Cc: Andrew Murray, linux-embedded, linux-kernel, Andrew Murray
On Fri, 10 Jun 2011 15:00:38 +0200 (CEST)
Jiri Kosina <jkosina@suse.cz> wrote:
>
> Adding Arjan to CC. Arjan, if you are OK with the patch, I can take
> it through my tree with your Ack.
I don't mind the patch, although I feel pitty for those having to work
with such broken systems...
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Bootgraph.pl bug fix
2011-06-11 3:21 ` Arjan van de Ven
@ 2011-06-12 22:05 ` Jiri Kosina
0 siblings, 0 replies; 4+ messages in thread
From: Jiri Kosina @ 2011-06-12 22:05 UTC (permalink / raw)
To: Arjan van de Ven
Cc: Andrew Murray, linux-embedded, linux-kernel, Andrew Murray
On Fri, 10 Jun 2011, Arjan van de Ven wrote:
> > Adding Arjan to CC. Arjan, if you are OK with the patch, I can take
> > it through my tree with your Ack.
>
> I don't mind the patch, although I feel pitty for those having to work
> with such broken systems...
Indeed.
Thanks, I have applied it with your Acked-by.
--
Jiri Kosina
SUSE Labs
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-06-12 22:05 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-06-09 21:40 [PATCH] Bootgraph.pl bug fix Andrew Murray
2011-06-10 13:00 ` Jiri Kosina
2011-06-11 3:21 ` Arjan van de Ven
2011-06-12 22:05 ` Jiri Kosina
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).