New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 835959 link

Starred by 7 users

Issue metadata

Status: Available
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Make sure Xwayland loads swrast_dri.so from the proper location

Project Member Reported by chirantan@chromium.org, Apr 23 2018

Issue description

Right now mesa hardcodes the location for the swrast_dri.so library to be /usr/lib64/dri.  This is because we set it to that location in the configure step of the mesa ebuild (--with-dri-driverdir).  We should make sure that mesa instead loads this library from the /opt/google location.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 23 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/containers/cros-container-guest-tools/+/fc405aaa878f99554876b312bee1d930ff8cdc94

commit fc405aaa878f99554876b312bee1d930ff8cdc94
Author: Chirantan Ekbote <chirantan@chromium.org>
Date: Mon Apr 23 20:23:58 2018

cros-sommelier-config: Add symlinks for swrast_dri.so

Xwayland hardcodes the search location for swrast_dri.so to be
/usr/lib64/dri/swrast_dri.so since that directory is where all the dri
drivers for chrome os are installed.

Have the cros-sommelier-config package install a symlink in this
location that points back to the /opt/google location where the library
is actually installed.

This should be reverted once we have a proper fix for mesa to load the
library from the right location.

BUG=chromium:835959
TEST=`glxinfo` works and `xdpyinfo | grep GLX` shows output inside the
     container

Change-Id: I0cae0249a01be37468e75e0dce46395d5036633a
Signed-off-by: Chirantan Ekbote <chirantan@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1024878

[modify] https://crrev.com/fc405aaa878f99554876b312bee1d930ff8cdc94/cros-sommelier-config/prerm
[modify] https://crrev.com/fc405aaa878f99554876b312bee1d930ff8cdc94/cros-sommelier-config/postinst

Project Member

Comment 2 by bugdroid1@chromium.org, Apr 24 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/ded52d335490f1c84c3754c5bfb1c231a3819cf9

commit ded52d335490f1c84c3754c5bfb1c231a3819cf9
Author: Chirantan Ekbote <chirantan@chromium.org>
Date: Tue Apr 24 01:09:21 2018

termina_container_tools: Pull in swrast_dri.so

Xwayland uses dlopen to open swrast_dri.so.  This means that lddtree
doesn't know that it needs to be pulled in.  Explicitly add that library
and its dependencies to the list of packages installed in the final
directory.

BUG=chromium:835959
TEST=`glxinfo` and `xdpyinfo | grep GLX` both work inside the container

Change-Id: I8191923115e10bfe283cac62c8ac5c3cc9fd6bef
Signed-off-by: Chirantan Ekbote <chirantan@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1024951
Reviewed-by: David Reveman <reveman@chromium.org>

[rename] https://crrev.com/ded52d335490f1c84c3754c5bfb1c231a3819cf9/chromeos-base/termina_container_tools/termina_container_tools-0.0.1-r4.ebuild
[modify] https://crrev.com/ded52d335490f1c84c3754c5bfb1c231a3819cf9/chromeos-base/termina_container_tools/termina_container_tools-0.0.1.ebuild

Owner: davidri...@chromium.org
https://chromium-review.googlesource.com/c/chromiumos/platform2/+/1403896 should fix this for both swrast and virtio_gpu which point the other changes should be revertable.

Sign in to add a comment