i965/sbe: fix number of inputs for active components
authorIago Toral Quiroga <itoral@igalia.com>
Thu, 1 Mar 2018 06:59:42 +0000 (07:59 +0100)
committerIago Toral Quiroga <itoral@igalia.com>
Thu, 1 Mar 2018 09:55:12 +0000 (10:55 +0100)
commitbc73016703f8f2815e000f1c100532cf6e13cd3c
treef4e03ca1c5b4c87b3cd5ec9f5fd3ec7e7d5209bf
parentc27f5419f6f6aa6d51b44a99b6738fba70873604
i965/sbe: fix number of inputs for active components

In 16631ca30ea6 we fixed gen9 active components to account for padded
inputs in the URB, which we can have with SSO programs. To do that,
instead of going through the bitfield of inputs (which doesn't include
padding information), we compute the number of inputs from the size
of the URB entry.

Unfortunately, there are some special inputs that are not stored in
the URB and that we also need to account for. These special inputs
are identified and handled during calculate_attr_overrides().

Instead of keeping track of the exact number of inputs, we just
program active components for all possible inputs like we do in
anvil.

This fixes a regression in a WebGL program that uses Point Sprite
functionality (specifically, VARYING_SLOT_PNTC).

v2:
 - Add 'Fixes' tag (Mark Janes)
 - make no_vue_inputs int instead of uint32_t, and add const qualifier
   to num_inputs variable (Ian)

v3:
 - Do not try to count inputs correctly, just program all input
   slots like we do in anvil (Ken)

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105224
Fixes: 16631ca30ea6 (i965/sbe: fix active components for SSO programs with over 16 inputs)
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/genX_state_upload.c