[Mesa-dev] [PATCH] docs/features: don't list EXT extensions in a list for KHR/ARB/OES extensions

[Ian Romanick]

On 5/17/19 6:24 AM, Eric Engestrom wrote:
> On 2019-05-16 at 18:34, Ian Romanick <idr at freedesktop.org> wrote:
>> On 5/15/19 7:39 AM, Gert Wollny wrote:
>>> How about moving these extensions to another (new) section? I think it
>>> is nice to have a one-stop place to find out what is supported. 
>>
>> Given the existence of mesamatrix.net, is that useful?
> 
> mesamatrix.net is nothing more than a pretty parser for this file. If you remove
> the information from this file, it won't be on the website anymore either ;-)
> 
>> When we started
>> this file, the purpose was to track work that people were doing to avoid
>> collisions and track progress towards closing the functionality gap with
>> the rest of the industry.  There's not a lot of new functionality work
>> being done, and there's not much of a functionality gap with the rest of
>> the industry.
>>
>> Given that it's unlikely there will ever be another GL version, ARB
>> extension, KHR extension, or OES extension, I'm honestly not sure how
>> much value this file has at all.
> 
> This file has contained other things as well for a while, which is why it was eventually renamed from gl3.txt to features.txt a few years ago.

When I made commit f926cf5bd0a ("docs: Rename GL3.txt to features.txt")
in 2016, it was because we finished OpenGL 3.x, and had been using the
file to track progress on OpenGL 4.x and OpenGL ES 3.x features for some
time.

> I don't have a stake in this steak, but to me this issue with this patch is that I don't see what's gained by removing this information?

I think Marek's point is that there are zero other EXT / vendor
extensions in features.txt that are not also part of some OpenGL or
OpenGL ES version.  I think if reviewers had been paying attention to
features.txt, none of these would have landed in the first place.
Looking at the logs, it seems that many of these changes were either
unreviewed or were reviewed by Marek.  There's some irony there. :)

I also believe that leaving these odd ball extensions invites more
clutter in this file.

>>> Best, 
>>> Gert
>>>
>>> On Di, 2019-05-14 at 16:07 -0400, Marek Olšák wrote:
>>>> From: Marek Olšák <marek.olsak at amd.com>
>>>>
>>>> ---
>>>>  docs/features.txt | 10 ----------
>>>>  1 file changed, 10 deletions(-)
>>>>
>>>> diff --git a/docs/features.txt b/docs/features.txt
>>>> index 38d6186dbe1..b1799550a0c 100644
>>>> --- a/docs/features.txt
>>>> +++ b/docs/features.txt
>>>> @@ -309,30 +309,20 @@ Khronos, ARB, and OES extensions that are not
>>>> part of any OpenGL or OpenGL ES ve
>>>>    GL_ARB_seamless_cubemap_per_texture                   DONE
>>>> (freedreno, i965, nvc0, radeonsi, r600, softpipe, swr, virgl)
>>>>    GL_ARB_shader_ballot                                  DONE
>>>> (i965/gen8+, nvc0, radeonsi)
>>>>    GL_ARB_shader_clock                                   DONE
>>>> (i965/gen7+, nv50, nvc0, r600, radeonsi, virgl)
>>>>    GL_ARB_shader_stencil_export                          DONE
>>>> (i965/gen9+, r600, radeonsi, softpipe, llvmpipe, swr, virgl)
>>>>    GL_ARB_shader_viewport_layer_array                    DONE
>>>> (i965/gen6+, nvc0, radeonsi)
>>>>    GL_ARB_sparse_buffer                                  DONE
>>>> (radeonsi/CIK+)
>>>>    GL_ARB_sparse_texture                                 not started
>>>>    GL_ARB_sparse_texture2                                not started
>>>>    GL_ARB_sparse_texture_clamp                           not started
>>>>    GL_ARB_texture_filter_minmax                          not started
>>>> -  GL_EXT_memory_object                                  DONE
>>>> (radeonsi)
>>>> -  GL_EXT_memory_object_fd                               DONE
>>>> (radeonsi)
>>>> -  GL_EXT_memory_object_win32                            not started
>>>> -  GL_EXT_render_snorm                                   DONE (i965,
>>>> radeonsi)
>>>> -  GL_EXT_semaphore                                      DONE
>>>> (radeonsi)
>>>> -  GL_EXT_semaphore_fd                                   DONE
>>>> (radeonsi)
>>>> -  GL_EXT_semaphore_win32                                not started
>>>> -  GL_EXT_sRGB_write_control                             DONE (all
>>>> drivers that support GLES 3.0+)
>>>> -  GL_EXT_texture_norm16                                 DONE
>>>> (freedreno, i965, r600, radeonsi, nvc0)
>>>> -  GL_EXT_texture_sRGB_R8                                DONE (all
>>>> drivers that support GLES 3.0+)
>>>>    GL_KHR_blend_equation_advanced_coherent               DONE
>>>> (i965/gen9+)
>>>>    GL_KHR_texture_compression_astc_hdr                   DONE
>>>> (i965/bxt)
>>>>    GL_KHR_texture_compression_astc_sliced_3d             DONE
>>>> (i965/gen9+, radeonsi)
>>>>    GL_OES_depth_texture_cube_map                         DONE (all
>>>> drivers that support GLSL 1.30+)
>>>>    GL_OES_EGL_image                                      DONE (all
>>>> drivers)
>>>>    GL_OES_EGL_image_external                             DONE (all
>>>> drivers)
>>>>    GL_OES_EGL_image_external_essl3                       DONE (all
>>>> drivers)
>>>>    GL_OES_required_internalformat                        DONE (all
>>>> drivers)
>>>>    GL_OES_surfaceless_context                            DONE (all
>>>> drivers)
>>>>    GL_OES_texture_compression_astc                       DONE (core
>>>> only)
>>>
>>> _______________________________________________
>>> mesa-dev mailing list
>>> mesa-dev at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>>>
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev