This also uses an obvious default value in the getters in case there's
a GL error and the value isn't written to the pointer.
---
 tests/spec/arb_texture_buffer_object/minmax.c |   28 +--
 tests/spec/gl-2.1/minmax.c                    |  199 +++++-------------
 tests/spec/gl-3.0/minmax.c                    |  278 +++++-------------------
 tests/spec/gl-3.1/minmax.c                    |  280 +++++--------------------
 tests/util/CMakeLists.gl.txt                  |    1 +
 tests/util/minmax-test.c                      |  249 ++++++++++++++++++++++
 tests/util/minmax-test.h                      |   33 +++
 7 files changed, 447 insertions(+), 621 deletions(-)
 create mode 100644 tests/util/minmax-test.c
 create mode 100644 tests/util/minmax-test.h

diff --git a/tests/spec/arb_texture_buffer_object/minmax.c 
b/tests/spec/arb_texture_buffer_object/minmax.c
index 36dd6b6..698e73d 100644
--- a/tests/spec/arb_texture_buffer_object/minmax.c
+++ b/tests/spec/arb_texture_buffer_object/minmax.c
@@ -26,6 +26,7 @@
  */
 
 #include "piglit-util.h"
+#include "minmax-test.h"
 
 int piglit_width = 32;
 int piglit_height = 32;
@@ -38,34 +39,13 @@ piglit_display(void)
        return PIGLIT_FAIL;
 }
 
-static bool pass = true;
-
-static void
-min_test_i(GLenum token, GLint min, const char *name)
-{
-       GLint val = 0;
-
-       glGetIntegerv(token, &val);
-
-       if (val < min) {
-               fprintf(stderr, "%-50s %8d %8d (ERROR)\n",
-                       name, min, val);
-               pass = false;
-       } else {
-               printf("%-50s %8d %8d\n", name, min, val);
-       }
-}
-
-#define MIN_INTEGER_TEST(token, min) min_test_i(token, min, #token)
-
 void
 piglit_init(int argc, char **argv)
 {
        piglit_require_extension("GL_ARB_texture_buffer_object");
 
-       printf("%-50s %8s %8s\n", "token", "minimum", "value");
-
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_BUFFER_SIZE, 65536);
+       piglit_print_minmax_header();
+       piglit_test_min_int(GL_MAX_TEXTURE_BUFFER_SIZE, 65536);
 
-       piglit_report_result(pass ? PIGLIT_PASS : PIGLIT_FAIL);
+       piglit_report_result(piglit_minmax_pass ? PIGLIT_PASS : PIGLIT_FAIL);
 }
diff --git a/tests/spec/gl-2.1/minmax.c b/tests/spec/gl-2.1/minmax.c
index 7073968..64f4a09 100644
--- a/tests/spec/gl-2.1/minmax.c
+++ b/tests/spec/gl-2.1/minmax.c
@@ -27,6 +27,7 @@
  */
 
 #include "piglit-util.h"
+#include "minmax-test.h"
 
 int piglit_width = 32;
 int piglit_height = 32;
@@ -39,170 +40,66 @@ piglit_display(void)
        return PIGLIT_FAIL;
 }
 
-static bool pass = true;
-
-static void
-min_test_i(GLenum token, GLint min, const char *name)
-{
-       GLint vals[2];
-
-       glGetIntegerv(token, vals);
-
-       if (vals[0] < min) {
-               fprintf(stderr, "%-50s %8d %8d (ERROR)\n",
-                       name, min, vals[0]);
-               pass = false;
-       } else {
-               printf("%-50s %8d %8d\n", name, min, vals[0]);
-       }
-}
-
-static void
-min_test_f(GLenum token, GLfloat min, const char *name)
+void
+piglit_init(int argc, char **argv)
 {
-       GLfloat val;
-
-       glGetFloatv(token, &val);
-
-       if (val < min) {
-               fprintf(stderr, "%-50s %8f %8f (ERROR)\n",
-                       name, min, val);
-               pass = false;
-       } else {
-               printf("%-50s %8f %8f\n", name, min, val);
-       }
-}
+       piglit_require_gl_version(21);
 
-/* All the size requirements happen to only require a range covering
- * [1.0, 1.0].
- */
-static void
-size_range_test(GLenum token, const char *name)
-{
-       GLfloat vals[2];
+       piglit_print_minmax_header();
 
-       glGetFloatv(token, vals);
+       piglit_test_min_int(GL_MAX_LIGHTS, 8);
+       piglit_test_min_int(GL_MAX_CLIP_PLANES, 6);
+       if (piglit_is_extension_supported("GL_ARB_imaging"))
+               piglit_test_min_int(GL_MAX_COLOR_MATRIX_STACK_DEPTH, 2);
+       piglit_test_min_int(GL_MAX_MODELVIEW_STACK_DEPTH, 32);
+       piglit_test_min_int(GL_MAX_PROJECTION_STACK_DEPTH, 2);
+       piglit_test_min_int(GL_MAX_TEXTURE_STACK_DEPTH, 2);
+       piglit_test_min_int(GL_SUBPIXEL_BITS, 4);
 
-       if (vals[0] > 1.0 || vals[1] < 1.0) {
-               fprintf(stderr, "%-50s %8s  %.1f-%.1f (ERROR)\n",
-                       name, "1-1", vals[0], vals[1]);
-               pass = false;
-       } else {
-               printf("%-50s %8s  %.1f-%.1f\n",
-                      name, "1-1", vals[0], vals[1]);
-       }
-}
+       piglit_test_min_int(GL_MAX_3D_TEXTURE_SIZE, 16);
+       piglit_test_min_int(GL_MAX_TEXTURE_SIZE, 64);
+       piglit_test_min_float(GL_MAX_TEXTURE_LOD_BIAS, 2.0);
+       piglit_test_min_int(GL_MAX_CUBE_MAP_TEXTURE_SIZE, 16);
+       piglit_test_min_int(GL_MAX_PIXEL_MAP_TABLE, 32);
+       piglit_test_min_int(GL_MAX_NAME_STACK_DEPTH, 64);
+       piglit_test_min_int(GL_MAX_LIST_NESTING, 64);
+       piglit_test_min_int(GL_MAX_EVAL_ORDER, 8);
 
-static void
-test_oq_bits()
-{
-       GLint dims[2];
-       GLint minbits, oqbits;
-
-       /* From the GL 2.1 specification:
-        *
-        *     "The number of query counter bits may be zero, in which
-        *      case the counter contains no useful
-        *      information. Otherwise, the minimum number of bits
-        *      allowed is a function of the implementation’s maximum
-        *      viewport dimensions (MAX_VIEWPORT_DIMS). In this case,
-        *      the counter must be able to represent at least two
-        *      overdraws for every pixel in the viewport The formula
-        *      to compute the allowable minimum value (where n is the
-        *      minimum number of bits) is:
-        *
-        *      n = min{32, log2(maxViewportWidth ∗ maxViewportHeight * 2}"
-        */
-
-       glGetIntegerv(GL_MAX_VIEWPORT_DIMS, dims);
-       minbits = log2((float)dims[0] * dims[1] * 2);
-       if (minbits > 32)
-               minbits = 32;
-
-       glGetQueryiv(GL_SAMPLES_PASSED, GL_QUERY_COUNTER_BITS, &oqbits);
-       if (oqbits == 0 || oqbits >= minbits) {
-               printf("%-50s   0 / %2d %8d\n",
-                      "GL_QUERY_COUNTER_BITS(GL_SAMPLES_PASSED)",
-                      minbits, oqbits);
-       } else {
-               fprintf(stderr,
-                       "%-50s   0 / %2d %8d\n",
-                       "GL_QUERY_COUNTER_BITS(GL_SAMPLES_PASSED)",
-                       minbits, oqbits);
-               pass = false;
-       }
-}
+       piglit_test_min_viewport_dimensions();
 
