[PATCH i-g-t] tests/intel/intel_sysfs_debugfs: Remove xe-gt
Peter Senna Tschudin
peter.senna at linux.intel.com
Thu Apr 10 09:03:15 UTC 2025
The intel_sysfs_debugfs test now includes functionality to read all Xe
debugfs files, making the older xe-gt test redundant. Additionally,
xe-gt causes issues when testing Virtual Functions (VFs) in SR-IOV
setups, as some of the debugfs files it expects are not present for VFs.
Rather than extending the overlapping and problematic xe-gt, this commit
removes it entirely.
Cc: marcin.bernatowicz at intel.com
Cc: matthew.brost at intel.com
Cc: pravalika.gurram at intel.com
Cc: kamil.konieczny at linux.intel.com
Signed-off-by: Peter Senna Tschudin <peter.senna at linux.intel.com>
---
tests/intel/intel_sysfs_debugfs.c | 66 -------------------------------
1 file changed, 66 deletions(-)
diff --git a/tests/intel/intel_sysfs_debugfs.c b/tests/intel/intel_sysfs_debugfs.c
index 6beb94109..431934aee 100644
--- a/tests/intel/intel_sysfs_debugfs.c
+++ b/tests/intel/intel_sysfs_debugfs.c
@@ -322,63 +322,6 @@ xe_test_base(int fd, struct drm_xe_query_config *config)
xe_validate_entries(fd, "", expected_files, ARRAY_SIZE(expected_files));
}
-/**
- * SUBTEST: xe-gt
- * Description: Check all gt debugfs devnodes
- * TODO: add support for ``force_reset`` entries
- */
-static void
-xe_test_gt(int fd, int gt_id)
-{
- char name[256];
- static const char * const expected_files[] = {
- "uc",
- "steering",
- "topology",
- "sa_info",
- "hw_engines",
- "pat",
- "mocs",
-// "force_reset"
- "ggtt",
- "register-save-restore",
- "workarounds",
- "default_lrc_rcs",
- "default_lrc_ccs",
- "default_lrc_bcs",
- "default_lrc_vcs",
- "default_lrc_vecs",
- "hwconfig"
-
- };
- static const char * const expected_files_uc[] = {
- "huc_info",
- "guc_log",
- "guc_info",
-// "guc_ct_selftest"
- };
-
- for (int i = 0; i < ARRAY_SIZE(expected_files); i++) {
- sprintf(name, "gt%d/%s", gt_id, expected_files[i]);
- igt_assert(igt_debugfs_exists(fd, name, O_RDONLY));
- if (igt_debugfs_is_dir(fd, expected_files[i], gt_id))
- continue;
- igt_debugfs_dump(fd, name);
- }
-
- for (int i = 0; i < ARRAY_SIZE(expected_files_uc); i++) {
- sprintf(name, "gt%d/uc/%s", gt_id, expected_files_uc[i]);
- igt_assert(igt_debugfs_exists(fd, name, O_RDONLY));
- igt_debugfs_dump(fd, name);
- }
-
- sprintf(name, "/gt%d", gt_id);
- xe_validate_entries(fd, name, expected_files, ARRAY_SIZE(expected_files));
-
- sprintf(name, "/gt%d/uc", gt_id);
- xe_validate_entries(fd, name, expected_files_uc, ARRAY_SIZE(expected_files_uc));
-}
-
/**
* SUBTEST: xe-forcewake
* Description: Check forcewake debugfs devnode
@@ -475,15 +418,6 @@ igt_main_args("", long_options, help_str, opt_handler, NULL)
xe_test_base(fd, xe_config(fd));
}
- igt_describe("Check all gt debugfs devnodes");
- igt_subtest("xe-gt") {
- xe_for_each_gt(fd, gt) {
- snprintf(devnode, sizeof(devnode), "gt%d", gt);
- igt_require(igt_debugfs_exists(fd, devnode, O_RDONLY));
- xe_test_gt(fd, gt);
- }
- }
-
igt_describe("Check forcewake debugfs devnode");
igt_subtest("xe-forcewake") {
xe_test_forcewake(fd);
--
2.43.0
More information about the igt-dev
mailing list