Re: [PATCH 24/24] drm/xe/eudebug: Disable SVM in Xe for Eudebug

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

On Thu, Apr 30, 2026 at 01:51:20PM +0300, Mika Kuoppala wrote:
> From: Maciej Patelczyk <maciej.patelczyk@intel.com>
> 
> Missing SVM pagefault intergration. Temporary exclude SVM in Xe until
> there is SVM support in EU Debugger.
> 
> Cc: Matthew Brost <matthew.brost@intel.com>
> Cc: Gustavo Sousa <gustavo.sousa@intel.com>
> Signed-off-by: Maciej Patelczyk <maciej.patelczyk@intel.com>
> Signed-off-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> ---
>  drivers/gpu/drm/xe/Kconfig | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/Kconfig b/drivers/gpu/drm/xe/Kconfig
> index 78d4673665b4..542198a60ed9 100644
> --- a/drivers/gpu/drm/xe/Kconfig
> +++ b/drivers/gpu/drm/xe/Kconfig
> @@ -81,7 +81,7 @@ config DRM_XE_GPUSVM
>  	depends on DRM_XE
>  	depends on !UML
>  	depends on ZONE_DEVICE
> -	default y
> +	default n

If EU debugger and SVM are mutual exclusive, you will need to do the
inverse of this - default DRM_XE_EUDEBUG to false. I don't think a new
feature can turn off an existing one.

Perhaps this will be non-issue by the time of merging though as the SVM
interaction will be fixed.

Matt

>  	select DEVICE_PRIVATE
>  	select DRM_GPUSVM
>  	help
> @@ -132,6 +132,7 @@ config DRM_XE_FORCE_PROBE
>  config DRM_XE_EUDEBUG
>  	bool "Enable gdb debugger support (eudebug)"
>  	depends on DRM_XE
> +	depends on !DRM_XE_GPUSVM
>  	default y
>  	help
>  	  Choose this option if you want to add support for debugger (gdb) to
> @@ -139,6 +140,9 @@ config DRM_XE_EUDEBUG
>  	  With debugger support, Xe will provide interface for a debugger to
>  	  process to track, inspect and modify resources.
>  
> +comment "Gdb debugger support (eudebug) disabled for CPU to GPU address mirroring"
> +	depends on DRM_XE_GPUSVM
> +
>  menu "drm/Xe Debugging"
>  depends on DRM_XE
>  depends on EXPERT
> -- 
> 2.43.0
>