[Mesa-dev] [PATCH 09/12] mesa: add _mesa_get_version, a ctx-independent variant of _mesa_compute_version

Ian Romanick idr at freedesktop.org
Tue Aug 5 11:06:33 PDT 2014


This patch is

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

On 08/03/2014 08:18 AM, Marek Olšák wrote:
> From: Marek Olšák <marek.olsak at amd.com>
> 
> ---
>  src/mesa/main/version.c | 274 ++++++++++++++++++++++++++----------------------
>  src/mesa/main/version.h |   4 +
>  2 files changed, 152 insertions(+), 126 deletions(-)
> 
> diff --git a/src/mesa/main/version.c b/src/mesa/main/version.c
> index c98e30c..d9124c3 100644
> --- a/src/mesa/main/version.c
> +++ b/src/mesa/main/version.c
> @@ -200,31 +200,32 @@ _mesa_override_glsl_version(struct gl_constants *consts)
>  /**
>   * Examine enabled GL extensions to determine GL version.
>   */
> -static void
> -compute_version(struct gl_context *ctx)
> +static GLuint
> +compute_version(const struct gl_extensions *extensions,
> +                const struct gl_constants *consts, gl_api api)
>  {
>     GLuint major, minor;
>  
> -   const GLboolean ver_1_3 = (ctx->Extensions.ARB_texture_border_clamp &&
> -                              ctx->Extensions.ARB_texture_cube_map &&
> -                              ctx->Extensions.ARB_texture_env_combine &&
> -                              ctx->Extensions.ARB_texture_env_dot3);
> +   const GLboolean ver_1_3 = (extensions->ARB_texture_border_clamp &&
> +                              extensions->ARB_texture_cube_map &&
> +                              extensions->ARB_texture_env_combine &&
> +                              extensions->ARB_texture_env_dot3);
>     const GLboolean ver_1_4 = (ver_1_3 &&
> -                              ctx->Extensions.ARB_depth_texture &&
> -                              ctx->Extensions.ARB_shadow &&
> -                              ctx->Extensions.ARB_texture_env_crossbar &&
> -                              ctx->Extensions.EXT_blend_color &&
> -                              ctx->Extensions.EXT_blend_func_separate &&
> -                              ctx->Extensions.EXT_blend_minmax &&
> -                              ctx->Extensions.EXT_point_parameters);
> +                              extensions->ARB_depth_texture &&
> +                              extensions->ARB_shadow &&
> +                              extensions->ARB_texture_env_crossbar &&
> +                              extensions->EXT_blend_color &&
> +                              extensions->EXT_blend_func_separate &&
> +                              extensions->EXT_blend_minmax &&
> +                              extensions->EXT_point_parameters);
>     const GLboolean ver_1_5 = (ver_1_4 &&
> -                              ctx->Extensions.ARB_occlusion_query);
> +                              extensions->ARB_occlusion_query);
>     const GLboolean ver_2_0 = (ver_1_5 &&
> -                              ctx->Extensions.ARB_point_sprite &&
> -                              ctx->Extensions.ARB_vertex_shader &&
> -                              ctx->Extensions.ARB_fragment_shader &&
> -                              ctx->Extensions.ARB_texture_non_power_of_two &&
> -                              ctx->Extensions.EXT_blend_equation_separate &&
> +                              extensions->ARB_point_sprite &&
> +                              extensions->ARB_vertex_shader &&
> +                              extensions->ARB_fragment_shader &&
> +                              extensions->ARB_texture_non_power_of_two &&
> +                              extensions->EXT_blend_equation_separate &&
>  
>  			      /* Technically, 2.0 requires the functionality
>  			       * of the EXT version.  Enable 2.0 if either
> @@ -232,61 +233,61 @@ compute_version(struct gl_context *ctx)
>  			       * driver that only exposes the ATI extension
>  			       * will fallback to software when necessary.
>  			       */
> -			      (ctx->Extensions.EXT_stencil_two_side
> -			       || ctx->Extensions.ATI_separate_stencil));
> +			      (extensions->EXT_stencil_two_side
> +			       || extensions->ATI_separate_stencil));
>     const GLboolean ver_2_1 = (ver_2_0 &&
> -                              ctx->Extensions.EXT_pixel_buffer_object &&
> -                              ctx->Extensions.EXT_texture_sRGB);
> +                              extensions->EXT_pixel_buffer_object &&
> +                              extensions->EXT_texture_sRGB);
>     const GLboolean ver_3_0 = (ver_2_1 &&
> -                              ctx->Const.GLSLVersion >= 130 &&
> -                              (ctx->Const.MaxSamples >= 4 || ctx->Const.FakeSWMSAA) &&
> -                              (ctx->API == API_OPENGL_CORE ||
> -                               ctx->Extensions.ARB_color_buffer_float) &&
> -                              ctx->Extensions.ARB_depth_buffer_float &&
> -                              ctx->Extensions.ARB_half_float_vertex &&
> -                              ctx->Extensions.ARB_map_buffer_range &&
> -                              ctx->Extensions.ARB_shader_texture_lod &&
> -                              ctx->Extensions.ARB_texture_float &&
> -                              ctx->Extensions.ARB_texture_rg &&
> -                              ctx->Extensions.ARB_texture_compression_rgtc &&
> -                              ctx->Extensions.EXT_draw_buffers2 &&
> -                              ctx->Extensions.ARB_framebuffer_object &&
> -                              ctx->Extensions.EXT_framebuffer_sRGB &&
> -                              ctx->Extensions.EXT_packed_float &&
> -                              ctx->Extensions.EXT_texture_array &&
> -                              ctx->Extensions.EXT_texture_shared_exponent &&
> -                              ctx->Extensions.EXT_transform_feedback &&
> -                              ctx->Extensions.NV_conditional_render);
> +                              consts->GLSLVersion >= 130 &&
> +                              (consts->MaxSamples >= 4 || consts->FakeSWMSAA) &&
> +                              (api == API_OPENGL_CORE ||
> +                               extensions->ARB_color_buffer_float) &&
> +                              extensions->ARB_depth_buffer_float &&
> +                              extensions->ARB_half_float_vertex &&
> +                              extensions->ARB_map_buffer_range &&
> +                              extensions->ARB_shader_texture_lod &&
> +                              extensions->ARB_texture_float &&
> +                              extensions->ARB_texture_rg &&
> +                              extensions->ARB_texture_compression_rgtc &&
> +                              extensions->EXT_draw_buffers2 &&
> +                              extensions->ARB_framebuffer_object &&
> +                              extensions->EXT_framebuffer_sRGB &&
> +                              extensions->EXT_packed_float &&
> +                              extensions->EXT_texture_array &&
> +                              extensions->EXT_texture_shared_exponent &&
> +                              extensions->EXT_transform_feedback &&
> +                              extensions->NV_conditional_render);
>     const GLboolean ver_3_1 = (ver_3_0 &&
> -                              ctx->Const.GLSLVersion >= 140 &&
> -                              ctx->Extensions.ARB_draw_instanced &&
> -                              ctx->Extensions.ARB_texture_buffer_object &&
> -                              ctx->Extensions.ARB_uniform_buffer_object &&
> -                              ctx->Extensions.EXT_texture_snorm &&
> -                              ctx->Extensions.NV_primitive_restart &&
> -                              ctx->Extensions.NV_texture_rectangle &&
> -                              ctx->Const.Program[MESA_SHADER_VERTEX].MaxTextureImageUnits >= 16);
> +                              consts->GLSLVersion >= 140 &&
> +                              extensions->ARB_draw_instanced &&
> +                              extensions->ARB_texture_buffer_object &&
> +                              extensions->ARB_uniform_buffer_object &&
> +                              extensions->EXT_texture_snorm &&
> +                              extensions->NV_primitive_restart &&
> +                              extensions->NV_texture_rectangle &&
> +                              consts->Program[MESA_SHADER_VERTEX].MaxTextureImageUnits >= 16);
>     const GLboolean ver_3_2 = (ver_3_1 &&
> -                              ctx->Const.GLSLVersion >= 150 &&
> -                              ctx->Extensions.ARB_depth_clamp &&
> -                              ctx->Extensions.ARB_draw_elements_base_vertex &&
> -                              ctx->Extensions.ARB_fragment_coord_conventions &&
> -                              ctx->Extensions.EXT_provoking_vertex &&
> -                              ctx->Extensions.ARB_seamless_cube_map &&
> -                              ctx->Extensions.ARB_sync &&
> -                              ctx->Extensions.ARB_texture_multisample &&
> -                              ctx->Extensions.EXT_vertex_array_bgra);
> +                              consts->GLSLVersion >= 150 &&
> +                              extensions->ARB_depth_clamp &&
> +                              extensions->ARB_draw_elements_base_vertex &&
> +                              extensions->ARB_fragment_coord_conventions &&
> +                              extensions->EXT_provoking_vertex &&
> +                              extensions->ARB_seamless_cube_map &&
> +                              extensions->ARB_sync &&
> +                              extensions->ARB_texture_multisample &&
> +                              extensions->EXT_vertex_array_bgra);
>     const GLboolean ver_3_3 = (ver_3_2 &&
> -                              ctx->Const.GLSLVersion >= 330 &&
> -                              ctx->Extensions.ARB_blend_func_extended &&
> -                              ctx->Extensions.ARB_explicit_attrib_location &&
> -                              ctx->Extensions.ARB_instanced_arrays &&
> -                              ctx->Extensions.ARB_occlusion_query2 &&
> -                              ctx->Extensions.ARB_shader_bit_encoding &&
> -                              ctx->Extensions.ARB_texture_rgb10_a2ui &&
> -                              ctx->Extensions.ARB_timer_query &&
> -                              ctx->Extensions.ARB_vertex_type_2_10_10_10_rev &&
> -                              ctx->Extensions.EXT_texture_swizzle);
> +                              consts->GLSLVersion >= 330 &&
> +                              extensions->ARB_blend_func_extended &&
> +                              extensions->ARB_explicit_attrib_location &&
> +                              extensions->ARB_instanced_arrays &&
> +                              extensions->ARB_occlusion_query2 &&
> +                              extensions->ARB_shader_bit_encoding &&
> +                              extensions->ARB_texture_rgb10_a2ui &&
> +                              extensions->ARB_timer_query &&
> +                              extensions->ARB_vertex_type_2_10_10_10_rev &&
> +                              extensions->EXT_texture_swizzle);
>                                /* ARB_sampler_objects is always enabled in mesa */
>  
>     if (ver_3_3) {
> @@ -330,74 +331,90 @@ compute_version(struct gl_context *ctx)
>        minor = 2;
>     }
>  
> -   ctx->Version = major * 10 + minor;
> -
> -   create_version_string(ctx, "");
> +   return major * 10 + minor;
>  }
>  
> -static void
> -compute_version_es1(struct gl_context *ctx)
> +static GLuint
> +compute_version_es1(const struct gl_extensions *extensions)
>  {
>     /* OpenGL ES 1.0 is derived from OpenGL 1.3 */
> -   const GLboolean ver_1_0 = (ctx->Extensions.ARB_texture_env_combine &&
> -                              ctx->Extensions.ARB_texture_env_dot3);
> +   const GLboolean ver_1_0 = (extensions->ARB_texture_env_combine &&
> +                              extensions->ARB_texture_env_dot3);
>     /* OpenGL ES 1.1 is derived from OpenGL 1.5 */
>     const GLboolean ver_1_1 = (ver_1_0 &&
> -                              ctx->Extensions.EXT_point_parameters);
> +                              extensions->EXT_point_parameters);
>  
>     if (ver_1_1) {
> -      ctx->Version = 11;
> +      return 11;
>     } else if (ver_1_0) {
> -      ctx->Version = 10;
> +      return 10;
>     } else {
> -      _mesa_problem(ctx, "Incomplete OpenGL ES 1.0 support.");
> +      return 0;
>     }
> -
> -   create_version_string(ctx, "OpenGL ES-CM ");
>  }
>  
> -static void
> -compute_version_es2(struct gl_context *ctx)
> +static GLuint
> +compute_version_es2(const struct gl_extensions *extensions)
>  {
>     /* OpenGL ES 2.0 is derived from OpenGL 2.0 */
> -   const GLboolean ver_2_0 = (ctx->Extensions.ARB_texture_cube_map &&
> -                              ctx->Extensions.EXT_blend_color &&
> -                              ctx->Extensions.EXT_blend_func_separate &&
> -                              ctx->Extensions.EXT_blend_minmax &&
> -                              ctx->Extensions.ARB_vertex_shader &&
> -                              ctx->Extensions.ARB_fragment_shader &&
> -                              ctx->Extensions.ARB_texture_non_power_of_two &&
> -                              ctx->Extensions.EXT_blend_equation_separate);
> +   const GLboolean ver_2_0 = (extensions->ARB_texture_cube_map &&
> +                              extensions->EXT_blend_color &&
> +                              extensions->EXT_blend_func_separate &&
> +                              extensions->EXT_blend_minmax &&
> +                              extensions->ARB_vertex_shader &&
> +                              extensions->ARB_fragment_shader &&
> +                              extensions->ARB_texture_non_power_of_two &&
> +                              extensions->EXT_blend_equation_separate);
>     /* FINISHME: This list isn't quite right. */
> -   const GLboolean ver_3_0 = (ctx->Extensions.ARB_half_float_vertex &&
> -                              ctx->Extensions.ARB_internalformat_query &&
> -                              ctx->Extensions.ARB_map_buffer_range &&
> -                              ctx->Extensions.ARB_shader_texture_lod &&
> -                              ctx->Extensions.ARB_texture_float &&
> -                              ctx->Extensions.ARB_texture_rg &&
> -                              ctx->Extensions.ARB_texture_compression_rgtc &&
> -                              ctx->Extensions.EXT_draw_buffers2 &&
> -                              /* ctx->Extensions.ARB_framebuffer_object && */
> -                              ctx->Extensions.EXT_framebuffer_sRGB &&
> -                              ctx->Extensions.EXT_packed_float &&
> -                              ctx->Extensions.EXT_texture_array &&
> -                              ctx->Extensions.EXT_texture_shared_exponent &&
> -                              ctx->Extensions.EXT_transform_feedback &&
> -                              ctx->Extensions.NV_conditional_render &&
> -                              ctx->Extensions.ARB_draw_instanced &&
> -                              ctx->Extensions.ARB_uniform_buffer_object &&
> -                              ctx->Extensions.EXT_texture_snorm &&
> -                              ctx->Extensions.NV_primitive_restart &&
> -                              ctx->Extensions.OES_depth_texture_cube_map);
> +   const GLboolean ver_3_0 = (extensions->ARB_half_float_vertex &&
> +                              extensions->ARB_internalformat_query &&
> +                              extensions->ARB_map_buffer_range &&
> +                              extensions->ARB_shader_texture_lod &&
> +                              extensions->ARB_texture_float &&
> +                              extensions->ARB_texture_rg &&
> +                              extensions->ARB_texture_compression_rgtc &&
> +                              extensions->EXT_draw_buffers2 &&
> +                              /* extensions->ARB_framebuffer_object && */
> +                              extensions->EXT_framebuffer_sRGB &&
> +                              extensions->EXT_packed_float &&
> +                              extensions->EXT_texture_array &&
> +                              extensions->EXT_texture_shared_exponent &&
> +                              extensions->EXT_transform_feedback &&
> +                              extensions->NV_conditional_render &&
> +                              extensions->ARB_draw_instanced &&
> +                              extensions->ARB_uniform_buffer_object &&
> +                              extensions->EXT_texture_snorm &&
> +                              extensions->NV_primitive_restart &&
> +                              extensions->OES_depth_texture_cube_map);
>     if (ver_3_0) {
> -      ctx->Version = 30;
> +      return 30;
>     } else if (ver_2_0) {
> -      ctx->Version = 20;
> +      return 20;
>     } else {
> -      _mesa_problem(ctx, "Incomplete OpenGL ES 2.0 support.");
> +      return 0;
>     }
> +}
>  
> -   create_version_string(ctx, "OpenGL ES ");
> +GLuint
> +_mesa_get_version(const struct gl_extensions *extensions,
> +                  struct gl_constants *consts, gl_api api)
> +{
> +   switch (api) {
> +   case API_OPENGL_COMPAT:
> +      /* Disable GLSL 1.40 and later for legacy contexts.
> +       * This disallows creation of the GL 3.1 compatibility context. */
> +      if (consts->GLSLVersion > 130) {
> +         consts->GLSLVersion = 130;
> +      }
> +      /* fall through */
> +   case API_OPENGL_CORE:
> +      return compute_version(extensions, consts, api);
> +   case API_OPENGLES:
> +      return compute_version_es1(extensions);
> +   case API_OPENGLES2:
> +      return compute_version_es2(extensions);
> +   }
> +   return 0;
>  }
>  
>  /**
> @@ -411,23 +428,28 @@ _mesa_compute_version(struct gl_context *ctx)
>     if (ctx->Version)
>        return;
>  
> +   ctx->Version = _mesa_get_version(&ctx->Extensions, &ctx->Const, ctx->API);
> +
>     switch (ctx->API) {
>     case API_OPENGL_COMPAT:
> -      /* Disable GLSL 1.40 and later for legacy contexts.
> -       * This disallows creation of the GL 3.1 compatibility context. */
> -      if (ctx->Const.GLSLVersion > 130) {
> -         ctx->Const.GLSLVersion = 130;
> -      }
> -      /* fall through */
>     case API_OPENGL_CORE:
> -      compute_version(ctx);
> +      create_version_string(ctx, "");
>        break;
> +
>     case API_OPENGLES:
> -      compute_version_es1(ctx);
> +      if (!ctx->Version) {
> +         _mesa_problem(ctx, "Incomplete OpenGL ES 1.0 support.");
> +         return;
> +      }
> +      create_version_string(ctx, "OpenGL ES-CM ");
>        break;
> +
>     case API_OPENGLES2:
> -      compute_version_es2(ctx);
> +      if (!ctx->Version) {
> +         _mesa_problem(ctx, "Incomplete OpenGL ES 2.0 support.");
> +         return;
> +      }
> +      create_version_string(ctx, "OpenGL ES ");
>        break;
>     }
> -
>  }
> diff --git a/src/mesa/main/version.h b/src/mesa/main/version.h
> index 8dcd6a0..d6459ad 100644
> --- a/src/mesa/main/version.h
> +++ b/src/mesa/main/version.h
> @@ -30,6 +30,10 @@
>  #include "mtypes.h"
>  
>  
> +extern GLuint
> +_mesa_get_version(const struct gl_extensions *extensions,
> +                  struct gl_constants *consts, gl_api api);
> +
>  extern void
>  _mesa_compute_version(struct gl_context *ctx);
>  
> 



More information about the mesa-dev mailing list