[PATCH] drm/amdgpu: Use the TGID for trace_amdgpu_vm_update_ptes

[Christian König]

Am 02.02.23 um 17:21 schrieb Friedrich Vock:
> The pid field corresponds to the result of gettid() in userspace.
> However, userspace cannot reliably attribute PTE events to processes
> with just the thread id. This patch allows userspace to easily
> attribute PTE update events to specific processes by comparing this
> field with the result of getpid().
>
> For attributing events to specific threads, the thread id is also
> contained in the common fields of each trace event.
>
> Signed-off-by: Friedrich Vock <friedrich.vock at gmx.de>

Ah, yes that makes more sense. Reviewed-by: Christian König 
<christian.koenig at amd.com>

Alex do you pick this up or should I take care of it?

Thanks,
Christian.

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c
> index b5f3bba851db..01e42bdd8e4e 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c
> @@ -974,7 +974,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.pid,
> +						    vm->task_info.tgid,
>   						    vm->immediate.fence_context);
>   			amdgpu_vm_pte_update_flags(params, to_amdgpu_bo_vm(pt),
>   						   cursor.level, pe_start, dst,
> --
> 2.39.1
>