Re: [PATCH 1/3] drm/xe/configfs: Don't touch survivability_mode on fini

["Summers, Stuart" <stuart.summers@intel.com>]

On Tue, 2025-09-02 at 15:17 +0200, Michal Wajdeczko wrote:
> This is a user controlled configfs attribute, we should not
> modify that outside the configfs attr.store() implementation.
> 
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> Cc: Riana Tauro <riana.tauro@intel.com>

Yeah I agree, makes sense.

Reviewed-by: Stuart Summers <stuart.summers@intel.com>

> ---
>  drivers/gpu/drm/xe/xe_survivability_mode.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_survivability_mode.c
> b/drivers/gpu/drm/xe/xe_survivability_mode.c
> index 53c5af4b810c..79426ea46861 100644
> --- a/drivers/gpu/drm/xe/xe_survivability_mode.c
> +++ b/drivers/gpu/drm/xe/xe_survivability_mode.c
> @@ -180,7 +180,6 @@ static void xe_survivability_mode_fini(void *arg)
>         struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
>         struct device *dev = &pdev->dev;
>  
> -       xe_configfs_clear_survivability_mode(pdev);
>         sysfs_remove_file(&dev->kobj,
> &dev_attr_survivability_mode.attr);
>  }
>