Thread (21 messages) 21 messages, 3 authors, 2021-06-28

Re: [PATCH v3 1/8] media: hantro: Trace hevc hw cycles performance register

From: Ezequiel Garcia <hidden>
Date: 2021-06-22 12:49:16
Also in: linux-media, linux-rockchip, lkml

On Tue, 2021-06-22 at 14:23 +0200, Benjamin Gaignard wrote:
[..]
quoted
quoted
+
+TRACE_EVENT(hantro_hevc_perf,
+       TP_PROTO(struct hantro_ctx *ctx, u32 hw_cycles),
+
+       TP_ARGS(ctx, hw_cycles),
+
+       TP_STRUCT__entry(
+               __field(int, minor)
+               __field(u32, hw_cycles)
+       ),
+
+       TP_fast_assign(
+               __entry->minor = ctx->fh.vdev->minor;
Tracking performance per minor doesn't seem useful,
we'd like to track per-fd (i.e. per context).
This part of the driver doesn't know for which fd the decoding job is done
so impossible to add it there.
Maybe you can explore using struct v4l2_m2m_ctx.
There's an RFC where this is discussed:

https://lore.kernel.org/linux-media/20210517183801.1255496-1-emil.l.velikov@gmail.com/ (local)

Kindly,
Ezequiel


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help