mesa/st: hardcode the viewport bounds range
authorIlia Mirkin <imirkin@alum.mit.edu>
Tue, 11 Feb 2014 05:26:32 +0000 (00:26 -0500)
committerDave Airlie <airlied@redhat.com>
Thu, 13 Feb 2014 02:44:36 +0000 (12:44 +1000)
The bound range is disconnected from the viewport dimensions. This is
the relevant bit from glViewportArray:

"""
The location of the viewport's bottom left corner, given by (x, y) is
clamped to be within the implementaiton-dependent viewport bounds range.
The viewport bounds range [min, max] can be determined by calling glGet
with argument GL_VIEWPORT_BOUNDS_RANGE. Viewport width and height are
silently clamped to a range that depends on the implementation. To query
this range, call glGet with argument GL_MAX_VIEWPORT_DIMS.
"""

Just set it to +/-16384, as that is the minimum required by
ARB_viewport_array and the value that all current drivers provide.

Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Signed-off-by: Dave Airlie <airlied@redhat.com>
src/mesa/state_tracker/st_extensions.c

index 1b3b2205fef2452631d7ab23150e6254d79a904a..e43e7b44ec5ba7c0ecc3a0cf1ebc391274d713dd 100644 (file)
@@ -782,8 +782,8 @@ void st_init_extensions(struct st_context *st)
    if (ctx->API == API_OPENGL_CORE) {
       ctx->Const.MaxViewports = screen->get_param(screen, PIPE_CAP_MAX_VIEWPORTS);
       if (ctx->Const.MaxViewports >= 16) {
-         ctx->Const.ViewportBounds.Min = -(float)ctx->Const.MaxViewportWidth;
-         ctx->Const.ViewportBounds.Max = ctx->Const.MaxViewportWidth;
+         ctx->Const.ViewportBounds.Min = -16384.0;
+         ctx->Const.ViewportBounds.Max = 16384.0;
          ctx->Extensions.ARB_viewport_array = GL_TRUE;
       }
    }