[Mesa-dev] [PATCH 1/2] glapi: add GL_ARB_program_interface_query skeleton

Ian Romanick idr at freedesktop.org
Wed Feb 18 12:09:35 PST 2015


On 02/17/2015 03:27 AM, Tapani Pälli wrote:
> Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
> ---
>  src/mapi/glapi/gen/ARB_program_interface_query.xml | 109 +++++++++++++++++++++
>  src/mapi/glapi/gen/gl_API.xml                      |   2 +
>  src/mapi/glapi/gen/gl_genexec.py                   |   1 +
>  src/mesa/Makefile.sources                          |   2 +
>  src/mesa/main/program_resource.c                   |  70 +++++++++++++
>  src/mesa/main/program_resource.h                   |  59 +++++++++++
>  6 files changed, 243 insertions(+)
>  create mode 100644 src/mapi/glapi/gen/ARB_program_interface_query.xml
>  create mode 100644 src/mesa/main/program_resource.c
>  create mode 100644 src/mesa/main/program_resource.h
> 
> diff --git a/src/mapi/glapi/gen/ARB_program_interface_query.xml b/src/mapi/glapi/gen/ARB_program_interface_query.xml
> new file mode 100644
> index 0000000..c042dbd
> --- /dev/null
> +++ b/src/mapi/glapi/gen/ARB_program_interface_query.xml
> @@ -0,0 +1,109 @@
> +<?xml version="1.0"?>
> +<!DOCTYPE OpenGLAPI SYSTEM "gl_API.dtd">
> +
> +<OpenGLAPI>
> +
> +<category name="GL_ARB_program_interface_query" number="134">
> +
> +    <enum name="UNIFORM"                                         value="0x92E1"/>
> +    <enum name="UNIFORM_BLOCK"                                   value="0x92E2"/>
> +    <enum name="PROGRAM_INPUT"                                   value="0x92E3"/>
> +    <enum name="PROGRAM_OUTPUT"                                  value="0x92E4"/>
> +    <enum name="BUFFER_VARIABLE"                                 value="0x92E5"/>
> +    <enum name="SHADER_STORAGE_BLOCK"                            value="0x92E6"/>
> +    <enum name="ATOMIC_COUNTER_BUFFER"                           value="0x92C0"/>
> +    <enum name="VERTEX_SUBROUTINE"                               value="0x92E8"/>
> +    <enum name="TESS_CONTROL_SUBROUTINE"                         value="0x92E9"/>
> +    <enum name="TESS_EVALUATION_SUBROUTINE"                      value="0x92EA"/>
> +    <enum name="GEOMETRY_SUBROUTINE"                             value="0x92EB"/>
> +    <enum name="FRAGMENT_SUBROUTINE"                             value="0x92EC"/>
> +    <enum name="COMPUTE_SUBROUTINE"                              value="0x92ED"/>
> +    <enum name="VERTEX_SUBROUTINE_UNIFORM"                       value="0x92EE"/>
> +    <enum name="TESS_CONTROL_SUBROUTINE_UNIFORM"                 value="0x92EF"/>
> +    <enum name="TESS_EVALUATION_SUBROUTINE_UNIFORM"              value="0x92F0"/>
> +    <enum name="GEOMETRY_SUBROUTINE_UNIFORM"                     value="0x92F1"/>
> +    <enum name="FRAGMENT_SUBROUTINE_UNIFORM"                     value="0x92F2"/>
> +    <enum name="COMPUTE_SUBROUTINE_UNIFORM"                      value="0x92F3"/>
> +    <enum name="TRANSFORM_FEEDBACK_VARYING"                      value="0x92F4"/>
> +    <enum name="ACTIVE_RESOURCES"                                value="0x92F5"/>
> +    <enum name="MAX_NAME_LENGTH"                                 value="0x92F6"/>
> +    <enum name="MAX_NUM_ACTIVE_VARIABLES"                        value="0x92F7"/>
> +    <enum name="MAX_NUM_COMPATIBLE_SUBROUTINES"                  value="0x92F8"/>
> +    <enum name="NAME_LENGTH"                                     value="0x92F9"/>
> +    <enum name="TYPE"                                            value="0x92FA"/>
> +    <enum name="ARRAY_SIZE"                                      value="0x92FB"/>
> +    <enum name="OFFSET"                                          value="0x92FC"/>
> +    <enum name="BLOCK_INDEX"                                     value="0x92FD"/>
> +    <enum name="ARRAY_STRIDE"                                    value="0x92FE"/>
> +    <enum name="MATRIX_STRIDE"                                   value="0x92FF"/>
> +    <enum name="IS_ROW_MAJOR"                                    value="0x9300"/>
> +    <enum name="ATOMIC_COUNTER_BUFFER_INDEX"                     value="0x9301"/>
> +    <enum name="BUFFER_BINDING"                                  value="0x9302"/>
> +    <enum name="BUFFER_DATA_SIZE"                                value="0x9303"/>
> +    <enum name="NUM_ACTIVE_VARIABLES"                            value="0x9304"/>
> +    <enum name="ACTIVE_VARIABLES"                                value="0x9305"/>
> +    <enum name="REFERENCED_BY_VERTEX_SHADER"                     value="0x9306"/>
> +    <enum name="REFERENCED_BY_TESS_CONTROL_SHADER"               value="0x9307"/>
> +    <enum name="REFERENCED_BY_TESS_EVALUATION_SHADER"            value="0x9308"/>
> +    <enum name="REFERENCED_BY_GEOMETRY_SHADER"                   value="0x9309"/>
> +    <enum name="REFERENCED_BY_FRAGMENT_SHADER"                   value="0x930A"/>
> +    <enum name="REFERENCED_BY_COMPUTE_SHADER"                    value="0x930B"/>
> +    <enum name="TOP_LEVEL_ARRAY_SIZE"                            value="0x930C"/>
> +    <enum name="TOP_LEVEL_ARRAY_STRIDE"                          value="0x930D"/>
> +    <enum name="LOCATION"                                        value="0x930E"/>
> +    <enum name="LOCATION_INDEX"                                  value="0x930F"/>
> +    <enum name="IS_PER_PATCH"                                    value="0x92E7"/>
> +    <enum name="NUM_COMPATIBLE_SUBROUTINES"                      value="0x8E4A"/>
> +    <enum name="COMPATIBLE_SUBROUTINES"                          value="0x8E4B"/>
> +
> +    <function name="GetProgramInterfaceiv" offset="assign">
> +        <param name="program" type="GLuint"/>
> +        <param name="programInterface" type="GLenum"/>
> +        <param name="pname" type="GLenum"/>
> +        <param name="params" type="GLintptr"/>
> +    </function>
> +
> +    <function name="GetProgramResourceIndex" offset="assign">
> +        <param name="program" type="GLuint"/>
> +        <param name="programInterface" type="GLenum"/>
> +        <param name="name" type="const GLchar *"/>
> +        <return type="GLuint"/>
> +    </function>
> +
> +    <function name="GetProgramResourceName" offset="assign">
> +        <param name="program" type="GLuint"/>
> +        <param name="programInterface" type="GLenum"/>
> +        <param name="index" type="GLuint"/>
> +        <param name="bufSize" type="GLsizei "/>
> +        <param name="length" type="GLsizei *" output="true"/>
> +        <param name="name" type="const GLchar *" output="true"/>
> +    </function>
> +
> +    <function name="GetProgramResourceiv" offset="assign">
> +        <param name="program" type="GLuint"/>
> +        <param name="programInterface" type="GLenum"/>
> +        <param name="index" type="GLuint"/>
> +        <param name="propCount" type="GLsizei "/>
> +        <param name="props" type="const GLenum *" output="true"/>
> +        <param name="bufSize" type="GLsizei "/>
> +        <param name="length" type="GLsizei *" output="true"/>
> +        <param name="params" type="GLintptr" output="true"/>
> +    </function>
> +
> +    <function name="GetProgramResourceLocation" offset="assign">
> +        <param name="program" type="GLuint"/>
> +        <param name="programInterface" type="GLenum"/>
> +        <param name="name" type="const GLchar *"/>
> +        <return type="GLint"/>
> +    </function>
> +
> +    <function name="GetProgramResourceLocationIndex" offset="assign">
> +        <param name="program" type="GLuint"/>
> +        <param name="programInterface" type="GLenum"/>
> +        <param name="name" type="const GLchar *"/>
> +        <return type="GLint"/>
> +    </function>
> +
> +</category>
> +
> +</OpenGLAPI>
> diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml
> index 17bf62a..0c20b99 100644
> --- a/src/mapi/glapi/gen/gl_API.xml
> +++ b/src/mapi/glapi/gen/gl_API.xml
> @@ -8337,6 +8337,8 @@
>  
>  <!-- ARB extensions #134...#138 -->
>  
> +<xi:include href="ARB_program_interface_query.xml" xmlns:xi="http://www.w3.org/2001/XInclude"/>
> +
>  <xi:include href="ARB_texture_buffer_range.xml" xmlns:xi="http://www.w3.org/2001/XInclude"/>
>  
>  <!-- 140. GL_ARB_texture_query_levels -->
> diff --git a/src/mapi/glapi/gen/gl_genexec.py b/src/mapi/glapi/gen/gl_genexec.py
> index d479e66..7151f0d 100644
> --- a/src/mapi/glapi/gen/gl_genexec.py
> +++ b/src/mapi/glapi/gen/gl_genexec.py
> @@ -92,6 +92,7 @@ header = """/**
>  #include "main/pixelstore.h"
>  #include "main/points.h"
>  #include "main/polygon.h"
> +#include "main/program_resource.h"
>  #include "main/querymatrix.h"
>  #include "main/queryobj.h"
>  #include "main/readpix.h"
> diff --git a/src/mesa/Makefile.sources b/src/mesa/Makefile.sources
> index b5912b7..df535fc 100644
> --- a/src/mesa/Makefile.sources
> +++ b/src/mesa/Makefile.sources
> @@ -158,6 +158,8 @@ MAIN_FILES = \
>  	main/rastpos.c \
>  	main/rastpos.h \
>  	main/readpix.c \
> +	main/program_resource.c \
> +	main/program_resource.h \
>  	main/readpix.h \
>  	main/remap.c \
>  	main/remap.h \
> diff --git a/src/mesa/main/program_resource.c b/src/mesa/main/program_resource.c
> new file mode 100644
> index 0000000..a5b114b
> --- /dev/null
> +++ b/src/mesa/main/program_resource.c
> @@ -0,0 +1,70 @@
> +/*
> + * Mesa 3-D graphics library
> + *
> + * Copyright (C) 2015 Intel Corporation.  All Rights Reserved.
> + *
> + * 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 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 "program_resource.h"
> +
> +void GLAPIENTRY
> +_mesa_GetProgramInterfaceiv(GLuint program, GLenum programInterface,
> +                            GLenum pname, GLintptr params)
> +{
> +}

If these functions are going to remain skeletons for awhile, they should
generate GL_INVALID_OPERATION.

> +
> +GLuint GLAPIENTRY
> +_mesa_GetProgramResourceIndex(GLuint program, GLenum programInterface,
> +                              const GLchar *name)
> +{
> +   return 0;
> +}
> +
> +void GLAPIENTRY
> +_mesa_GetProgramResourceName(GLuint program, GLenum programInterface,
> +                             GLuint index, GLsizei bufSize, GLsizei *length,
> +                             const GLchar *name)
> +{
> +   *length = 0;
> +}
> +
> +void GLAPIENTRY
> +_mesa_GetProgramResourceiv(GLuint program, GLenum programInterface,
> +                           GLuint index, GLsizei propCount,
> +                           const GLenum *props, GLsizei bufSize,
> +                           GLsizei *length, GLintptr params)
> +{
> +   *length = 0;
> +}
> +
> +GLint GLAPIENTRY
> +_mesa_GetProgramResourceLocation(GLuint program, GLenum programInterface,
> +                                 const GLchar *name)
> +{
> +   return -1;
> +}
> +
> +GLint GLAPIENTRY
> +_mesa_GetProgramResourceLocationIndex(GLuint program, GLenum programInterface,
> +                                      const GLchar *name)
> +{
> +   return -1;
> +}
> diff --git a/src/mesa/main/program_resource.h b/src/mesa/main/program_resource.h
> new file mode 100644
> index 0000000..edac3a3
> --- /dev/null
> +++ b/src/mesa/main/program_resource.h
> @@ -0,0 +1,59 @@
> +/*
> + * Mesa 3-D graphics library
> + *
> + * Copyright (C) 2015 Intel Corporation.  All Rights Reserved.
> + *
> + * 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 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.
> + *
> + */
> +
> +#ifndef PROGRAM_RESOURCE_H
> +#define PROGRAM_RESOURCE_H
> +
> +#include "glheader.h"
> +
> +struct gl_context;
> +
> +extern void GLAPIENTRY
> +_mesa_GetProgramInterfaceiv(GLuint program, GLenum programInterface,
> +                            GLenum pname, GLintptr params);
> +
> +extern GLuint GLAPIENTRY
> +_mesa_GetProgramResourceIndex(GLuint program, GLenum programInterface,
> +                              const GLchar *name);
> +
> +extern void GLAPIENTRY
> +_mesa_GetProgramResourceName(GLuint program, GLenum programInterface,
> +                             GLuint index, GLsizei bufSize, GLsizei *length,
> +                             const GLchar *name);
> +extern void GLAPIENTRY
> +_mesa_GetProgramResourceiv(GLuint program, GLenum programInterface,
> +                           GLuint index, GLsizei propCount,
> +                           const GLenum *props, GLsizei bufSize,
> +                           GLsizei *length, GLintptr params);
> +
> +extern GLint GLAPIENTRY
> +_mesa_GetProgramResourceLocation(GLuint program, GLenum programInterface,
> +                                 const GLchar *name);
> +
> +extern GLint GLAPIENTRY
> +_mesa_GetProgramResourceLocationIndex(GLuint program, GLenum programInterface,
> +                                      const GLchar *name);
> +
> +#endif
> 



More information about the mesa-dev mailing list