Hi Slavomir,
One minor nit of this patch.
On Wed, 14 Nov 2018 17:43:28 +0200
kaslevs@xxxxxxxxxx wrote:
> diff --git a/lib/traceevent/event-parse.c b/lib/traceevent/event-parse.c
> index 078ed5e..2e8031c 100644
> --- a/lib/traceevent/event-parse.c
> +++ b/lib/traceevent/event-parse.c
> @@ -5411,7 +5411,7 @@ void tep_event_info(struct trace_seq *s, struct tep_event_format *event,
>
> static bool is_timestamp_in_us(char *trace_clock, bool use_trace_clock)
> {
> - if (!use_trace_clock)
> + if (!trace_clock || !use_trace_clock)
> return true;
>
> if (!strcmp(trace_clock, "local") || !strcmp(trace_clock, "global")
This should be a separate patch, and it needs to be pushed to the Linux
kernel. Can you work with Tzvetomir in doing it?
Thanks!
-- Steve
|