On 23/06/2026 14:34, Philipp Stanner wrote:
dma_fence_driver_name() reports that a driver whose fence is already
signaled is a "detached-driver". This is incorrect, the driver might
still be loaded and running.

Rename that string to something more descriptive.

Signed-off-by: Philipp Stanner <[email protected]>
---
Just compile-tested.
---
  drivers/dma-buf/dma-fence.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/dma-buf/dma-fence.c b/drivers/dma-buf/dma-fence.c
index c7ea1e75d38a..9c3dfa7b0066 100644
--- a/drivers/dma-buf/dma-fence.c
+++ b/drivers/dma-buf/dma-fence.c
@@ -1173,7 +1173,7 @@ const char __rcu *dma_fence_driver_name(struct dma_fence 
*fence)
        if (!dma_fence_test_signaled_flag(fence))
                return (const char __rcu *)ops->get_driver_name(fence);
        else
-               return (const char __rcu *)"detached-driver";
+               return (const char __rcu *)"driver-whose-fence-is-signaled";

My logic was "driver detached from the fence". :)

Anyway, no strong opinion apart to be mindful on whether some output becomes overly verbose.

Regards,

Tvrtko

  }
  EXPORT_SYMBOL(dma_fence_driver_name);

Reply via email to