drm/amdgpu: remove unecessary parameters from trace_amdgpu_vm_update_ptes

Those can be determined by the parameter structure as well.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Timur Kristóf <timur.kristof@gmail.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
Christian König 2026-05-26 14:52:39 +02:00 committed by Alex Deucher
parent e12b99d7e1
commit 1a8eb6ef24
2 changed files with 6 additions and 9 deletions

View File

@ -327,9 +327,8 @@ TRACE_EVENT(amdgpu_vm_update_ptes,
TP_PROTO(struct amdgpu_vm_update_params *p,
uint64_t start, uint64_t end,
unsigned int nptes, uint64_t dst,
uint64_t incr, uint64_t flags,
pid_t pid, uint64_t vm_ctx),
TP_ARGS(p, start, end, nptes, dst, incr, flags, pid, vm_ctx),
uint64_t incr, uint64_t flags),
TP_ARGS(p, start, end, nptes, dst, incr, flags),
TP_STRUCT__entry(
__field(u64, start)
__field(u64, end)
@ -349,8 +348,9 @@ TRACE_EVENT(amdgpu_vm_update_ptes,
__entry->flags = flags;
__entry->incr = incr;
__entry->nptes = nptes;
__entry->pid = pid;
__entry->vm_ctx = vm_ctx;
__entry->pid = p->vm->task_info ?
p->vm->task_info->tgid : 0;
__entry->vm_ctx = p->vm->immediate.fence_context;
for (i = 0; i < nptes; ++i) {
u64 addr = p->pages_addr ? amdgpu_vm_map_gart(
p->pages_addr, dst) : dst;

View File

@ -882,7 +882,6 @@ int amdgpu_vm_ptes_update(struct amdgpu_vm_update_params *params,
entry_end = min(entry_end, end);
do {
struct amdgpu_vm *vm = params->vm;
uint64_t upd_end = min(entry_end, frag_end);
unsigned int nptes = (upd_end - frag_start) >> shift;
uint64_t upd_flags = flags | AMDGPU_PTE_FRAG(frag);
@ -894,9 +893,7 @@ int amdgpu_vm_ptes_update(struct amdgpu_vm_update_params *params,
trace_amdgpu_vm_update_ptes(params, frag_start, upd_end,
min(nptes, 32u), dst, incr,
upd_flags,
vm->task_info ? vm->task_info->tgid : 0,
vm->immediate.fence_context);
upd_flags);
amdgpu_vm_pte_update_flags(params, to_amdgpu_bo_vm(pt),
cursor.level, pe_start, dst,
nptes, incr, upd_flags);