drm/imagination: Add support for trace points

Add support for workload submission trace points.

Co-developed-by: Donald Robson <donald.robson@imgtec.com>
Signed-off-by: Donald Robson <donald.robson@imgtec.com>
Signed-off-by: Alexandru Dadu <alexandru.dadu@imgtec.com>
Reviewed-by: Matt Coster <matt.coster@imgtec.com>
Link: https://patch.msgid.link/20260513-b4-pvr-trace-points-v1-1-81222d1a4c99@imgtec.com
Signed-off-by: Matt Coster <matt.coster@imgtec.com>

diff --git a/drivers/gpu/drm/imagination/Makefile b/drivers/gpu/drm/imagination/Makefile
index 1222a14262e4a2e5ccf236232ff3446baaa569bd..a5c5e386f16a7cb63fc4b97470505dc537bd458a 100644 (file)
--- a/drivers/gpu/drm/imagination/Makefile
+++ b/drivers/gpu/drm/imagination/Makefile
@@ -32,6 +32,9 @@ powervr-y := \
 powervr-$(CONFIG_DEBUG_FS) += \
        pvr_debugfs.o
 
+powervr-$(CONFIG_TRACEPOINTS) += \
+       pvr_trace_points.o
+
 obj-$(CONFIG_DRM_POWERVR) += powervr.o
 
 obj-$(CONFIG_DRM_POWERVR_KUNIT_TEST) += pvr_test.o

diff --git a/drivers/gpu/drm/imagination/pvr_drv.c b/drivers/gpu/drm/imagination/pvr_drv.c
index 268900464ab693a239be529458931f1aa49474cc..b20c462bcba0ba4822e4f772467a0e8fe3e07514 100644 (file)
--- a/drivers/gpu/drm/imagination/pvr_drv.c
+++ b/drivers/gpu/drm/imagination/pvr_drv.c
@@ -14,6 +14,7 @@
 #include "pvr_rogue_defs.h"
 #include "pvr_rogue_fwif_client.h"
 #include "pvr_rogue_fwif_shared.h"
+#include "pvr_trace.h"
 #include "pvr_vm.h"
 
 #include <uapi/drm/pvr_drm.h>
@@ -1150,6 +1151,8 @@ pvr_ioctl_submit_jobs(struct drm_device *drm_dev, void *raw_args,
        int idx;
        int err;
 
+       trace_pvr_job_submit_ioctl(pvr_dev, args->jobs.count);
+
        if (!drm_dev_enter(drm_dev, &idx))
                return -EIO;
 

diff --git a/drivers/gpu/drm/imagination/pvr_job.c b/drivers/gpu/drm/imagination/pvr_job.c
index dd9f5df01e0803c20c5c8a1d38fb008682fbf0cc..b8a58d817009b69db90c05231e7870209d0baf73 100644 (file)
--- a/drivers/gpu/drm/imagination/pvr_job.c
+++ b/drivers/gpu/drm/imagination/pvr_job.c
@@ -14,6 +14,7 @@
 #include "pvr_stream.h"
 #include "pvr_stream_defs.h"
 #include "pvr_sync.h"
+#include "pvr_trace.h"
 
 #include <drm/drm_exec.h>
 #include <drm/drm_gem.h>
@@ -510,6 +511,8 @@ static int pvr_job_data_init(struct pvr_device *pvr_dev,
                }
 
                job_data_out[i].sync_op_count = job_args[i].sync_ops.count;
+
+               trace_pvr_job_create(pvr_dev, job_data_out[i].job, job_data_out[i].sync_op_count);
        }
 
        return 0;

diff --git a/drivers/gpu/drm/imagination/pvr_queue.c b/drivers/gpu/drm/imagination/pvr_queue.c
index 822e6259674f37777e9d53ecd9a20d6c9c7f5c21..7ed60e1c1a866296410a993d00bbdde34376f4a9 100644 (file)
--- a/drivers/gpu/drm/imagination/pvr_queue.c
+++ b/drivers/gpu/drm/imagination/pvr_queue.c
@@ -10,6 +10,7 @@
 #include "pvr_drv.h"
 #include "pvr_job.h"
 #include "pvr_queue.h"
+#include "pvr_trace.h"
 #include "pvr_vm.h"
 
 #include "pvr_rogue_fwif_client.h"
@@ -725,6 +726,8 @@ static void pvr_queue_submit_job_to_cccb(struct pvr_job *job)
                cmd->partial_render_geom_frag_fence.value = job->done_fence->seqno - 1;
        }
 
+       trace_pvr_job_submit_fw(job);
+
        /* Submit job to FW */
        pvr_cccb_write_command_with_header(cccb, job->fw_ccb_cmd_type, job->cmd_len, job->cmd,
                                           job->id, job->id);
@@ -1198,6 +1201,8 @@ void pvr_queue_job_cleanup(struct pvr_job *job)
 
        if (job->base.s_fence)
                drm_sched_job_cleanup(&job->base);
+
+       trace_pvr_job_done(job);
 }
 
 /**

diff --git a/drivers/gpu/drm/imagination/pvr_trace.h b/drivers/gpu/drm/imagination/pvr_trace.h
new file mode 100644 (file)
index 0000000..ffbbde3
--- /dev/null
+++ b/drivers/gpu/drm/imagination/pvr_trace.h
@@ -0,0 +1,113 @@
+/* SPDX-License-Identifier: GPL-2.0 OR MIT */
+/* Copyright (c) 2026 Imagination Technologies Ltd. */
+
+#undef TRACE_SYSTEM
+#define TRACE_SYSTEM pvr
+
+#if !defined(PVR_TRACE_H) || defined(TRACE_HEADER_MULTI_READ)
+#define PVR_TRACE_H
+
+#include "pvr_context.h"
+#include "pvr_device.h"
+#include "pvr_fw.h"
+#include "pvr_hwrt.h"
+#include "pvr_job.h"
+#include "pvr_sync.h"
+
+#include <linux/stringify.h>
+#include <linux/types.h>
+#include <linux/tracepoint.h>
+
+/*
+ * NOTE:
+ * When adding trace points, or extra data to existing ones - you must capture
+ * all the data in the TP_fast_assign section that you wish to use in the
+ * TP_printk section. This is because the printk is performed on demand from the
+ * captured data when you `cat /sys/kernel/tracing/trace` and by the time this
+ * happens any pointers you captured will likely no longer point to valid data.
+ */
+
+/* Job submit */
+
+TRACE_EVENT(pvr_job_submit_ioctl,
+           TP_PROTO(struct pvr_device *pvr_dev, u32 count),
+           TP_ARGS(pvr_dev, count),
+           TP_STRUCT__entry(__field(struct pvr_device *, pvr_dev)
+                            __field(u32, count)),
+           TP_fast_assign(__entry->pvr_dev = pvr_dev;
+                          __entry->count = count;),
+           TP_printk("pvr_dev=%p count=%u",
+                     __entry->pvr_dev,
+                     __entry->count)
+);
+
+#define PVR_JOB_TYPE_TO_STR(val)                                         \
+       __print_symbolic(val,                                            \
+                        { DRM_PVR_JOB_TYPE_GEOMETRY, "geometry" },      \
+                        { DRM_PVR_JOB_TYPE_FRAGMENT, "fragment" },      \
+                        { DRM_PVR_JOB_TYPE_COMPUTE, "compute" },        \
+                        { DRM_PVR_JOB_TYPE_TRANSFER_FRAG, "transfer" })
+
+TRACE_EVENT(pvr_job_create,
+           TP_PROTO(struct pvr_device *pvr_dev, struct pvr_job *job,
+                    u32 sync_op_count),
+           TP_ARGS(pvr_dev, job, sync_op_count),
+           TP_STRUCT__entry(__field(struct pvr_device *, pvr_dev)
+                            __field(struct pvr_context *, ctx)
+                            __field(struct pvr_fw_object *, fw_obj)
+                            __field(u32, fw_addr)
+                            __field(u32, hwrt_addr)
+                            __field(struct pvr_job *, job)
+                            __field(enum drm_pvr_job_type, job_type)
+                            __field(u32, sync_op_count)),
+           TP_fast_assign(__entry->pvr_dev = pvr_dev;
+                          __entry->ctx = job->ctx;
+                          __entry->fw_obj = job->ctx->fw_obj;
+                          pvr_fw_object_get_fw_addr(job->ctx->fw_obj, &__entry->fw_addr);
+                          __entry->hwrt_addr = job->hwrt ?
+                                               job->hwrt->fw_obj->fw_addr_offset :
+                                               0;
+                          __entry->job = job;
+                          __entry->job_type = job->type;
+                          __entry->sync_op_count = sync_op_count;),
+           TP_printk("pvr_dev=%p ctx=%p fw_obj=%p fw_addr=0x%x hwrt_addr=0x%x job=%p job_type=%s sync_op_count=%u",
+                     __entry->pvr_dev,
+                     __entry->ctx,
+                     __entry->fw_obj,
+                     __entry->fw_addr,
+                     __entry->hwrt_addr,
+                     __entry->job,
+                     PVR_JOB_TYPE_TO_STR(__entry->job_type),
+                     __entry->sync_op_count)
+);
+
+#undef PVR_JOB_TYPE_TO_STR
+
+TRACE_EVENT(pvr_job_submit_fw,
+           TP_PROTO(struct pvr_job *job),
+           TP_ARGS(job),
+           TP_STRUCT__entry(__field(struct pvr_job *, job)
+                            __field(u32, done_seqno)),
+           TP_fast_assign(__entry->job = job;
+                          __entry->done_seqno = job->done_fence->seqno;),
+           TP_printk("job=%p done_seqno=%u",
+                     __entry->job,
+                     __entry->done_seqno)
+);
+
+TRACE_EVENT(pvr_job_done,
+           TP_PROTO(struct pvr_job *job),
+           TP_ARGS(job),
+           TP_STRUCT__entry(__field(struct pvr_job *, job)),
+           TP_fast_assign(__entry->job = job;),
+           TP_printk("job=%p", __entry->job)
+);
+
+#endif /* PVR_TRACE_H */
+
+/* This part must be outside protection */
+#undef TRACE_INCLUDE_PATH
+#undef TRACE_INCLUDE_FILE
+#define TRACE_INCLUDE_PATH ../../drivers/gpu/drm/imagination
+#define TRACE_INCLUDE_FILE pvr_trace
+#include <trace/define_trace.h>

diff --git a/drivers/gpu/drm/imagination/pvr_trace_points.c b/drivers/gpu/drm/imagination/pvr_trace_points.c
new file mode 100644 (file)
index 0000000..71b53f2
--- /dev/null
+++ b/drivers/gpu/drm/imagination/pvr_trace_points.c
@@ -0,0 +1,7 @@
+// SPDX-License-Identifier: GPL-2.0 OR MIT
+/* Copyright (c) 2026 Imagination Technologies Ltd. */
+
+#ifndef __CHECKER__
+#define CREATE_TRACE_POINTS
+#include "pvr_trace.h"
+#endif