diff options
author | Christian König <christian.koenig@amd.com> | 2018-10-30 08:04:37 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2018-11-05 14:21:29 -0500 |
commit | 9340c36ca5f80e0a75856cf0aeeb9e25ce7f9574 (patch) | |
tree | 0e7b59142570be9d9f448423e654b86a51069117 /drivers/gpu/drm/amd/amdgpu | |
parent | e0519696cc31c111f11e7c67bd663dbb88d6673f (diff) |
drm/amdgpu: use ring name instead of idx in traces
Further remove using the ring index in messages and traces.
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h index e9bf70e2ac51..626abca770a0 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | |||
@@ -218,6 +218,7 @@ TRACE_EVENT(amdgpu_vm_grab_id, | |||
218 | TP_ARGS(vm, ring, job), | 218 | TP_ARGS(vm, ring, job), |
219 | TP_STRUCT__entry( | 219 | TP_STRUCT__entry( |
220 | __field(u32, pasid) | 220 | __field(u32, pasid) |
221 | __string(ring, ring->name) | ||
221 | __field(u32, ring) | 222 | __field(u32, ring) |
222 | __field(u32, vmid) | 223 | __field(u32, vmid) |
223 | __field(u32, vm_hub) | 224 | __field(u32, vm_hub) |
@@ -227,14 +228,14 @@ TRACE_EVENT(amdgpu_vm_grab_id, | |||
227 | 228 | ||
228 | TP_fast_assign( | 229 | TP_fast_assign( |
229 | __entry->pasid = vm->pasid; | 230 | __entry->pasid = vm->pasid; |
230 | __entry->ring = ring->idx; | 231 | __assign_str(ring, ring->name) |
231 | __entry->vmid = job->vmid; | 232 | __entry->vmid = job->vmid; |
232 | __entry->vm_hub = ring->funcs->vmhub, | 233 | __entry->vm_hub = ring->funcs->vmhub, |
233 | __entry->pd_addr = job->vm_pd_addr; | 234 | __entry->pd_addr = job->vm_pd_addr; |
234 | __entry->needs_flush = job->vm_needs_flush; | 235 | __entry->needs_flush = job->vm_needs_flush; |
235 | ), | 236 | ), |
236 | TP_printk("pasid=%d, ring=%u, id=%u, hub=%u, pd_addr=%010Lx needs_flush=%u", | 237 | TP_printk("pasid=%d, ring=%s, id=%u, hub=%u, pd_addr=%010Lx needs_flush=%u", |
237 | __entry->pasid, __entry->ring, __entry->vmid, | 238 | __entry->pasid, __get_str(ring), __entry->vmid, |
238 | __entry->vm_hub, __entry->pd_addr, __entry->needs_flush) | 239 | __entry->vm_hub, __entry->pd_addr, __entry->needs_flush) |
239 | ); | 240 | ); |
240 | 241 | ||
@@ -366,20 +367,20 @@ TRACE_EVENT(amdgpu_vm_flush, | |||
366 | uint64_t pd_addr), | 367 | uint64_t pd_addr), |
367 | TP_ARGS(ring, vmid, pd_addr), | 368 | TP_ARGS(ring, vmid, pd_addr), |
368 | TP_STRUCT__entry( | 369 | TP_STRUCT__entry( |
369 | __field(u32, ring) | 370 | __string(ring, ring->name) |
370 | __field(u32, vmid) | 371 | __field(u32, vmid) |
371 | __field(u32, vm_hub) | 372 | __field(u32, vm_hub) |
372 | __field(u64, pd_addr) | 373 | __field(u64, pd_addr) |
373 | ), | 374 | ), |
374 | 375 | ||
375 | TP_fast_assign( | 376 | TP_fast_assign( |
376 | __entry->ring = ring->idx; | 377 | __assign_str(ring, ring->name) |
377 | __entry->vmid = vmid; | 378 | __entry->vmid = vmid; |
378 | __entry->vm_hub = ring->funcs->vmhub; | 379 | __entry->vm_hub = ring->funcs->vmhub; |
379 | __entry->pd_addr = pd_addr; | 380 | __entry->pd_addr = pd_addr; |
380 | ), | 381 | ), |
381 | TP_printk("ring=%u, id=%u, hub=%u, pd_addr=%010Lx", | 382 | TP_printk("ring=%s, id=%u, hub=%u, pd_addr=%010Lx", |
382 | __entry->ring, __entry->vmid, | 383 | __get_str(ring), __entry->vmid, |
383 | __entry->vm_hub,__entry->pd_addr) | 384 | __entry->vm_hub,__entry->pd_addr) |
384 | ); | 385 | ); |
385 | 386 | ||