On 22 May 2012 19:42, Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>> wrote: <div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <div class="im">This test varifies that GL_POINT_SMOOTH is ignored in case of multisample points. </div>Note: This test passes on AMD and NVIDIA drivers. V2: Point smoothing is applied uniformly to all attached buffers. So, It is redundant to test on depth and stencil buffers. </blockquote><div> Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <div class="im"> Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> --- tests/all.tests | 6 + .../ext_framebuffer_multisample/CMakeLists.gl.txt | 1 + </div> .../ext_framebuffer_multisample/point-smooth.cpp | 160 ++++++++++++++++++++ 3 files changed, 167 insertions(+), 0 deletions(-) <div class="im"> create mode 100644 tests/spec/ext_framebuffer_multisample/point-smooth.cpp diff --git a/tests/all.tests b/tests/all.tests </div>index 710da95..b8a1e2b 100644 --- a/tests/all.tests +++ b/tests/all.tests @@ -1351,6 +1351,12 @@ for num_samples in (2, 4, 8, 16, 32): <div class="im"> test_name) ext_framebuffer_multisample[test_name] = PlainExecTest(executable) +for num_samples in (2, 4, 8, 16, 32): </div>+ test_name = ' ' .join(['point-smooth', str(num_samples)]) <div class="im">+ executable = 'ext_framebuffer_multisample-{0} -auto'.format( + test_name) + ext_framebuffer_multisample[test_name] = PlainExecTest(executable) + 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 </div>index fdb6a7a..9d5bc91 100644 <div class="im">--- a/tests/spec/ext_framebuffer_multisample/CMakeLists.gl.txt +++ b/tests/spec/ext_framebuffer_multisample/CMakeLists.gl.txt @@ -21,6 +21,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-point-smooth common.cpp point-smooth.cpp) piglit_add_executable (ext_framebuffer_multisample-renderbuffer-samples renderbuffer-samples.c) piglit_add_executable (ext_framebuffer_multisample-renderbufferstorage-samples renderbufferstorage-samples.c) piglit_add_executable (ext_framebuffer_multisample-samples samples.c) diff --git a/tests/spec/ext_framebuffer_multisample/point-smooth.cpp b/tests/spec/ext_framebuffer_multisample/point-smooth.cpp new file mode 100644 </div>index 0000000..2fe4953 --- /dev/null +++ b/tests/spec/ext_framebuffer_multisample/point-smooth.cpp @@ -0,0 +1,160 @@ <div><div class="h5">+/* + * 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. + */ + +#include "common.h" + +/** + * \file point-smooth.cpp + * + * Page 122 (in the PDF) of the OpenGL 3.0 spec says: + * "If MULTISAMPLE is enabled, and the value of SAMPLE BUFFERS is one, + * then points are rasterized using the following algorithm, regardless + * of whether point antialias-ing (POINT SMOOTH) is enabled or disabled". + * + * This test operates by drawing a test image with GL_POINT_SMOOTH + * enabled in an MSAA buffer. Blit it in to left half of window system + * framebuffer. + * + * Draw same image second time with GL_POINT_SMOOTH disabled. Blit it + * in to right half of window system framebuffer. This is the reference + * image. + * + * To verify that GL_POINT_SMOOTH doesn't affect MSAA, compare the two + * halves of default framebuffer. There should be no difference. + * + */ +int piglit_width = 512; int piglit_height = 256; +int piglit_window_mode = + GLUT_DOUBLE | GLUT_RGBA | GLUT_ALPHA | GLUT_DEPTH | GLUT_STENCIL; + +const int pattern_width = 256; const int pattern_height = 256; + +Fbo test_fbo; +TestPattern *test_pattern = NULL; </div></div><div class="im">+GLbitfield buffer_to_test; + +void +print_usage_and_exit(char *prog_name) +{ </div>+ printf("Usage: %s <num_samples>\n", <div class="im">+ prog_name); + piglit_report_result(PIGLIT_FAIL); +} </div><div class="im">+ +void +piglit_init(int argc, char **argv) +{ + int num_samples; </div>+ if (argc < 2) <div class="im">+ print_usage_and_exit(argv[0]); + { + char *endptr = NULL; + num_samples = strtol(argv[1], &endptr, 0); + if (endptr != argv[1] + strlen(argv[1])) + print_usage_and_exit(argv[0]); + } + + piglit_require_gl_version(30); </div>+ glClear (GL_COLOR_BUFFER_BIT); <div class="im">+ + /* 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); + </div>+ buffer_to_test = GL_COLOR_BUFFER_BIT; <div class="im">+ test_pattern = new Points(); </div>+ test_pattern->compile(); <div><div class="h5">+ + test_fbo.init(num_samples, pattern_width, pattern_height, + true /* combine_depth_stencil */, + false /* attach_texture */); + + /* Blending is required to test smooth points */ + glEnable (GL_BLEND); + glBlendFunc (GL_SRC_ALPHA_SATURATE, GL_ONE); +} + +enum piglit_result +piglit_display() +{ + bool pass = true; + float proj[4][4] = { + { 1, 0, 0, 0 }, + { 0, 1, 0, 0 }, + { 0, 0, 1, 0 }, + { 0, 0, 0, 1 } }; + /* Draw test pattern in multisample test_fbo with GL_POINT_SMOOTH + * disabled. + */ + glBindFramebuffer(GL_DRAW_FRAMEBUFFER, test_fbo.handle); + glClear(buffer_to_test); + test_fbo.set_viewport(); + test_pattern->draw(proj); + + /* Blit test_fbo to the right half of window system framebuffer. + * This is the reference image. + */ </div></div>+ glBindFramebuffer(GL_READ_FRAMEBUFFER, test_fbo.handle); <div class="im">+ 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); + + /* Draw test pattern in mulisample test_fbo with GL_POINT_SMOOTH + * enabled + */ + glEnable (GL_POINT_SMOOTH); + glHint( GL_POINT_SMOOTH_HINT, GL_NICEST ); + glDisable (GL_DEPTH_TEST); + + glBindFramebuffer(GL_DRAW_FRAMEBUFFER, test_fbo.handle); + test_fbo.set_viewport(); + test_pattern->draw(proj); + + glDisable(GL_POINT_SMOOTH); + + /* Now blit test_fbo to the left half of window system framebuffer. + * This is the test image. + */ </div>+ glBindFramebuffer(GL_READ_FRAMEBUFFER, test_fbo.handle); <div class="HOEnZb"><div class="h5">+ glBindFramebuffer(GL_DRAW_FRAMEBUFFER, 0); + glBlitFramebuffer(0, 0, pattern_width, pattern_height, + 0, 0, pattern_width, pattern_height, + GL_COLOR_BUFFER_BIT, GL_NEAREST); + + /* Check that the left and right halves of the screen match. If they + * don't, then GL_POINT_SMOOTH is not ignored with multisample + * rendering. + */ + glBindFramebuffer(GL_READ_FRAMEBUFFER, 0); + pass = piglit_probe_rect_halves_equal_rgba(0, 0, piglit_width, + piglit_height) && pass; + + pass = piglit_check_gl_error(GL_NO_ERROR) && pass; + + piglit_present_results(); + + return pass ? PIGLIT_PASS : PIGLIT_FAIL; +} -- 1.7.7.6 </div></div></blockquote></div>