From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> This Kconfig option was added to protect the implementation specific internals from user expectations but so far it was mostly hassle. Remove it so it is possible to debug request submission on any kernel anywhere. This adds around 4k to default i915.ko build but should have no performance effects due inactive tracepoints being no-op-ed out and out- of-line. Users should remember tracepoints which are close to low level i915 implementation details are subject to change and cannot be guaranteed. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> --- drivers/gpu/drm/i915/Kconfig.debug | 11 ----------- drivers/gpu/drm/i915/i915_trace.h | 20 -------------------- 2 files changed, 31 deletions(-) diff --git a/drivers/gpu/drm/i915/Kconfig.debug b/drivers/gpu/drm/i915/Kconfig.debug index 9de8b1c51a5c..058094235329 100644 --- a/drivers/gpu/drm/i915/Kconfig.debug +++ b/drivers/gpu/drm/i915/Kconfig.debug @@ -132,17 +132,6 @@ config DRM_I915_SELFTEST_BROKEN If in doubt, say "N". -config DRM_I915_LOW_LEVEL_TRACEPOINTS - bool "Enable low level request tracing events" - depends on DRM_I915 - default n - help - Choose this option to turn on low level request tracing events. - This provides the ability to precisely monitor engine utilisation - and also analyze the request dependency resolving timeline. - - If in doubt, say "N". - config DRM_I915_DEBUG_VBLANK_EVADE bool "Enable extra debug warnings for vblank evasion" depends on DRM_I915 diff --git a/drivers/gpu/drm/i915/i915_trace.h b/drivers/gpu/drm/i915/i915_trace.h index 4a6a15075afa..c0352a1b036c 100644 --- a/drivers/gpu/drm/i915/i915_trace.h +++ b/drivers/gpu/drm/i915/i915_trace.h @@ -679,7 +679,6 @@ DEFINE_EVENT(i915_request, i915_request_add, TP_ARGS(rq) ); -#if defined(CONFIG_DRM_I915_LOW_LEVEL_TRACEPOINTS) DEFINE_EVENT(i915_request, i915_request_submit, TP_PROTO(struct i915_request *rq), TP_ARGS(rq) @@ -751,25 +750,6 @@ TRACE_EVENT(i915_request_out, __entry->global_seqno, __entry->completed) ); -#else -#if !defined(TRACE_HEADER_MULTI_READ) -static inline void -trace_i915_request_submit(struct i915_request *rq) -{ -} - -static inline void -trace_i915_request_in(struct i915_request *rq, unsigned int port) -{ -} - -static inline void -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), -- 2.17.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx