Ping.
2016-10-06 19:50 GMT+02:00 Gustaw Smolarczyk <[email protected]>: > It's supposed to be how much at least we want to grow the cs, not the > minimum size of the cs after growth. > > v2: Unbreak use_ib_bos. > Don't mask the ib_size when !use_ib_bos, since it's not needed. > --- > src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c | 11 +++++++---- > 1 file changed, 7 insertions(+), 4 deletions(-) > > diff --git a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c > b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c > index dedc778..c07c092 100644 > --- a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c > +++ b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c > @@ -178,10 +178,6 @@ radv_amdgpu_cs_create(struct radeon_winsys *ws, > static void radv_amdgpu_cs_grow(struct radeon_winsys_cs *_cs, size_t > min_size) > { > struct radv_amdgpu_cs *cs = radv_amdgpu_cs(_cs); > - uint64_t ib_size = MAX2(min_size * 4 + 16, cs->base.max_dw * 4 * 2); > - > - /* max that fits in the chain size field. */ > - ib_size = MIN2(ib_size, 0xfffff); > > if (cs->failed) { > cs->base.cdw = 0; > @@ -189,6 +185,8 @@ static void radv_amdgpu_cs_grow(struct radeon_winsys_cs > *_cs, size_t min_size) > } > > if (!cs->ws->use_ib_bos) { > + uint64_t ib_size = MAX2((cs->base.cdw + min_size) * 4 + 16, > + cs->base.max_dw * 4 * 2); > uint32_t *new_buf = realloc(cs->base.buf, ib_size); > if (new_buf) { > cs->base.buf = new_buf; > @@ -200,6 +198,11 @@ static void radv_amdgpu_cs_grow(struct radeon_winsys_cs > *_cs, size_t min_size) > return; > } > > + uint64_t ib_size = MAX2(min_size * 4 + 16, cs->base.max_dw * 4 * 2); > + > + /* max that fits in the chain size field. */ > + ib_size = MIN2(ib_size, 0xfffff); > + > while (!cs->base.cdw || (cs->base.cdw & 7) != 4) > cs->base.buf[cs->base.cdw++] = 0xffff1000; > > -- > 2.10.0 > _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
