From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Wu Subject: Re: Stupid NVIDIA 3D vgaarb.c patch Date: Tue, 23 Sep 2014 16:22:17 +0200 Message-ID: <20771253.xd3Bmja4Qg@al> References: Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: C =?ISO-8859-1?Q?Bergstr=F6m?= Cc: Linus Torvalds , David Airlie , Alex Williamson , Benjamin Herrenschmidt , DRI , Linux Kernel Mailing List List-Id: dri-devel@lists.freedesktop.org On Tuesday 23 September 2014 03:52:48 C Bergstr=F6m wrote: > Here's where I originally found it > https://github.com/Bumblebee-Project/Bumblebee/issues/159 > (Adding Peter to cc chain) >=20 > I guess there's already a bug id and some (snarky?) comments > https://bugzilla.kernel.org/show_bug.cgi?id=3D63641 > ------- > This is for Aorus X3 laptop with NVIDIA GTX 870m card >=20 > On Tue, Sep 23, 2014 at 3:43 AM, Linus Torvalds < > torvalds@linux-foundation.org> wrote: >=20 > > Adding proper people and mailing lists.. > > > > The PCI_CLASS_DISPLAY_VGA test goes back to the very beginning by > > BenH, and I have no idea if adding PCI_CLASS_DISPLAY_3D is > > appropriate, but hopefully somebody does. The fact that it makes > > things work certainly argues fairly convincingly for it, but I want > > some backup here. > > > > Dave, BenH? > > > > Christopher(?), can you please also specify which laptop etc. And t= he > > patch itself seems to have come from somebody else, unless you're > > Lekensteyn. So we'd want to get the provenance of that too. If I understood this correctly, VGA arbitration is used to deal with mu= ltiple users of a fixed IO memory address. I don't know whether modern softwar= e still use it, but if they do, then it probably makes sense to include the 3D controller class as some Nvidia graphics card are configured with this = class. Unfortunately I cannot provide more details than this as I don't know t= he specifics. Feel free to take this patch and make it suitable for inclus= ion. I don't have this kind of hardware, but mr. C is certainly not the only o= ne with this. Kind regards, Peter > > Linus > > > > On Mon, Sep 22, 2014 at 1:28 PM, C Bergstr=F6m > > wrote: > > > Hi Linus, > > > > > > I don't know who the original author is and I can have one of the= distro > > > graphics friends review it, but I need this patch below to get NV= IDIA > > > drivers to work (without using any Intel graphics) on my laptop > > > http://pastebin.com/wpmFi38k > > > > > > Sorry - I know this is not the proper way to submit a patch, but = it's > > > trivial and I'm not a kernel dev. > > > > > > Thanks > > > > > > ./C > > --=20 Kind regards, Peter https://lekensteyn.nl