Mesa (main): freedreno/a6xx: Split VFD_FETCH[] if needed
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Sun Jul 3 23:44:47 UTC 2022
Module: Mesa
Branch: main
Commit: 8b197f4ba751187b5ba19b1e31f03e5b6c58f356
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=8b197f4ba751187b5ba19b1e31f03e5b6c58f356
Author: Rob Clark <robdclark at chromium.org>
Date: Thu Jun 30 10:06:21 2022 -0700
freedreno/a6xx: Split VFD_FETCH[] if needed
Avoid overflowing max pkt4 size by splitting VFD_FETCH[] emit.
Otherwise the maximum size of 32 VBOs would overflow and wrap to
zero.
Signed-off-by: Rob Clark <robdclark at chromium.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17317>
---
src/gallium/drivers/freedreno/a6xx/fd6_emit.c | 18 ++++++++++++++----
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/freedreno/a6xx/fd6_emit.c b/src/gallium/drivers/freedreno/a6xx/fd6_emit.c
index bb2249aad7f..7fa2754bfb3 100644
--- a/src/gallium/drivers/freedreno/a6xx/fd6_emit.c
+++ b/src/gallium/drivers/freedreno/a6xx/fd6_emit.c
@@ -555,12 +555,22 @@ build_vbo_state(struct fd6_emit *emit) assert_dt
{
const struct fd_vertex_state *vtx = emit->vtx;
+ /* Limit PKT4 size, because at max count (32) we would overflow the
+ * size of the PKT4 size field:
+ */
+ const unsigned maxcnt = 16;
+ const unsigned cnt = vtx->vertexbuf.count;
+ const unsigned dwords = (cnt * 4) /* per vbo: reg64 + two reg32 */
+ + (1 + cnt / maxcnt); /* PKT4 hdr every 16 vbo's */
+
struct fd_ringbuffer *ring = fd_submit_new_ringbuffer(
- emit->ctx->batch->submit, 4 * (1 + vtx->vertexbuf.count * 4),
- FD_RINGBUFFER_STREAMING);
+ emit->ctx->batch->submit, 4 * dwords, FD_RINGBUFFER_STREAMING);
- OUT_PKT4(ring, REG_A6XX_VFD_FETCH(0), 4 * vtx->vertexbuf.count);
- for (int32_t j = 0; j < vtx->vertexbuf.count; j++) {
+ for (int32_t j = 0; j < cnt; j++) {
+ if ((j % maxcnt) == 0) {
+ unsigned sz = MIN2(maxcnt, cnt - j);
+ OUT_PKT4(ring, REG_A6XX_VFD_FETCH(j), 4 * sz);
+ }
const struct pipe_vertex_buffer *vb = &vtx->vertexbuf.vb[j];
struct fd_resource *rsc = fd_resource(vb->buffer.resource);
if (rsc == NULL) {
More information about the mesa-commit
mailing list