From: Nareshkumar Gollakoti <naresh.kumar.g@intel.com>
To: intel-xe@lists.freedesktop.org
Cc: tejas.upadhyay@intel.com,
Nareshkumar Gollakoti <naresh.kumar.g@intel.com>
Subject: [PATCH v6 6/7] drm/xe: capture L3 node status registers in devcoredump
Date: Tue, 27 Oct 2026 10:40:23 +0530 [thread overview]
Message-ID: <20261027051024.465587-7-naresh.kumar.g@intel.com> (raw)
In-Reply-To: <20261027051024.465587-1-naresh.kumar.g@intel.com>
Add L3_NODE_IDLE and L3STAT_LBCF_GFX register definitions and include
them in the devcoredump snapshot as a node status section.
Extend the register descriptor to mark MCR registers so the snapshot
code can read them through xe_gt_mcr_unicast_read_any(), then print and
free the captured node status registers alongside the existing dump
sections.
v2:(Sashiko)
- change L3 node registers as XE_REG_MCR type
- identify mcr register type from reg.mcr
- As generic capture function ensure VF check this will not execute
on VF Mode.
v4:(Tejas)
- Add L3_NODE type into the enum as changed/added in initial patch
v5:
- Rebase
Signed-off-by: Nareshkumar Gollakoti <naresh.kumar.g@intel.com>
---
drivers/gpu/drm/xe/regs/xe_gt_regs.h | 3 +++
drivers/gpu/drm/xe/xe_devcoredump.c | 21 ++++++++++++++++++++-
drivers/gpu/drm/xe/xe_devcoredump_types.h | 2 ++
3 files changed, 25 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/xe/regs/xe_gt_regs.h b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
index f884fc2e38aaf..28331505bb34e 100644
--- a/drivers/gpu/drm/xe/regs/xe_gt_regs.h
+++ b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
@@ -454,6 +454,9 @@
#define L3_ESC_MASK REG_BIT(0)
#define L3_ESC(value) REG_FIELD_PREP(L3_ESC_MASK, value)
+#define L3_NODE_IDLE XE_REG_MCR(0xb0b0)
+#define L3STAT_LBCF_GFX XE_REG_MCR(0xb128)
+
#define XEHP_L3NODEARBCFG XE_REG_MCR(0xb0b4)
#define XEHP_LNESPARE REG_BIT(19)
diff --git a/drivers/gpu/drm/xe/xe_devcoredump.c b/drivers/gpu/drm/xe/xe_devcoredump.c
index 90e16c89dd963..714b63e9653df 100644
--- a/drivers/gpu/drm/xe/xe_devcoredump.c
+++ b/drivers/gpu/drm/xe/xe_devcoredump.c
@@ -121,6 +121,11 @@ static const struct xe_reg_desc xe_dbg_gam_pf_reglist[] = {
XE_REG_DESC_MCR(PAGE_FLTRPTQ_TAIL, "PAGE_FLTRPTQ_TAIL"),
};
+static const struct xe_reg_desc xe_dbg_l3_node_reglist[] = {
+ XE_REG_DESC_MCR(L3_NODE_IDLE, "L3_NODE_IDLE"),
+ XE_REG_DESC_MCR(L3STAT_LBCF_GFX, "L3STAT_LBCF_GFX"),
+};
+
/**
* struct xe_gt_regset - Struct for register descriptor list for GT
*
@@ -141,6 +146,7 @@ struct xe_gt_regset {
* @XE_DBG_REGSET_RPM: RPM register set
* @XE_DBG_REGSET_GUC: GUC register set
* @XE_DBG_REGSET_GAM_PF: GAM Pagefault registers set
+ * @XE_DBG_REGSET_L3_NODE: L3 Node register Set
* @XE_DBG_REGSET_MAX: Number of debug register set types
*/
enum xe_dbg_regset_type {
@@ -148,6 +154,7 @@ enum xe_dbg_regset_type {
XE_DBG_REGSET_RPM,
XE_DBG_REGSET_GUC,
XE_DBG_REGSET_GAM_PF,
+ XE_DBG_REGSET_L3_NODE,
XE_DBG_REGSET_MAX,
};
@@ -174,6 +181,10 @@ static const struct xe_gt_regset xe_dbg_regsets[XE_DBG_REGSET_MAX] = {
.gt[XE_GT_TYPE_MAIN] = { xe_dbg_gam_pf_reglist,
ARRAY_SIZE(xe_dbg_gam_pf_reglist) },
},
+ [XE_DBG_REGSET_L3_NODE] = {
+ .gt[XE_GT_TYPE_MAIN] = { xe_dbg_l3_node_reglist,
+ ARRAY_SIZE(xe_dbg_l3_node_reglist) },
+ },
};
static void xe_dbg_reg_snapshot_print(struct drm_printer *p,
@@ -352,6 +363,8 @@ static ssize_t __xe_devcoredump_read(char *buffer, ssize_t count,
xe_gam_pf_snapshot_print(&p, "GAM PF Report", ss->gam_pf_report);
+ xe_dbg_reg_snapshot_print(&p, "Node Status", ss->ns);
+
drm_puts(&p, "\n**** Contexts ****\n");
xe_guc_exec_queue_snapshot_print(ss->ge, &p);
@@ -409,6 +422,9 @@ static void xe_devcoredump_snapshot_free(struct xe_devcoredump_snapshot *ss)
xe_dbg_reg_snapshot_free(ss->gam_pf_report);
ss->gam_pf_report = NULL;
+ xe_dbg_reg_snapshot_free(ss->ns);
+ ss->ns = NULL;
+
xe_guc_exec_queue_snapshot_free(ss->ge);
ss->ge = NULL;
@@ -609,6 +625,10 @@ static void xe_devcoredump_capture_gt_regs(struct xe_devcoredump_snapshot *ss,
other_gt,
XE_DBG_REGSET_GAM_PF,
XE_GT_TYPE_MAIN);
+ ss->ns = xe_dbg_gt_snapshot_capture(gt,
+ other_gt,
+ XE_DBG_REGSET_L3_NODE,
+ XE_GT_TYPE_MAIN);
}
static void devcoredump_snapshot(struct xe_devcoredump *coredump,
@@ -660,7 +680,6 @@ static void devcoredump_snapshot(struct xe_devcoredump *coredump,
if (job)
ss->job = xe_sched_job_snapshot_capture(job);
ss->vm = xe_vm_snapshot_capture(q->vm);
-
xe_engine_snapshot_capture_for_queue(q);
}
diff --git a/drivers/gpu/drm/xe/xe_devcoredump_types.h b/drivers/gpu/drm/xe/xe_devcoredump_types.h
index fba3d23040b9e..e4fb3e12e068e 100644
--- a/drivers/gpu/drm/xe/xe_devcoredump_types.h
+++ b/drivers/gpu/drm/xe/xe_devcoredump_types.h
@@ -110,6 +110,8 @@ struct xe_devcoredump_snapshot {
/** @gam_pf_report: GAM PF report snapshot */
struct xe_dbg_reg_snapshot *gam_pf_report;
+ /** @ns: Node status snapshot */
+ struct xe_dbg_reg_snapshot *ns;
/** @ge: GuC Submission Engine snapshot */
struct xe_guc_submit_exec_queue_snapshot *ge;
--
2.43.0
next prev parent reply other threads:[~2026-10-27 5:13 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-27 5:10 [PATCH v6 0/7] drm/xe: Capture additional HW register state in devcoredump Nareshkumar Gollakoti
2026-10-27 5:10 ` [PATCH v6 1/7] drm/xe/devcoredump: Capture GT fuse registers " Nareshkumar Gollakoti
2026-10-28 13:30 ` Upadhyay, Tejas
2026-10-27 5:10 ` [PATCH v6 2/7] drm/xe/devcoredump: Add GuC register snapshot to devcoredump Nareshkumar Gollakoti
2026-10-27 5:10 ` [PATCH v6 3/7] drm/xe/guc: Print register addresses in capture snapshot output Nareshkumar Gollakoti
2026-10-27 5:10 ` [PATCH v6 4/7] drm/xe: dump GAM page fault report registers in devcoredump Nareshkumar Gollakoti
2026-10-27 5:21 ` sashiko-bot
2026-10-28 13:36 ` Upadhyay, Tejas
2026-10-27 5:10 ` [PATCH v6 5/7] drm/xe/guc: add TDL, SLICE gfx registers to capture list Nareshkumar Gollakoti
2026-10-27 5:10 ` Nareshkumar Gollakoti [this message]
2026-10-27 5:19 ` [PATCH v6 6/7] drm/xe: capture L3 node status registers in devcoredump sashiko-bot
2026-10-27 5:10 ` [PATCH v6 7/7] drm/xe/guc: capture additional engine state registers Nareshkumar Gollakoti
2026-10-27 5:21 ` sashiko-bot
2026-10-28 13:46 ` Upadhyay, Tejas
2026-10-27 6:02 ` ✗ CI.checkpatch: warning for drm/xe: Capture additional HW register state in devcoredump (rev7) Patchwork
2026-10-27 6:04 ` ✓ CI.KUnit: success " Patchwork
2026-10-27 6:44 ` ✓ Xe.CI.BAT: " Patchwork
2026-10-27 11:24 ` ✗ Xe.CI.FULL: failure " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20261027051024.465587-7-naresh.kumar.g@intel.com \
--to=naresh.kumar.g@intel.com \
--cc=intel-xe@lists.freedesktop.org \
--cc=tejas.upadhyay@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox