Re: [PATCH 2/2] drm/xe: Use DRM_GPUVM_RESV_PROTECTED for gpuvm

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Sat, Dec 09, 2023 at 03:49:17PM +0100, Thomas Hellström wrote:
> Use DRM_GPUVM_RESV_PROTECTED to use corse-grained locking for the
> evict and external object list.
> Since we are already holding the relevant RESV locks, for now at least,
> we don't need the fine-grained locking.
> 
> Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> ---
>  drivers/gpu/drm/xe/xe_vm.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_vm.c b/drivers/gpu/drm/xe/xe_vm.c
> index 2c35395ff5d4..c0fcb49e9137 100644
> --- a/drivers/gpu/drm/xe/xe_vm.c
> +++ b/drivers/gpu/drm/xe/xe_vm.c
> @@ -1231,8 +1231,8 @@ struct xe_vm *xe_vm_create(struct xe_device *xe, u32 flags)
>  		goto err_no_resv;
>  	}
>  
> -	drm_gpuvm_init(&vm->gpuvm, "Xe VM", 0, &xe->drm, vm_resv_obj,
> -		       0, vm->size, 0, 0, &gpuvm_ops);
> +	drm_gpuvm_init(&vm->gpuvm, "Xe VM", DRM_GPUVM_RESV_PROTECTED, &xe->drm,
> +		       vm_resv_obj, 0, vm->size, 0, 0, &gpuvm_ops);
>  
>  	drm_gem_object_put(vm_resv_obj);
>  
> -- 
> 2.42.0
>