[Mesa-dev] [PATCH 21/25] mesa: Add extension enable and table entry for OES_shader_image_atomic.

[Ian Romanick]

On 08/17/2015 09:56 AM, Ilia Mirkin wrote:
> On Mon, Aug 17, 2015 at 12:46 PM, Francisco Jerez <currojerez at riseup.net> wrote:
>> ---
>>  src/mesa/main/extensions.c | 1 +
>>  src/mesa/main/mtypes.h     | 1 +
>>  2 files changed, 2 insertions(+)
>>
>> diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
>> index d934d19..38c7bc8 100644
>> --- a/src/mesa/main/extensions.c
>> +++ b/src/mesa/main/extensions.c
>> @@ -319,6 +319,7 @@ static const struct extension extension_table[] = {
>>     { "GL_OES_query_matrix",                        o(dummy_true),                                   ES1,       2003 },
>>     { "GL_OES_read_format",                         o(dummy_true),                              GL | ES1,       2003 },
>>     { "GL_OES_rgb8_rgba8",                          o(dummy_true),                                   ES1 | ES2, 2005 },
>> +   { "GL_OES_shader_image_atomic",                 o(OES_shader_image_atomic),                      ES3,       2015 },
>>     { "GL_OES_single_precision",                    o(dummy_true),                                   ES1,       2003 },
>>     { "GL_OES_standard_derivatives",                o(OES_standard_derivatives),                           ES2, 2005 },
>>     { "GL_OES_stencil1",                            o(dummy_false),                     DISABLE,                2005 },
>> diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
>> index 9964fb4..b489f8c 100644
>> --- a/src/mesa/main/mtypes.h
>> +++ b/src/mesa/main/mtypes.h
>> @@ -3970,6 +3970,7 @@ struct gl_extensions
>>     GLboolean OES_draw_texture;
>>     GLboolean OES_depth_texture_cube_map;
>>     GLboolean OES_EGL_image_external;
>> +   GLboolean OES_shader_image_atomic;
> 
> Does this add any functionality over ARB_shader_image_load_store? I
> guess extension bools are cheap, but... seems unnecessary.

Yes and no.  In terms of functionality, all desktop hardware that can do
GL_ARB_shader_image_load_store can do this extension.  However, this
extension has one annoying bit of text:

    "OpenGL ES 3.1 and GLSL ES 3.10 are required."

We may need an ES31 tag.  Right now several drivers support OpenGL ES
3.0, and they could incorrectly advertise this extension.  This may be a
transient state, so it may not matter much in the long run.

>>     GLboolean OES_texture_float;
>>     GLboolean OES_texture_float_linear;
>>     GLboolean OES_texture_half_float;
>> --
>> 2.4.6
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev