Re: [PATCH 1/5] drm: add interface to get drm devices on the system v2

From: "Kai Wasserbäch" <kai@dev.carbon-project.org>
To: "Zhou, Jammy" <Jammy.Zhou@amd.com>,
	Emil Velikov <emil.l.velikov@gmail.com>,
	Daniel Vetter <daniel@ffwll.ch>
Cc: ML dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 1/5] drm: add interface to get drm devices on the system v2
Date: Fri, 14 Aug 2015 09:59:23 +0200	[thread overview]
Message-ID: <55CD9FDB.2010809@dev.carbon-project.org> (raw)
In-Reply-To: <FD5696F5CDD4B04D92C97DB96167DECA16222BE2@SCYBEXDAG04.amd.com>

[-- Attachment #1.1: Type: text/plain, Size: 1511 bytes --]

Zhou, Jammy wrote on 14.08.2015 07:59:
> We tried several different ways already for the enumeration interface (libpciaccess, libudev, etc). But we ran into some problems with these options for example when run Steam games which ships 32bit libraries (including libudev) in the steam runtime, so finally we decided to use sysfs directly to avoid introducing some additional dependencies into libdrm.

The reason sounds wrong. There was a similar discussion over at Mesa. I think
you (as in hardware/driver vendors like AMD/Intel/Nvidia) need to push Valve (or
the game devs through Valve or directly) to fix their setup. Steam runtime is
fine and all, but please only pre-load it, if needed (ie. library foo is missing
on the system and can't be installed through the package manager). IIRC the
VMWare guys said in the Mesa discussion, they have a script in place for their
virtualisation products, that checks whether a library needs to be loaded from
their "baseline directory" or from the system.

Working around a bug/design flaw in Steam's Linux version doesn't sound like a
supportable solution in the long run. As long as you let them get away with
that, you will face this problem over and over with different libraries. (For me
it's usually libstdc++ (needed by LLVM), libncurses and a few X(CB) libraries I
need to remove from Steam, before anything works. Though I do have script for
that, that I can run after every upgrade, this is not a solution for everyone.)

Cheers,
Kai

[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 630 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel