From 23930d501849d7c2209017224dde29d7f194e110 Mon Sep 17 00:00:00 2001 From: Jonathan Cavitt Date: Wed, 30 Sep 2026 04:56:06 +0800 Subject: [PATCH] drm/xe/vm: Do not dma_fence_put zero fences in err_out path It is not necessary to iterate over and dma_fence_put current_fence fences in the err_out path of ops_execute because current_fences cannot be non-zero when jumping to that section of code. Caught by static analysis. Signed-off-by: Jonathan Cavitt Cc: Matthew Brost Cc: Himal Prasad Ghimiray Cc: Shuicheng Lin Cc: Maarten Lankhorst Reviewed-by: Matthew Brost Signed-off-by: Matthew Brost Link: https://patch.msgid.link/20260929205606.1076177-1-jonathan.cavitt@intel.com Signed-off-by: Rodrigo Vivi --- drivers/gpu/drm/xe/xe_vm.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/xe/xe_vm.c b/drivers/gpu/drm/xe/xe_vm.c index 77cbffa625ab..03e0c06ec82c 100644 --- a/drivers/gpu/drm/xe/xe_vm.c +++ b/drivers/gpu/drm/xe/xe_vm.c @@ -3575,8 +3575,6 @@ static struct dma_fence *ops_execute(struct xe_vm *vm, err_out: xe_pt_update_ops_abort(xe, vops); - while (current_fence) - dma_fence_put(fences[--current_fence]); kfree(fences); kfree(cf);