diff options
| author | Samuel Pitoiset <[email protected]> | 2018-10-30 11:41:20 +0100 |
|---|---|---|
| committer | Samuel Pitoiset <[email protected]> | 2018-10-31 09:21:28 +0100 |
| commit | 57982b683bcbcfecba07c7d3dcbacd82e4ced614 (patch) | |
| tree | 83c9d8c4931a1f189a2b11e85602f0f108b229fa | |
| parent | ae3aecd07f038e1b4e927b7f942c6e865b973a7d (diff) | |
radv/winsys: do not assign last submission when chained path failed
I don't think we want to wait for something that hasn't been
correctly submitted. This is similar to the fallback path.
Signed-off-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Dave Airlie <[email protected]>
| -rw-r--r-- | src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c index 75f1c47fd90..abc4f3903d3 100644 --- a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c +++ b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c @@ -840,12 +840,15 @@ static int radv_amdgpu_winsys_cs_submit_chained(struct radeon_winsys_ctx *_ctx, if (bo_list) amdgpu_bo_list_destroy(bo_list); + if (r) + return r; + if (fence) radv_amdgpu_request_to_fence(ctx, fence, &request); radv_assign_last_submit(ctx, &request); - return r; + return 0; } static int radv_amdgpu_winsys_cs_submit_fallback(struct radeon_winsys_ctx *_ctx, |