[Mesa-dev] [PATCH v4 2/6] radeonsi: make some si_descriptors fields 32-bit

Samuel Pitoiset samuel.pitoiset at gmail.com
Mon Aug 21 14:50:35 UTC 2017


The number of bindless descriptors is dynamic and we definitely
have to support more than 256 slots.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
---
 src/gallium/drivers/radeonsi/si_state.h | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/gallium/drivers/radeonsi/si_state.h b/src/gallium/drivers/radeonsi/si_state.h
index bce4066308..2b3c37fa16 100644
--- a/src/gallium/drivers/radeonsi/si_state.h
+++ b/src/gallium/drivers/radeonsi/si_state.h
@@ -234,7 +234,7 @@ struct si_descriptors {
 	/* The size of one descriptor. */
 	ubyte element_dw_size;
 	/* The maximum number of descriptors. */
-	ubyte num_elements;
+	uint32_t num_elements;
 
 	/* Offset in CE RAM */
 	uint16_t ce_offset;
@@ -243,16 +243,16 @@ struct si_descriptors {
 	 * range, direct uploads to memory will be used instead. This basically
 	 * governs switching between onchip (CE) and offchip (upload) modes.
 	 */
-	ubyte first_ce_slot;
-	ubyte num_ce_slots;
+	uint32_t first_ce_slot;
+	uint32_t num_ce_slots;
 
 	/* Slots that are used by currently-bound shaders.
 	 * With CE: It determines which slots are dumped to L2.
 	 *          It doesn't skip uploads to CE RAM.
 	 * Without CE: It determines which slots are uploaded.
 	 */
-	ubyte first_active_slot;
-	ubyte num_active_slots;
+	uint32_t first_active_slot;
+	uint32_t num_active_slots;
 
 	/* Whether CE is used to upload this descriptor array. */
 	bool uses_ce;
-- 
2.14.1



More information about the mesa-dev mailing list