Mesa (main): radeonsi: remove redundant setting scratch_state atom dirty

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Fri Jul 16 23:21:51 UTC 2021


Module: Mesa
Branch: main
Commit: be9ca622478ac4ef1eda9c1ce6244d7bed7e02f9
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=be9ca622478ac4ef1eda9c1ce6244d7bed7e02f9

Author: Yogesh mohan marimuthu <yogesh.mohanmarimuthu at amd.com>
Date:   Tue Jun 15 14:55:40 2021 +0530

radeonsi: remove redundant setting scratch_state atom dirty

Whenever scratch buffer is allocated, current spi_tmpring_size and
previous spi_tmpring_size cannot be same and hence scratch_state will be
set dirty as part of "if (spi_tmpring_size != sctx->spi_tmpring_size)".
Removing redundant dirty bit sat while allocating scratch buffer.

Signed-off-by: Yogesh mohan marimuthu <yogesh.mohanmarimuthu at amd.com>
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer at amd.com>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11900>

---

 src/gallium/drivers/radeonsi/si_state_shaders.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c b/src/gallium/drivers/radeonsi/si_state_shaders.c
index 5ef033ed6a5..51db88a6188 100644
--- a/src/gallium/drivers/radeonsi/si_state_shaders.c
+++ b/src/gallium/drivers/radeonsi/si_state_shaders.c
@@ -3809,7 +3809,6 @@ static bool si_update_spi_tmpring_size(struct si_context *sctx)
          if (!sctx->scratch_buffer)
             return false;
 
-         si_mark_atom_dirty(sctx, &sctx->atoms.s.scratch_state);
          si_context_add_resource_size(sctx, &sctx->scratch_buffer->b.b);
       }
 



More information about the mesa-commit mailing list