From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Hounschell Subject: Re: [Intel-gfx] Intel graphics drm issue? Date: Sun, 14 Oct 2012 07:39:21 -0400 Message-ID: <507AA469.5070100@cfl.rr.com> References: <507812E8.3060607@compro.net> <20121012231459.166e29fd@neptune.home> <5079B97C.7020401@cfl.rr.com> <5079BE5C.8020401@cfl.rr.com> <507A91EA.2030205@cfl.rr.com> Reply-To: dmarkh@cfl.rr.com Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Daniel Vetter Cc: markh@compro.net, =?ISO-8859-1?Q?Bruno_Pr=E9mont?= , Intel Graphics Development , linux-kernel@vger.kernel.org List-Id: intel-gfx@lists.freedesktop.org On 10/14/2012 07:26 AM, Daniel Vetter wrote: > On Sun, Oct 14, 2012 at 12:20 PM, Mark Hounschell wrote: >> Hi Daniel, >> >> cat /proc/cmdline >> root=/dev/disk/by-id/ata-INTEL_SSDSC2CW060A3_CVCV205106EB060AGN-part4 >> video=1024x768 noresume splash=silent quiet apm=off nomodeset vga=normal >> drm.debug=0xe >> >> dmesg is attached > >>>From the dmesg: > > [ 1.833195] [drm:drm_pci_agp_init] *ERROR* Cannot initialize the > agpgart module. > > Which is the root-cause preventing i915.ko from loading. Now with 3.7 > we /shouldn't/ all that agp crap any longer on snb/ivb, so I'm a bit > confused why it still tries to initialize it. Can you maybe attach > your .config? > Did you mean 3.6 above? I used 3.6.2? If I'm using the right kernel I will apply the patch and try. Let me know. > Also, we need to employ some dirty runtime tricks to no longer require > agp on newer chips, since we still need to keep it around for > backwards-compat with old userspace on old chips. Can you please check > with the below diff whether this hack works? > > Thanks, Daniel > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index a7837e5..1152502 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -842,6 +842,7 @@ i915_pci_probe(struct pci_dev *pdev, const struct > pci_device_id *ent) > if (intel_info->gen != 3) { > driver.driver_features &= > ~(DRIVER_USE_AGP | DRIVER_REQUIRE_AGP); > + printk("dropping agp requirement\n"); > } else if (!intel_agp_enabled) { > DRM_ERROR("drm/i915 can't work without intel_agp module!\n"); > return -ENODEV; > Doesn't this patch only add a printk? Is that what you want me to do? Mark