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

[Riana Tauro <riana.tauro@intel.com>]

On 9/2/2025 11:34 PM, Lucas De Marchi wrote:
> On Tue, Sep 02, 2025 at 03:17:42PM +0200, Michal Wajdeczko wrote:
>> This is a user controlled configfs attribute, we should not
>> modify that outside the configfs attr.store() implementation.
> 
> agreed... this was the first user of configfs and I don't think it was
> the right to do it. However for such a change we should propagate it to
> stable too. Riana, any possible issue in userspace? With this change
> userspace should do a rmdir() before proceeding so it drops the
> configuration.

Survivability mode is enabled in early probe and is used for firmware 
flashing by admin. If this is not cleared while exiting the mode the 
driver keeps entering survivability mode on every unbind/bind till admin 
clears the mode in configfs.

rmdir is general and can affect other entries. For survivability mode,
it's better to retain this clear.

Thanks
Riana
> 
> Lucas De Marchi
> 
>>
>> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
>> Cc: Lucas De Marchi <lucas.demarchi@intel.com>
>> Cc: Riana Tauro <riana.tauro@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);
>> }
>>
>> -- 
>> 2.47.1
>>