Re: [PATCH] drm/imagination: Add support for trace points

[Matt Coster <Matt.Coster@imgtec.com>]

[-- Attachment #1.1: Type: text/plain, Size: 9806 bytes --]

On 13/05/2026 13:00, Alexandru Dadu wrote:
> 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>

> ---
>  drivers/gpu/drm/imagination/Makefile           |   3 +
>  drivers/gpu/drm/imagination/pvr_drv.c          |   3 +
>  drivers/gpu/drm/imagination/pvr_job.c          |   3 +
>  drivers/gpu/drm/imagination/pvr_queue.c        |   5 ++
>  drivers/gpu/drm/imagination/pvr_trace.h        | 113 +++++++++++++++++++++++++
>  drivers/gpu/drm/imagination/pvr_trace_points.c |   7 ++
>  6 files changed, 134 insertions(+)
> 
> diff --git a/drivers/gpu/drm/imagination/Makefile b/drivers/gpu/drm/imagination/Makefile
> index 1222a14262e4..a5c5e386f16a 100644
> --- 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 268900464ab6..b20c462bcba0 100644
> --- 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 dd9f5df01e08..b8a58d817009 100644
> --- 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 d10a13173f0f..3c2331e834fb 100644
> --- 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);
> @@ -1193,6 +1196,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
> index 000000000000..ffbbde3b2bcc
> --- /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
> index 000000000000..71b53f267af0
> --- /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
> 
> ---
> base-commit: 921578cf373981eba92774a27f7644a11282cf89
> change-id: 20260512-b4-pvr-trace-points-4d3b66116e15
> 
> Best regards,
> --
> Alexandru Dadu <alexandru.dadu@imgtec.com>
> 


-- 
Matt Coster
E: matt.coster@imgtec.com

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]