From a29ef75565e14f785c071ac99044300003168fa8 Mon Sep 17 00:00:00 2001 From: Samuel Pitoiset Date: Wed, 28 Jun 2017 18:46:31 +0200 Subject: [PATCH] radeonsi: make some si_descriptors fields 32-bit MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The number of bindless descriptors is dynamic and we definitely have to support more than 256 slots. Signed-off-by: Samuel Pitoiset Reviewed-by: Marek Olšák --- 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 bce40663085..2b3c37fa16d 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;