-#define MAX_INTEGER_TEST(token, max) max_test_i(token, max, #token)
-#define MIN_INTEGER_TEST(token, min) min_test_i(token, min, #token)
-#define MIN_FLOAT_TEST(token, min) min_test_f(token, min, #token)
-#define TEST_SIZE_RANGE(token) size_range_test(token, #token)
+       piglit_test_min_int(GL_MAX_ATTRIB_STACK_DEPTH, 16);
+       piglit_test_min_int(GL_MAX_CLIENT_ATTRIB_STACK_DEPTH, 16);
 
-void
-piglit_init(int argc, char **argv)
-{
-       piglit_require_gl_version(21);
+       piglit_test_range_float(GL_ALIASED_POINT_SIZE_RANGE, 1, 1);
+       piglit_test_range_float(GL_SMOOTH_POINT_SIZE_RANGE, 1, 1);
+       piglit_test_range_float(GL_ALIASED_LINE_WIDTH_RANGE, 1, 1);
+       piglit_test_range_float(GL_SMOOTH_LINE_WIDTH_RANGE, 1, 1);
 
-       printf("%-50s %8s %8s\n", "token", "minimum", "value");
+       piglit_test_oq_bits();
 
-       MIN_INTEGER_TEST(GL_MAX_LIGHTS, 8);
-       MIN_INTEGER_TEST(GL_MAX_CLIP_PLANES, 6);
-       if (piglit_is_extension_supported("GL_ARB_imaging"))
-               MIN_INTEGER_TEST(GL_MAX_COLOR_MATRIX_STACK_DEPTH, 2);
-       MIN_INTEGER_TEST(GL_MAX_MODELVIEW_STACK_DEPTH, 32);
-       MIN_INTEGER_TEST(GL_MAX_PROJECTION_STACK_DEPTH, 2);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_STACK_DEPTH, 2);
-       MIN_INTEGER_TEST(GL_SUBPIXEL_BITS, 4);
-
-       MIN_INTEGER_TEST(GL_MAX_3D_TEXTURE_SIZE, 16);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_SIZE, 64);
-       MIN_FLOAT_TEST(GL_MAX_TEXTURE_LOD_BIAS, 2.0);
-       MIN_INTEGER_TEST(GL_MAX_CUBE_MAP_TEXTURE_SIZE, 16);
-       MIN_INTEGER_TEST(GL_MAX_PIXEL_MAP_TABLE, 32);
-       MIN_INTEGER_TEST(GL_MAX_NAME_STACK_DEPTH, 64);
-       MIN_INTEGER_TEST(GL_MAX_LIST_NESTING, 64);
-       MIN_INTEGER_TEST(GL_MAX_EVAL_ORDER, 8);
-
-       /* FINISHME:
-        *
-        *     "The maximum viewport dimensions must be greater than
-        *      or equal to the visible dimensions of the display
-        *      being rendered to."
-        *
-        * Surely the screen is at least 1024x768, right?
-        */
-       MIN_INTEGER_TEST(GL_MAX_VIEWPORT_DIMS, 1024);
-
-       MIN_INTEGER_TEST(GL_MAX_ATTRIB_STACK_DEPTH, 16);
-       MIN_INTEGER_TEST(GL_MAX_CLIENT_ATTRIB_STACK_DEPTH, 16);
-
-       TEST_SIZE_RANGE(GL_ALIASED_POINT_SIZE_RANGE);
-       TEST_SIZE_RANGE(GL_SMOOTH_POINT_SIZE_RANGE);
-       TEST_SIZE_RANGE(GL_ALIASED_LINE_WIDTH_RANGE);
-       TEST_SIZE_RANGE(GL_SMOOTH_LINE_WIDTH_RANGE);
-
-       test_oq_bits();
-
-       MIN_INTEGER_TEST(GL_AUX_BUFFERS, 0);
+       piglit_test_min_int(GL_AUX_BUFFERS, 0);
 
        if (piglit_is_extension_supported("GL_ARB_imaging")) {
                /* FINISHME: GL_MAX_CONVOLUTION_WIDTH */
                /* FINISHME: GL_MAX_CONVOLUTION_HEIGHT */
        }
 
-       MIN_INTEGER_TEST(GL_SAMPLE_BUFFERS, 0);
-       MIN_INTEGER_TEST(GL_SAMPLES, 0);
-
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_UNITS, 2);
-       MIN_INTEGER_TEST(GL_MAX_VERTEX_ATTRIBS, 16);
-       MIN_INTEGER_TEST(GL_MAX_VERTEX_UNIFORM_COMPONENTS, 512);
-       MIN_INTEGER_TEST(GL_MAX_VARYING_COMPONENTS, 32);
-       MIN_INTEGER_TEST(GL_MAX_COMBINED_TEXTURE_IMAGE_UNITS, 2);
-       MIN_INTEGER_TEST(GL_MAX_VERTEX_TEXTURE_IMAGE_UNITS, 0);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_IMAGE_UNITS, 2);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_COORDS, 2);
-       MIN_INTEGER_TEST(GL_MAX_FRAGMENT_UNIFORM_COMPONENTS, 64);
-       MIN_INTEGER_TEST(GL_MAX_DRAW_BUFFERS, 1);
-
-       piglit_report_result(pass ? PIGLIT_PASS : PIGLIT_FAIL);
+       piglit_test_min_int(GL_SAMPLE_BUFFERS, 0);
+       piglit_test_min_int(GL_SAMPLES, 0);
+
+       piglit_test_min_int(GL_MAX_TEXTURE_UNITS, 2);
+       piglit_test_min_int(GL_MAX_VERTEX_ATTRIBS, 16);
+       piglit_test_min_int(GL_MAX_VERTEX_UNIFORM_COMPONENTS, 512);
+       piglit_test_min_int(GL_MAX_VARYING_COMPONENTS, 32);
+       piglit_test_min_int(GL_MAX_COMBINED_TEXTURE_IMAGE_UNITS, 2);
+       piglit_test_min_int(GL_MAX_VERTEX_TEXTURE_IMAGE_UNITS, 0);
+       piglit_test_min_int(GL_MAX_TEXTURE_IMAGE_UNITS, 2);
+       piglit_test_min_int(GL_MAX_TEXTURE_COORDS, 2);
+       piglit_test_min_int(GL_MAX_FRAGMENT_UNIFORM_COMPONENTS, 64);
+       piglit_test_min_int(GL_MAX_DRAW_BUFFERS, 1);
+
+       if (!piglit_check_gl_error(GL_NO_ERROR))
+               piglit_report_result(PIGLIT_FAIL);
+
+       piglit_report_result(piglit_minmax_pass ? PIGLIT_PASS : PIGLIT_FAIL);
 }
