[Intel-gfx] [PATCH] trace: Fixup logic inversion on setting trace_global_clock defaults

Steven Rostedt rostedt at goodmis.org
Thu Apr 5 11:48:28 UTC 2018


On Wed,  4 Apr 2018 22:24:50 +0100
Chris Wilson <chris at chris-wilson.co.uk> wrote:

> In commit 932066a15335 ("tracing: Default to using trace_global_clock if
> sched_clock is unstable"), the logic for deciding to override the
> default clock if unstable was reversed from the earlier posting. I was
> trying to reduce the width of the message by using an early return
> rather than a if-block, but reverted back to using the if-block and
> accidentally left the predicate inverted.

Hey, the inversion found some issues with global clock interacting with
lockdep, so it was a good bug to have ;-)

-- Steve


More information about the Intel-gfx mailing list