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

[Marek Olšák]

On Mon, Jun 6, 2016 at 11:10 AM, Michel Dänzer <michel at daenzer.net> wrote:
> On 04.06.2016 00:10, Marek Olšák wrote:
>> 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.
>
> Why do released versions of xserver/glamor have to work with unreleased
> versions of Mesa?

Because it's my use case. (I don't compile X and I guess neither do
many other people)

>
>
>> - keep the current eglBindAPI behavior forever
>
> So because we haven't been following the EGL spec, allowing broken EGL
> apps to work by accident, we have to preserve that bug forever? I'm not
> buying it.

Like I said, it's my use case. I can't really change this.

Marek