[Piglit] [PATCH] ubo/dsa: add rendering test

Ilia Mirkin imirkin at alum.mit.edu
Sat Oct 31 21:45:05 PDT 2015


This exposes (a class of) segfaults in nouveau

Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
---

This is the first dsa buffer test. It's crazy that there are no tests
for any of this stuff!

 tests/all.py                                       |   2 +
 .../arb_uniform_buffer_object/CMakeLists.gl.txt    |   1 +
 .../spec/arb_uniform_buffer_object/rendering-dsa.c | 223 +++++++++++++++++++++
 3 files changed, 226 insertions(+)
 create mode 100644 tests/spec/arb_uniform_buffer_object/rendering-dsa.c

diff --git a/tests/all.py b/tests/all.py
index f21b6b9..55114cf 100644
--- a/tests/all.py
+++ b/tests/all.py
@@ -3525,6 +3525,8 @@ with profile.group_manager(
       'referenced-by-shader')
     g(['arb_uniform_buffer_object-rendering'], 'rendering')
     g(['arb_uniform_buffer_object-rendering', 'offset'], 'rendering-offset')
+    g(['arb_uniform_buffer_object-rendering-dsa'], 'rendering-dsa')
+    g(['arb_uniform_buffer_object-rendering-dsa', 'offset'], 'rendering-dsa-offset')
     g(['arb_uniform_buffer_object-row-major'], 'row-major')
     g(['arb_uniform_buffer_object-uniformblockbinding'], 'uniformblockbinding')
 
