[Mesa-dev] [PATCH 05/22] i965: correctly assign SamplerCount of INTERFACE_DESCRIPTOR_DATA

[Lionel Landwerlin]

I'm genuinely surprised we didn't noticed this problem before :|

Fixes: 71bfb44005bf ("i965: Port brw_cs_state tracked state to genxml.")
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Cc: "17.2" <mesa-stable at lists.freedesktop.org>

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 | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c
> index bbb47ea..32c7d22 100644
> --- a/src/mesa/drivers/dri/i965/genX_state_upload.c
> +++ b/src/mesa/drivers/dri/i965/genX_state_upload.c
> @@ -4197,7 +4197,7 @@ genX(upload_cs_state)(struct brw_context *brw)
>      const struct GENX(INTERFACE_DESCRIPTOR_DATA) idd = {
>         .KernelStartPointer = brw->cs.base.prog_offset,
>         .SamplerStatePointer = stage_state->sampler_offset,
> -      .SamplerCount = DIV_ROUND_UP(stage_state->sampler_count, 4) >> 2,
> +      .SamplerCount = DIV_ROUND_UP(stage_state->sampler_count, 4),
>         .BindingTablePointer = stage_state->bind_bo_offset,
>         .BindingTableEntryCount = prog_data->binding_table.size_bytes / 4,
>         .ConstantURBEntryReadLength = cs_prog_data->push.per_thread.regs,