[Intel-gfx] [PATCH 26/33] drm/i915: Remove the intel_engine_notify tracepoint
Tvrtko Ursulin
tvrtko.ursulin at linux.intel.com
Fri Jan 25 14:10:38 UTC 2019
On 25/01/2019 02:29, Chris Wilson wrote:
> The global seqno is defunct and so we have no meaningful indicator of
> forward progress for an engine. You need to listen to the request
> signaling tracepoints instead.
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---
> drivers/gpu/drm/i915/i915_irq.c | 2 --
> drivers/gpu/drm/i915/i915_trace.h | 25 -------------------------
> 2 files changed, 27 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
> index 7e56611b3d60..bedac6c14928 100644
> --- a/drivers/gpu/drm/i915/i915_irq.c
> +++ b/drivers/gpu/drm/i915/i915_irq.c
> @@ -1211,8 +1211,6 @@ static void notify_ring(struct intel_engine_cs *engine)
> wake_up_process(tsk);
>
> rcu_read_unlock();
> -
> - trace_intel_engine_notify(engine, wait);
> }
>
> static void vlv_c0_read(struct drm_i915_private *dev_priv,
> diff --git a/drivers/gpu/drm/i915/i915_trace.h b/drivers/gpu/drm/i915/i915_trace.h
> index 43da14f08dc0..eab313c3163c 100644
> --- a/drivers/gpu/drm/i915/i915_trace.h
> +++ b/drivers/gpu/drm/i915/i915_trace.h
> @@ -752,31 +752,6 @@ trace_i915_request_out(struct i915_request *rq)
> #endif
> #endif
>
> -TRACE_EVENT(intel_engine_notify,
> - TP_PROTO(struct intel_engine_cs *engine, bool waiters),
> - TP_ARGS(engine, waiters),
> -
> - TP_STRUCT__entry(
> - __field(u32, dev)
> - __field(u16, class)
> - __field(u16, instance)
> - __field(u32, seqno)
> - __field(bool, waiters)
> - ),
> -
> - TP_fast_assign(
> - __entry->dev = engine->i915->drm.primary->index;
> - __entry->class = engine->uabi_class;
> - __entry->instance = engine->instance;
> - __entry->seqno = intel_engine_get_seqno(engine);
> - __entry->waiters = waiters;
> - ),
> -
> - TP_printk("dev=%u, engine=%u:%u, seqno=%u, waiters=%u",
> - __entry->dev, __entry->class, __entry->instance,
> - __entry->seqno, __entry->waiters)
> -);
> -
> DEFINE_EVENT(i915_request, i915_request_retire,
> TP_PROTO(struct i915_request *rq),
> TP_ARGS(rq)
>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
I think I checked that trace_dma_fence_singal will allow me to fix
trace.pl.. I think I have.
Regards,
Tvrtko
More information about the Intel-gfx
mailing list