Mesa (master): r600g, radeonsi: don't add streamout.num_dw_for_end twice

Marek Olšák mareko at kemper.freedesktop.org
Tue Mar 11 18:05:34 UTC 2014


Module: Mesa
Branch: master
Commit: 958ef47a6dea652d79a676625b69b3dec6042395
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=958ef47a6dea652d79a676625b69b3dec6042395

Author: Marek Olšák <marek.olsak at amd.com>
Date:   Sun Mar  9 22:10:27 2014 +0100

r600g,radeonsi: don't add streamout.num_dw_for_end twice

It's already added in need_cs_space.

Also don't calculate anything if there are no buffers.

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>

---

 src/gallium/drivers/radeon/r600_streamout.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/gallium/drivers/radeon/r600_streamout.c b/src/gallium/drivers/radeon/r600_streamout.c
index a556e3a..d9b1d1c 100644
--- a/src/gallium/drivers/radeon/r600_streamout.c
+++ b/src/gallium/drivers/radeon/r600_streamout.c
@@ -79,6 +79,9 @@ void r600_streamout_buffers_dirty(struct r600_common_context *rctx)
 	unsigned num_bufs_appended = util_bitcount(rctx->streamout.enabled_mask &
 						   rctx->streamout.append_bitmask);
 
+	if (!num_bufs)
+		return;
+
 	rctx->streamout.num_dw_for_end =
 		12 + /* flush_vgt_streamout */
 		num_bufs * 8 + /* STRMOUT_BUFFER_UPDATE */
@@ -99,8 +102,7 @@ void r600_streamout_buffers_dirty(struct r600_common_context *rctx)
 	begin->num_dw +=
 		num_bufs_appended * 8 + /* STRMOUT_BUFFER_UPDATE */
 		(num_bufs - num_bufs_appended) * 6 + /* STRMOUT_BUFFER_UPDATE */
-		(rctx->family > CHIP_R600 && rctx->family < CHIP_RS780 ? 2 : 0) + /* SURFACE_BASE_UPDATE */
-		rctx->streamout.num_dw_for_end;
+		(rctx->family > CHIP_R600 && rctx->family < CHIP_RS780 ? 2 : 0); /* SURFACE_BASE_UPDATE */
 
 	begin->dirty = true;
 }




More information about the mesa-commit mailing list