lkml.org 
[lkml]   [2011]   [Jul]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: i915_gem_inactive_shrink fault
From
Date
On Mon, 2011-07-11 at 04:27 +0800, Keith Packard wrote:
> On Tue, 05 Jul 2011 16:59:24 +0800, Lin Ming <ming.m.lin@intel.com> wrote:
>
>
> > WARNING: at /linux-2.6/drivers/gpu/drm/i915/intel_display.c:826 assert_fdi_tx+0xcc/0xd8()
> > Hardware name: To be filled by O.E.M.
> > FDI TX state assertion failure (expected off, current on)
> > Modules linked in:
> > Pid: 1, comm: swapper Not tainted 3.0.0-rc6-tip-mlin+ #571
> > Call Trace:
> > [<ffffffff81037802>] warn_slowpath_common+0x7e/0x96
> > [<ffffffff810378ae>] warn_slowpath_fmt+0x41/0x43
> > [<ffffffff813637c4>] assert_fdi_tx+0xcc/0xd8
> > [<ffffffff8136f68a>] ironlake_crtc_disable+0x3de/0xdbe
>
> That's weird -- this test happens after the FDI has been explicitly
> disabled.
>
> > [drm:init_ring_common] *ERROR* failed to set render ring head to zero ctl ffffffff head ffffffff tail ffffffff start ffffffff
> > [drm:init_ring_common] *ERROR* render ring initialization failed ctl ffffffff head ffffffff tail ffffffff start ffffffff
> > [drm:init_ring_common] *ERROR* failed to set bsd ring head to zero ctl ffffffff head ffffffff tail ffffffff start ffffffff
> > [drm:init_ring_common] *ERROR* bsd ring initialization failed ctl ffffffff head ffffffff tail ffffffff start ffffffff
> > [drm:i915_driver_load] *ERROR* failed to init modeset
>
> That looks like the hardware just cannot be mapped at all.
>
> > BUG: unable to handle kernel NULL pointer dereference at 0000000000000060
>
> > [<ffffffff81355191>] i915_gem_inactive_shrink+0x2b/0x18d
>
> Failing to load, the i915 driver isn't cleaning up and is leaving its
> shrinker in place. That part, at least, should be easy to fix. That
> doesn't explain why the driver cannot access the hardware.
>
> Here's a patch which cleans up a bunch of failure-path issues. It should
> prevent the BUG, but it won't help load the driver on your hardware. I'd
> like to see this patch tested and then we can go figure out why your
> hardware doesn't work with the driver.

Tested and it fixes the fault.

Thanks,
Lin Ming

>
> From 138df88a83f36e3ba8709d7a8a88d3b536dbc70b Mon Sep 17 00:00:00 2001
> From: Keith Packard <keithp@keithp.com>
> Date: Sun, 10 Jul 2011 13:12:17 -0700
> Subject: [PATCH] drm/i915: Clean up i915_driver_load failure path
>
> i915_driver_load adds a write-combining MTRR region for the GTT
> aperture to improve memory speeds through the aperture. If
> i915_driver_load fails after this, it would not have cleaned up the
> MTRR. This shouldn't cause any problems, except for consuming an MTRR
> register. Still, it's best to clean up completely in the failure path,
> which is easily done by calling mtrr_del if the mtrr was successfully
> allocated.
>
> i915_driver_load calls i915_gem_load which register
> i915_gem_inactive_shrink. If i915_driver_load fails after calling
> i915_gem_load, the shrinker will be left registered. When called, it
> will access freed memory and crash. The fix is to unregister the shrinker in the
> failure path using code duplicated from i915_driver_unload.
>
> After failing to initialize the GTT (which cannot happen, btw,
> intel_gtt_get returns a fixed (non-NULL) value), it tries to free the
> uninitialized WC IO mapping. Fixed this by changing the target from
> out_iomapfree to out_rmmap
>
> Signed-off-by: Keith Packard <keithp@keithp.com>
> ---
> drivers/gpu/drm/i915/i915_dma.c | 14 +++++++++++---
> 1 files changed, 11 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c
> index e178702..296fbd6 100644
> --- a/drivers/gpu/drm/i915/i915_dma.c
> +++ b/drivers/gpu/drm/i915/i915_dma.c
> @@ -1943,7 +1943,7 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags)
> if (!dev_priv->mm.gtt) {
> DRM_ERROR("Failed to initialize GTT\n");
> ret = -ENODEV;
> - goto out_iomapfree;
> + goto out_rmmap;
> }
>
> agp_size = dev_priv->mm.gtt->gtt_mappable_entries << PAGE_SHIFT;
> @@ -1987,7 +1987,7 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags)
> if (dev_priv->wq == NULL) {
> DRM_ERROR("Failed to create our workqueue.\n");
> ret = -ENOMEM;
> - goto out_iomapfree;
> + goto out_mtrrfree;
> }
>
> /* enable GEM by default */
> @@ -2074,13 +2074,21 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags)
> return 0;
>
> out_gem_unload:
> + if (dev_priv->mm.inactive_shrinker.shrink)
> + unregister_shrinker(&dev_priv->mm.inactive_shrinker);
> +
> if (dev->pdev->msi_enabled)
> pci_disable_msi(dev->pdev);
>
> intel_teardown_gmbus(dev);
> intel_teardown_mchbar(dev);
> destroy_workqueue(dev_priv->wq);
> -out_iomapfree:
> +out_mtrrfree:
> + if (dev_priv->mm.gtt_mtrr >= 0) {
> + mtrr_del(dev_priv->mm.gtt_mtrr, dev->agp->base,
> + dev->agp->agp_info.aper_size * 1024 * 1024);
> + dev_priv->mm.gtt_mtrr = -1;
> + }
> io_mapping_free(dev_priv->mm.gtt_mapping);
> out_rmmap:
> pci_iounmap(dev->pdev, dev_priv->regs);
> --
> 1.7.5.4
>
>




\
 
 \ /
  Last update: 2011-07-11 03:11    [W:0.084 / U:1.900 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site