diff --git a/tests/spec/gl-3.0/minmax.c b/tests/spec/gl-3.0/minmax.c
index c2175de..0e40ee1 100644
--- a/tests/spec/gl-3.0/minmax.c
+++ b/tests/spec/gl-3.0/minmax.c
@@ -27,6 +27,7 @@
  */
 
 #include "piglit-util.h"
+#include "minmax-test.h"
 
 int piglit_width = 32;
 int piglit_height = 32;
@@ -39,246 +40,81 @@ piglit_display(void)
        return PIGLIT_FAIL;
 }
 
-static bool pass = true;
-
-static void
-max_test_i(GLenum token, GLint max, const char *name)
-{
-       GLint val[2];
-
-       glGetIntegerv(token, val);
-
-       if (val[0] > max) {
-               fprintf(stderr, "%-50s %8d %8d (ERROR)\n",
-                       name, max, val[0]);
-               pass = false;
-       } else {
-               printf("%-50s %8d %8d\n", name, max, val[0]);
-       }
-}
-
-static void
-min_test_i(GLenum token, GLint min, const char *name)
-{
-       GLint val;
-
-       glGetIntegerv(token, &val);
-
-       if (val < min) {
-               fprintf(stderr, "%-50s %8d %8d (ERROR)\n",
-                       name, min, val);
-               pass = false;
-       } else {
-               printf("%-50s %8d %8d\n", name, min, val);
-       }
-}
-
-static void
-min_test_f(GLenum token, GLfloat min, const char *name)
-{
-       GLfloat val;
-
-       glGetFloatv(token, &val);
-
-       if (val < min) {
-               fprintf(stderr, "%-50s %8f %8f (ERROR)\n",
-                       name, min, val);
-               pass = false;
-       } else {
-               printf("%-50s %8f %8f\n", name, min, val);
-       }
-}
-
-/* All the size requirements happen to only require a range covering
- * [1.0, 1.0].
- */
-static void
-size_range_test(GLenum token, const char *name)
-{
-       GLfloat vals[2];
-
-       glGetFloatv(token, vals);
-
-       if (vals[0] > 1.0 || vals[1] < 1.0) {
-               fprintf(stderr, "%-50s %8s  %.1f-%.1f (ERROR)\n",
-                       name, "1-1", vals[0], vals[1]);
-               pass = false;
-       } else {
-               printf("%-50s %8s  %.1f-%.1f\n",
-                      name, "1-1", vals[0], vals[1]);
-       }
-}
-
-static void
-test_oq_bits()
-{
-       GLint dims[2];
-       GLint minbits, oqbits;
-
-       /* From the GL 3.0 specification, page 329:
-        *
-        *     "If pname is QUERY_COUNTER_BITS, the
-        *      implementation-dependent number of query counter bits
-        *      may be zero, in which case the counter contains no
-        *      useful information.
-        *
-        *      For occlusion queries (SAMPLES PASSED), if the number
-        *      of bits is non-zero, the minimum number of bits
-        *      allowed is a function of the implementation’s maximum
-        *      viewport dimensions (MAX VIEWPORT DIMS). The counter
-        *      must be able to represent at least two overdraws for
-        *      every pixel in the viewport. The formula to compute
-        *      the allowable minimum value (where n is the minimum
-        *      number of bits) is
-        *
-        *      n = min{32, log2(maxViewportWidth ∗ maxViewportHeight * 2}"
-        */
-
-       glGetIntegerv(GL_MAX_VIEWPORT_DIMS, dims);
-       minbits = log2((float)dims[0] * dims[1] * 2);
-       if (minbits > 32)
-               minbits = 32;
-
-       glGetQueryiv(GL_SAMPLES_PASSED, GL_QUERY_COUNTER_BITS, &oqbits);
-       if (oqbits == 0 || oqbits >= minbits) {
-               printf("%-50s   0 / %2d %8d\n",
-                      "GL_QUERY_COUNTER_BITS(GL_SAMPLES_PASSED)",
-                      minbits, oqbits);
-       } else {
-               fprintf(stderr,
-                       "%-50s   0 / %2d %8d\n",
-                       "GL_QUERY_COUNTER_BITS(GL_SAMPLES_PASSED)",
-                       minbits, oqbits);
-               pass = false;
-       }
-}
-
-static void
-test_tf_bits(GLenum target)
-{
-       GLint bits = -1;
-       const char *name;
-
-       if (target == GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN)
-               name = "GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN bits";
-       else
-               name = "GL_PRIMITIVES_GENERATED bits";
-
-       /* From the GL 3.0 specification, page 329:
-        *
-        *     "If pname is QUERY_COUNTER_BITS, the
-        *      implementation-dependent number of query counter bits
-        *      may be zero, in which case the counter contains no
-        *      useful information.
-        *
-        *      For primitive queries (PRIMITIVES GENERATED and
-        *      TRANSFORM FEEDBACK PRIMITIVES WRITTEN) if the number
-        *      of bits is non-zero, the minimum number of bits
-        *      allowed is 32."
-        */
-
-       glGetQueryiv(target, GL_QUERY_COUNTER_BITS, &bits);
-       if (bits == 0 || bits >= 32) {
-               printf("%-50s %8s %8d\n", name, "0 / 32", bits);
-       } else {
-               fprintf(stderr, "%-50s %8s %8d (ERROR)\n",
-                       name, "0 / 32", bits);
-               pass = false;
-       }
-}
-
-#define MAX_INTEGER_TEST(token, max) max_test_i(token, max, #token)
-#define MIN_INTEGER_TEST(token, min) min_test_i(token, min, #token)
-#define MIN_FLOAT_TEST(token, min) min_test_f(token, min, #token)
-#define TEST_SIZE_RANGE(token) size_range_test(token, #token)
-
 void
 piglit_init(int argc, char **argv)
 {
-       int rb_size;
-
        piglit_require_gl_version(30);
 
-       printf("%-50s %8s %8s\n", "token", "minimum", "value");
+       piglit_print_minmax_header();
 
        /* These should be in the section with "Minimum Value" but
         * appear in the section with "Initial Value".
         */
-       MIN_INTEGER_TEST(GL_MAX_TRANSFORM_FEEDBACK_INTERLEAVED_COMPONENTS, 64);
-       MIN_INTEGER_TEST(GL_MAX_TRANSFORM_FEEDBACK_SEPARATE_ATTRIBS, 4);
-       MIN_INTEGER_TEST(GL_MAX_TRANSFORM_FEEDBACK_SEPARATE_COMPONENTS, 4);
+       piglit_test_min_int(GL_MAX_TRANSFORM_FEEDBACK_INTERLEAVED_COMPONENTS, 
64);
+       piglit_test_min_int(GL_MAX_TRANSFORM_FEEDBACK_SEPARATE_ATTRIBS, 4);
+       piglit_test_min_int(GL_MAX_TRANSFORM_FEEDBACK_SEPARATE_COMPONENTS, 4);
 
-       MIN_INTEGER_TEST(GL_MAX_LIGHTS, 8);
-       MIN_INTEGER_TEST(GL_MAX_CLIP_PLANES, 6);
+       piglit_test_min_int(GL_MAX_LIGHTS, 8);
+       piglit_test_min_int(GL_MAX_CLIP_PLANES, 6);
        if (piglit_is_extension_supported("GL_ARB_imaging"))
-               MIN_INTEGER_TEST(GL_MAX_COLOR_MATRIX_STACK_DEPTH, 2);
-       MIN_INTEGER_TEST(GL_MAX_MODELVIEW_STACK_DEPTH, 32);
-       MIN_INTEGER_TEST(GL_MAX_PROJECTION_STACK_DEPTH, 2);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_STACK_DEPTH, 2);
-       MIN_INTEGER_TEST(GL_SUBPIXEL_BITS, 4);
-       MIN_INTEGER_TEST(GL_MAX_3D_TEXTURE_SIZE, 256);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_SIZE, 1024);
-       MIN_FLOAT_TEST(GL_MAX_TEXTURE_LOD_BIAS, 2.0);
-       MIN_INTEGER_TEST(GL_MAX_ARRAY_TEXTURE_LAYERS, 256);
-       MIN_INTEGER_TEST(GL_MAX_CUBE_MAP_TEXTURE_SIZE, 1024);
-       MIN_INTEGER_TEST(GL_MAX_RENDERBUFFER_SIZE, 1024);
-       MIN_INTEGER_TEST(GL_MAX_PIXEL_MAP_TABLE, 32);
-       MIN_INTEGER_TEST(GL_MAX_NAME_STACK_DEPTH, 64);
-       MIN_INTEGER_TEST(GL_MAX_LIST_NESTING, 64);
-       MIN_INTEGER_TEST(GL_MAX_EVAL_ORDER, 8);
-
-       /* FINISHME:
-        *
-        *     "The maximum viewport dimensions must be greater than
-        *      or equal to the larger of the visible dimensions of
-        *      the display being rendered to (if a display exists),
-        *      and the largest renderbuffer image which can be
-        *      successfully created and attached to a framebuffer
-        *      object (see chapter 4).  INVALID VALUE is generated if
-        *      either w or h is negative."
-        *
-        * We're only looking at RB limits here.
-        */
-       glGetIntegerv(GL_MAX_RENDERBUFFER_SIZE, &rb_size);
-       MIN_INTEGER_TEST(GL_MAX_VIEWPORT_DIMS, rb_size);
-
-       MIN_INTEGER_TEST(GL_MAX_ATTRIB_STACK_DEPTH, 16);
-       MIN_INTEGER_TEST(GL_MAX_CLIENT_ATTRIB_STACK_DEPTH, 16);
-
-       TEST_SIZE_RANGE(GL_ALIASED_POINT_SIZE_RANGE);
-       TEST_SIZE_RANGE(GL_SMOOTH_POINT_SIZE_RANGE);
-       TEST_SIZE_RANGE(GL_ALIASED_LINE_WIDTH_RANGE);
-       TEST_SIZE_RANGE(GL_SMOOTH_LINE_WIDTH_RANGE);
-
-       test_tf_bits(GL_PRIMITIVES_GENERATED);
-       test_tf_bits(GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN);
-       test_oq_bits();
+               piglit_test_min_int(GL_MAX_COLOR_MATRIX_STACK_DEPTH, 2);
+       piglit_test_min_int(GL_MAX_MODELVIEW_STACK_DEPTH, 32);
+       piglit_test_min_int(GL_MAX_PROJECTION_STACK_DEPTH, 2);
+       piglit_test_min_int(GL_MAX_TEXTURE_STACK_DEPTH, 2);
+       piglit_test_min_int(GL_SUBPIXEL_BITS, 4);
+       piglit_test_min_int(GL_MAX_3D_TEXTURE_SIZE, 256);
+       piglit_test_min_int(GL_MAX_TEXTURE_SIZE, 1024);
+       piglit_test_min_float(GL_MAX_TEXTURE_LOD_BIAS, 2.0);
+       piglit_test_min_int(GL_MAX_ARRAY_TEXTURE_LAYERS, 256);
+       piglit_test_min_int(GL_MAX_CUBE_MAP_TEXTURE_SIZE, 1024);
+       piglit_test_min_int(GL_MAX_RENDERBUFFER_SIZE, 1024);
+       piglit_test_min_int(GL_MAX_PIXEL_MAP_TABLE, 32);
+       piglit_test_min_int(GL_MAX_NAME_STACK_DEPTH, 64);
+       piglit_test_min_int(GL_MAX_LIST_NESTING, 64);
+       piglit_test_min_int(GL_MAX_EVAL_ORDER, 8);
+
+       piglit_test_min_viewport_dimensions();
+
+       piglit_test_min_int(GL_MAX_ATTRIB_STACK_DEPTH, 16);
+       piglit_test_min_int(GL_MAX_CLIENT_ATTRIB_STACK_DEPTH, 16);
+
+       piglit_test_range_float(GL_ALIASED_POINT_SIZE_RANGE, 1, 1);
+       piglit_test_range_float(GL_SMOOTH_POINT_SIZE_RANGE, 1, 1);
+       piglit_test_range_float(GL_ALIASED_LINE_WIDTH_RANGE, 1, 1);
+       piglit_test_range_float(GL_SMOOTH_LINE_WIDTH_RANGE, 1, 1);
+
+       piglit_test_tf_bits(GL_PRIMITIVES_GENERATED);
+       piglit_test_tf_bits(GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN);
+       piglit_test_oq_bits();
 
        if (piglit_is_extension_supported("GL_ARB_imaging")) {
                /* FINISHME: GL_MAX_CONVOLUTION_WIDTH */
                /* FINISHME: GL_MAX_CONVOLUTION_HEIGHT */
        }
 
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_UNITS, 2);
-       MIN_INTEGER_TEST(GL_MAX_VERTEX_ATTRIBS, 16);
-       MIN_INTEGER_TEST(GL_MAX_VERTEX_UNIFORM_COMPONENTS, 1024);
-       MIN_INTEGER_TEST(GL_MAX_VARYING_COMPONENTS, 64);
-       MIN_INTEGER_TEST(GL_MAX_COMBINED_TEXTURE_IMAGE_UNITS, 16);
-       MIN_INTEGER_TEST(GL_MAX_VERTEX_TEXTURE_IMAGE_UNITS, 16);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_IMAGE_UNITS, 16);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_COORDS, 8);
-       MIN_INTEGER_TEST(GL_MAX_FRAGMENT_UNIFORM_COMPONENTS, 1024);
+       piglit_test_min_int(GL_MAX_TEXTURE_UNITS, 2);
+       piglit_test_min_int(GL_MAX_VERTEX_ATTRIBS, 16);
+       piglit_test_min_int(GL_MAX_VERTEX_UNIFORM_COMPONENTS, 1024);
+       piglit_test_min_int(GL_MAX_VARYING_COMPONENTS, 64);
+       piglit_test_min_int(GL_MAX_COMBINED_TEXTURE_IMAGE_UNITS, 16);
+       piglit_test_min_int(GL_MAX_VERTEX_TEXTURE_IMAGE_UNITS, 16);
+       piglit_test_min_int(GL_MAX_TEXTURE_IMAGE_UNITS, 16);
+       piglit_test_min_int(GL_MAX_TEXTURE_COORDS, 8);
+       piglit_test_min_int(GL_MAX_FRAGMENT_UNIFORM_COMPONENTS, 1024);
+
+       piglit_test_max_int(GL_MIN_PROGRAM_TEXEL_OFFSET, -8);
+       piglit_test_min_int(GL_MAX_PROGRAM_TEXEL_OFFSET, 7);
 
