| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20250720000146.1405060-3-olvaffe@gmail.com>
Date: Sat, 19 Jul 2025 17:01:39 -0700
From: Chia-I Wu <olvaffe@...il.com>
To: Boris Brezillon <boris.brezillon@...labora.com>,
Steven Price <steven.price@....com>,
Liviu Dudau <liviu.dudau@....com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>,
Simona Vetter <simona@...ll.ch>,
linux-kernel@...r.kernel.org,
dri-devel@...ts.freedesktop.org
Subject: [PATCH 2/9] drm/panthor: capture GPU state for devcoredump
Capture interesting GPU_CONTROL regs for devcoredump.
Signed-off-by: Chia-I Wu <olvaffe@...il.com>
---
drivers/gpu/drm/panthor/panthor_coredump.c | 85 ++++++++++++++++++++++
drivers/gpu/drm/panthor/panthor_coredump.h | 16 ++++
drivers/gpu/drm/panthor/panthor_regs.h | 6 ++
drivers/gpu/drm/panthor/panthor_sched.c | 6 ++
4 files changed, 113 insertions(+)
diff --git a/drivers/gpu/drm/panthor/panthor_coredump.c b/drivers/gpu/drm/panthor/panthor_coredump.c
index 767f3327e3e8..a41d0bbcb4f1 100644
--- a/drivers/gpu/drm/panthor/panthor_coredump.c
+++ b/drivers/gpu/drm/panthor/panthor_coredump.c
@@ -7,11 +7,13 @@
#include <generated/utsrelease.h>
#include <linux/devcoredump.h>
#include <linux/err.h>
+#include <linux/pm_runtime.h>
#include <linux/slab.h>
#include <linux/timekeeping.h>
#include "panthor_coredump.h"
#include "panthor_device.h"
+#include "panthor_regs.h"
#include "panthor_sched.h"
/**
@@ -19,6 +21,7 @@
*/
enum panthor_coredump_mask {
PANTHOR_COREDUMP_GROUP = BIT(0),
+ PANTHOR_COREDUMP_GPU = BIT(1),
};
/**
@@ -46,6 +49,7 @@ struct panthor_coredump {
u32 mask;
struct panthor_coredump_group_state group;
+ struct panthor_coredump_gpu_state gpu;
/* @data: Serialized coredump data. */
void *data;
@@ -78,6 +82,63 @@ static const char *reason_str(enum panthor_coredump_reason reason)
}
}
+static void print_gpu(struct drm_printer *p,
+ const struct panthor_coredump_gpu_state *gpu,
+ const struct drm_panthor_gpu_info *info)
+{
+ drm_puts(p, "gpu:\n");
+ drm_printf(p, " GPU_ID: 0x%x\n", info->gpu_id);
+ drm_printf(p, " L2_FEATURES: 0x%x\n", info->l2_features);
+ drm_printf(p, " CORE_FEATURES: 0x%x\n", info->core_features);
+ drm_printf(p, " TILER_FEATURES: 0x%x\n", info->tiler_features);
+ drm_printf(p, " MEM_FEATURES: 0x%x\n", info->mem_features);
+ drm_printf(p, " MMU_FEATURES: 0x%x\n", info->mmu_features);
+ drm_printf(p, " AS_PRESENT: 0x%x\n", info->as_present);
+ drm_printf(p, " CSF_ID: 0x%x\n", info->csf_id);
+ drm_printf(p, " MMU_FEATURES: 0x%x\n", info->mmu_features);
+
+ if (gpu) {
+ drm_printf(p, " GPU_STATUS: 0x%x\n", gpu->gpu_status);
+ drm_printf(p, " GPU_FAULTSTATUS: 0x%x\n",
+ gpu->gpu_faultstatus);
+ drm_printf(p, " GPU_FAULTADDRESS: 0x%llx\n",
+ gpu->gpu_faultaddress);
+ drm_printf(p, " L2_CONFIG: 0x%x\n", gpu->l2_config);
+ }
+
+ drm_printf(p, " THREAD_MAX_THREADS: 0x%x\n", info->max_threads);
+ drm_printf(p, " THREAD_MAX_WORKGROUP_SIZE: 0x%x\n",
+ info->thread_max_workgroup_size);
+ drm_printf(p, " THREAD_MAX_BARRIER_SIZE: 0x%x\n",
+ info->thread_max_barrier_size);
+ drm_printf(p, " THREAD_FEATURES: 0x%x\n", info->thread_features);
+ drm_printf(p, " TEXTURE_FEATURES_0: 0x%x\n",
+ info->texture_features[0]);
+ drm_printf(p, " TEXTURE_FEATURES_1: 0x%x\n",
+ info->texture_features[1]);
+ drm_printf(p, " TEXTURE_FEATURES_2: 0x%x\n",
+ info->texture_features[2]);
+ drm_printf(p, " TEXTURE_FEATURES_3: 0x%x\n",
+ info->texture_features[3]);
+
+ if (gpu) {
+ drm_printf(p, " DOORBELL_FEATURES: 0x%x\n",
+ gpu->doorbell_features);
+ }
+
+ drm_printf(p, " SHADER_PRESENT: 0x%llx\n", info->shader_present);
+ drm_printf(p, " TILER_PRESENT: 0x%llx\n", info->tiler_present);
+ drm_printf(p, " L2_PRESENT: 0x%llx\n", info->l2_present);
+ drm_printf(p, " REVIDR: 0x%x\n", info->gpu_rev);
+ drm_printf(p, " AMBA_FEATURES: 0x%x\n", info->coherency_features);
+
+ if (gpu) {
+ drm_printf(p, " AMBA_ENABLE: 0x%x\n", gpu->amba_enable);
+ drm_printf(p, " MCU_STATUS: 0x%x\n", gpu->mcu_status);
+ drm_printf(p, " MCU_FEATURES: 0x%x\n", gpu->mcu_features);
+ }
+}
+
static void print_group(struct drm_printer *p,
const struct panthor_coredump_group_state *group)
{
@@ -111,6 +172,10 @@ static void print_cd(struct drm_printer *p, const struct panthor_coredump *cd)
if (cd->mask & PANTHOR_COREDUMP_GROUP)
print_group(p, &cd->group);
+
+ /* many gpu states are static and are captured in drm_panthor_gpu_info */
+ print_gpu(p, cd->mask & PANTHOR_COREDUMP_GPU ? &cd->gpu : NULL,
+ &cd->ptdev->gpu_info);
}
static void process_cd(struct panthor_device *ptdev,
@@ -137,6 +202,19 @@ static void process_cd(struct panthor_device *ptdev,
print_cd(&p, cd);
}
+static void capture_gpu(struct panthor_device *ptdev,
+ struct panthor_coredump_gpu_state *gpu)
+{
+ gpu->gpu_status = gpu_read(ptdev, GPU_STATUS);
+ gpu->gpu_faultstatus = gpu_read(ptdev, GPU_FAULT_STATUS);
+ gpu->gpu_faultaddress = gpu_read64(ptdev, GPU_FAULT_ADDR);
+ gpu->l2_config = gpu_read(ptdev, GPU_L2_CONFIG);
+ gpu->doorbell_features = gpu_read(ptdev, GPU_DOORBELL_FEATURES);
+ gpu->amba_enable = gpu_read(ptdev, GPU_COHERENCY_PROTOCOL);
+ gpu->mcu_status = gpu_read(ptdev, MCU_STATUS);
+ gpu->mcu_features = gpu_read(ptdev, MCU_FEATURES);
+}
+
static void capture_cd(struct panthor_device *ptdev,
struct panthor_coredump *cd, struct panthor_group *group)
{
@@ -146,6 +224,13 @@ static void capture_cd(struct panthor_device *ptdev,
panthor_group_capture_coredump(group, &cd->group);
cd->mask |= PANTHOR_COREDUMP_GROUP;
}
+
+ /* remaining states require the device to be powered on */
+ if (!pm_runtime_active(ptdev->base.dev))
+ return;
+
+ capture_gpu(ptdev, &cd->gpu);
+ cd->mask |= PANTHOR_COREDUMP_GPU;
}
static void panthor_coredump_free(void *data)
diff --git a/drivers/gpu/drm/panthor/panthor_coredump.h b/drivers/gpu/drm/panthor/panthor_coredump.h
index dd1fe1c2e175..9e30c02ab962 100644
--- a/drivers/gpu/drm/panthor/panthor_coredump.h
+++ b/drivers/gpu/drm/panthor/panthor_coredump.h
@@ -40,6 +40,22 @@ struct panthor_coredump_group_state {
int csg_id;
};
+/**
+ * struct panthor_coredump_gpu_state - Coredump GPU state
+ *
+ * Interesting GPU_CONTROL regs.
+ */
+struct panthor_coredump_gpu_state {
+ u32 gpu_status;
+ u32 gpu_faultstatus;
+ u64 gpu_faultaddress;
+ u32 l2_config;
+ u32 doorbell_features;
+ u32 amba_enable;
+ u32 mcu_status;
+ u32 mcu_features;
+};
+
#ifdef CONFIG_DEV_COREDUMP
struct panthor_coredump *
diff --git a/drivers/gpu/drm/panthor/panthor_regs.h b/drivers/gpu/drm/panthor/panthor_regs.h
index 48bbfd40138c..062f939e075c 100644
--- a/drivers/gpu/drm/panthor/panthor_regs.h
+++ b/drivers/gpu/drm/panthor/panthor_regs.h
@@ -65,6 +65,8 @@
#define GPU_FAULT_STATUS 0x3C
#define GPU_FAULT_ADDR 0x40
+#define GPU_L2_CONFIG 0x48
+
#define GPU_PWR_KEY 0x50
#define GPU_PWR_KEY_UNLOCK 0x2968A819
#define GPU_PWR_OVERRIDE0 0x54
@@ -81,6 +83,8 @@
#define GPU_TEXTURE_FEATURES(n) (0xB0 + ((n) * 4))
+#define GPU_DOORBELL_FEATURES 0xC0
+
#define GPU_SHADER_PRESENT 0x100
#define GPU_TILER_PRESENT 0x110
#define GPU_L2_PRESENT 0x120
@@ -126,6 +130,8 @@
#define MCU_STATUS_HALT 2
#define MCU_STATUS_FATAL 3
+#define MCU_FEATURES 0x708
+
/* Job Control regs */
#define JOB_INT_RAWSTAT 0x1000
#define JOB_INT_CLEAR 0x1004
diff --git a/drivers/gpu/drm/panthor/panthor_sched.c b/drivers/gpu/drm/panthor/panthor_sched.c
index eb45b5ad9774..a9fd71fa984b 100644
--- a/drivers/gpu/drm/panthor/panthor_sched.c
+++ b/drivers/gpu/drm/panthor/panthor_sched.c
@@ -3670,6 +3670,7 @@ static void panthor_sched_coredump_locked(struct panthor_device *ptdev,
struct panthor_group *group)
{
struct panthor_coredump *cd;
+ int pm_active;
lockdep_assert_held(&ptdev->scheduler->lock);
@@ -3678,7 +3679,12 @@ static void panthor_sched_coredump_locked(struct panthor_device *ptdev,
if (!cd)
return;
+ pm_active = pm_runtime_get_if_active(ptdev->base.dev);
+
panthor_coredump_capture(cd, group);
+
+ if (pm_active == 1)
+ pm_runtime_put(ptdev->base.dev);
}
void panthor_group_capture_coredump(const struct panthor_group *group,
--
2.50.0.727.gbf7dc18ff4-goog
Powered by blists - more mailing lists