drm/xe: Introduce xe_gt_dbg_printer()
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Mon, 9 Dec 2024 23:27:36 +0000 (15:27 -0800)
committerLucas De Marchi <lucas.demarchi@intel.com>
Wed, 11 Dec 2024 15:28:08 +0000 (07:28 -0800)
We didn't have GT-oriented debug level printer as it was hard to
correctly show actual callsite annotation. But this is now doable
from commit c2ef66e9ad88 ("drm/print: Improve drm_dbg_printer").

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20241209232739.147417-3-lucas.demarchi@intel.com
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
drivers/gpu/drm/xe/xe_gt_printk.h

index 5dc71394372d63db4d7a8c7937078056c2fbc0fc..11da0228cea71f7ff1b44aeddee77dfc36a9c0c2 100644 (file)
@@ -60,6 +60,21 @@ static inline void __xe_gt_printfn_info(struct drm_printer *p, struct va_format
        xe_gt_info(gt, "%pV", vaf);
 }
 
+static inline void __xe_gt_printfn_dbg(struct drm_printer *p, struct va_format *vaf)
+{
+       struct xe_gt *gt = p->arg;
+       struct drm_printer dbg;
+
+       /*
+        * The original xe_gt_dbg() callsite annotations are useless here,
+        * redirect to the tweaked drm_dbg_printer() instead.
+        */
+       dbg = drm_dbg_printer(&gt_to_xe(gt)->drm, DRM_UT_DRIVER, NULL);
+       dbg.origin = p->origin;
+
+       drm_printf(&dbg, "GT%u: %pV", gt->info.id, vaf);
+}
+
 /**
  * xe_gt_err_printer - Construct a &drm_printer that outputs to xe_gt_err()
  * @gt: the &xe_gt pointer to use in xe_gt_err()
@@ -90,4 +105,20 @@ static inline struct drm_printer xe_gt_info_printer(struct xe_gt *gt)
        return p;
 }
 
+/**
+ * xe_gt_dbg_printer - Construct a &drm_printer that outputs like xe_gt_dbg()
+ * @gt: the &xe_gt pointer to use in xe_gt_dbg()
+ *
+ * Return: The &drm_printer object.
+ */
+static inline struct drm_printer xe_gt_dbg_printer(struct xe_gt *gt)
+{
+       struct drm_printer p = {
+               .printfn = __xe_gt_printfn_dbg,
+               .arg = gt,
+               .origin = (const void *)_THIS_IP_,
+       };
+       return p;
+}
+
 #endif