Mesa (master): st/mesa: MAX_VARYING is the max supported number of patch varyings, not min

Ilia Mirkin imirkin at kemper.freedesktop.org
Fri Feb 3 01:31:16 UTC 2017


Module: Mesa
Branch: master
Commit: 7d3f9ed71c71637a91ecf15f50dbe7578a65d57e
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7d3f9ed71c71637a91ecf15f50dbe7578a65d57e

Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Wed Jan 25 22:31:58 2017 -0500

st/mesa: MAX_VARYING is the max supported number of patch varyings, not min

This fixes
GL45-CTS.tessellation_shader.tessellation_shader_tessellation.max_in_out_attributes
on nouveau. We only support 30 patch varyings (as 2 vec4 slots end up
being used for tess level settings), but were getting 32 exposed.

Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
Cc: "13.0 17.0" <mesa-stable at lists.freedesktop.org>

---

 src/mesa/state_tracker/st_extensions.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/state_tracker/st_extensions.c b/src/mesa/state_tracker/st_extensions.c
index 4d46006..293814e 100644
--- a/src/mesa/state_tracker/st_extensions.c
+++ b/src/mesa/state_tracker/st_extensions.c
@@ -348,7 +348,7 @@ void st_init_limits(struct pipe_screen *screen,
    c->MaxGeometryTotalOutputComponents =
       screen->get_param(screen, PIPE_CAP_MAX_GEOMETRY_TOTAL_OUTPUT_COMPONENTS);
    c->MaxTessPatchComponents =
-      MAX2(screen->get_param(screen, PIPE_CAP_MAX_SHADER_PATCH_VARYINGS),
+      MIN2(screen->get_param(screen, PIPE_CAP_MAX_SHADER_PATCH_VARYINGS),
            MAX_VARYING) * 4;
 
    c->MinProgramTexelOffset =




More information about the mesa-commit mailing list