[PATCH 1/2] drm/amdgpu: make trace format uniform csv name=value

Christian König deathsimple at vodafone.de
Sat Feb 25 09:33:42 UTC 2017


Am 24.02.2017 um 19:20 schrieb Andres Rodriguez:
> Most of the traces have uniform format except for two of them. Having
> all the traces match makes it simple to run awk on the ftrace output.
>
> Signed-off-by: Andres Rodriguez <andresx7 at gmail.com>

Reviewed-by: Christian König <christian.koenig at amd.com>.

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
> index a18ae1e..01623d1 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
> @@ -142,7 +142,7 @@ TRACE_EVENT(amdgpu_sched_run_job,
>   			   __entry->ring_name = job->ring->name;
>   			   __entry->num_ibs = job->num_ibs;
>   			   ),
> -	    TP_printk("adev=%p, sched_job=%p, first ib=%p, sched fence=%p, ring name:%s, num_ibs:%u",
> +	    TP_printk("adev=%p, sched_job=%p, first ib=%p, sched fence=%p, ring name=%s, num_ibs=%u",
>   		      __entry->adev, __entry->sched_job, __entry->ib,
>   		      __entry->fence, __entry->ring_name, __entry->num_ibs)
>   );
> @@ -359,7 +359,7 @@ TRACE_EVENT(amdgpu_ttm_bo_move,
>   			__entry->new_placement = new_placement;
>   			__entry->old_placement = old_placement;
>   			),
> -	    TP_printk("bo=%p from:%d to %d with size = %Ld",
> +	    TP_printk("bo=%p, from=%d, to=%d, size=%Ld",
>   			__entry->bo, __entry->old_placement,
>   			__entry->new_placement, __entry->bo_size)
>   );




More information about the amd-gfx mailing list