Skip to content

Commit

Permalink
drm/i915: Kill the get_fence tracepoint
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Chris Wilson committed Dec 2, 2010
1 parent c6748e0 commit 60de2ba
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 25 deletions.
3 changes: 0 additions & 3 deletions drivers/gpu/drm/i915/i915_gem.c
Original file line number Diff line number Diff line change
Expand Up @@ -2603,9 +2603,6 @@ i915_gem_object_get_fence(struct drm_i915_gem_object *obj,
break;
}

trace_i915_gem_object_get_fence(obj,
obj->fence_reg,
obj->tiling_mode);
return ret;
}

Expand Down
22 changes: 0 additions & 22 deletions drivers/gpu/drm/i915/i915_trace.h
Original file line number Diff line number Diff line change
Expand Up @@ -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),
Expand Down

0 comments on commit 60de2ba

Please sign in to comment.