drm/i915: Kill the get_fence tracepoint
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 12 Nov 2010 15:53:11 +0000 (15:53 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Thu, 2 Dec 2010 10:20:47 +0000 (10:20 +0000)
As the tracepoint is now decoupled from when the actual register is
assigned and was never complemented by detailing when the object lost
its fence, it has outlived its limited usefulness. Profiling the actual
stalls is a far more profitable venture anyway.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
drivers/gpu/drm/i915/i915_gem.c
drivers/gpu/drm/i915/i915_trace.h

index 9e036010e3fc408c797d984c75cc625087301173..d99212fe54edfd8bfbac6153d572fa1f1f10e433 100644 (file)
@@ -2603,9 +2603,6 @@ update:
                break;
        }
 
-       trace_i915_gem_object_get_fence(obj,
-                                       obj->fence_reg,
-                                       obj->tiling_mode);
        return ret;
 }
 
index 1df7262ae0775dceeb12d0466712fea7330b4632..7f0fc3ed61aa73e49a8340d6e17673bdc3df2fe2 100644 (file)
@@ -80,28 +80,6 @@ TRACE_EVENT(i915_gem_object_change_domain,
                      __entry->read_domains, __entry->write_domain)
 );
 
-TRACE_EVENT(i915_gem_object_get_fence,
-
-           TP_PROTO(struct drm_i915_gem_object *obj, int fence, int tiling_mode),
-
-           TP_ARGS(obj, fence, tiling_mode),
-
-           TP_STRUCT__entry(
-                            __field(struct drm_i915_gem_object *, obj)
-                            __field(int, fence)
-                            __field(int, tiling_mode)
-                            ),
-
-           TP_fast_assign(
-                          __entry->obj = obj;
-                          __entry->fence = fence;
-                          __entry->tiling_mode = tiling_mode;
-                          ),
-
-           TP_printk("obj=%p, fence=%d, tiling=%d",
-                     __entry->obj, __entry->fence, __entry->tiling_mode)
-);
-
 DECLARE_EVENT_CLASS(i915_gem_object,
 
            TP_PROTO(struct drm_i915_gem_object *obj),