Re: [PATCH v3 2/2] drm/xe: Select CONFIG_DEVICE_PRIVATE when DRM_XE_GPUSVM is selected

[Matthew Auld <matthew.auld@intel.com>]

On 21/01/2026 09:10, Thomas Hellström wrote:
> CONFIG_DEVICE_PRIVATE is a prerequisite for DRM_XE_GPUSVM.
> Explicitly select it so that DRM_XE_GPUSVM is not unintentionally
> left out from distro configs not explicitly enabling
> CONFIG_DEVICE_PRIVATE.
> 
> v2:
> - Select also CONFIG_ZONE_DEVICE since it's needed by
>    CONFIG_DEVICE_PRIVATE.
> v3:
> - Depend on CONFIG_ZONE_DEVICE rather than selecting it.
> 
> Cc: Matthew Auld <matthew.auld@intel.com>
> Cc: Matthew Brost <matthew.brost@intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Cc: <dri-devel@lists.freedesktop.org>
> Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>

Reviewed-by: Matthew Auld <matthew.auld@intel.com>

> ---
>   drivers/gpu/drm/xe/Kconfig | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/Kconfig b/drivers/gpu/drm/xe/Kconfig
> index c34be1be155b..4d7dcaff2b91 100644
> --- a/drivers/gpu/drm/xe/Kconfig
> +++ b/drivers/gpu/drm/xe/Kconfig
> @@ -80,8 +80,9 @@ config DRM_XE_GPUSVM
>   	bool "Enable CPU to GPU address mirroring"
>   	depends on DRM_XE
>   	depends on !UML
> -	depends on DEVICE_PRIVATE
> +	depends on ZONE_DEVICE
>   	default y
> +	select DEVICE_PRIVATE
>   	select DRM_GPUSVM
>   	help
>   	  Enable this option if you want support for CPU to GPU address