drm/i915: improve i915_wait_request_begin trace
authorBen Widawsky <ben@bwidawsk.net>
Thu, 24 May 2012 22:03:09 +0000 (15:03 -0700)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 25 May 2012 07:55:15 +0000 (09:55 +0200)
commitf3fd37683cc406ffaccf097de0433130c85c8651
treeb3fdd17c52001bfd48edc339819f6b9a7a0022af
parent5c81fe85dad3c8c2bcec03e3fc2bfd4ea198763c
drm/i915: improve i915_wait_request_begin trace

The trace events adds whether or not the wait was blocking. Blocking in
this case means to hold struct_mutex (ie. no new work can be submitted
during the wait). The information is inherently racy.

The blocking information is racy since mutex_is_locked doesn't check
that the current thread holds the lock. The only other option would be
to pass the boolean information of whether or not the class was blocking
down through the stack which is less desirable.

v2: Don't do a trace event per loop. (Chris)
Only get blocking/non-blocking info (Chris)

v3: updated comment in code as well as commit msg (Daniel)
Add "(NB)" to trace information to remind us in 6 months (Ben)

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
Reviewed-by: Eugeni Dodonov <eugeni.dodonov@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_trace.h