Re: [PATCH] drm/xe: remove unmatched xe_vm_unlock() from __xe_exec_queue_init()

[Matthew Brost <matthew.brost@intel.com>]

On Fri, May 30, 2025 at 03:56:27PM +0200, Maciej Patelczyk wrote:
> There is unmatched xe_vm_unlock() in the __xe_exec_queue_init().
> Leftover from commit fbeaad071a98 ("drm/xe: Create LRC BO without VM")
> 
> Fixes: fbeaad071a98 ("drm/xe: Create LRC BO without VM")
> Signed-off-by: Maciej Patelczyk <maciej.patelczyk@intel.com>
> Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>

Reviewed-by: Matthew Brost <matthew.brost@intel.com>
 
> ---
>  drivers/gpu/drm/xe/xe_exec_queue.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_exec_queue.c b/drivers/gpu/drm/xe/xe_exec_queue.c
> index 32c6923df7c3..fee22358cc09 100644
> --- a/drivers/gpu/drm/xe/xe_exec_queue.c
> +++ b/drivers/gpu/drm/xe/xe_exec_queue.c
> @@ -114,7 +114,6 @@ static struct xe_exec_queue *__xe_exec_queue_alloc(struct xe_device *xe,
>  
>  static int __xe_exec_queue_init(struct xe_exec_queue *q)
>  {
> -	struct xe_vm *vm = q->vm;
>  	int i, err;
>  	u32 flags = 0;
>  
> @@ -136,7 +135,7 @@ static int __xe_exec_queue_init(struct xe_exec_queue *q)
>  		q->lrc[i] = xe_lrc_create(q->hwe, q->vm, SZ_16K, q->msix_vec, flags);
>  		if (IS_ERR(q->lrc[i])) {
>  			err = PTR_ERR(q->lrc[i]);
> -			goto err_unlock;
> +			goto err_lrc;
>  		}
>  	}
>  
> @@ -146,9 +145,6 @@ static int __xe_exec_queue_init(struct xe_exec_queue *q)
>  
>  	return 0;
>  
> -err_unlock:
> -	if (vm)
> -		xe_vm_unlock(vm);
>  err_lrc:
>  	for (i = i - 1; i >= 0; --i)
>  		xe_lrc_put(q->lrc[i]);
> -- 
> 2.35.0
>