[Piglit] [PATCH] Test case on rendering with only fragment shader but no vertex shader.
Zhao, Jian J
jian.j.zhao at intel.com
Wed Nov 2 02:07:46 PDT 2011
Hi Paul,
Thanks for your suggestion. My answer is inline.
Best regards
ZhaoJian
From: Paul Berry [mailto:stereotype441 at gmail.com]
Sent: Wednesday, November 02, 2011 12:20 AM
To: Zhao, Jian J
Cc: piglit at lists.freedesktop.org
Subject: Re: [Piglit] [PATCH] Test case on rendering with only fragment shader but no vertex shader.
Before I get to specific comments, are you familiar with Piglit's "shader runner" framework (look at any of the Piglit files that end in ".shader_test"). It seems like this test could be rewritten as a shader runner test--that would make it shorter and more readable, and save a compilation step.
It's a good idea. But unfortunately I can't reproduce it with shader_runner, I have a look and find it is because it uses glDrawArraysInstancedARB instead of glDrawArrays or just simple glBegin and glEnd. I don't know what is glDrawArraysInstancedARB. What's it difference from glDrawArrays? So I just updated my original patch file.
Other comments follow...
On 1 November 2011 03:59, <jian.j.zhao at intel.com<mailto:jian.j.zhao at intel.com>> wrote:
From: Jian Zhao <jian.j.zhao at intel.com<mailto:jian.j.zhao at intel.com>>
Per the GL C Specification on glLinkProgram, it should use fixed function pipeline if there is no vertex shader or fragment shader. The spec is as following:
"If program contains shader objects
of type GL_VERTEX_SHADER but does not
contain shader objects of type
GL_FRAGMENT_SHADER, the vertex shader will
be linked against the implicit interface for fixed functionality
fragment processing. Similarly, if
program contains shader objects of type
GL_FRAGMENT_SHADER but it does not contain
shader objects of type GL_VERTEX_SHADER,
the fragment shader will be linked against the implicit
interface for fixed functionality vertex processing."
When quoting from the spec, can you site the particular version of the GL spec that you're quoting, and the section number you're quoting from? That will make it easier to figure out what went wrong if this test ever fails.
I quoted this from OpenGL2.1 Reference Page on glLinkProgram at http://www.opengl.org/sdk/docs/man/ I will add it.
---
tests/shaders/CMakeLists.gl.txt | 1 +
tests/shaders/glsl-no-vertex-shader-compiled.c | 165 ++++++++++++++++++++++++
2 files changed, 166 insertions(+), 0 deletions(-)
create mode 100644 tests/shaders/glsl-no-vertex-shader-compiled.c
diff --git a/tests/shaders/CMakeLists.gl.txt b/tests/shaders/CMakeLists.gl.txt
index 3dce256..d009395 100644
--- a/tests/shaders/CMakeLists.gl.txt
+++ b/tests/shaders/CMakeLists.gl.txt
@@ -28,6 +28,7 @@ add_executable (fp-condition_codes-01 fp-condition_codes-01.c)
add_executable (fp-lit-mask fp-lit-mask.c)
add_executable (fp-lit-src-equals-dst fp-lit-src-equals-dst.c)
add_executable (fp-fog fp-fog.c)
+add_executable (glsl-no-vertex-shader-compiled glsl-no-vertex-shader-compiled.c)
add_executable (fp-formats fp-formats.c)
add_executable (fp-fragment-position fp-fragment-position.c)
add_executable (fp-generic fp-generic.c)
diff --git a/tests/shaders/glsl-no-vertex-shader-compiled.c b/tests/shaders/glsl-no-vertex-shader-compiled.c
new file mode 100644
index 0000000..0f90f6b
--- /dev/null
+++ b/tests/shaders/glsl-no-vertex-shader-compiled.c
@@ -0,0 +1,165 @@
+/*
+ * Copyright (c) 2011 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.
+ *
+ * Authors:
+ * Jian Zhao <jian.j.zhao at intel.com<mailto:jian.j.zhao at intel.com>>
+ *
+ */
+
+/** @file glsl-no-vertex-shader-compiled.c
+ *
+ * Tests that OpenGL shader should work well when only have fragment shader(or vertext shader), but now if without vertext shader it doesn't work.
+ In the OpenGL spec for glLinkProgram, as following says:
+ "If program contains shader objects
+ of type GL_VERTEX_SHADER but does not
+ contain shader objects of type
+ GL_FRAGMENT_SHADER, the vertex shader will
+ be linked against the implicit interface for fixed functionality
+ fragment processing. Similarly, if
+ program contains shader objects of type
+ GL_FRAGMENT_SHADER but it does not contain
+ shader objects of type GL_VERTEX_SHADER,
+ the fragment shader will be linked against the implicit
+ interface for fixed functionality vertex processing."
+ *
+ */
+
+#include "piglit-util.h"
+#include "piglit-framework.h"
+
+int piglit_width = 100, piglit_height = 100;
+int piglit_window_mode = GLUT_RGBA | GLUT_DOUBLE;
+
+static GLint prog;
+
+static const char fs_vector_template[] =
+"#version 120\n"
+"void myfunc2(void); \n"
+"void myfunc3(void); \n"
+"void main() \n"
+"{ \n"
+" gl_FragColor = gl_Color; \n"
+" myfunc2(); \n"
+"} \n"
+"void myfunc2(void) \n"
+"{ \n"
+" gl_FragColor = vec4(0.2,0.2,0.2, 1.0); \n"
+" myfunc3(); \n"
+"} \n"
+"void myfunc3(void) \n"
+"{ \n"
+" gl_FragColor = vec4(0.2,0.8,0.2,1.0); \n"
+"} \n";
+
Are you trying to test something about the behavior of function calls here? If so, that should be split into its own test, with a comment explaining what you're trying to accomplish. If not, we should use the simplest possible fragment shader that allows us to test what we're trying to test, e.g.:
void main()
{
gl_FragColor = vec4(0.2, 0.8, 0.2, 1.0);
}
Yes. You are right! I will trim it down and send a new version. Thanks very much!
+
+enum piglit_result
+piglit_display(void)
+{
+ GLboolean result = PIGLIT_PASS;
+ GLfloat expectedColor[4] = {0.2, 0.8, 0.2, 1.0};
+
+ glClearColor(1.0, 0.1, 0.1, 0.0);
+ glClear(GL_COLOR_BUFFER_BIT );
+
+ glColor4f(0.0, 1.0, 0.0, 0.0);
+ glBegin(GL_QUADS);
+ glVertex3f(0.0, 0.0, 0.0);
+ glVertex3f(0.5, 0.0, 0.0);
+ glVertex3f(0.5, 0.5, 0.0);
+ glVertex3f(0.0, 0.5, 0.0);
+ glEnd();
+
+ result = piglit_probe_pixel_rgba(piglit_width /2, piglit_height / 2,
+ expectedColor) ? PIGLIT_PASS : PIGLIT_FAIL;
+ glutSwapBuffers();
+
+ return result;
+}
+
+static GLuint compile_shader(GLenum shaderType, const char * text)
+{
+ GLuint shader;
+ GLint status;
+
+ shader = glCreateShaderObjectARB(shaderType);
+ glShaderSourceARB(shader, 1, (const GLchar **)&text, NULL);
+ glCompileShaderARB(shader);
+
+ glGetObjectParameterivARB(shader, GL_OBJECT_COMPILE_STATUS_ARB, &status);
+ if (!status) {
+ GLchar log[1000];
+ GLsizei len;
+ glGetInfoLogARB(shader, 1000, &len, log);
+ fprintf(stderr, "Error: problem compiling shader: %s\n", log);
+ piglit_report_result(PIGLIT_FAIL);
+ }
+
+ return shader;
+}
+
+static GLuint link_program(GLuint fs)
+{
+ GLuint program;
+ GLint status;
+
+ program = glCreateProgramObjectARB();
+
+ if (fs)
+ glAttachObjectARB(program, fs);
+
+ glLinkProgramARB(program);
+ glGetObjectParameterivARB(program, GL_OBJECT_LINK_STATUS_ARB, &status);
+ if (!status) {
+ GLchar log[1000];
+ GLsizei len;
+ glGetInfoLogARB(program, 1000, &len, log);
+ fprintf(stderr, "Error: problem linking program: %s\n", log);
+ piglit_report_result(PIGLIT_FAIL);
+ }
+
+ return program;
+}
+
+void
+piglit_init(int argc, char **argv)
+{
+ GLint fs;
+ char *fs_buffer;
+
+ if (!GLEW_VERSION_2_0 || !GLEW_ARB_shader_objects || !GLEW_ARB_vertex_shader || !GLEW_ARB_fragment_shader) {
+ printf("Requires ARB_shader_objects and ARB_{vertex,fragment}_shader\n");
+ piglit_report_result(PIGLIT_SKIP);
+ }
+
+ piglit_ortho_projection(piglit_width, piglit_height, GL_FALSE);
+
+ fs_buffer = (char *) malloc(sizeof(char) * (strlen(fs_vector_template)+1));
+ strcpy(fs_buffer, fs_vector_template);
+ fs = compile_shader(GL_FRAGMENT_SHADER_ARB, fs_buffer);
+
+ prog = link_program(fs);
+
+ glUseProgramObjectARB(prog);
+
+ if(fs_buffer)
+ free(fs_buffer);
+}
--
1.7.0.1
_______________________________________________
Piglit mailing list
Piglit at lists.freedesktop.org<mailto:Piglit at lists.freedesktop.org>
http://lists.freedesktop.org/mailman/listinfo/piglit
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20111102/1034fcef/attachment-0001.htm>
More information about the Piglit
mailing list