drm/xe: improve readability of debugfs engine info output

Improve the readability of the info debugfs output by replacing
raw numeric engine masks with human-readable engine and class
names.

Also print per-GT engine capability data in a form that is easier
for a human to interpret directly, and as a side effect simpler
for IGT tests to use when validating engine capabilities.

Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>
Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Xin Wang <x.wang@intel.com>
Link: https://patch.msgid.link/20260402180552.24121-2-x.wang@intel.com
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>

diff --git a/drivers/gpu/drm/xe/xe_debugfs.c b/drivers/gpu/drm/xe/xe_debugfs.c
index 844cfafe1ec7943fb0a43a772e2c84cdf4424fda..490d7ef7e81201062827336f14b971e0a3843de8 100644 (file)
--- a/drivers/gpu/drm/xe/xe_debugfs.c
+++ b/drivers/gpu/drm/xe/xe_debugfs.c
@@ -15,9 +15,11 @@
 #include "xe_bo.h"
 #include "xe_device.h"
 #include "xe_force_wake.h"
+#include "xe_gt.h"
 #include "xe_gt_debugfs.h"
 #include "xe_gt_printk.h"
 #include "xe_guc_ads.h"
+#include "xe_hw_engine.h"
 #include "xe_mmio.h"
 #include "xe_pm.h"
 #include "xe_psmi.h"
@@ -61,6 +63,37 @@ static struct xe_device *node_to_xe(struct drm_info_node *node)
        return to_xe_device(node->minor->dev);
 }
 
+static void print_engine_class_mask(struct drm_printer *p, u16 mask)
+{
+       if (!mask) {
+               drm_printf(p, " none\n");
+               return;
+       }
+
+       for (enum xe_engine_class ec = 0; ec < XE_ENGINE_CLASS_MAX; ec++) {
+               if (mask & BIT(ec))
+                       drm_printf(p, " %s", xe_hw_engine_class_to_str(ec));
+       }
+       drm_printf(p, "\n");
+}
+
+static void print_engine_mask(struct drm_printer *p, struct xe_gt *gt, u64 mask)
+{
+       struct xe_hw_engine *hwe;
+       enum xe_hw_engine_id id;
+
+       if (!mask) {
+               drm_printf(p, " none\n");
+               return;
+       }
+
+       for_each_hw_engine(hwe, gt, id) {
+               if (mask & BIT_ULL(id))
+                       drm_printf(p, " %s", hwe->name);
+       }
+       drm_printf(p, "\n");
+}
+
 static int info(struct seq_file *m, void *data)
 {
        struct xe_device *xe = node_to_xe(m->private);
@@ -91,10 +124,10 @@ static int info(struct seq_file *m, void *data)
        for_each_gt(gt, xe, id) {
                drm_printf(&p, "gt%d force wake %d\n", id,
                           xe_force_wake_ref(gt_to_fw(gt), XE_FW_GT));
-               drm_printf(&p, "gt%d engine_mask 0x%llx\n", id,
-                          gt->info.engine_mask);
-               drm_printf(&p, "gt%d multi_queue_engine_class_mask 0x%x\n", id,
-                          gt->info.multi_queue_engine_class_mask);
+               drm_printf(&p, "gt%d engines", id);
+               print_engine_mask(&p, gt, gt->info.engine_mask);
+               drm_printf(&p, "gt%d multi_queue_engine_classes", id);
+               print_engine_class_mask(&p, gt->info.multi_queue_engine_class_mask);
        }
 
        return 0;