[Mesa-dev] [PATCH 04/22] i965: assign BindingTableEntryCount of INTERFACE_DESCRIPTOR_DATA

Lionel Landwerlin lionel.g.landwerlin at intel.com
Mon Sep 25 12:09:38 UTC 2017


On 25/09/17 11:34, kevin.rogovin at intel.com wrote:
> From: Kevin Rogovin <kevin.rogovin at intel.com>
>
> Signed-off-by: Kevin Rogovin <kevin.rogovin at intel.com>
> ---
>   src/mesa/drivers/dri/i965/genX_state_upload.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c
> index 6127616..bbb47ea 100644
> --- a/src/mesa/drivers/dri/i965/genX_state_upload.c
> +++ b/src/mesa/drivers/dri/i965/genX_state_upload.c
> @@ -4199,6 +4199,7 @@ genX(upload_cs_state)(struct brw_context *brw)
>         .SamplerStatePointer = stage_state->sampler_offset,
>         .SamplerCount = DIV_ROUND_UP(stage_state->sampler_count, 4) >> 2,
>         .BindingTablePointer = stage_state->bind_bo_offset,
> +      .BindingTableEntryCount = prog_data->binding_table.size_bytes / 4,

I'm not sure this should be divided by 4. The documentation doesn't 
seemto say it behaves like SamplerCount.

>         .ConstantURBEntryReadLength = cs_prog_data->push.per_thread.regs,
>         .NumberofThreadsinGPGPUThreadGroup = cs_prog_data->threads,
>         .SharedLocalMemorySize = encode_slm_size(GEN_GEN,




More information about the mesa-dev mailing list