diff --git a/tests/spec/arb_uniform_buffer_object/CMakeLists.gl.txt b/tests/spec/arb_uniform_buffer_object/CMakeLists.gl.txt
index 2f66fc0..541a100 100644
--- a/tests/spec/arb_uniform_buffer_object/CMakeLists.gl.txt
+++ b/tests/spec/arb_uniform_buffer_object/CMakeLists.gl.txt
@@ -38,6 +38,7 @@ piglit_add_executable (arb_uniform_buffer_object-negative-getactiveuniformblocki
 piglit_add_executable (arb_uniform_buffer_object-negative-getactiveuniformsiv negative-getactiveuniformsiv.c)
 piglit_add_executable (arb_uniform_buffer_object-referenced-by-shader referenced-by-shader.c)
 piglit_add_executable (arb_uniform_buffer_object-rendering rendering.c)
+piglit_add_executable (arb_uniform_buffer_object-rendering-dsa rendering-dsa.c)
 piglit_add_executable (arb_uniform_buffer_object-row-major row-major.c)
 piglit_add_executable (arb_uniform_buffer_object-uniformblockbinding uniformblockbinding.c)
 
diff --git a/tests/spec/arb_uniform_buffer_object/rendering-dsa.c b/tests/spec/arb_uniform_buffer_object/rendering-dsa.c
new file mode 100644
index 0000000..a0cb5f7
--- /dev/null
+++ b/tests/spec/arb_uniform_buffer_object/rendering-dsa.c
@@ -0,0 +1,223 @@
+/*
+ * Copyright (c) 2014 VMware, Inc.
+ *
+ * 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 rendering-dsa.c
+ *
+ * Test rendering with UBOs.  We draw four squares with different positions,
+ * sizes, rotations and colors where those parameters come from UBOs.
+ */
+
+#include "piglit-util-gl.h"
+
+PIGLIT_GL_TEST_CONFIG_BEGIN
+
+	config.supports_gl_core_version = 31;
+	config.window_visual = PIGLIT_GL_VISUAL_DOUBLE | PIGLIT_GL_VISUAL_RGBA;
+
+PIGLIT_GL_TEST_CONFIG_END
+
+static const char vert_shader_text[] =
+	"#version 140\n"
+	"in vec4 piglit_vertex;\n"
+	"layout(std140) uniform;\n"
+	"uniform ub_pos_size { vec2 pos; float size; };\n"
+	"uniform ub_rot {float rotation; };\n"
+	"\n"
+	"void main()\n"
+	"{\n"
+	"   mat2 m;\n"
+	"   m[0][0] = m[1][1] = cos(rotation); \n"
+	"   m[0][1] = sin(rotation); \n"
+	"   m[1][0] = -m[0][1]; \n"
+	"   gl_Position.xy = m * piglit_vertex.xy * vec2(size) + pos;\n"
+	"   gl_Position.zw = vec2(0, 1);\n"
+	"}\n";
+
+static const char frag_shader_text[] =
+	"#version 140\n"
+	"layout(std140) uniform;\n"
+	"uniform ub_color { vec4 color; float color_scale; };\n"
+	"out vec4 col;\n"
+	"\n"
+	"void main()\n"
+	"{\n"
+	"   col = color * color_scale;\n"
+	"}\n";
+
+#define NUM_SQUARES 4
+#define NUM_UBOS 3
+
+/* Square positions and sizes */
+static const float pos_size[NUM_SQUARES][3] = {
+	{ -0.5, -0.5, 0.1 },
+	{  0.5, -0.5, 0.2 },
+	{ -0.5, 0.5, 0.3 },
+	{  0.5, 0.5, 0.4 }
+};
+
+/* Square color and color_scales */
+static const float color[NUM_SQUARES][8] = {
+	{ 2.0, 0.0, 0.0, 1.0,   0.50, 0.0, 0.0, 0.0 },
+	{ 0.0, 4.0, 0.0, 1.0,   0.25, 0.0, 0.0, 0.0 },
+	{ 0.0, 0.0, 5.0, 1.0,   0.20, 0.0, 0.0, 0.0 },
+	{ 0.2, 0.2, 0.2, 0.2,   5.00, 0.0, 0.0, 0.0 }
+};
+
+/* Square rotations */
+static const float rotation[NUM_SQUARES] = {
+	0.0,
+	0.1,
+	0.2,
+	0.3
+};
+
+static GLuint prog;
+static GLuint buffers[NUM_UBOS];
+static GLint alignment;
+static bool test_buffer_offset = false;
+
+
+static void
+setup_ubos(void)
+{
+	static const char *names[NUM_UBOS] = {
+		"ub_pos_size",
+		"ub_color",
+		"ub_rot"
+	};
+	int i;
+
+	glGetIntegerv(GL_UNIFORM_BUFFER_OFFSET_ALIGNMENT, &alignment);
+	printf("GL_UNIFORM_BUFFER_OFFSET_ALIGNMENT = %d\n", alignment);
+
+	if (test_buffer_offset) {
+		printf("Testing buffer offset %d\n", alignment);
+	}
+	else {
+		/* we use alignment as the offset */
+		alignment = 0;
+	}
+
+	glCreateBuffers(NUM_UBOS, buffers);
+
+	for (i = 0; i < NUM_UBOS; i++) {
+		GLint index, size;
+
+		/* query UBO index */
+		index = glGetUniformBlockIndex(prog, names[i]);
+
+		/* query UBO size */
+		glGetActiveUniformBlockiv(prog, index,
+					  GL_UNIFORM_BLOCK_DATA_SIZE, &size);
+
+		printf("UBO %s: index = %d, size = %d\n",
+		       names[i], index, size);
+
+		/* Allocate UBO */
+		glNamedBufferData(buffers[i], size + alignment,
+                                  NULL, GL_DYNAMIC_DRAW);
+
+		/* Attach UBO */
+		glBindBufferRange(GL_UNIFORM_BUFFER, i, buffers[i],
+				  alignment,  /* offset */
+				  size);
+		glUniformBlockBinding(prog, index, i);
+
+		if (!piglit_check_gl_error(GL_NO_ERROR))
+			piglit_report_result(PIGLIT_FAIL);
+	}
+}
+
+
+void
+piglit_init(int argc, char **argv)
+{
+	piglit_require_extension("GL_ARB_direct_state_access");
+
+	if (argc > 1 && strcmp(argv[1], "offset") == 0) {
+		test_buffer_offset = true;
+	}
+
+	prog = piglit_build_simple_program(vert_shader_text, frag_shader_text);
+	assert(prog);
+	glUseProgram(prog);
+
+	setup_ubos();
+
+	glClearColor(0.2, 0.2, 0.2, 0.2);
+}
+
+
+static bool
+probe(int x, int y, int color_index)
+{
+	float expected[4];
+
+	/* mul color by color_scale */
+	expected[0] = color[color_index][0] * color[color_index][4];
+	expected[1] = color[color_index][1] * color[color_index][4];
+	expected[2] = color[color_index][2] * color[color_index][4];
+	expected[3] = color[color_index][3] * color[color_index][4];
+
+	return piglit_probe_pixel_rgba(x, y, expected);
+}
+
+
+enum piglit_result
+piglit_display(void)
+{
+	bool pass = true;
+	int x0 = piglit_width / 4;
+	int x1 = piglit_width * 3 / 4;
+	int y0 = piglit_height / 4;
+	int y1 = piglit_height * 3 / 4;
+	int i;
+
+	glViewport(0, 0, piglit_width, piglit_height);
+
+	glClear(GL_COLOR_BUFFER_BIT);
+
+	for (i = 0; i < NUM_SQUARES; i++) {
+		/* Load UBO data, at offset=alignment */
+		glNamedBufferSubData(buffers[0], alignment, sizeof(pos_size[0]),
+                                     pos_size[i]);
+		glNamedBufferSubData(buffers[1], alignment, sizeof(color[0]),
+                                     color[i]);
+		glNamedBufferSubData(buffers[2], alignment, sizeof(rotation[0]),
+                                     &rotation[i]);
+
+		if (!piglit_check_gl_error(GL_NO_ERROR))
+			return PIGLIT_FAIL;
+
+		piglit_draw_rect(-1, -1, 2, 2);
+	}
+
+	pass = probe(x0, y0, 0) && pass;
+	pass = probe(x1, y0, 1) && pass;
+	pass = probe(x0, y1, 2) && pass;
+	pass = probe(x1, y1, 3) && pass;
+
+	piglit_present_results();
+
+	return pass ? PIGLIT_PASS : PIGLIT_FAIL;
+}
-- 
2.4.10



More information about the Piglit mailing list