Re: [PATCH v10 08/27] drm/xe: Remove ifdef in DRM_GPUVA_OP_DRIVER svm subop checking

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

On Thu, Sep 03, 2026 at 05:59:32PM +0300, Mika Kuoppala wrote:
> Always emit the DRM_GPUVA_OP_DRIVER case so the previous break is
> effective and we never fall through to the default warning.
> This also allows adding more subops in the future without ifdef dancing.
> 
> Cc: Matthew Auld <matthew.auld@intel.com>
> Signed-off-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> Reviewed-by: Matthew Auld <matthew.auld@intel.com>

Please let's resend this as a standalone patch, get it in and reduce
this pile.

> ---
>  drivers/gpu/drm/xe/xe_pt.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_pt.c b/drivers/gpu/drm/xe/xe_pt.c
> index 5d990c1c3740..b7749eda6490 100644
> --- a/drivers/gpu/drm/xe/xe_pt.c
> +++ b/drivers/gpu/drm/xe/xe_pt.c
> @@ -1622,7 +1622,6 @@ static int op_check_svm_userptr(struct xe_vm *vm, struct xe_vma_op *op,
>  			err = vma_check_userptr(vm, gpuva_to_vma(op->base.prefetch.va), pt_update);
>  		}
>  		break;
> -#if IS_ENABLED(CONFIG_DRM_XE_GPUSVM)
>  	case DRM_GPUVA_OP_DRIVER:
>  		if (op->subop == XE_VMA_SUBOP_MAP_RANGE) {
>  			struct xe_svm_range *range = op->map_range.range;
> @@ -1637,7 +1636,6 @@ static int op_check_svm_userptr(struct xe_vm *vm, struct xe_vma_op *op,
>  			}
>  		}
>  		break;
> -#endif
>  	default:
>  		drm_warn(&vm->xe->drm, "NOT POSSIBLE");
>  	}
> -- 
> 2.53.0
>