drm/i915: Kill the get_fence tracepoint

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>

diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index 9e03601..d99212f 100644 (file)
--- a/drivers/gpu/drm/i915/i915_gem.c
+++ b/drivers/gpu/drm/i915/i915_gem.c
@@ -2603,9 +2603,6 @@ update:
                break;
        }
 
-       trace_i915_gem_object_get_fence(obj,
-                                       obj->fence_reg,
-                                       obj->tiling_mode);
        return ret;
 }
 

diff --git a/drivers/gpu/drm/i915/i915_trace.h b/drivers/gpu/drm/i915/i915_trace.h
index 1df7262..7f0fc3e 100644 (file)
--- a/drivers/gpu/drm/i915/i915_trace.h
+++ b/drivers/gpu/drm/i915/i915_trace.h
@@ -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),