[Intel-gfx] [PATCH 2/2] drm/i915: Remove defunct trace points

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Wed Oct 4 09:39:08 UTC 2017


On 03/10/2017 13:50, Chris Wilson wrote:
> trace_i915_gem_evict_everything and trace_i915_gem_ring_flush stopped
> being used when their parent functions were removed.
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> ---
>   drivers/gpu/drm/i915/i915_trace.h | 72 +++++++++------------------------------
>   1 file changed, 17 insertions(+), 55 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_trace.h b/drivers/gpu/drm/i915/i915_trace.h
> index cca73636a747..9cab91ddeb79 100644
> --- a/drivers/gpu/drm/i915/i915_trace.h
> +++ b/drivers/gpu/drm/i915/i915_trace.h
> @@ -539,38 +539,6 @@ TRACE_EVENT(i915_gem_evict,
>   		      __entry->flags & PIN_MAPPABLE ? ", mappable" : "")
>   );
>   
> -TRACE_EVENT(i915_gem_evict_everything,
> -	    TP_PROTO(struct drm_device *dev),
> -	    TP_ARGS(dev),
> -
> -	    TP_STRUCT__entry(
> -			     __field(u32, dev)
> -			    ),
> -
> -	    TP_fast_assign(
> -			   __entry->dev = dev->primary->index;
> -			  ),
> -
> -	    TP_printk("dev=%d", __entry->dev)
> -);
> -
> -TRACE_EVENT(i915_gem_evict_vm,
> -	    TP_PROTO(struct i915_address_space *vm),
> -	    TP_ARGS(vm),
> -
> -	    TP_STRUCT__entry(
> -			     __field(u32, dev)
> -			     __field(struct i915_address_space *, vm)
> -			    ),
> -
> -	    TP_fast_assign(
> -			   __entry->dev = vm->i915->drm.primary->index;
> -			   __entry->vm = vm;
> -			  ),
> -
> -	    TP_printk("dev=%d, vm=%p", __entry->dev, __entry->vm)
> -);
> -
>   TRACE_EVENT(i915_gem_evict_node,
>   	    TP_PROTO(struct i915_address_space *vm, struct drm_mm_node *node, unsigned int flags),
>   	    TP_ARGS(vm, node, flags),
> @@ -599,6 +567,23 @@ TRACE_EVENT(i915_gem_evict_node,
>   		      __entry->color, __entry->flags)
>   );
>   
> +TRACE_EVENT(i915_gem_evict_vm,
> +	    TP_PROTO(struct i915_address_space *vm),
> +	    TP_ARGS(vm),
> +
> +	    TP_STRUCT__entry(
> +			     __field(u32, dev)
> +			     __field(struct i915_address_space *, vm)
> +			    ),
> +
> +	    TP_fast_assign(
> +			   __entry->dev = vm->i915->drm.primary->index;
> +			   __entry->vm = vm;
> +			  ),
> +
> +	    TP_printk("dev=%d, vm=%p", __entry->dev, __entry->vm)
> +);
> +
>   TRACE_EVENT(i915_gem_ring_sync_to,
>   	    TP_PROTO(struct drm_i915_gem_request *to,
>   		     struct drm_i915_gem_request *from),
> @@ -649,29 +634,6 @@ TRACE_EVENT(i915_gem_request_queue,
>   		      __entry->flags)
>   );
>   
> -TRACE_EVENT(i915_gem_ring_flush,
> -	    TP_PROTO(struct drm_i915_gem_request *req, u32 invalidate, u32 flush),
> -	    TP_ARGS(req, invalidate, flush),
> -
> -	    TP_STRUCT__entry(
> -			     __field(u32, dev)
> -			     __field(u32, ring)
> -			     __field(u32, invalidate)
> -			     __field(u32, flush)
> -			     ),
> -
> -	    TP_fast_assign(
> -			   __entry->dev = req->i915->drm.primary->index;
> -			   __entry->ring = req->engine->id;
> -			   __entry->invalidate = invalidate;
> -			   __entry->flush = flush;
> -			   ),
> -
> -	    TP_printk("dev=%u, ring=%x, invalidate=%04x, flush=%04x",
> -		      __entry->dev, __entry->ring,
> -		      __entry->invalidate, __entry->flush)
> -);
> -
>   DECLARE_EVENT_CLASS(i915_gem_request,
>   	    TP_PROTO(struct drm_i915_gem_request *req),
>   	    TP_ARGS(req),
> 

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>

Regards,

Tvrtko


More information about the Intel-gfx mailing list