[Mesa-dev] [PATCH] i965: Do not store SRC after 0 on component control.
Anuj Phogat
anuj.phogat at gmail.com
Wed Aug 23 22:41:22 UTC 2017
On Wed, Aug 23, 2017 at 2:59 PM, Rafael Antognolli
<rafael.antognolli at intel.com> wrote:
> The PRM SKL-Vol 2b-05.16 says:
>
> "Within a VERTEX_ELEMENT_STATE structure, if a Component Control
> field is set to something other than VFCOMP_STORE_SRC, no
> higher-numbered Component Control fields may be set to
> VFCOMP_STORE_SRC. In other words, only trailing components can be set
> to something other than VFCOMP_STORE_SRC."
>
> Since we set the component 1 to VFCOMP_STORE_0 on gen8+, and
> VFCOMP_STORE_IID on gen5+, and we are not using components 2 and 3,
> let's also set them to VFCOMP_STORE_0.
>
> Signed-off-by: Rafael Antognolli <rafael.antognolli at intel.com>
> ---
> src/intel/blorp/blorp_genX_exec.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/intel/blorp/blorp_genX_exec.h b/src/intel/blorp/blorp_genX_exec.h
> index 93534169ef..524736fbc0 100644
> --- a/src/intel/blorp/blorp_genX_exec.h
> +++ b/src/intel/blorp/blorp_genX_exec.h
> @@ -395,8 +395,8 @@ blorp_emit_vertex_elements(struct blorp_batch *batch,
> #else
> .Component1Control = VFCOMP_STORE_0,
> #endif
> - .Component2Control = VFCOMP_STORE_SRC,
> - .Component3Control = VFCOMP_STORE_SRC,
> + .Component2Control = VFCOMP_STORE_0,
> + .Component3Control = VFCOMP_STORE_0,
> #if GEN_GEN <= 5
> .DestinationElementOffset = slot * 4,
> #endif
> --
> 2.13.5
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>
More information about the mesa-dev
mailing list