-       MAX_INTEGER_TEST(GL_MIN_PROGRAM_TEXEL_OFFSET, -8);
-       MIN_INTEGER_TEST(GL_MAX_PROGRAM_TEXEL_OFFSET, 7);
+       piglit_test_min_int(GL_AUX_BUFFERS, 0);
+       piglit_test_min_int(GL_MAX_DRAW_BUFFERS, 8);
 
-       MIN_INTEGER_TEST(GL_AUX_BUFFERS, 0);
-       MIN_INTEGER_TEST(GL_MAX_DRAW_BUFFERS, 8);
+       piglit_test_min_int(GL_SAMPLE_BUFFERS, 0);
+       piglit_test_min_int(GL_SAMPLES, 0);
+       piglit_test_min_int(GL_MAX_COLOR_ATTACHMENTS, 8);
+       piglit_test_min_int(GL_MAX_SAMPLES, 4);
 
-       MIN_INTEGER_TEST(GL_SAMPLE_BUFFERS, 0);
-       MIN_INTEGER_TEST(GL_SAMPLES, 0);
-       MIN_INTEGER_TEST(GL_MAX_COLOR_ATTACHMENTS, 8);
-       MIN_INTEGER_TEST(GL_MAX_SAMPLES, 4);
+       if (!piglit_check_gl_error(GL_NO_ERROR))
+               piglit_report_result(PIGLIT_FAIL);
 
