[Mesa-dev] [PATCH] i965: Fix anisotropic filtering for mag filter

[Kenneth Graunke]

On Tuesday, June 27, 2017 10:37:16 AM PDT Rafael Antognolli wrote:
> From: Eero Tamminen <eero.t.tamminen at intel.com>
> 
> Commit f8d69beed49c64f883bb8ffb28d4960306baf575 moving sampler
> handling to genxml messed up change done by commit
> 6a7c5257cac23cd9767aa4bc8fdab68925b11157.
> 
> This broke rendering in SynMark CSDof and TexFilterAniso tests.
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101607
> 
> Thanks to Kevin, who spotted the actual typo!
> ---
>  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 d65b468..06b9cd5 100644
> --- a/src/mesa/drivers/dri/i965/genX_state_upload.c
> +++ b/src/mesa/drivers/dri/i965/genX_state_upload.c
> @@ -4551,7 +4551,7 @@ genX(update_sampler_state)(struct brw_context *brw,
>     if (sampler->MaxAnisotropy > 1.0f) {
>        if (samp_st.MinModeFilter == MAPFILTER_LINEAR)
>           samp_st.MinModeFilter = MAPFILTER_ANISOTROPIC;
> -      if (samp_st.MinModeFilter == MAPFILTER_LINEAR)
> +      if (samp_st.MagModeFilter == MAPFILTER_LINEAR)
>           samp_st.MagModeFilter = MAPFILTER_ANISOTROPIC;
>  
>        if (sampler->MaxAnisotropy > 2.0f) {
> 

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170627/7ea7bf49/attachment-0001.sig>