[PATCH i-g-t v4 0/5] Refactor DRM Debug Severity Handling for

Pranay Samala pranay.samala at intel.com
Wed Jun 4 05:30:41 UTC 2025


This update refactors the debug mask handling in the test framework for
improved readability and addresses logical inconsistencies. The test now 
leverages the IGT_CI_RUN environment variable to control the debug mask 
dynamically, according to the test's internal requirements.

Additionally, the exit handler has been modified to restore the original
debug mask after the test completes, ensuring consistency. These changes
enhance the flexibility and robustness of the DRM debug logic, making it
easier to control the debug mask throughout the test.

v2:
- Implementated to allow flexible bitmask usage during execution. (Jani)
- Update the commit message (Jani)
- Using mask to update instead of a single category (Jani)
- To handle nested updates, exit_handler is set before the update
  process (Jani)

v3:
- Refactored for readability and address logical comment.

v4:
- Remove custom command-line option handler (Jani & Kunal)
- Have different masks documented (Kunal)
- Use an environment variable for CI jobs where disk space is tight (Kunal)
- Improve variable name for original debug mask saving (Kunal)
- Rename debug_level parameter to mask (Kunal)
 
Cc: Leo Li <sunpeng.li at amd.com>
Cc: Jani Nikula <jani.nikula at intel.com>
Cc: Uma Shankar <uma.shankar at intel.com>
Cc: Kunal Joshi <kunal1.joshi at intel.com>
Cc: Karthik B S <karthik.b.s at intel.com>
Cc: Ramanaidu Naladala <ramanaidu.naladala at intel.com>
Cc: Sameer Lattannavar <sameer.lattannavar at intel.com>

Fixes: 56b91193b825 ("lib/igt_sysfs: Implement dynamic adjustment of debug log level")
Fixes: a2ab0ec12ef4 ("tests/kms_atomic_transition: Reducing debug loglevel dynamically")
Fixes: 4baeb7397d71 ("tests/intel/kms_dp_linktrain_fallback: Reduce debug loglevel dynamically")
Fixes: 7a8a3744466f ("tests/kms_cursor_legacy: Reduce debug loglevel dynamically")

Pranay Samala (5):
  lib/igt_sysfs: Usage of Original debug mask to read/reset
  lib/igt_sysfs: Rename debug level APIs/variables to debug mask
  lib/igt_sysfs: Update new debug mask requested by user
  tests/kms: Use CI environment variable
  lib/igt_sysfs: Move igt_exit_handler into test

 lib/igt_sysfs.c                         | 63 +++++++++++++------------
 lib/igt_sysfs.h                         | 20 ++++++--
 tests/intel/kms_dp_linktrain_fallback.c | 22 +++++----
 tests/kms_atomic_transition.c           | 23 +++++----
 tests/kms_cursor_legacy.c               | 22 +++++----
 5 files changed, 89 insertions(+), 61 deletions(-)

-- 
2.34.1



More information about the igt-dev mailing list