-       piglit_report_result(pass ? PIGLIT_PASS : PIGLIT_FAIL);
+       piglit_report_result(piglit_minmax_pass ? PIGLIT_PASS : PIGLIT_FAIL);
 }
diff --git a/tests/spec/gl-3.1/minmax.c b/tests/spec/gl-3.1/minmax.c
index 4a83e9d..d59c3cc 100644
--- a/tests/spec/gl-3.1/minmax.c
+++ b/tests/spec/gl-3.1/minmax.c
@@ -27,6 +27,7 @@
  */
 
 #include "piglit-util.h"
+#include "minmax-test.h"
 
 int piglit_width = 32;
 int piglit_height = 32;
@@ -39,234 +40,63 @@ piglit_display(void)
        return PIGLIT_FAIL;
 }
 
-static bool pass = true;
-
-static void
-report_int(const char *name, GLint requirement, GLint val, bool error)
-{
-       if (error) {
-               fprintf(stderr, "%-50s %8d %8d (ERROR)\n",
-                       name, requirement, val);
-               pass = false;
-       } else {
-               printf("%-50s %8d %8d\n", name, requirement, val);
-       }
-}
-
-static void
-max_test_i(GLenum token, GLint max, const char *name)
-{
-       GLint val = 0;
-
-       glGetIntegerv(token, &val);
-
-       report_int(name, val, max, val > max);
-}
-
-static void
-min_test_i(GLenum token, GLint min, const char *name)
-{
-       GLint val = 0;
-
-       glGetIntegerv(token, &val);
-
-       report_int(name, val, min, val < min);
-}
-
-static void
-min_test_f(GLenum token, GLfloat min, const char *name)
-{
-       GLfloat val = 0.0;
-
-       glGetFloatv(token, &val);
-
-       if (val < min) {
-               fprintf(stderr, "%-50s %8f %8f (ERROR)\n",
-                       name, min, val);
-               pass = false;
-       } else {
-               printf("%-50s %8f %8f\n", name, min, val);
-       }
-}
-
-/* All the size requirements happen to only require a range covering
- * [1.0, 1.0].
- */
-static void
-size_range_test(GLenum token, const char *name)
-{
-       GLfloat vals[2];
-
-       glGetFloatv(token, vals);
-
-       if (vals[0] > 1.0 || vals[1] < 1.0) {
-               fprintf(stderr, "%-50s %8s  %.1f-%.1f (ERROR)\n",
-                       name, "1-1", vals[0], vals[1]);
-               pass = false;
-       } else {
-               printf("%-50s %8s  %.1f-%.1f\n",
-                      name, "1-1", vals[0], vals[1]);
-       }
-}
-
-static void
-test_oq_bits()
-{
-       GLint dims[2];
-       GLint minbits, oqbits;
-
-       /* From the GL 3.0 specification, page 329:
-        *
-        *     "If pname is QUERY_COUNTER_BITS, the
-        *      implementation-dependent number of query counter bits
-        *      may be zero, in which case the counter contains no
-        *      useful information.
-        *
-        *      For occlusion queries (SAMPLES PASSED), if the number
-        *      of bits is non-zero, the minimum number of bits
-        *      allowed is a function of the implementation’s maximum
-        *      viewport dimensions (MAX VIEWPORT DIMS). The counter
-        *      must be able to represent at least two overdraws for
-        *      every pixel in the viewport. The formula to compute
-        *      the allowable minimum value (where n is the minimum
-        *      number of bits) is
-        *
-        *      n = min{32, log2(maxViewportWidth ∗ maxViewportHeight * 2}"
-        */
-
-       glGetIntegerv(GL_MAX_VIEWPORT_DIMS, dims);
-       minbits = log2((float)dims[0] * dims[1] * 2);
-       if (minbits > 32)
-               minbits = 32;
-
-       glGetQueryiv(GL_SAMPLES_PASSED, GL_QUERY_COUNTER_BITS, &oqbits);
-       if (oqbits == 0 || oqbits >= minbits) {
-               printf("%-50s   0 / %2d %8d\n",
-                      "GL_QUERY_COUNTER_BITS(GL_SAMPLES_PASSED)",
-                      minbits, oqbits);
-       } else {
-               fprintf(stderr,
-                       "%-50s   0 / %2d %8d\n",
-                       "GL_QUERY_COUNTER_BITS(GL_SAMPLES_PASSED)",
-                       minbits, oqbits);
-               pass = false;
-       }
-}
-
-static void
-test_tf_bits(GLenum target)
-{
-       GLint bits = -1;
-       const char *name;
-
-       if (target == GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN)
-               name = "GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN bits";
-       else
-               name = "GL_PRIMITIVES_GENERATED bits";
-
-       /* From the GL 3.0 specification, page 329:
-        *
-        *     "If pname is QUERY_COUNTER_BITS, the
-        *      implementation-dependent number of query counter bits
-        *      may be zero, in which case the counter contains no
-        *      useful information.
-        *
-        *      For primitive queries (PRIMITIVES GENERATED and
-        *      TRANSFORM FEEDBACK PRIMITIVES WRITTEN) if the number
-        *      of bits is non-zero, the minimum number of bits
-        *      allowed is 32."
-        */
-
-       glGetQueryiv(target, GL_QUERY_COUNTER_BITS, &bits);
-       if (bits == 0 || bits >= 32) {
-               printf("%-50s %8s %8d\n", name, "0 / 32", bits);
-       } else {
-               fprintf(stderr, "%-50s %8s %8d (ERROR)\n",
-                       name, "0 / 32", bits);
-               pass = false;
-       }
-}
-
-#define MAX_INTEGER_TEST(token, max) max_test_i(token, max, #token)
-#define MIN_INTEGER_TEST(token, min) min_test_i(token, min, #token)
-#define MIN_FLOAT_TEST(token, min) min_test_f(token, min, #token)
-#define TEST_SIZE_RANGE(token) size_range_test(token, #token)
-
 void
 piglit_init(int argc, char **argv)
 {
-       int rb_size, dims[2] = { 0 };
        int vuniforms = 0, vblocks = 0;
        int funiforms = 0, fblocks = 0;
        int blocksize = 0;
 
        piglit_require_gl_version(31);
 
-       printf("%-50s %8s %8s\n", "token", "minimum", "value");
+       piglit_print_minmax_header();
 
        /* These should be in the section with "Minimum Value" but
         * appear in the section with "Initial Value".
         */
-       MIN_INTEGER_TEST(GL_MAX_TRANSFORM_FEEDBACK_INTERLEAVED_COMPONENTS, 64);
-       MIN_INTEGER_TEST(GL_MAX_TRANSFORM_FEEDBACK_SEPARATE_ATTRIBS, 4);
-       MIN_INTEGER_TEST(GL_MAX_TRANSFORM_FEEDBACK_SEPARATE_COMPONENTS, 4);
-
-       MIN_INTEGER_TEST(GL_MAX_CLIP_DISTANCES, 8);
-       MIN_INTEGER_TEST(GL_SUBPIXEL_BITS, 4);
-       MIN_INTEGER_TEST(GL_MAX_3D_TEXTURE_SIZE, 256);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_SIZE, 1024);
-       MIN_INTEGER_TEST(GL_MAX_ARRAY_TEXTURE_LAYERS, 256);
-       MIN_FLOAT_TEST(GL_MAX_TEXTURE_LOD_BIAS, 2.0);
-       MIN_INTEGER_TEST(GL_MAX_CUBE_MAP_TEXTURE_SIZE, 1024);
-       MIN_INTEGER_TEST(GL_MAX_RENDERBUFFER_SIZE, 1024);
-
-       /* FINISHME:
-        *
-        *     "The maximum viewport dimensions must be greater than
-        *      or equal to the larger of the visible dimensions of
-        *      the display being rendered to (if a display exists),
-        *      and the largest renderbuffer image which can be
-        *      successfully created and attached to a framebuffer
-        *      object (see chapter 4).  INVALID VALUE is generated if
-        *      either w or h is negative."
-        *
-        * We're only looking at RB limits here.
-        */
-       glGetIntegerv(GL_MAX_RENDERBUFFER_SIZE, &rb_size);
-       glGetIntegerv(GL_MAX_VIEWPORT_DIMS, dims);
-       report_int("GL_MAX_VIEWPORT_DIMS[0]",
-                  rb_size, dims[0], dims[0] < rb_size);
-       report_int("GL_MAX_VIEWPORT_DIMS[1]",
-                  rb_size, dims[1], dims[1] < rb_size);
-
-       TEST_SIZE_RANGE(GL_POINT_SIZE_RANGE);
-       TEST_SIZE_RANGE(GL_ALIASED_LINE_WIDTH_RANGE);
-       TEST_SIZE_RANGE(GL_SMOOTH_LINE_WIDTH_RANGE);
-       MIN_INTEGER_TEST(GL_NUM_COMPRESSED_TEXTURE_FORMATS, 4);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_BUFFER_SIZE, 65536);
-       MIN_INTEGER_TEST(GL_MAX_RECTANGLE_TEXTURE_SIZE, 1024);
-
-       test_tf_bits(GL_PRIMITIVES_GENERATED);
-       test_tf_bits(GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN);
-       test_oq_bits();
-
-       MIN_INTEGER_TEST(GL_MAX_VERTEX_ATTRIBS, 16);
-       MIN_INTEGER_TEST(GL_MAX_VERTEX_UNIFORM_COMPONENTS, 1024);
-       MIN_INTEGER_TEST(GL_MAX_VARYING_COMPONENTS, 64);
-       MIN_INTEGER_TEST(GL_MAX_COMBINED_TEXTURE_IMAGE_UNITS, 32);
-       MIN_INTEGER_TEST(GL_MAX_VERTEX_TEXTURE_IMAGE_UNITS, 16);
-       MIN_INTEGER_TEST(GL_MAX_TEXTURE_IMAGE_UNITS, 16);
-       MIN_INTEGER_TEST(GL_MAX_FRAGMENT_UNIFORM_COMPONENTS, 1024);
-
-       MAX_INTEGER_TEST(GL_MIN_PROGRAM_TEXEL_OFFSET, -8);
-       MIN_INTEGER_TEST(GL_MAX_PROGRAM_TEXEL_OFFSET, 7);
-
-       MIN_INTEGER_TEST(GL_MAX_VERTEX_UNIFORM_BLOCKS, 12);
-       MIN_INTEGER_TEST(GL_MAX_FRAGMENT_UNIFORM_BLOCKS, 12);
-       MIN_INTEGER_TEST(GL_MAX_COMBINED_UNIFORM_BLOCKS, 24);
-       MIN_INTEGER_TEST(GL_MAX_UNIFORM_BUFFER_BINDINGS, 24);
-       MIN_INTEGER_TEST(GL_MAX_UNIFORM_BLOCK_SIZE, 16384);
-       MIN_INTEGER_TEST(GL_UNIFORM_BUFFER_OFFSET_ALIGNMENT, 1);
+       piglit_test_min_int(GL_MAX_TRANSFORM_FEEDBACK_INTERLEAVED_COMPONENTS, 
64);
+       piglit_test_min_int(GL_MAX_TRANSFORM_FEEDBACK_SEPARATE_ATTRIBS, 4);
+       piglit_test_min_int(GL_MAX_TRANSFORM_FEEDBACK_SEPARATE_COMPONENTS, 4);
+
+       piglit_test_min_int(GL_MAX_CLIP_DISTANCES, 8);
+       piglit_test_min_int(GL_SUBPIXEL_BITS, 4);
+       piglit_test_min_int(GL_MAX_3D_TEXTURE_SIZE, 256);
+       piglit_test_min_int(GL_MAX_TEXTURE_SIZE, 1024);
+       piglit_test_min_int(GL_MAX_ARRAY_TEXTURE_LAYERS, 256);
+       piglit_test_min_float(GL_MAX_TEXTURE_LOD_BIAS, 2.0);
+       piglit_test_min_int(GL_MAX_CUBE_MAP_TEXTURE_SIZE, 1024);
+       piglit_test_min_int(GL_MAX_RENDERBUFFER_SIZE, 1024);
+
+       piglit_test_min_viewport_dimensions();
+
+       piglit_test_range_float(GL_POINT_SIZE_RANGE, 1, 1);
+       piglit_test_range_float(GL_ALIASED_LINE_WIDTH_RANGE, 1, 1);
+       piglit_test_range_float(GL_SMOOTH_LINE_WIDTH_RANGE, 1, 1);
+       piglit_test_min_int(GL_NUM_COMPRESSED_TEXTURE_FORMATS, 4);
+       piglit_test_min_int(GL_MAX_TEXTURE_BUFFER_SIZE, 65536);
+       piglit_test_min_int(GL_MAX_RECTANGLE_TEXTURE_SIZE, 1024);
+
+       piglit_test_tf_bits(GL_PRIMITIVES_GENERATED);
+       piglit_test_tf_bits(GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN);
+       piglit_test_oq_bits();
+
+       piglit_test_min_int(GL_MAX_VERTEX_ATTRIBS, 16);
+       piglit_test_min_int(GL_MAX_VERTEX_UNIFORM_COMPONENTS, 1024);
+       piglit_test_min_int(GL_MAX_VARYING_COMPONENTS, 64);
+       piglit_test_min_int(GL_MAX_COMBINED_TEXTURE_IMAGE_UNITS, 32);
+       piglit_test_min_int(GL_MAX_VERTEX_TEXTURE_IMAGE_UNITS, 16);
+       piglit_test_min_int(GL_MAX_TEXTURE_IMAGE_UNITS, 16);
+       piglit_test_min_int(GL_MAX_FRAGMENT_UNIFORM_COMPONENTS, 1024);
+
+       piglit_test_max_int(GL_MIN_PROGRAM_TEXEL_OFFSET, -8);
+       piglit_test_min_int(GL_MAX_PROGRAM_TEXEL_OFFSET, 7);
+
+       piglit_test_min_int(GL_MAX_VERTEX_UNIFORM_BLOCKS, 12);
+       piglit_test_min_int(GL_MAX_FRAGMENT_UNIFORM_BLOCKS, 12);
+       piglit_test_min_int(GL_MAX_COMBINED_UNIFORM_BLOCKS, 24);
+       piglit_test_min_int(GL_MAX_UNIFORM_BUFFER_BINDINGS, 24);
+       piglit_test_min_int(GL_MAX_UNIFORM_BLOCK_SIZE, 16384);
+       piglit_test_min_int(GL_UNIFORM_BUFFER_OFFSET_ALIGNMENT, 1);
 
        glGetIntegerv(GL_MAX_VERTEX_UNIFORM_BLOCKS, &vblocks);
        glGetIntegerv(GL_MAX_VERTEX_UNIFORM_COMPONENTS, &vuniforms);
