Re: [PATCH 4/5] tracing/fastboot: Add a time field on the sched_switch entry

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Frédéric Weisbecker
Date: Saturday, October 18, 2008 - 3:34 pm

2008/10/18 Tim Bird <tim.bird@am.sony.com>:

Actually the problem is not really to have a zero-based timestamp but I need
to have a timestamp which is the same than the one used by the boot tracer.

If the boot tracer had only to capture one time per entry, I would use
the internal
timestamp of the ring-buffer. But the boot tracer must capture two
times: the call
time of an initcall and the time it returns.

At the same time I want to know the schedule events that occur during
the initcalls
and then the latency they raise. For that I need to have the same
timestamp than the boot
tracer to have a consistent view of these schedule events inside the initcalls.

But you just gave me a new idea :-)
If I split the initcall in two parts: the call and the return, I will
be able to drop these
boot-tracer-specific timestamps and just use the ring-buffer timestamp used
when an entry is appended into the buffer.
And so I can use the usual sched_switch events whithout additional field.

Seems to me really better.

But there is one thing that changes with this new way. It seems that
the ring buffer's timestamp
is based on the sched-clock and so I guess that's why it doesn't start
when the cpu boot but some
times after.

Arjan would you agree with this new time capture system?
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 4/5] tracing/fastboot: Add a time field on the sche ..., Frederic Weisbecker, (Sat Oct 11, 1:20 pm)
Re: [PATCH 4/5] tracing/fastboot: Add a time field on the ..., Frédéric Weisbecker, (Sat Oct 11, 1:58 pm)
Re: [PATCH 4/5] tracing/fastboot: Add a time field on the ..., Frédéric Weisbecker, (Sun Oct 12, 4:29 am)
Re: [PATCH 4/5] tracing/fastboot: Add a time field on the ..., Frédéric Weisbecker, (Mon Oct 13, 1:10 am)
Re: [PATCH 4/5] tracing/fastboot: Add a time field on the ..., Frédéric Weisbecker, (Sat Oct 18, 3:34 pm)