On 05/02/2026 21:31, Maíra Canal wrote:
Return distinct timeline names ("vc4-bin" and "vc4-render") from
vc4_fence_get_timeline_name() based on the fence's queue, instead of
the generic "vc4-v3d" for all fences. This improves debuggability when
inspecting fence state.

Signed-off-by: Maíra Canal <[email protected]>
---
  drivers/gpu/drm/vc4/vc4_fence.c | 11 ++++++++++-
  1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/vc4/vc4_fence.c b/drivers/gpu/drm/vc4/vc4_fence.c
index 
72aa6b8ba4b6b6bd8affe2c73827bd127b18225e..32a2c78fb632b1d2c647402dba434e7168922dfb
 100644
--- a/drivers/gpu/drm/vc4/vc4_fence.c
+++ b/drivers/gpu/drm/vc4/vc4_fence.c
@@ -47,7 +47,16 @@ static const char *vc4_fence_get_driver_name(struct 
dma_fence *fence)
static const char *vc4_fence_get_timeline_name(struct dma_fence *fence)
  {
-       return "vc4-v3d";
+       struct vc4_fence *f = to_vc4_fence(fence);
+
+       switch (f->queue) {
+       case VC4_BIN:
+               return "vc4-bin";
+       case VC4_RENDER:
+               return "vc4-render";
+       default:
+               return NULL;

NULL could be problematic for strscpy in sync_file.c. I suggest "unknown" or something like that.

Regards,

Tvrtko

+       }
  }
const struct dma_fence_ops vc4_fence_ops = {


Reply via email to