@@ -278,8 +108,8 @@ piglit_init(int argc, char **argv)
         * above, with realistic minimum values.  This appears to be a
         * typo and was dropped in 3.2.
         */
-       /* MIN_INTEGER_TEST(GL_MAX_VERTEX_UNIFORM_COMPONENTS, 1); */
-       /* MIN_INTEGER_TEST(GL_MAX_FRAGMENT_UNIFORM_COMPONENTS, 1); */
+       /* piglit_test_min_int(GL_MAX_VERTEX_UNIFORM_COMPONENTS, 1); */
+       /* piglit_test_min_int(GL_MAX_FRAGMENT_UNIFORM_COMPONENTS, 1); */
 
        /* In this case, the "1" in the table refers to the footnote:
         *
@@ -288,21 +118,21 @@ piglit_init(int argc, char **argv)
         *      MAX_stage_UNIFORM_BLOCK_SIZE +
         *      MAX_stage_UNIFORM_COMPONENTS"
         */
-       MIN_INTEGER_TEST(GL_MAX_COMBINED_VERTEX_UNIFORM_COMPONENTS,
+       piglit_test_min_int(GL_MAX_COMBINED_VERTEX_UNIFORM_COMPONENTS,
                         vblocks * blocksize + vuniforms);
-       MIN_INTEGER_TEST(GL_MAX_COMBINED_FRAGMENT_UNIFORM_COMPONENTS,
+       piglit_test_min_int(GL_MAX_COMBINED_FRAGMENT_UNIFORM_COMPONENTS,
                         fblocks * blocksize + funiforms);
 
-       MIN_INTEGER_TEST(GL_MAX_DRAW_BUFFERS, 8);
+       piglit_test_min_int(GL_MAX_DRAW_BUFFERS, 8);
 
-       MIN_INTEGER_TEST(GL_SAMPLE_BUFFERS, 0);
-       MIN_INTEGER_TEST(GL_SAMPLES, 0);
+       piglit_test_min_int(GL_SAMPLE_BUFFERS, 0);
+       piglit_test_min_int(GL_SAMPLES, 0);
 
-       MIN_INTEGER_TEST(GL_MAX_COLOR_ATTACHMENTS, 8);
-       MIN_INTEGER_TEST(GL_MAX_SAMPLES, 4);
+       piglit_test_min_int(GL_MAX_COLOR_ATTACHMENTS, 8);
+       piglit_test_min_int(GL_MAX_SAMPLES, 4);
 
        if (!piglit_check_gl_error(GL_NO_ERROR))
                piglit_report_result(PIGLIT_FAIL);
 
-       piglit_report_result(pass ? PIGLIT_PASS : PIGLIT_FAIL);
+       piglit_report_result(piglit_minmax_pass ? PIGLIT_PASS : PIGLIT_FAIL);
 }
diff --git a/tests/util/CMakeLists.gl.txt b/tests/util/CMakeLists.gl.txt
index 7790e4f..3d08b97 100644
--- a/tests/util/CMakeLists.gl.txt
+++ b/tests/util/CMakeLists.gl.txt
@@ -14,6 +14,7 @@ set(UTIL_SOURCES
        piglit-util-gl.c
        piglit-vbo.cpp
        sized-internalformats.c
+       minmax-test.c
        )
 
 IF(BUILD_GLX_TESTS)
diff --git a/tests/util/minmax-test.c b/tests/util/minmax-test.c
new file mode 100644
index 0000000..4fa2d98
--- /dev/null
+++ b/tests/util/minmax-test.c
@@ -0,0 +1,249 @@
+/* Copyright © 2011-2012 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ */
+
+/** @file minmax-test.c
+ *
+ * Helpers for performing minimuTest for the minimum maximum values in section 
6.2 "State Tables"
+ * of the GL 3.0 spec.
+ */
+
+#define _GNU_SOURCE
+#include <string.h>
+
+#include "piglit-util.h"
+#include "minmax-test.h"
+
+bool piglit_minmax_pass = true;
+
+void piglit_print_minmax_header(void)
+{
+       printf("%-50s %8s %8s\n", "token", "minimum", "value");
+}
+
+static void
+piglit_report_int(const char *name, GLint limit, GLint val, bool pass)
+{
+       if (pass) {
+               printf("%-50s %8d %8d\n", name, limit, val);
+       } else {
+               fprintf(stderr, "%-50s %8d %8d (ERROR)\n",
+                       name, limit, val);
+               piglit_minmax_pass = false;
+       }
+}
+
+static void
+piglit_report_float(const char *name, GLfloat limit, GLfloat val, bool pass)
+{
+       if (pass) {
+               printf("%-50s %8.1f %8.1f\n", name, limit, val);
+       } else {
+               fprintf(stderr, "%-50s %8f %8f (ERROR)\n",
+                       name, limit, val);
+               piglit_minmax_pass = false;
+       }
+}
+
+static void
+piglit_test_int(GLenum token, GLint limit, bool max)
+{
+       const char *name = piglit_get_gl_enum_name(token);
+       GLint val = 9999;
+
+       glGetIntegerv(token, &val);
+
+       piglit_report_int(name, limit, val,
+                         (max && val <= limit) ||
+                         (!max && val >= limit));
+}
+
+void piglit_test_min_int(GLenum token, GLint min)
+{
+       piglit_test_int(token, min, false);
+}
+
+void piglit_test_max_int(GLenum token, GLint max)
+{
+       piglit_test_int(token, max, true);
+}
+
+
+static void
+piglit_test_float(GLenum token, GLfloat limit, bool max)
+{
+       const char *name = piglit_get_gl_enum_name(token);
+       GLfloat val = -9999;
+
+       glGetFloatv(token, &val);
+
+       piglit_report_float(name, limit, val,
+                           (max && val <= limit) ||
+                           (!max && val >= limit));
+}
+
+void piglit_test_min_float(GLenum token, GLfloat min)
+{
+       piglit_test_float(token, min, false);
+}
+
+void piglit_test_max_float(GLenum token, GLfloat max)
+{
+       piglit_test_float(token, max, true);
+}
+
+/** Tests tha the range referneced by the token covers at least low-high. */
+void piglit_test_range_float(GLenum token, GLfloat low, GLfloat high)
+{
+       const char *name = piglit_get_gl_enum_name(token);
+       char *temp;
+       GLfloat vals[2] = {9999, 9999};
+
+       glGetFloatv(token, vals);
+
+       asprintf(&temp, "%s[0]", name);
+       piglit_report_float(temp, low, vals[0], vals[0] <= low);
+       free(temp);
+
+       asprintf(&temp, "%s[1]", name);
+       piglit_report_float(temp, high, vals[1], vals[1] >= high);
+       free(temp);
+}
+
+void piglit_test_min_viewport_dimensions(void)
+{
+       int min_w, min_h;
+       GLint dims[2] = {9999, 9999};
+
+       if (piglit_get_gl_version() < 30) {
+               /* FINISHME:
+                *
+                *     "The maximum viewport dimensions must be
+                *      greater than or equal to the larger of the
+                *      visible dimensions of the display being
+                *      rendered to (if a display exists), and the
+                *      largest renderbuffer image which can be
+                *      successfully created and attached to a
+                *      framebuffer object (see chapter 4).  INVALID
+                *      VALUE is generated if either w or h is
+                *      negative."
+                *
+                * We're only looking at RB limits here.
+                */
+               int rb_size = 9999;
+               glGetIntegerv(GL_MAX_RENDERBUFFER_SIZE, &rb_size);
+
+               min_w = rb_size;
+               min_h = rb_size;
+       } else {
+               /* FINISHME:
+                *
+                *     "The maximum viewport dimensions must be
+                *      greater than or equal to the visible
+                *      dimensions of the display being rendered to."
+                *
+                * Surely the screen is at least 1024x768, right?
+                */
+               min_w = 1024;
+               min_h = 768;
+       }
+
+       glGetIntegerv(GL_MAX_VIEWPORT_DIMS, dims);
+
+       piglit_report_int("GL_MAX_VIEWPORT_DIMS[0]", min_w, dims[0],
+                         dims[0] >= min_w);
+       piglit_report_int("GL_MAX_VIEWPORT_DIMS[1]", min_h, dims[1],
+                         dims[1] >= min_h);
+}
+
+void
+piglit_test_oq_bits()
+{
+       GLint dims[2] = {9999, 9999};
+       GLint minbits, oqbits = 9999;
+
+       /* From the GL 2.1 specification:
+        *
+        *     "The number of query counter bits may be zero, in which
+        *      case the counter contains no useful
+        *      information. Otherwise, the minimum number of bits
+        *      allowed is a function of the implementation’s maximum
+        *      viewport dimensions (MAX_VIEWPORT_DIMS). In this case,
+        *      the counter must be able to represent at least two
+        *      overdraws for every pixel in the viewport The formula
+        *      to compute the allowable minimum value (where n is the
+        *      minimum number of bits) is:
+        *
+        *      n = min{32, log2(maxViewportWidth ∗ maxViewportHeight * 2}"
+        */
+
+       glGetIntegerv(GL_MAX_VIEWPORT_DIMS, dims);
+       minbits = log2((float)dims[0] * dims[1] * 2);
+       if (minbits > 32)
+               minbits = 32;
+
+       glGetQueryiv(GL_SAMPLES_PASSED, GL_QUERY_COUNTER_BITS, &oqbits);
+       if (oqbits == 0 || oqbits >= minbits) {
+               printf("%-50s   0 / %2d %8d\n",
+                      "GL_QUERY_COUNTER_BITS(GL_SAMPLES_PASSED)",
+                      minbits, oqbits);
+       } else {
+               fprintf(stderr,
+                       "%-50s   0 / %2d %8d\n",
+                       "GL_QUERY_COUNTER_BITS(GL_SAMPLES_PASSED)",
+                       minbits, oqbits);
+               piglit_minmax_pass = false;
+       }
+}
+
+void
+piglit_test_tf_bits(GLenum target)
+{
+       GLint bits = 9999;
+       const char *name;
+
+       if (target == GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN)
+               name = "GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN bits";
+       else
+               name = "GL_PRIMITIVES_GENERATED bits";
+
+       /* From the GL 3.0 specification, page 329:
+        *
+        *     "If pname is QUERY_COUNTER_BITS, the
+        *      implementation-dependent number of query counter bits
+        *      may be zero, in which case the counter contains no
+        *      useful information.
+        *
+        *      For primitive queries (PRIMITIVES GENERATED and
+        *      TRANSFORM FEEDBACK PRIMITIVES WRITTEN) if the number
+        *      of bits is non-zero, the minimum number of bits
+        *      allowed is 32."
+        */
+
+       glGetQueryiv(target, GL_QUERY_COUNTER_BITS, &bits);
+       if (bits == 0 || bits >= 32) {
+               printf("%-50s %8s %8d\n", name, "0 / 32", bits);
+       } else {
+               fprintf(stderr, "%-50s %8s %8d (ERROR)\n",
+                       name, "0 / 32", bits);
+               piglit_minmax_pass = false;
+       }
+}
diff --git a/tests/util/minmax-test.h b/tests/util/minmax-test.h
new file mode 100644
index 0000000..698b9e6
--- /dev/null
+++ b/tests/util/minmax-test.h
@@ -0,0 +1,33 @@
+/* Copyright © 2012 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ */
+
+extern bool piglit_minmax_pass;
+
+void piglit_print_minmax_header(void);
+void piglit_test_min_int(GLenum token, GLint val);
+void piglit_test_max_int(GLenum token, GLint val);
+void piglit_test_min_float(GLenum token, GLfloat val);
+void piglit_test_max_float(GLenum token, GLfloat val);
+void piglit_test_range_float(GLenum token, GLfloat low, GLfloat high);
+void piglit_test_min_viewport_dimensions();
+void piglit_test_oq_bits(void);
+void piglit_test_tf_bits(GLenum target);
-- 
1.7.10

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to