Re: [PATCH] drm/xe: Reorder 'Get pages failed' message

["Ghimiray, Himal Prasad" <himal.prasad.ghimiray@intel.com>]

On 10-06-2025 10:26, Matthew Brost wrote:
> Print the error from get pages failing, not the cast to -ENODATA.

Reviewed-by: Himal Prasad Ghimiray <himal.prasad.ghimiray@intel.com>>
> Signed-off-by: Matthew Brost <matthew.brost@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 18f967ce1f1a..d18807b92b18 100644
> --- a/drivers/gpu/drm/xe/xe_vm.c
> +++ b/drivers/gpu/drm/xe/xe_vm.c
> @@ -2924,10 +2924,10 @@ static int prefetch_ranges(struct xe_vm *vm, struct xe_vma_op *op)
>   
>   		err = xe_svm_range_get_pages(vm, svm_range, &ctx);
>   		if (err) {
> -			if (err == -EOPNOTSUPP || err == -EFAULT || err == -EPERM)
> -				err = -ENODATA;
>   			drm_dbg(&vm->xe->drm, "Get pages failed, asid=%u, gpusvm=%p, errno=%pe\n",
>   				vm->usm.asid, &vm->svm.gpusvm, ERR_PTR(err));
> +			if (err == -EOPNOTSUPP || err == -EFAULT || err == -EPERM)
> +				err = -ENODATA;
>   			return err;
>   		}
>   		xe_svm_range_debug(svm_range, "PREFETCH - RANGE GET PAGES DONE");