<p dir="ltr"><br>
On Mar 28, 2016 10:36 PM, "Kenneth Graunke" <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> wrote:<br>
><br>
> There is no linear filtering for integer formats, so we should always<br>
> be using CLAMP_TO_EDGE mode.<br>
><br>
> Fixes 46 dEQP cases on Ivybridge.<br>
><br>
> This workaround doesn't appear to be necessary on later hardware, and<br>
> I haven't found any documentation mentioning errata in this area.<br>
> However, it seems harmless to apply generally, so we may as well.</p>
<p dir="ltr">Seems reasonable</p>
<p dir="ltr">Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>></p>
<p dir="ltr">> Signed-off-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
> ---<br>
> src/mesa/drivers/dri/i965/brw_sampler_state.c | 3 ++-<br>
> 1 file changed, 2 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/brw_sampler_state.c b/src/mesa/drivers/dri/i965/brw_sampler_state.c<br>
> index 7bd21f7..7e44cf5 100644<br>
> --- a/src/mesa/drivers/dri/i965/brw_sampler_state.c<br>
> +++ b/src/mesa/drivers/dri/i965/brw_sampler_state.c<br>
> @@ -463,7 +463,8 @@ brw_update_sampler_state(struct brw_context *brw,<br>
> /* Cube maps must use the same wrap mode for all three coordinate<br>
> * dimensions. Prior to Haswell, only CUBE and CLAMP are valid.<br>
> */<br>
> - if (tex_cube_map_seamless || sampler->CubeMapSeamless) {<br>
> + if ((tex_cube_map_seamless || sampler->CubeMapSeamless) &&<br>
> + !is_integer_format) {<br>
> wrap_s = BRW_TEXCOORDMODE_CUBE;<br>
> wrap_t = BRW_TEXCOORDMODE_CUBE;<br>
> wrap_r = BRW_TEXCOORDMODE_CUBE;<br>
> --<br>
> 2.7.4<br>
><br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</p>