Bug 62434 - [bisected] 3284.073] (EE) AIGLX error: dlopen of /usr/lib/xorg/modules/dri/r600_dri.so failed (/usr/lib/libllvmradeon9.2.0.so: undefined symbol: lp_build_tgsi_intrinsic)
Summary: [bisected] 3284.073] (EE) AIGLX error: dlopen of /usr/lib/xorg/modules/dri/r6...
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/Gallium/r600 (show other bugs)
Version: git
Hardware: x86-64 (AMD64) Linux (All)
: medium blocker
Assignee: Default DRI bug account
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-17 11:45 UTC by LoneVVolf
Modified: 2013-03-20 08:45 UTC (History)
4 users (show)

See Also:
i915 platform:
i915 features:


Attachments
build output (215.68 KB, text/plain)
2013-03-17 12:22 UTC, LoneVVolf
Details
complete Xorg.0.log (40.77 KB, text/plain)
2013-03-17 12:23 UTC, LoneVVolf
Details
Possible patch (1.02 KB, patch)
2013-03-17 14:47 UTC, Andreas Boll
Details | Splinter Review

Description LoneVVolf 2013-03-17 11:45:32 UTC
Upon starting x loading r600_dri.so fails

first bad commit : http://cgit.freedesktop.org/mesa/mesa/commit/?id=f70c3853513637fa6ed38e75f73d472a9fa61213

-------------------------------------------------------------
  ./autogen.sh --prefix=/usr \
    --sysconfdir=/etc \
    --with-dri-driverdir=/usr/lib/xorg/modules/dri \
    --with-gallium-drivers=r300,r600,radeonsi,swrast \
    --with-dri-drivers=swrast \
    --enable-gallium-llvm \
    --enable-egl \
    --enable-gallium-egl \
    --with-egl-platforms=x11,drm,wayland \
    --enable-shared-glapi \
    --enable-gbm \
    --enable-glx-tls \
    --enable-dri \
    --enable-glx \
    --enable-osmesa \
    --enable-gles1 \
    --enable-gles2 \
    --enable-texture-float \
    --enable-xa \
    --enable-vdpau \
    --enable-gallium-gbm \
    --enable-gallium-g3dvl \
    --enable-xvmc \
    --enable-xorg \
    --enable-r600-llvm-compiler \
    --with-llvm-shared-libs

---------------------------------------------------
I've chosen blocker as severity, as this commit is a candidate for the 9.1 branch
Comment 1 Maarten Lankhorst 2013-03-17 11:48:15 UTC
Can you be more specific on how it fails?
Comment 2 LoneVVolf 2013-03-17 12:22:25 UTC
X can't initialize the driver and falls back to software rastering.

excerpt from Xorg.0.log :

[ 12609.742] (EE) AIGLX error: dlopen of /usr/lib/xorg/modules/dri/r600_dri.so failed (/usr/lib/libllvmradeon9.2.0.so: undefined symbol: lp_build_tgsi_intrinsic)
[ 12609.742] (EE) AIGLX: reverting to software rendering
[ 12609.742] (II) AIGLX: Screen 0 is not DRI capable
[ 12609.746] (II) AIGLX: Loaded and initialized swrast


additional info :

OS : Archlinux x86_64
kernel 3.8.3 , https://www.archlinux.org/packages/testing/x86_64/linux/
xf86-video-ati 7.1.0, https://www.archlinux.org/packages/testing/x86_64/xf86-video-ati/

I'll attach the build output and complete Xorg.0.log
Comment 3 LoneVVolf 2013-03-17 12:22:59 UTC
Created attachment 76641 [details]
build output
Comment 4 LoneVVolf 2013-03-17 12:23:25 UTC
Created attachment 76642 [details]
complete Xorg.0.log
Comment 5 Andreas Boll 2013-03-17 14:47:28 UTC
Created attachment 76652 [details] [review]
Possible patch

