[Piglit] [PATCH 4/4] builtin-gl-sample-mask: Add testing for fbo with renderbuffer attachment

[Anuj Phogat]

Existing test only covers the drawing to texture attachment with a
sample mask. This patch extends the test to include drawing to
renderbuffer attachment. This might help capture the bugs where
drawing to a texture differ from drawing to a renderbuffer.

Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
---
 .../execution/builtin-gl-sample-mask.cpp           | 54 ++++++++++++++++++----
 1 file changed, 44 insertions(+), 10 deletions(-)

diff --git a/tests/spec/arb_sample_shading/execution/builtin-gl-sample-mask.cpp b/tests/spec/arb_sample_shading/execution/builtin-gl-sample-mask.cpp
index bd71bac..6585ab4 100644
--- a/tests/spec/arb_sample_shading/execution/builtin-gl-sample-mask.cpp
+++ b/tests/spec/arb_sample_shading/execution/builtin-gl-sample-mask.cpp
@@ -45,7 +45,7 @@ PIGLIT_GL_TEST_CONFIG_END
 
 static int  num_samples;
 static unsigned prog_0, prog_1;
-static Fbo multisampled_tex;
+static Fbo multisampled_fbo, multisampled_tex;
 
 static void
 print_usage_and_exit(char *prog_name)
@@ -156,6 +156,8 @@ piglit_init(int argc, char **argv)
 		piglit_report_result(PIGLIT_SKIP);
 
 	FboConfig msConfig(num_samples, pattern_width, pattern_height);
+	multisampled_fbo.setup(msConfig);
+
         msConfig.attach_texture = true;
 	multisampled_tex.setup(msConfig);
 
@@ -165,31 +167,63 @@ piglit_init(int argc, char **argv)
 	}
 }
 
-enum piglit_result
-piglit_display()
+bool
+test_builtin_sample_mask(Fbo ms_fbo)
 {
-	bool pass = true;
-	GLint samples;
-        GLfloat expected[4] = {0.0, 1.0, 0.0, 1.0};
+	int samples;
+	bool result = true;
+	float expected[4] = {0.0, 1.0, 0.0, 1.0};
 
 	glUseProgram(prog_0);
-	glBindFramebuffer(GL_DRAW_FRAMEBUFFER, multisampled_tex.handle);
+	glBindFramebuffer(GL_DRAW_FRAMEBUFFER, ms_fbo.handle);
 	glGetIntegerv(GL_SAMPLES, &samples);
 	glClear(GL_COLOR_BUFFER_BIT);
 	glUniform1i(glGetUniformLocation(prog_0, "samples"), samples);
         piglit_draw_rect(-1, -1, 2, 2);
 
+	if(!ms_fbo.config.attach_texture) {
+		/* Blit the framebuffer with multisample renderbuffer attachment
+		 * into the framebuffer with multisample texture attachment.
+		 */
+		glBindFramebuffer(GL_READ_FRAMEBUFFER, ms_fbo.handle);
+		glBindFramebuffer(GL_DRAW_FRAMEBUFFER, multisampled_tex.handle);
+		glClear(GL_COLOR_BUFFER_BIT);
+		glBlitFramebuffer(0, 0,
+				  ms_fbo.config.width,
+				  ms_fbo.config.height,
+				  0, 0,
+				  ms_fbo.config.width,
+				  ms_fbo.config.height,
+				  GL_COLOR_BUFFER_BIT, GL_NEAREST);
+	}
+
+	glBindFramebuffer(GL_READ_FRAMEBUFFER, multisampled_tex.handle);
 	glBindFramebuffer(GL_DRAW_FRAMEBUFFER, piglit_winsys_fbo);
 	glClear(GL_COLOR_BUFFER_BIT);
+
 	glUseProgram(prog_1);
 	glUniform1i(glGetUniformLocation(prog_1, "tex"), 0);
 	glUniform1i(glGetUniformLocation(prog_1, "samples"), samples);
 	piglit_draw_rect(-1, -1, 2, 2);
 	
 	glBindFramebuffer(GL_READ_FRAMEBUFFER, piglit_winsys_fbo);
-	pass = piglit_probe_rect_rgba(0, 0, pattern_width,
-                                      pattern_width, expected)
-               && pass;
+	result = piglit_probe_rect_rgba(0, 0, pattern_width,
+					pattern_width, expected)
+		 && result;
 	piglit_present_results();
+	printf("FBO attachment = %s, result = %s\n",
+	       ms_fbo.config.attach_texture ?
+	       "TEXTURE" :
+	       "RENDERBUFFER",
+	       result ? "pass" : "fail");
+	return result;
+}
+
+enum piglit_result
+piglit_display()
+{
+	bool pass = true;
+	pass = test_builtin_sample_mask(multisampled_tex) && pass;
+	pass = test_builtin_sample_mask(multisampled_fbo) && pass;
 	return pass ? PIGLIT_PASS : PIGLIT_FAIL;
 }
-- 
1.8.3.1