[Mesa-dev] [PATCH] egl: Do not export private symbols

Chad Versace chad.versace at linux.intel.com
Thu Aug 8 15:13:10 PDT 2013


On 08/06/2013 05:30 PM, Jakob Bornecrantz wrote:
> On Wed, Aug 7, 2013 at 2:00 AM, Chad Versace
> <chad.versace at linux.intel.com>wrote:
>
>> On 08/06/2013 03:02 PM, Jakob Bornecrantz wrote:
>>
>>> On Wed, Aug 7, 2013 at 12:00 AM, Jakob Bornecrantz <wallbraker at gmail.com
>>>> wrote:
>>>
>>>   On Tue, Aug 6, 2013 at 9:21 PM, Chad Versace <
>>>> chad.versace at linux.intel.com
>>>>
>>>>> wrote:
>>>>>
>>>>
>>>>   libEGL was incorrectly exporting *all* symbols, public and private.
>>>>> This patch adds -fvisibility=hidden to libEGL's linker flags to ensure
>>>>> that only symbols annotated with __attribute__((visibility("**
>>>>> default")))
>>>>> get exported.
>>>>>
>>>>> Sanity-checked on X11/EGL with Piglit, and on Wayland by running
>>>>> weston-gears.
>>>>>
>>>>>
>>>> You need to test this patch on a non-inbuilt driver, egl_gallium or
>>>> egl_glx,
>>>> none of which require you to have any hardware. Or force egl_dri2 to not
>>>> be built in.
>>>>
>>>>
>>> Do that and you have my
>>> Reviewed-by: Jakob Bornecrantz <jakob at vmware.com>
>>>
>>
>> Ok. I'll run this on egl_gallium or egl_glx, and then report back.
>
>
> Either is fine, tho I have no idea if egl_glx even works.

All looks good. I tested es2gears_x11 using egl_gallium with Gallium's
swrast driver. I also verified with gdb that those drivers were actually
used.

I'll commit to master now.

By the way, I talked to krh today, and he suggested that we delete egl_glx
rather than allow it to bitrot.



More information about the mesa-dev mailing list