On 21 April 2016 at 13:14, Emil Velikov <emil.l.veli...@gmail.com> wrote: > Annotate the only symbol that should be provided statically as PUBLIC > and add VISIBILITY_CFLAGS to hide everything else. > > Note: with this in place we still have multiple symbols exported. Those > will be fixed with follow up commits. > > Signed-off-by: Emil Velikov <emil.veli...@collabora.com> > --- > Does anyone know the section name that covers this? Afaict one could > (should?) have a layer section in their json file with a functions > vkGetInstanceProcAddr... entry point. Right ? > > -Emil > > src/intel/vulkan/Makefile.am | 5 ++++- > src/intel/vulkan/anv_device.c | 5 ++--- > 2 files changed, 6 insertions(+), 4 deletions(-) > > diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am > index edbe213..bac5993 100644 > --- a/src/intel/vulkan/Makefile.am > +++ b/src/intel/vulkan/Makefile.am > @@ -66,7 +66,10 @@ AM_CPPFLAGS = \ > -I$(top_builddir)/src/compiler/nir \ > -I$(top_builddir)/src/intel > > -libvulkan_intel_la_CFLAGS = $(CFLAGS) -DVK_USE_PLATFORM_XCB_KHR > -Wno-override-init -msse2 > +libvulkan_intel_la_CFLAGS = \ > + $(VISIBILITY_CFLAGS) \ > + $(CFLAGS) \ > + -DVK_USE_PLATFORM_XCB_KHR -Wno-override-init -msse2 > > VULKAN_SOURCES = \ > anv_allocator.c \ > diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c > index 00edd95..d6e74d3 100644 > --- a/src/intel/vulkan/anv_device.c > +++ b/src/intel/vulkan/anv_device.c > @@ -641,13 +641,12 @@ PFN_vkVoidFunction anv_GetInstanceProcAddr( > return anv_lookup_entrypoint(pName); > } > > -/* The loader wants us to expose a second GetInstanceProcAddr function > - * to work around certain LD_PRELOAD issues seen in apps. > - */ > +PUBLIC > VKAPI_ATTR PFN_vkVoidFunction VKAPI_CALL vk_icdGetInstanceProcAddr( > VkInstance instance, > const char* pName); > Jason, Chad,
Can I bother you with this one as well ? From my rather quick look at Vulkan spec, the vendor driver should not export anything but the vkGetInstanceProcAddr symbol and any extensions. As the actual symbol name here isn't the standard one - we should create a layer section in the json and wire the two in there. Correct ? Do we want to keep the stubs for VK_DEBUG_MARKER_EXT (patch 2/2) or just nuke it ? Thanks Emil P.S. A s/an/anv/ fix in the shortlog has been applied. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
