[Piglit] [PATCH 3/3] msaa: Test proper functioning when no color buffer is present.

Jordan Justen jljusten at gmail.com
Fri Jul 13 17:14:30 PDT 2012


1-3: Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>

One question is below...

On Thu, Jul 12, 2012 at 5:53 PM, Paul Berry <stereotype441 at gmail.com> wrote:
> This test reproduces a problem in Mesa's i965 driver as of 7/12/12
> (commit 1a06e84): when no color buffer is present, but a multisampled
> depth or stencil buffer is present, Mesa fails to set up the rendering
> pipeline for multisampling.
>
> The test includes variants for testing both the stencil buffer and the
> depth buffer.  The depth buffer test may be run in a "computed" mode,
> in which case the depth values are output by the fragment shader (this
> is an important corner case for i965).
>
> Verified on nVidia's proprietary binary driver for Linux.
> ---
>  tests/all.tests                                    |    9 +
>  .../ext_framebuffer_multisample/CMakeLists.gl.txt  |    1 +
>  .../spec/ext_framebuffer_multisample/no-color.cpp  |  232 ++++++++++++++++++++
>  3 files changed, 242 insertions(+), 0 deletions(-)
>  create mode 100644 tests/spec/ext_framebuffer_multisample/no-color.cpp
>
> diff --git a/tests/all.tests b/tests/all.tests
> index b0c9355..46f6a9b 100644
> --- a/tests/all.tests
> +++ b/tests/all.tests
> @@ -1473,6 +1473,15 @@ for num_samples in MSAA_SAMPLE_COUNTS:
>                  ext_framebuffer_multisample[test_name] = PlainExecTest(
>                          executable)
>
> +for num_samples in MSAA_SAMPLE_COUNTS:
> +        for test_type in ('depth', 'depth-computed', 'stencil'):
> +                for buffer_config in ('combined', 'separate', 'single'):
> +                        test_name = ' '.join(['no-color', str(num_samples),
> +                                              test_type, buffer_config])
> +                        executable = 'ext_framebuffer_multisample-{0} -auto'.format(
> +                                test_name)
> +                        ext_framebuffer_multisample[test_name] = PlainExecTest(executable)

At first I tried to run:
./piglit-run.py -t ext_framebuffer_multisample-no-color

But, "ext_framebuffer_multisample-" is not part of the name. Should it be?

-Jordan

> +
>  ext_framebuffer_object = Group()
>  spec['EXT_framebuffer_object'] = ext_framebuffer_object
>  add_fbo_stencil_tests(ext_framebuffer_object, 'GL_STENCIL_INDEX1')
> diff --git a/tests/spec/ext_framebuffer_multisample/CMakeLists.gl.txt b/tests/spec/ext_framebuffer_multisample/CMakeLists.gl.txt
> index b2b7b51..027e4c5 100644
> --- a/tests/spec/ext_framebuffer_multisample/CMakeLists.gl.txt
> +++ b/tests/spec/ext_framebuffer_multisample/CMakeLists.gl.txt
> @@ -28,6 +28,7 @@ piglit_add_executable (ext_framebuffer_multisample-negative-copyteximage negativ
>  piglit_add_executable (ext_framebuffer_multisample-negative-max-samples negative-max-samples.c)
>  piglit_add_executable (ext_framebuffer_multisample-negative-mismatched-samples negative-mismatched-samples.c)
>  piglit_add_executable (ext_framebuffer_multisample-negative-readpixels negative-readpixels.c)
> +piglit_add_executable (ext_framebuffer_multisample-no-color no-color.cpp common.cpp)
>  piglit_add_executable (ext_framebuffer_multisample-point-smooth common.cpp point-smooth.cpp)
>  piglit_add_executable (ext_framebuffer_multisample-polygon-smooth common.cpp polygon-smooth.cpp)
>  piglit_add_executable (ext_framebuffer_multisample-polygon-stipple common.cpp polygon-stipple.cpp)
> diff --git a/tests/spec/ext_framebuffer_multisample/no-color.cpp b/tests/spec/ext_framebuffer_multisample/no-color.cpp
> new file mode 100644
> index 0000000..b7ded7c
> --- /dev/null
> +++ b/tests/spec/ext_framebuffer_multisample/no-color.cpp
> @@ -0,0 +1,232 @@
> +/*
> + * 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.
> + */
> +
> +/**
> + * \file no-color.cpp
> + *
> + * This test verifies proper operation of multisampled FBOs that lack
> + * a color buffer.  It operates by performing identical draw
> + * operations in an FBO that lacks a color buffer and an FBO that has
> + * a color buffer, and verifying that the resulting output is the
> + * same.
> + *
> + * The test can run in any of the following modes:
> + *
> + * - depth: test depth buffer behaviour, using a fragment shader that
> + *   does not compute depth.
> + *
> + * - depth-computed: test depth buffer behaviour, using a fragment
> + *   shader that *does* compute depth.
> + *
> + * - stencil: test stencil buffer beahviour.
> + *
> + * It can also be configured to use either a combined depth/stencil
> + * buffer, separate depth/stencil buffers, or just a single
> + * depth/stencil buffer depending on the type of test.
> + *
> + * The test operates by performing the following steps:
> + *
> + * 1. Draw a test pattern to a multisampled FBO that lacks a color
> + *    buffer (let's call this the "test FBO").
> + *
> + * 2. Blit the test pattern to a multisampled FBO that has a full
> + *    complement of color, depth, and stencil buffers (let's call this
> + *    the "manifest FBO").
> + *
> + * 3. Do a "manifest" operation to cause colors to be drawn that are
> + *    dependent upon the contents of the depth or stencil buffer.
> + *
> + * 4. Blit the color buffer from the manifest FBO to the screen.  This
> + *    is the test image, and is shown in the left half of the piglit
> + *    window.
> + *
> + * 5. Draw the test pattern again, but this time draw it directly to
> + *    the manifest FBO.
> + *
> + * 6. Do a "manifest" operation again.
> + *
> + * 7. Blit the color buffer to the screen.  This is the reference
> + *    image, and is shown in the right half of the piglit window.
> + *
> + * 8. Compare the test and reference images to make sure they match.
> + */
> +
> +#include "common.h"
> +
> +PIGLIT_GL_TEST_MAIN(
> +    512 /* window_width */,
> +    256 /* window_height */,
> +    GLUT_DOUBLE | GLUT_RGBA | GLUT_ALPHA)
> +
> +namespace {
> +
> +const int pattern_width = 256, pattern_height = 256;
> +
> +GLenum buffer_to_test;
> +Fbo test_fbo;
> +Fbo manifest_fbo;
> +ManifestProgram *manifest_program;
> +TestPattern *test_pattern;
> +
> +void
> +print_usage_and_exit(char *prog_name)
> +{
> +       printf("Usage: %s <num_samples> <test_type> <buffer_config?\n"
> +              "  where <test_type> is one of:\n"
> +              "    depth: test fixed pipeline depth\n"
> +              "    depth-computed: test depth value computed by a shader\n"
> +              "    stencil: test stencil\n"
> +              "  and <buffer_config> is one of:\n"
> +              "    combined: use a single combined depth/stencil buffer\n"
> +              "    separate: use separate depth and stencil buffers\n"
> +              "    single: use just a single buffer (depth or stencil)\n",
> +              prog_name);
> +       piglit_report_result(PIGLIT_FAIL);
> +}
> +
> +extern "C" void
> +piglit_init(int argc, char **argv)
> +{
> +       if (argc != 4)
> +               print_usage_and_exit(argv[0]);
> +
> +       /* 1st arg: num_samples */
> +       char *endptr = NULL;
> +       int num_samples = strtol(argv[1], &endptr, 0);
> +       if (endptr != argv[1] + strlen(argv[1]))
> +               print_usage_and_exit(argv[0]);
> +
> +       /* 2nd arg: test_type */
> +       if (strcmp(argv[2], "depth") == 0) {
> +               buffer_to_test = GL_DEPTH_BUFFER_BIT;
> +               manifest_program = new ManifestDepth;
> +               test_pattern = new DepthSunburst(false /* compute_depth */);
> +       } else if (strcmp(argv[2], "depth-computed") == 0) {
> +               buffer_to_test = GL_DEPTH_BUFFER_BIT;
> +               manifest_program = new ManifestDepth;
> +               test_pattern = new DepthSunburst(true /* compute_depth */);
> +       } else if (strcmp(argv[2], "stencil") == 0) {
> +               buffer_to_test = GL_STENCIL_BUFFER_BIT;
> +               manifest_program = new ManifestStencil;
> +               test_pattern = new StencilSunburst;
> +       } else {
> +               print_usage_and_exit(argv[0]);
> +       }
> +
> +       /* 3rd arg: buffer_config */
> +       FboConfig test_fbo_config(num_samples, pattern_width, pattern_height);
> +       test_fbo_config.color_internalformat = GL_NONE;
> +       if (strcmp(argv[3], "combined") == 0) {
> +               test_fbo_config.combine_depth_stencil = true;
> +       } else if (strcmp(argv[3], "separate") == 0) {
> +               test_fbo_config.combine_depth_stencil = false;
> +       } else if (strcmp(argv[3], "single") == 0) {
> +               test_fbo_config.combine_depth_stencil = false;
> +               if (buffer_to_test == GL_DEPTH_BUFFER_BIT)
> +                       test_fbo_config.stencil_internalformat = GL_NONE;
> +               else
> +                       test_fbo_config.depth_internalformat = GL_NONE;
> +       } else {
> +               print_usage_and_exit(argv[0]);
> +       }
> +
> +       piglit_require_gl_version(30);
> +       piglit_require_GLSL_version(130);
> +
> +       /* Skip the test if num_samples > GL_MAX_SAMPLES */
> +       GLint max_samples;
> +       glGetIntegerv(GL_MAX_SAMPLES, &max_samples);
> +       if (num_samples > max_samples)
> +               piglit_report_result(PIGLIT_SKIP);
> +
> +       test_fbo.setup(test_fbo_config);
> +       manifest_fbo.setup(FboConfig(num_samples, pattern_width,
> +                                    pattern_height));
> +       manifest_program->compile();
> +       test_pattern->compile();
> +
> +       if (!piglit_check_gl_error(GL_NO_ERROR))
> +               piglit_report_result(PIGLIT_FAIL);
> +}
> +
> +extern "C" enum piglit_result
> +piglit_display()
> +{
> +       bool pass = true;
> +
> +       /* Draw the test pattern into test_fbo. */
> +       glBindFramebuffer(GL_DRAW_FRAMEBUFFER, test_fbo.handle);
> +       test_fbo.set_viewport();
> +       test_pattern->draw(TestPattern::no_projection);
> +
> +       /* Blit the test pattern to manifest_fbo. */
> +       glBindFramebuffer(GL_READ_FRAMEBUFFER, test_fbo.handle);
> +       glBindFramebuffer(GL_DRAW_FRAMEBUFFER, manifest_fbo.handle);
> +       glBlitFramebuffer(0, 0, pattern_width, pattern_height,
> +                         0, 0, pattern_width, pattern_height,
> +                         buffer_to_test, GL_NEAREST);
> +
> +       /* Manifest the pattern so that it is reflected in color
> +        * values in manifest_fbo.
> +        */
> +       manifest_program->run();
> +
> +       /* Blit the color buffer from manifest_fbo to the screen.
> +        * This is the test image.
> +        */
> +       glBindFramebuffer(GL_READ_FRAMEBUFFER, manifest_fbo.handle);
> +       glBindFramebuffer(GL_DRAW_FRAMEBUFFER, 0);
> +       glBlitFramebuffer(0, 0, pattern_width, pattern_height,
> +                         0, 0, pattern_width, pattern_height,
> +                         GL_COLOR_BUFFER_BIT, GL_NEAREST);
> +
> +       /* Draw the test pattern into manifest_fbo. */
> +       glBindFramebuffer(GL_DRAW_FRAMEBUFFER, manifest_fbo.handle);
> +       manifest_fbo.set_viewport();
> +       test_pattern->draw(TestPattern::no_projection);
> +
> +       /* Manifest the pattern so that it is reflected in color
> +        * values in manifest_fbo.
> +        */
> +       manifest_program->run();
> +
> +       /* Blit the color buffer from manifest_fbo to the screen.
> +        * This is the reference image.
> +        */
> +       glBindFramebuffer(GL_READ_FRAMEBUFFER, manifest_fbo.handle);
> +       glBindFramebuffer(GL_DRAW_FRAMEBUFFER, 0);
> +       glBlitFramebuffer(0, 0, pattern_width, pattern_height,
> +                         pattern_width, 0, 2*pattern_width, pattern_height,
> +                         GL_COLOR_BUFFER_BIT, GL_NEAREST);
> +
> +       /* Compare the test and reference images */
> +       glBindFramebuffer(GL_READ_FRAMEBUFFER, 0);
> +       pass = piglit_probe_rect_halves_equal_rgba(0, 0, 2*pattern_width,
> +                                                  pattern_height) && pass;
> +
> +       piglit_present_results();
> +
> +       return pass ? PIGLIT_PASS : PIGLIT_FAIL;
> +}
> +
> +} /* Anonymous namespace */
> --
> 1.7.7.6
>


More information about the Piglit mailing list