diff options
| author | Marek Olšák <[email protected]> | 2019-02-14 00:38:37 -0500 |
|---|---|---|
| committer | Marek Olšák <[email protected]> | 2019-04-23 11:36:31 -0400 |
| commit | 694e320643f7fc6a4f1a9cf3e4ac274c37cccdfb (patch) | |
| tree | be3c98e691f13aeb013f5d2fb516a94a1adb966c | |
| parent | 1807f6cfe9037a29b99c14c71b915b4776c729e0 (diff) | |
winsys/amdgpu: clean up and remove nonsensical assertion
The assertion considers max_dw from the current IB in the chain, but
big_ib_buffer is a buffer for the next IB, which can be smaller.
Tested-by: Dieter Nützel <[email protected]>
Acked-by: Nicolai Hähnle <[email protected]>
| -rw-r--r-- | src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c index 83b04ff53af..4a2377f7e09 100644 --- a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c +++ b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c @@ -1030,8 +1030,7 @@ static bool amdgpu_cs_check_space(struct radeon_cmdbuf *rcs, unsigned dw) va = amdgpu_winsys_bo(ib->big_ib_buffer)->va; /* This space was originally reserved. */ - rcs->current.max_dw += 4; - assert(ib->used_ib_space + 4 * rcs->current.max_dw <= ib->big_ib_buffer->size); + rcs->current.max_dw += cs_epilog_dw; /* Pad with NOPs and add INDIRECT_BUFFER packet */ while ((rcs->current.cdw & 7) != 4) |