On 9/23/26 21:33, Shahyan Soltani wrote:
> This is already handled in amdgpu_cs_vm_handling()

Actually it isn't handled there yet, but we need to add this.

Otherwise the CS after a GPU reset might be broken.

Regards,
Christian.

> and calling it this
> early is unnecessary. Removing it is required to allow the vm to
> initialize without waking up sleep.
> 
> Signed-off-by: Shahyan Soltani <[email protected]>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_seq64.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_seq64.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_seq64.c
> index 21a225b0116a..fbf9ec7d467b 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_seq64.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_seq64.c
> @@ -103,13 +103,6 @@ int amdgpu_seq64_map(struct amdgpu_device *adev, struct 
> amdgpu_vm *vm,
>               goto error;
>       }
>  
> -     r = amdgpu_vm_bo_update(adev, *bo_va, false);
> -     if (r) {
> -             DRM_ERROR("failed to do vm_bo_update on userq sem\n");
> -             amdgpu_vm_bo_del(adev, *bo_va);
> -             goto error;
> -     }
> -
>  error:
>       drm_exec_fini(&exec);
>       return r;

Reply via email to