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

Alex Deucher alexdeucher at gmail.com
Wed Mar 8 21:20:32 UTC 2017


On Sat, Feb 25, 2017 at 4:33 AM, Christian König
<deathsimple at vodafone.de> wrote:
> 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>.

Applied.  thanks!

Alex

>
>
>> ---
>>   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)
>>   );
>
>
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list