Re: [PATCH v2] drm/amdgpu/userq: update the vm task info during signal ioctl

2026-05-12 Thread Christian König
On 5/12/26 12:51, Sunil Khatri wrote:
> Pagefaults does not have process information correctly populated
> as vm->task is not set during vm_init but should be updated while
> real submission. So setting that up during signal_ioctl to get
> the correct submission process details.
> 
> Signed-off-by: Sunil Khatri 

Reviewed-by: Christian König 

> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c
> index 6000dee5ee3d..f62163917f70 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c
> @@ -802,6 +802,9 @@ amdgpu_userq_create(struct drm_file *filp, union 
> drm_amdgpu_userq *args)
>   goto clean_fence_driver;
>   }
>  
> + /* Update VM owner at userq submit-time for page-fault attribution. */
> + amdgpu_vm_set_task_info(&fpriv->vm);
> +
>   amdgpu_userq_ensure_ev_fence(&fpriv->userq_mgr, &fpriv->evf_mgr);
>  
>   /* don't map the queue if scheduling is halted */



[PATCH v2] drm/amdgpu/userq: update the vm task info during signal ioctl

2026-05-12 Thread Sunil Khatri
Pagefaults does not have process information correctly populated
as vm->task is not set during vm_init but should be updated while
real submission. So setting that up during signal_ioctl to get
the correct submission process details.

Signed-off-by: Sunil Khatri 
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c
index 6000dee5ee3d..f62163917f70 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c
@@ -802,6 +802,9 @@ amdgpu_userq_create(struct drm_file *filp, union 
drm_amdgpu_userq *args)
goto clean_fence_driver;
}
 
+   /* Update VM owner at userq submit-time for page-fault attribution. */
+   amdgpu_vm_set_task_info(&fpriv->vm);
+
amdgpu_userq_ensure_ev_fence(&fpriv->userq_mgr, &fpriv->evf_mgr);
 
/* don't map the queue if scheduling is halted */
-- 
2.34.1