[Mesa-dev] Mesa (master): Revert "egl: Check if API is supported when using eglBindAPI."

[Marek Olšák]

On Fri, Jun 3, 2016 at 4:33 PM, Dieter Nützel <Dieter at nuetzel-hh.de> wrote:
> Am 03.06.2016 11:47, schrieb Michel Dänzer:
>>
>> On 03.06.2016 18:34, Marek =?UNKNOWN?B?T2zFocOhaw==?= wrote:
>>>
>>> Module: Mesa
>>> Branch: master
>>> Commit: 8c361e84ad010552a42593fad4130befc58e9a6a
>>> URL:
>>> http://cgit.freedesktop.org/mesa/mesa/commit/?id=8c361e84ad010552a42593fad4130befc58e9a6a
>>>
>>> Author: Marek Olšák <marek.olsak at amd.com>
>>> Date:   Fri Jun  3 11:25:19 2016 +0200
>>>
>>> Revert "egl: Check if API is supported when using eglBindAPI."
>>>
>>> This reverts commit e8b38ca202fbe8c281aeb81a4b64256983f185e0.
>>>
>>> It broke Glamor for Gallium at least.
>>
>>
>> It exposed a bug in glamor, which is fixed by
>>
>> https://patchwork.freedesktop.org/patch/91214/
>
>
> So what route should we take?
>
> Wait for the distros to catch up and enable it then, again?
>
> I was fallen in this, too.
>
> openSUSE 13.2 / Leap 42.1
>
> /usr/bin/Xorg: symbol lookup error:
> /usr/lib64/xorg/modules/drivers/radeon_drv.so: undefined symbol:
> exaGetPixmapDriverPrivate

The commit caused Glamor to fail with:
(WW) glamor0: Failed to get GLSL version

We can't just kill Glamor support with a Mesa commit.

The only solutions are:
- add a workaround into libEGL specifically for Glamor
- keep the current eglBindAPI behavior forever

Marek