fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-06 13:48:06 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

panvk/csf: Emit INDEX_BUFFER[_SIZE] even for non-indexed draws

Browse source 
The index buffer and index buffer size don't affect whether or not we're
actually doing indexed rendering so we should just emit them whenever
they change.  Otherwise, if someone sets an index buffer and then does a
non-indexed draw and then an indexed draw, the first draw will clear the
dirty bits without setting the index buffer registers and the second
draw won't know to re-emit them.

Fixes: 5544d39f44 ("panvk: Add a CSF backend for panvk_queue/cmd_buffer")
Reviewed-by: Lars-Ivar Hesselberg Simonsen <lars-ivar.simonsen@arm.com>
Reviewed-by: Marc Alcala Prieto <marc.alcalaprieto@arm.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/40997>
This commit is contained in:
Faith Ekstrand 2025-11-07 10:07:14 -05:00 • committed by Marge Bot
parent 3d7d2115f8
commit 9c8e8ed655
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/panfrost/vulkan/csf/panvk_vX_cmd_draw.c
View file

@ -2101,13 +2101,12 @@ prepare_dcd(struct panvk_cmd_buffer *cmdbuf,
}
static void
prepare_index_buffer(struct panvk_cmd_buffer *cmdbuf,
struct panvk_draw_info *draw)
prepare_index_buffer(struct panvk_cmd_buffer *cmdbuf)
{
struct cs_builder *b =
panvk_get_cs_builder(cmdbuf, PANVK_SUBQUEUE_VERTEX_TILER);
if (draw->index.size && gfx_state_dirty(cmdbuf, IB)) {
if (gfx_state_dirty(cmdbuf, IB)) {
cs_move32_to(b, cs_sr_reg32(b, IDVS, INDEX_BUFFER_SIZE),
cmdbuf->state.gfx.ib.size);
@ -2300,7 +2299,7 @@ prepare_draw(struct panvk_cmd_buffer *cmdbuf, struct panvk_draw_info *draw)
return result;
cs_update_vt_ctx(b) {
prepare_index_buffer(cmdbuf, draw);
prepare_index_buffer(cmdbuf);
set_tiler_idvs_flags(b, cmdbuf, draw);