<div dir="ltr">On Thu, Aug 15, 2013 at 7:38 AM, Chia-I Wu <span dir="ltr"><<a href="mailto:olvaffe@gmail.com" target="_blank">olvaffe@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Aug 15, 2013 at 1:26 PM, Chia-I Wu <<a href="mailto:olvaffe@gmail.com">olvaffe@gmail.com</a>> wrote:<br>

> On Sat, Aug 10, 2013 at 2:56 AM, Marek Olšák <<a href="mailto:maraeo@gmail.com">maraeo@gmail.com</a>> wrote:<br>
>> Most importantly, this hides all LLVM symbols. They shouldn't clash<br>
>> with a different LLVM version used by apps (at least in theory).<br>
>><br>
>> $ nm -g --defined-only radeonsi_dri.so<br>
>> 01148f30 D __driDriverExtensions<br>
> I am not familiar with issues regarding LLVM symbols so I am fine with<br>
> the change if this is what needs to be done (except maybe use<br>
> -export-symbols-regex __driDriverExtensions to avoid the version<br>
> script?)<br>
><br>
> But I ran the nm command on ilo_dri.so, and almost all of the exported<br>
> symbols are from libdricommon or st/dri.  I think those two components<br>
> need VISIBILITY_CFLAGS in their AM_CFLAGS and __driDriverExtensions<br>
> needs to be marked as PUBLIC.  This way other gallium targets can<br>
> benefit.<br>
</div>There is no other gallium target that uses st/dri :)<br>
<br>
Anyway, in addition to controlling exported symbols using symbol<br>
files, I still like to see VISIBILITY_CFLAGS be added to st/dri and<br>
the dri targets, which directly list source files from libdrmcommon in<br>
their SOURCES.  Besides, it seems __driConfigOptions and<br>
__dri2ConfigOptions are also marked PUBLIC.  Do they need to be<br>
exported?<br></blockquote><div><br></div><div>I tested the v2 patch on vmwgfx worked but driconf complained after</div><div>adding the above two symbols to dri.versions and driconf worked so:</div><div><br></div><div>Tested-by: Jakob Bornecrantz <<a href="mailto:jakob@vmware.com">jakob@vmware.com</a>></div>
<div>Reviewed-by: Jakob Bornecrantz <<a href="mailto:jakob@vmware.com">jakob@vmware.com</a>></div></div></div></div>