* [PATCH] drm/imagination: Fix value of HWRT address in pvr_job_create tracepoint
@ 2026-07-20 15:07 Alessio Belle
2026-07-22 9:56 ` Brajesh Gupta
2026-07-24 8:02 ` Alessio Belle
0 siblings, 2 replies; 3+ messages in thread
From: Alessio Belle @ 2026-07-20 15:07 UTC (permalink / raw)
To: Luigi Santivetti, Brajesh Gupta, Frank Binns, Alexandru Dadu,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann,
David Airlie, Simona Vetter
Cc: dri-devel, linux-kernel, Alessio Belle
The pvr_job_create tracepoint was using pvr_fw_object::fw_addr_offset
to display the firmware address of the render target structure attached
to a job, but that's an offset into the firmware heap, not the expected
full address, which is more useful in general e.g. for cross referencing
against firmware logs.
Use pvr_fw_object_get_fw_addr() to get the full firmware address.
Fixes: c1079aebb4de ("drm/imagination: Add support for trace points")
Signed-off-by: Alessio Belle <alessio.belle@imgtec.com>
---
drivers/gpu/drm/imagination/pvr_trace.h | 16 +++++++++++++---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/imagination/pvr_trace.h b/drivers/gpu/drm/imagination/pvr_trace.h
index ffbbde3b2bcc..943d6b2418cb 100644
--- a/drivers/gpu/drm/imagination/pvr_trace.h
+++ b/drivers/gpu/drm/imagination/pvr_trace.h
@@ -41,6 +41,17 @@ TRACE_EVENT(pvr_job_submit_ioctl,
__entry->count)
);
+#define PVR_JOB_GET_HWRT_FW_ADDR(job) \
+ ({ \
+ struct pvr_job *_job = (job); \
+ u32 _hwrt_fw_addr = 0; \
+ \
+ if (_job && _job->hwrt) \
+ pvr_fw_object_get_fw_addr(_job->hwrt->fw_obj, &_hwrt_fw_addr); \
+ \
+ _hwrt_fw_addr; \
+ })
+
#define PVR_JOB_TYPE_TO_STR(val) \
__print_symbolic(val, \
{ DRM_PVR_JOB_TYPE_GEOMETRY, "geometry" }, \
@@ -64,9 +75,7 @@ TRACE_EVENT(pvr_job_create,
__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->hwrt_addr = PVR_JOB_GET_HWRT_FW_ADDR(job);
__entry->job = job;
__entry->job_type = job->type;
__entry->sync_op_count = sync_op_count;),
@@ -82,6 +91,7 @@ TRACE_EVENT(pvr_job_create,
);
#undef PVR_JOB_TYPE_TO_STR
+#undef PVR_JOB_GET_HWRT_FW_ADDR
TRACE_EVENT(pvr_job_submit_fw,
TP_PROTO(struct pvr_job *job),
---
base-commit: 17e2030f37600994440f875dc410615d5c66ee6d
change-id: 20260720-fix-pvr-job-trace-hwrt-32f54c0ec185
Best regards,
--
Alessio Belle <alessio.belle@imgtec.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/imagination: Fix value of HWRT address in pvr_job_create tracepoint
2026-07-20 15:07 [PATCH] drm/imagination: Fix value of HWRT address in pvr_job_create tracepoint Alessio Belle
@ 2026-07-22 9:56 ` Brajesh Gupta
2026-07-24 8:02 ` Alessio Belle
1 sibling, 0 replies; 3+ messages in thread
From: Brajesh Gupta @ 2026-07-22 9:56 UTC (permalink / raw)
To: Luigi Santivetti, tzimmermann, simona, airlied, Frank Binns,
Alessio Belle, maarten.lankhorst, Alexandru Dadu, mripard
Cc: dri-devel, linux-kernel
On Mon, 2026-07-20 at 16:07 +0100, Alessio Belle wrote:
> *** NOTE: This is an internal email from Imagination Technologies ***
>
>
>
>
> The pvr_job_create tracepoint was using pvr_fw_object::fw_addr_offset
> to display the firmware address of the render target structure attached
> to a job, but that's an offset into the firmware heap, not the expected
> full address, which is more useful in general e.g. for cross referencing
> against firmware logs.
>
> Use pvr_fw_object_get_fw_addr() to get the full firmware address.
>
> Fixes: c1079aebb4de ("drm/imagination: Add support for trace points")
> Signed-off-by: Alessio Belle <alessio.belle@imgtec.com>
> ---
> drivers/gpu/drm/imagination/pvr_trace.h | 16 +++++++++++++---
> 1 file changed, 13 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/imagination/pvr_trace.h b/drivers/gpu/drm/imagination/pvr_trace.h
> index ffbbde3b2bcc..943d6b2418cb 100644
> --- a/drivers/gpu/drm/imagination/pvr_trace.h
> +++ b/drivers/gpu/drm/imagination/pvr_trace.h
> @@ -41,6 +41,17 @@ TRACE_EVENT(pvr_job_submit_ioctl,
> __entry->count)
> );
>
> +#define PVR_JOB_GET_HWRT_FW_ADDR(job) \
> + ({ \
> + struct pvr_job *_job = (job); \
> + u32 _hwrt_fw_addr = 0; \
> + \
> + if (_job && _job->hwrt) \
> + pvr_fw_object_get_fw_addr(_job->hwrt->fw_obj, &_hwrt_fw_addr); \
> + \
> + _hwrt_fw_addr; \
> + })
> +
> #define PVR_JOB_TYPE_TO_STR(val) \
> __print_symbolic(val, \
> { DRM_PVR_JOB_TYPE_GEOMETRY, "geometry" }, \
> @@ -64,9 +75,7 @@ TRACE_EVENT(pvr_job_create,
> __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->hwrt_addr = PVR_JOB_GET_HWRT_FW_ADDR(job);
> __entry->job = job;
> __entry->job_type = job->type;
> __entry->sync_op_count = sync_op_count;),
> @@ -82,6 +91,7 @@ TRACE_EVENT(pvr_job_create,
> );
>
> #undef PVR_JOB_TYPE_TO_STR
> +#undef PVR_JOB_GET_HWRT_FW_ADDR
>
> TRACE_EVENT(pvr_job_submit_fw,
> TP_PROTO(struct pvr_job *job),
>
> ---
> base-commit: 17e2030f37600994440f875dc410615d5c66ee6d
> change-id: 20260720-fix-pvr-job-trace-hwrt-32f54c0ec185
>
> Best regards,
> --
> Alessio Belle <alessio.belle@imgtec.com>
>
Reviewed-by: Brajesh Gupta <brajesh.gupta@imgtec.com>
Thanks,
Brajesh
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/imagination: Fix value of HWRT address in pvr_job_create tracepoint
2026-07-20 15:07 [PATCH] drm/imagination: Fix value of HWRT address in pvr_job_create tracepoint Alessio Belle
2026-07-22 9:56 ` Brajesh Gupta
@ 2026-07-24 8:02 ` Alessio Belle
1 sibling, 0 replies; 3+ messages in thread
From: Alessio Belle @ 2026-07-24 8:02 UTC (permalink / raw)
To: Luigi Santivetti, Brajesh Gupta, Frank Binns, Alexandru Dadu,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann,
David Airlie, Simona Vetter, Alessio Belle
Cc: dri-devel, linux-kernel
On Mon, 20 Jul 2026 16:07:28 +0100, Alessio Belle wrote:
> The pvr_job_create tracepoint was using pvr_fw_object::fw_addr_offset
> to display the firmware address of the render target structure attached
> to a job, but that's an offset into the firmware heap, not the expected
> full address, which is more useful in general e.g. for cross referencing
> against firmware logs.
>
> Use pvr_fw_object_get_fw_addr() to get the full firmware address.
>
> [...]
Applied to drm-misc-fixes, thanks!
[1/1] drm/imagination: Fix value of HWRT address in pvr_job_create tracepoint
commit: 112badb7245059c176e3924f9cd9ce2933cb0082
Best regards,
--
Alessio Belle <alessio.belle@imgtec.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-07-24 8:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-07-20 15:07 [PATCH] drm/imagination: Fix value of HWRT address in pvr_job_create tracepoint Alessio Belle
2026-07-22 9:56 ` Brajesh Gupta
2026-07-24 8:02 ` Alessio Belle
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®