<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:x="urn:schemas-microsoft-com:office:excel" xmlns:p="urn:schemas-microsoft-com:office:powerpoint" xmlns:a="urn:schemas-microsoft-com:office:access" xmlns:dt="uuid:C2F41010-65B3-11d1-A29F-00AA00C14882" xmlns:s="uuid:BDC6E3F0-6DA3-11d1-A2A3-00AA00C14882" xmlns:rs="urn:schemas-microsoft-com:rowset" xmlns:z="#RowsetSchema" xmlns:b="urn:schemas-microsoft-com:office:publisher" xmlns:ss="urn:schemas-microsoft-com:office:spreadsheet" xmlns:c="urn:schemas-microsoft-com:office:component:spreadsheet" xmlns:odc="urn:schemas-microsoft-com:office:odc" xmlns:oa="urn:schemas-microsoft-com:office:activation" xmlns:html="http://www.w3.org/TR/REC-html40" xmlns:q="http://schemas.xmlsoap.org/soap/envelope/" xmlns:rtc="http://microsoft.com/officenet/conferencing" xmlns:D="DAV:" xmlns:Repl="http://schemas.microsoft.com/repl/" xmlns:mt="http://schemas.microsoft.com/sharepoint/soap/meetings/" xmlns:x2="http://schemas.microsoft.com/office/excel/2003/xml" xmlns:ppda="http://www.passport.com/NameSpace.xsd" xmlns:ois="http://schemas.microsoft.com/sharepoint/soap/ois/" xmlns:dir="http://schemas.microsoft.com/sharepoint/soap/directory/" xmlns:ds="http://www.w3.org/2000/09/xmldsig#" xmlns:dsp="http://schemas.microsoft.com/sharepoint/dsp" xmlns:udc="http://schemas.microsoft.com/data/udc" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:sub="http://schemas.microsoft.com/sharepoint/soap/2002/1/alerts/" xmlns:ec="http://www.w3.org/2001/04/xmlenc#" xmlns:sp="http://schemas.microsoft.com/sharepoint/" xmlns:sps="http://schemas.microsoft.com/sharepoint/soap/" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:udcs="http://schemas.microsoft.com/data/udc/soap" xmlns:udcxf="http://schemas.microsoft.com/data/udc/xmlfile" xmlns:udcp2p="http://schemas.microsoft.com/data/udc/parttopart" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns:st="" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=us-ascii"><meta name=Generator content="Microsoft Word 12 (filtered medium)"><style></style></head><body lang=ZH-CN link=blue vlink=purple><div class=WordSection1>Hi Paul, <o:p></o:p>  Thanks for your suggestion.  My answer is inline. <o:p></o:p><o:p> </o:p>Best regards<o:p></o:p>ZhaoJian<o:p></o:p><o:p> </o:p><div style='border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt'><div><div style='border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm'>From: Paul Berry [mailto:stereotype441@gmail.com] Sent: Wednesday, November 02, 2011 12:20 AM To: Zhao, Jian J Cc: piglit@lists.freedesktop.org Subject: Re: [Piglit] [PATCH] Test case on rendering with only fragment shader but no vertex shader.<o:p></o:p></div></div><o:p> </o:p>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.<o:p></o:p><o:p> </o:p>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. <o:p></o:p> Other comments follow... On 1 November 2011 03:59, <<a href="mailto:jian.j.zhao@intel.com">jian.j.zhao@intel.com</a>> wrote:<o:p></o:p><div><blockquote style='border:none;border-left:solid #CCCCCC 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm'>From: Jian Zhao <<a href="mailto:jian.j.zhao@intel.com">jian.j.zhao@intel.com</a>> 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."<o:p></o:p></blockquote><div> 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.  <o:p></o:p>I quoted this from OpenGL2.1 Reference Page on glLinkProgram at <a href="http://www.opengl.org/sdk/docs/man/">http://www.opengl.org/sdk/docs/man/</a> I will add it. <o:p></o:p><o:p> </o:p></div><blockquote style='border:none;border-left:solid #CCCCCC 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm'>---  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 © 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 <<a href="mailto:jian.j.zhao@intel.com">jian.j.zhao@intel.com</a>> + * + */ + +/** @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"; +<o:p></o:p></blockquote><div> 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); }  <o:p></o:p>Yes. You are right! I will trim it down and send a new version. Thanks very much! <o:p></o:p><o:p> </o:p><o:p> </o:p></div><blockquote style='border:none;border-left:solid #CCCCCC 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm'>+ +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 <a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a> <a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><o:p></o:p></blockquote></div><o:p> </o:p></div></div></body></html>