From: Rob Clark <robin.clark@oss.qualcomm.com>
To: dri-devel@lists.freedesktop.org
Cc: linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org,
Alexander Koskovich <akoskovich@pm.me>,
Jie Zhang <jie.zhang@oss.qualcomm.com>,
Akhil P Oommen <akhilpo@oss.qualcomm.com>,
Bhaskar Reddy Valluru <bvalluru@qti.qualcomm.com>,
Rob Clark <robin.clark@oss.qualcomm.com>,
Abhinav Kumar <abhinav.kumar@linux.dev>,
David Airlie <airlied@gmail.com>,
Dmitry Baryshkov <lumag@kernel.org>,
Jessica Zhang <jesszhan0024@gmail.com>,
Konrad Dybcio <konradybcio@kernel.org>,
linux-kernel@vger.kernel.org (open list),
Marijn Suijten <marijn.suijten@somainline.org>,
Sean Paul <sean@poorly.run>, Simona Vetter <simona@ffwll.ch>
Subject: [PATCH 00/12] drm/msm/a8xx: GPU state capture support
Date: Tue, 20 Oct 2026 13:46:14 -0700 [thread overview]
Message-ID: <20261020204728.28844-1-robin.clark@oss.qualcomm.com> (raw)
This series introduces gen8 devcoredump support. Compared to previous
gens, it:
1. Switches to generated tables. Compared to hand modifying headers from
kgsl (which were originally generated, then adjusted a bit by hand before
adding to kgsl tree. And then copied to drm/msm and lightly modified
some more in the process).
In the process, we also switch to generated nonctx reg tables and
cp_protect tables.
2. Aligns to the "snapshot" format that kgsl/etc use for hw state capture.
The non-generation specific bits (SMMU fault info, capturing relevant
BOs, etc) still works as before. Everything gen8 specific gets encoded
as "snapshot" sections, ascii85 encoded in the resulting devcoredump
file (similar to captured BOs).
3. Try to make it easier to at least compare kgsl snapshot code with
upstream, for porting fixes/etc. State capture is tedious and mind-
numbing, and the last thing you want to do when debugging a GPU hang
is going down the rabbit hole of having to first debug gpu state
capture.
However, I don't love kgsl_snapshot_add_section(). It does avoid
duplicating snapshot header stuff everywhere. But the ergonomics
are not great. So instead I went with "align with what I wish kgsl
looked like..
The mesa/crashdec side of this is at:
https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/45304
I'd appreciate tested-by's on Hawi, Maili, Milos, and Pakala, since I
don't have these devices at-hand to test myself. In particular, do
things look fine after the switch to generated nonctx regs. To test
devcoredump, you can use the msm_recovery test in igt.. or just:
cat /sys/kernel/debug/dri/*/gpu > /dev/null
NOTE: Uses $codename for generated file/table names, rather than continue
to perpetuate the equally meaningless gen8_x_y naming from kgsl into
upstream. At least the codenames are familiar upstream and this
removes an extra level of name translation.
Rob Clark (12):
drm/msm: Import adreno_snapshot.h
drm/msm/a8xx: Export aperture helpers
drm/msm/a8xx: Add a8xx_aperture_acquire_slice()
drm/msm/a8xx: Allow nonctx reglist to include GMU regs
drm/msm/a8xx: Stop getting GMEM_PROTECT from nonctx list
drm/msm/adreno: Fix PROTECT span length
drm/msm/adreno: add temporary __maybe_unused
drm/msm/a8xx: Import generated headers
drm/msm/a8xx: devcoredump state capture support
drm/msm/a8xx: Switch to generated nonctx regs
drm/msm/a8xx: Switch to generated protect regs
drm/msm/adreno: Remove temporary __maybe_unused
drivers/gpu/drm/msm/Makefile | 4 +-
drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 7 +
drivers/gpu/drm/msm/adreno/a6xx_gpu.h | 15 +-
drivers/gpu/drm/msm/adreno/a8xx_catalog.c | 668 +----
drivers/gpu/drm/msm/adreno/a8xx_gpu.c | 42 +-
drivers/gpu/drm/msm/adreno/a8xx_gpu_state.c | 1253 +++++++++
.../gpu/drm/msm/adreno/adreno_gen8_snapshot.h | 582 ++++
drivers/gpu/drm/msm/adreno/adreno_gpu.h | 2 +
drivers/gpu/drm/msm/adreno/adreno_snapshot.h | 526 ++++
.../drm/msm/adreno/catalog/adreno_glymur.h | 2315 ++++++++++++++++
.../gpu/drm/msm/adreno/catalog/adreno_hawi.h | 2410 +++++++++++++++++
.../drm/msm/adreno/catalog/adreno_kaanapali.h | 2339 ++++++++++++++++
.../gpu/drm/msm/adreno/catalog/adreno_maili.h | 1900 +++++++++++++
.../gpu/drm/msm/adreno/catalog/adreno_milos.h | 1994 ++++++++++++++
.../drm/msm/adreno/catalog/adreno_pakala.h | 2178 +++++++++++++++
15 files changed, 15583 insertions(+), 652 deletions(-)
create mode 100644 drivers/gpu/drm/msm/adreno/a8xx_gpu_state.c
create mode 100644 drivers/gpu/drm/msm/adreno/adreno_gen8_snapshot.h
create mode 100644 drivers/gpu/drm/msm/adreno/adreno_snapshot.h
create mode 100644 drivers/gpu/drm/msm/adreno/catalog/adreno_glymur.h
create mode 100644 drivers/gpu/drm/msm/adreno/catalog/adreno_hawi.h
create mode 100644 drivers/gpu/drm/msm/adreno/catalog/adreno_kaanapali.h
create mode 100644 drivers/gpu/drm/msm/adreno/catalog/adreno_maili.h
create mode 100644 drivers/gpu/drm/msm/adreno/catalog/adreno_milos.h
create mode 100644 drivers/gpu/drm/msm/adreno/catalog/adreno_pakala.h
--
2.56.0
next reply other threads:[~2026-10-20 20:47 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-20 20:46 Rob Clark [this message]
2026-10-20 20:46 ` [PATCH 01/12] drm/msm: Import adreno_snapshot.h Rob Clark
2026-10-20 20:46 ` [PATCH 02/12] drm/msm/a8xx: Export aperture helpers Rob Clark
2026-10-20 20:46 ` [PATCH 03/12] drm/msm/a8xx: Add a8xx_aperture_acquire_slice() Rob Clark
2026-10-20 20:46 ` [PATCH 04/12] drm/msm/a8xx: Allow nonctx reglist to include GMU regs Rob Clark
2026-10-20 20:46 ` [PATCH 05/12] drm/msm/a8xx: Stop getting GMEM_PROTECT from nonctx list Rob Clark
2026-10-20 20:46 ` [PATCH 06/12] drm/msm/adreno: Fix PROTECT span length Rob Clark
2026-10-20 20:46 ` [PATCH 07/12] drm/msm/adreno: add temporary __maybe_unused Rob Clark
2026-10-20 20:46 ` [PATCH 08/12] drm/msm/a8xx: Import generated headers Rob Clark
2026-10-20 20:46 ` [PATCH 09/12] drm/msm/a8xx: devcoredump state capture support Rob Clark
2026-10-20 20:46 ` [PATCH 10/12] drm/msm/a8xx: Switch to generated nonctx regs Rob Clark
2026-10-20 20:46 ` [PATCH 11/12] drm/msm/a8xx: Switch to generated protect regs Rob Clark
2026-10-20 20:46 ` [PATCH 12/12] drm/msm/adreno: Remove temporary __maybe_unused Rob Clark
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=20261020204728.28844-1-robin.clark@oss.qualcomm.com \
--to=robin.clark@oss.qualcomm.com \
--cc=abhinav.kumar@linux.dev \
--cc=airlied@gmail.com \
--cc=akhilpo@oss.qualcomm.com \
--cc=akoskovich@pm.me \
--cc=bvalluru@qti.qualcomm.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=freedreno@lists.freedesktop.org \
--cc=jesszhan0024@gmail.com \
--cc=jie.zhang@oss.qualcomm.com \
--cc=konradybcio@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lumag@kernel.org \
--cc=marijn.suijten@somainline.org \
--cc=sean@poorly.run \
--cc=simona@ffwll.ch \
/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
all inboxes | Powered by JetHome®