Make the implementation use the previously added helper for accessing the
driver and timeline name. This will enable more coverage later when
asserts will be added into them.

Signed-off-by: Tvrtko Ursulin <tvrtko.ursu...@igalia.com>
---
 drivers/dma-buf/dma-fence.c      | 9 +++++----
 include/trace/events/dma_fence.h | 4 ++--
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/drivers/dma-buf/dma-fence.c b/drivers/dma-buf/dma-fence.c
index 33661658f684..dc2456f68685 100644
--- a/drivers/dma-buf/dma-fence.c
+++ b/drivers/dma-buf/dma-fence.c
@@ -538,8 +538,8 @@ void dma_fence_release(struct kref *kref)
        if (WARN(!list_empty(&fence->cb_list) &&
                 !test_bit(DMA_FENCE_FLAG_SIGNALED_BIT, &fence->flags),
                 "Fence %s:%s:%llx:%llx released with pending signals!\n",
-                fence->ops->get_driver_name(fence),
-                fence->ops->get_timeline_name(fence),
+                dma_fence_driver_name(fence),
+                dma_fence_timeline_name(fence),
                 fence->context, fence->seqno)) {
                unsigned long flags;
 
@@ -983,8 +983,9 @@ EXPORT_SYMBOL(dma_fence_set_deadline);
 void dma_fence_describe(struct dma_fence *fence, struct seq_file *seq)
 {
        seq_printf(seq, "%s %s seq %llu %ssignalled\n",
-                  fence->ops->get_driver_name(fence),
-                  fence->ops->get_timeline_name(fence), fence->seqno,
+                  dma_fence_driver_name(fence),
+                  dma_fence_timeline_name(fence),
+                  fence->seqno,
                   dma_fence_is_signaled(fence) ? "" : "un");
 }
 EXPORT_SYMBOL(dma_fence_describe);
diff --git a/include/trace/events/dma_fence.h b/include/trace/events/dma_fence.h
index a4de3df8500b..84c83074ee81 100644
--- a/include/trace/events/dma_fence.h
+++ b/include/trace/events/dma_fence.h
@@ -16,8 +16,8 @@ DECLARE_EVENT_CLASS(dma_fence,
        TP_ARGS(fence),
 
        TP_STRUCT__entry(
-               __string(driver, fence->ops->get_driver_name(fence))
-               __string(timeline, fence->ops->get_timeline_name(fence))
+               __string(driver, dma_fence_driver_name(fence))
+               __string(timeline, dma_fence_timeline_name(fence))
                __field(unsigned int, context)
                __field(unsigned int, seqno)
        ),
-- 
2.48.0

Reply via email to