The implementation just tests if the fence is signaled or not and doesn't enable any further handling. On the other hand the callback to test if a fence is signaled or not is missing.
So remove the enable_signaling callback and implement the signaled callback instead. Signed-off-by: Christian König <christian.koe...@amd.com> --- drivers/gpu/drm/vmwgfx/vmwgfx_fence.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_fence.c b/drivers/gpu/drm/vmwgfx/vmwgfx_fence.c index 588d50ababf6..255ef677c712 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_fence.c +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_fence.c @@ -143,7 +143,7 @@ static const char *vmw_fence_get_timeline_name(struct dma_fence *f) return "svga"; } -static bool vmw_fence_enable_signaling(struct dma_fence *f) +static bool vmw_fence_signaled(struct dma_fence *f) { struct vmw_fence_obj *fence = container_of(f, struct vmw_fence_obj, base); @@ -153,9 +153,9 @@ static bool vmw_fence_enable_signaling(struct dma_fence *f) u32 seqno = vmw_fence_read(dev_priv); if (seqno - fence->base.seqno < VMW_FENCE_WRAP) - return false; + return true; - return true; + return false; } struct vmwgfx_wait_cb { @@ -251,7 +251,7 @@ static long vmw_fence_wait(struct dma_fence *f, bool intr, signed long timeout) static const struct dma_fence_ops vmw_fence_ops = { .get_driver_name = vmw_fence_get_driver_name, .get_timeline_name = vmw_fence_get_timeline_name, - .enable_signaling = vmw_fence_enable_signaling, + .signaled = vmw_fence_signaled, .wait = vmw_fence_wait, .release = vmw_fence_obj_destroy, }; -- 2.34.1