Does this patch help?
Comment 6 Andreas Boll 2013-03-17 14:57:00 UTC
Tom, could you look at this bug?
Comment 7 LoneVVolf 2013-03-17 19:13:36 UTC
yes, the patch in comment #5 solves the problem.

Looking at the 'bad' commit, it seems the changes in 
src/gallium/auxiliary/Makefile.am required a change in the 
src/gallium/drivers/radeon/Makefile.am ?

If so, i wonder if other drivers then radeon are affected also.
The patch looks more as a workaround then as a real solution.
Comment 8 Michel Dänzer 2013-03-18 11:40:23 UTC
(In reply to comment #5)
> Possible patch

This seems less than ideal, as it makes libllvmradeon duplicate lots of stuff from the driver *.so, and it grows from ~500K to ~10M for me.
Comment 9 LoneVVolf 2013-03-18 12:40:24 UTC
That appears to be correct,
in a git version build on march 11
libllvmradeon9.2.0.so is just 39 Kib .
Build with the patch it's size increases to 1.8 Mib !
Comment 10 Andreas Boll 2013-03-18 13:08:52 UTC
(In reply to comment #8)
> (In reply to comment #5)
> > Possible patch
> 
> This seems less than ideal, as it makes libllvmradeon duplicate lots of
> stuff from the driver *.so, and it grows from ~500K to ~10M for me.

Hmm we definitely need a shared libgallium.
That would reduce the size of all gallium drivers.

Or we could remove/revert the visibility flags from src/gallium/auxiliary/Makefile.am
Comment 11 Maarten Lankhorst 2013-03-18 14:19:23 UTC
The fix is correct, despite the size increase.
Comment 12 Maarten Lankhorst 2013-03-18 15:38:38 UTC
Reverting the gallium fix would reopen bug #59238
Comment 13 Fabio Pedretti 2013-03-18 17:10:07 UTC
Ubuntu is providing a shared libgallium, the patches are here:
http://anonscm.debian.org/gitweb/?p=pkg-xorg/lib/mesa.git;a=tree;f=debian/patches;hb=refs/heads/ubuntu%2B1
Comment 14 LoneVVolf 2013-03-18 19:54:22 UTC
Trying to summarize

I've looked a bit more at bug #59238, and if i understand correctly the problem with building xa tracker was that symbols were exported that are not supposed to be visible outside of the library itself ?

By adding the VISIBILITY_CFLAGS and VISIBILITY_CXXFLAGS changes in the f70c385 commit, the troublesome symbols were removed.
A consequence however was that symbols needed for r600_dri.so to call libllvmradeon9.2.0.so were no longer present.

Including $(top_builddir)/src/gallium/auxiliary/libgallium.la in src/gallium/drivers/radeon/Makefile.am makes sure all needed symbols are present, but has a size increase of libllvmradeon9.2.0 as downside.

To counter the size increase, switching to a shared libgallium is proposed.
Comment 15 Michel Dänzer 2013-03-19 10:04:10 UTC
Can you please fix this somehow for now? This is preventing radeonsi from working at all.
Comment 16 Andreas Boll 2013-03-19 10:58:22 UTC
(In reply to comment #15)
> Can you please fix this somehow for now? This is preventing radeonsi from
> working at all.

I've sent the proposed patch from comment #5 to the list:

http://lists.freedesktop.org/archives/mesa-dev/2013-March/036423.html
Comment 17 Michel Dänzer 2013-03-19 11:26:12 UTC
(In reply to comment #12)
> Reverting the gallium fix would reopen bug #59238

FWIW, that seems less of an issue to me than this bug or the side effects of the attached patch.
Comment 18 Maarten Lankhorst 2013-03-20 08:45:05 UTC
The size increase only happens if radeonsi is built too, and can be countered with the shared libgallium patch.

The original bug described here is fixed with these 2 commits:

36320bfa54b758b34 - radeon/llvm: Link against libgallium.la to fix an undefined symbol
7c3d8301afed46cf9 - radeon/llvm: Do not link against libgallium when building statically.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.