[PATCH i-g-t v3 5/5] tests/intel/xe_configfs: Add test to validate survivability mode

[Riana Tauro]

Hi Rodrigo

On 5/14/2025 7:54 PM, Rodrigo Vivi wrote:
> On Tue, May 13, 2025 at 07:58:25PM +0530, Riana Tauro wrote:
>> The test validates if survivability mode is enabled on supported
>> platforms when configured using configfs attribute.
>>
>> v2: make test platform specific
>>      check for presence of file for survivability mode
>>      use mode_t flags (Aravind)
>>
>> Signed-off-by: Riana Tauro <riana.tauro at intel.com>
>> ---
>>   tests/intel/xe_configfs.c | 100 ++++++++++++++++++++++++++++++++++++++
>>   tests/meson.build         |   1 +
>>   2 files changed, 101 insertions(+)
>>   create mode 100644 tests/intel/xe_configfs.c
>>
>> diff --git a/tests/intel/xe_configfs.c b/tests/intel/xe_configfs.c
>> new file mode 100644
>> index 000000000..4d244321c
>> --- /dev/null
>> +++ b/tests/intel/xe_configfs.c
>> @@ -0,0 +1,100 @@
>> +// SPDX-License-Identifier: MIT
>> +/*
>> + * Copyright © 2025 Intel Corporation
>> + */
>> +#include <limits.h>
>> +
>> +#include "igt.h"
>> +#include "igt_configfs.h"
>> +#include "igt_device.h"
>> +#include "igt_fs.h"
>> +#include "igt_kmod.h"
>> +#include "igt_sysfs.h"
>> +
>> +/**
>> + * TEST: Check configfs userspace API
>> + * Category: Core
>> + * Mega feature: General Core features
>> + * Sub-category: uapi
>> + * Functionality: configfs
>> + * Description: validate configfs entries
>> + * Test category: functionality test
>> + */
>> +
>> +static char bus_addr[NAME_MAX];
>> +
>> +static void restore(int sig)
>> +{
>> +	/* Restore after survivability mode */
>> +	igt_kmod_unbind("xe", bus_addr);
>> +	igt_kmod_bind("xe", bus_addr);
>> +}
>> +
>> +static void set_survivability_mode(int dir_fd, bool value)
>> +{
>> +	igt_kmod_unbind("xe", bus_addr);
>> +	igt_sysfs_set_boolean(dir_fd, "survivability_mode", value);
>> +	igt_kmod_bind("xe", bus_addr);
>> +}
>> +
>> +/**
>> + * SUBTEST: survivability-mode
>> + * Description: Validate survivability mode by setting configfs
>> + */
>> +static void test_survivability_mode(int dir_fd)
>> +{
>> +	char path[PATH_MAX];
>> +	int fd;
>> +
>> +	/* Enable survivability mode */
>> +	set_survivability_mode(dir_fd, true);
>> +
>> +	/* check presence of survivability mode sysfs */
>> +	snprintf(path, PATH_MAX, "/sys/bus/pci/devices/%s/survivability_mode", bus_addr);
>> +
>> +	fd = open(path, O_RDONLY);
>> +	igt_assert_f(fd >= 0, "Survivability mode not set\n");
>> +	close(fd);
>> +}
>> +
>> +static int create_device_configfs(int configfs_fd, int fd)
>> +{
>> +	int dir_fd;
>> +	struct pci_device *pci_dev;
>> +	mode_t mode = S_IRWXU | S_IRGRP | S_IXGRP | S_IROTH | S_IXOTH;
>> +
>> +	pci_dev = igt_device_get_pci_device(fd);
>> +	snprintf(bus_addr, sizeof(bus_addr), "%04x:%02x:%02x.%01x",
>> +		 pci_dev->domain, pci_dev->bus, pci_dev->dev, pci_dev->func);
>> +
>> +	dir_fd = igt_fs_create_dir(configfs_fd, bus_addr, mode);
>> +	igt_assert(dir_fd);
>> +
>> +	return dir_fd;
>> +}
> 
> This could probably be in /lib/ (in the future)
> 
> Let's do this later whenever we have other cases, and then
> think about the clean up with multiple users...

I had added the generic create directory in lib/igt_configfs in previous 
revision. Then moved it to igt_fs as it can be reused

Device specific configfs creation might not be useful for other configfs 
using igt(ex: vkms tests). We could add it under lib/xe/ if there are 
multiple users in the future.>
>> +
>> +igt_main
>> +{
>> +	int fd, configfs_fd, dir_fd;
>> +
>> +	igt_fixture {
>> +		fd = drm_open_driver(DRIVER_XE);
>> +		configfs_fd = igt_configfs_open("xe");
>> +		igt_require(configfs_fd != -1);
>> +		dir_fd = create_device_configfs(configfs_fd, fd);
>> +	}
>> +
>> +	igt_describe("Validate survivability mode");
>> +	igt_subtest("survivability-mode") {
>> +		igt_require(IS_BATTLEMAGE(intel_get_drm_devid(fd)));
>> +		igt_install_exit_handler(restore);
>> +		test_survivability_mode(dir_fd);
> 
> I believe the only thing confusing here is the name of this variable.
> I was expecting that to be the config_fs and it was hard to follow
> 
> So, perhaps s/dir_fd/configfs_device_fd/g ?!
Will add this

Thank you for the review

Thanks,
Riana>
> up to you:
> 
> Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> 
>> +	}
>> +
>> +	igt_fixture {
>> +		igt_fs_remove_dir(configfs_fd, bus_addr);
>> +		close(dir_fd);
>> +		close(configfs_fd);
>> +		close(fd);
>> +	}
>> +}
>> diff --git a/tests/meson.build b/tests/meson.build
>> index 20ddddb89..55bcf57ec 100644
>> --- a/tests/meson.build
>> +++ b/tests/meson.build
>> @@ -279,6 +279,7 @@ intel_xe_progs = [
>>   	'xe_compute',
>>   	'xe_compute_preempt',
>>   	'xe_copy_basic',
>> +	'xe_configfs',
>>   	'xe_dma_buf_sync',
>>   	'xe_drm_fdinfo',
>>   	'xe_eu_stall',
>> -- 
>> 2.47.1
>>