[PATCH] drm/xe: Don't treat SR-IOV platforms as reclaim unsafe

Bernatowicz, Marcin marcin.bernatowicz at linux.intel.com
Thu Feb 6 12:05:14 UTC 2025



On 2/5/2025 1:01 PM, Michal Wajdeczko wrote:
> Since commit a4d1c5d0b99b ("drm/xe/pf: Move VFs reprovisioning
> to worker") and commit 78d5d1e20d1d ("drm/xe/relay: Don't use
> GFP_KERNEL for new transactions") we should have no more lockdep
> dependencies on the reclaim path when running in the SRIOV mode
> so we believe that we can now mark SRIOV driver as reclaim safe.
> 
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
> Cc: Thomas Hellström <thomas.hellstrom at linux.intel.com>
> Cc: Jonathan Cavitt <jonathan.cavitt at intel.com>
> Cc: Matthew Brost <matthew.brost at intel.com>
> ---
>   drivers/gpu/drm/xe/xe_pm.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_pm.c b/drivers/gpu/drm/xe/xe_pm.c
> index 35dd1757b40b..12200be7b43d 100644
> --- a/drivers/gpu/drm/xe/xe_pm.c
> +++ b/drivers/gpu/drm/xe/xe_pm.c
> @@ -91,7 +91,7 @@ static struct lockdep_map xe_pm_runtime_nod3cold_map = {
>    */
>   bool xe_rpm_reclaim_safe(const struct xe_device *xe)
>   {
> -	return !xe->d3cold.capable && !xe->info.has_sriov;
> +	return !xe->d3cold.capable;
>   }

LGTM

Tested-by: Marcin Bernatowicz <marcin.bernatowicz at linux.intel.com>
Reviewed-by: Marcin Bernatowicz <marcin.bernatowicz at linux.intel.com>

>   
>   static void xe_rpm_lockmap_acquire(const struct xe_device *xe)



More information about the Intel-xe mailing list