[Mesa-dev] [PATCH 01/37] ABI-check: add a note that checking only T symbols is flacky

[Emil Velikov]

On 23 February 2017 at 17:34, Matt Turner <mattst88 at gmail.com> wrote:
> s/flacky/flakey/ in the title.
>
> On Thu, Feb 23, 2017 at 9:13 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>> From: Emil Velikov <emil.velikov at collabora.com>
>>
>> Note that all the symbols/ABI checks in tree do so :-\
>>
>> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
>> ---
>> Mostly an RFC since the es*api code might be going soon.
>
> Tell me more :)
>
Should have said "soon (tm)" ;-)

GLVND effectively makes these dead code/deprecated/unwanted. I've got
the EGL one locally, but I need to do some testing on it.

>> ---
>>  src/mapi/es2api/ABI-check | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/src/mapi/es2api/ABI-check b/src/mapi/es2api/ABI-check
>> index e0bf3c8314..e338408c7e 100755
>> --- a/src/mapi/es2api/ABI-check
>> +++ b/src/mapi/es2api/ABI-check
>> @@ -5,6 +5,10 @@
>>  # GL_EXT_multi_draw_arrays
>>  # GL_OES_EGL_image
>>
>> +# FIXME: checking only for T symbols in incomplete/wrong.
>
> s/in/is/ ?
>
> What's wrong with checking for T symbols? I don't know, and I think it
> should be in the commit message.

In general we want to track all symbols (but the platform specific
_edata and friends) since those are the ABI we expose. For example:
The DRI modules expose "B __driDriverExtensions", while glapi "B
_glapi_tls_Context" and "D _glapi_tls_Dispatch" amongst others.

Not to mention that libEGL/libgbm provides fluid ABI based on the
configure toggles. I.e. wl_drm_interface is always exposed by libgbm,
for libEGL things vary. That has some Wayland roots which I'm also
trying to resolve as well, just need some convincing.

Obviously special casing libGLES* doesn't bring much, even if they
expose only T (barring the platform specific of course) symbols.

Thanks
Emil