[Mesa-dev] [PATCH 2/2] r600g: use SIMPLE_FLOAT for blending to avoid NaNs in RTs

[Nicolai Hähnle]

On 06.11.2017 05:22, Ilia Mirkin wrote:
> Radeonsi also sets this flag.
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103544
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> ---
> 
> This needs testing with the fbo-float-nan piglit that was recently added. Just
> guessing that this is the right flag to set here.

Assuming that the test passes:

Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>


> 
>   src/gallium/drivers/r600/evergreen_state.c | 1 +
>   src/gallium/drivers/r600/r600_state.c      | 1 +
>   2 files changed, 2 insertions(+)
> 
> diff --git a/src/gallium/drivers/r600/evergreen_state.c b/src/gallium/drivers/r600/evergreen_state.c
> index 96eb35a9818..131778dea9f 100644
> --- a/src/gallium/drivers/r600/evergreen_state.c
> +++ b/src/gallium/drivers/r600/evergreen_state.c
> @@ -1211,6 +1211,7 @@ static void evergreen_set_color_surface_common(struct r600_context *rctx,
>   		S_028C70_COMP_SWAP(swap) |
>   		S_028C70_BLEND_CLAMP(blend_clamp) |
>   		S_028C70_BLEND_BYPASS(blend_bypass) |
> +		S_028C70_SIMPLE_FLOAT(1) |
>   		S_028C70_NUMBER_TYPE(ntype) |
>   		S_028C70_ENDIAN(endian);
>   
> diff --git a/src/gallium/drivers/r600/r600_state.c b/src/gallium/drivers/r600/r600_state.c
> index c21e8dabb1f..0c331537460 100644
> --- a/src/gallium/drivers/r600/r600_state.c
> +++ b/src/gallium/drivers/r600/r600_state.c
> @@ -898,6 +898,7 @@ static void r600_init_color_surface(struct r600_context *rctx,
>   		S_0280A0_COMP_SWAP(swap) |
>   		S_0280A0_BLEND_BYPASS(blend_bypass) |
>   		S_0280A0_BLEND_CLAMP(blend_clamp) |
> +		S_0280A0_SIMPLE_FLOAT(1) |
>   		S_0280A0_NUMBER_TYPE(ntype) |
>   		S_0280A0_ENDIAN(endian);
>   
> 


-- 
Lerne, wie die Welt wirklich ist,
Aber vergiss niemals, wie sie sein sollte.