All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v1 0/5] Redesign of the xe_debugfs test
@ 2025-11-13 11:46 Piórkowski, Piotr
  2025-11-13 11:46 ` [PATCH v1 1/5] lib/igt_dir: Allow scanning only the current directory Piórkowski, Piotr
                   ` (9 more replies)
  0 siblings, 10 replies; 15+ messages in thread
From: Piórkowski, Piotr @ 2025-11-13 11:46 UTC (permalink / raw)
  To: igt-dev; +Cc: Piotr Piórkowski

From: Piotr Piórkowski <piotr.piorkowski@intel.com>

As part of the work on patch [1], it became necessary to update the
xe_debugfs test to account for the new debugfs structure. However,
while reviewing the test, I concluded that a broader redesign would make
the test easier to maintain, extend, and adapt to future debugfs changes.

As part of this redesign, I propose replacing the old subtests with a new,
clearer layout. Instead of the previous subtests:
xe-base — validate debugfs root directory and check the contents of the info file
xe-forcewake — check the forcewake node
the new subtests would be:
root-dir — validate the debugfs root directory
tile-dir — validate the debugfs directory for each tile
info-read — validate the contents of the info file

The new subtests rely on a table-driven approach: all expected debugfs
nodes are defined in an array. This allows both conditional tests
(e.g., testing vram_mm only when VRAM is available) and dynamically
generated nodes (e.g., gt%u based on gt_mask, or tile%u based on
tile_mask).

[1]: https://patchwork.freedesktop.org/series/157089/

Piotr Piórkowski (5):
  lib/igt_dir: Allow scanning only the current directory
  lib/debugfs: Add debugfs helpers to open tile directory
  tests/intel/debugfs: Rewrite tests to use array-based required debugfs
    file lists
  tests/intel/debugfs: Add tile-dir subtest for per-tile debugfs
    validation
  tests/intel/debugfs: Move VRAM MM checks from root to tile-dir
    validation

 lib/igt_debugfs.c        |  32 +++
 lib/igt_debugfs.h        |   1 +
 lib/igt_dir.c            |   1 -
 tests/intel/xe_debugfs.c | 419 +++++++++++++++++++++++++++++----------
 4 files changed, 351 insertions(+), 102 deletions(-)

-- 
2.34.1


^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2025-11-20  9:21 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-13 11:46 [PATCH v1 0/5] Redesign of the xe_debugfs test Piórkowski, Piotr
2025-11-13 11:46 ` [PATCH v1 1/5] lib/igt_dir: Allow scanning only the current directory Piórkowski, Piotr
2025-11-19 14:32   ` Bernatowicz, Marcin
2025-11-13 11:46 ` [PATCH v1 2/5] lib/debugfs: Add debugfs helpers to open tile directory Piórkowski, Piotr
2025-11-20  9:10   ` Bernatowicz, Marcin
2025-11-13 11:46 ` [PATCH v1 3/5] tests/intel/debugfs: Rewrite tests to use array-based required debugfs file lists Piórkowski, Piotr
2025-11-20  8:58   ` Bernatowicz, Marcin
2025-11-13 11:46 ` [PATCH v1 4/5] tests/intel/debugfs: Add tile-dir subtest for per-tile debugfs validation Piórkowski, Piotr
2025-11-20  9:21   ` Bernatowicz, Marcin
2025-11-13 11:46 ` [PATCH v1 5/5] tests/intel/debugfs: Move VRAM MM checks from root to tile-dir validation Piórkowski, Piotr
2025-11-13 21:14 ` ✗ Xe.CI.BAT: failure for Redesign of the xe_debugfs test Patchwork
2025-11-13 23:45 ` ✓ i915.CI.BAT: success " Patchwork
2025-11-14  4:46 ` ✗ Xe.CI.Full: failure " Patchwork
2025-11-14 17:59 ` ✗ i915.CI.Full: " Patchwork
2025-11-18 16:56 ` [PATCH v1 0/5] " Peter Senna Tschudin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.