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

Jakob Bornecrantz wallbraker at gmail.com
Tue Aug 6 17:30:02 PDT 2013


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.

Cheers, Jakob.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-stable/attachments/20130807/05472efd/attachment.html>


More information about the mesa-stable mailing list