From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Kai_Wasserb=c3=a4ch?= Subject: Re: [PATCH 1/5] drm: add interface to get drm devices on the system v2 Date: Fri, 14 Aug 2015 10:26:30 +0200 Message-ID: <55CDA636.1080407@dev.carbon-project.org> References: <1439436825-16908-1-git-send-email-Jammy.Zhou@amd.com> <1439436825-16908-2-git-send-email-Jammy.Zhou@amd.com> <20150813150755.GD17734@phenom.ffwll.local> <55CD9FDB.2010809@dev.carbon-project.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0319435050==" Return-path: Received: from blackmesa.kw-serverwartung.de (blackmesa.kw-serverwartung.de [78.46.90.79]) by gabe.freedesktop.org (Postfix) with ESMTPS id 71F776E035 for ; Fri, 14 Aug 2015 01:26:39 -0700 (PDT) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Emil Velikov Cc: ML dri-devel List-Id: dri-devel@lists.freedesktop.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --===============0319435050== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="6bRQxrVCU5oe9dTWfGHQvPmJPhwDkEJmB" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --6bRQxrVCU5oe9dTWfGHQvPmJPhwDkEJmB Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Emil Velikov wrote on 14.08.2015 10:17: > On 14 August 2015 at 08:59, Kai Wasserb=C3=A4ch wrote: >> 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 o= ptions for example when run Steam games which ships 32bit libraries (incl= uding libudev) in the steam runtime, so finally we decided to use sysfs d= irectly 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 run= time 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). IIR= C the >> VMWare guys said in the Mesa discussion, they have a script in place f= or their >> virtualisation products, that checks whether a library needs to be loa= ded from >> their "baseline directory" or from the system. >> >> Working around a bug/design flaw in Steam's Linux version doesn't soun= d 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 librarie= s. (For me >> it's usually libstdc++ (needed by LLVM), libncurses and a few X(CB) li= braries I >> need to remove from Steam, before anything works. Though I do have scr= ipt for >> that, that I can run after every upgrade, this is not a solution for e= veryone.) >> > Helping and applying pressure to resolve the issue is the way to go. > But until that is resolved it's great to have a solution that does not > lead to a crash. It feels rude towards you and other users to > deliberately use the problematic combo and expect from you to remove > libfoo.so. Well, I'd rather remove stuff from Steam's runtime than burden you and ot= her developers with maintaing code that is unnecessrily ugly. (Though that's obviously just my opinion.) > When things get sorted out, we can easily replace this (a tad ugly > implementation) with libudev. As long as you allow this behaviour by working around it, I don't see Val= ve/game developers "invest" in a real solution (because it works now). Businesses= usually only move from a position, when there's outside pressure and a cl= ear advantage to do so (here: no bug reports about crashing games). Anyway, this was just my two cents and you can obviously decide in any wa= y you deem to be the best. --6bRQxrVCU5oe9dTWfGHQvPmJPhwDkEJmB Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- iQGcBAEBCgAGBQJVzaY7AAoJEKMJ12zh3lnS4wkMAIbpGESwhOuSkXINUcT/VR6e 4SEzVFR9ITPz0SU6MNf4upnYCK8llQUA+y9T8dPrjVXfhdwhoigsrAw/+Z33y7X2 KE/Kq0d8TquK0nOTzpKJnDnMlDILKGxs0C5y9EsFE9O7ElYqvg4R5BIjslrqTJtg 3/P0/+XyhbUcIFs05/f8EoIDrBxXVO4zp3sU9/M2ga+WV53wnjFJUOTrx99LMGwl UICjj6TibV4Olhy/WsEp6WFyDHuOSnxwJGe8EBWM/2nivdEAD2NH8HWotzRnosdN wVzNAI6x8bGEVcetGsrU1omuo03tmkZiBWx7G9lwyf4JBxXOTnMOBf4AuRvK9RJ2 qgDKxSez2dw74jkovU2yASpKpRO7EeSWhIuv4tjz2OOTf4aXg5tjBendcq5h3B7K 2gy2NiGKMM/cxEu4HZXJaGVGlS9aaVbMbd7f5LQprLV1wzdy7EPmzbDc7VGz2Syj PmA+idO+afhphr+pYHYoFkcmKhxlEyNhR3M7TgzoZA== =6Tfv -----END PGP SIGNATURE----- --6bRQxrVCU5oe9dTWfGHQvPmJPhwDkEJmB-- --===============0319435050== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0 cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK --===============0319435050==--