Re: [PATCH v2 2/2] trace: Add an option for boot clock as trace clock

From: Steven Rostedt
Date: Mon Nov 21 2016 - 17:26:20 EST


On Mon, 21 Nov 2016 14:12:50 -0800
Joel Fernandes <joelaf@xxxxxxxxxx> wrote:

> Unlike monotonic clock, boot clock as a trace clock will account for
> time spent in suspend useful for tracing suspend/resume. This uses
> earlier introduced infrastructure for using the fast boot clock.
>
> Cc: Steven Rostedt <rostedt@xxxxxxxxxxx>
> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: John Stultz <john.stultz@xxxxxxxxxx>
> Cc: Ingo Molnar <mingo@xxxxxxxxxx>
> Signed-off-by: Joel Fernandes <joelaf@xxxxxxxxxx>
> ---
> kernel/trace/trace.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c
> index 8696ce6..f7b64db 100644
> --- a/kernel/trace/trace.c
> +++ b/kernel/trace/trace.c
> @@ -1125,6 +1125,7 @@ static struct {
> { trace_clock, "perf", 1 },
> { ktime_get_mono_fast_ns, "mono", 1 },
> { ktime_get_raw_fast_ns, "mono_raw", 1 },
> + { ktime_get_boot_fast_ns, "boot", 1 },
> ARCH_TRACE_CLOCKS
> };
>

I'm fine with this change, but can you please update
Documentation/trace/ftrace.txt under trace_clock. I notice that mono
and mono_raw are missing too.

-- Steve