Re: Intel 915GM MCHBAR bug

[Pekka Enberg]

Woody Suwalski wrote:
> Andrew Morton wrote:
> > On Fri, 05 Jun 2009 11:02:21 +0300
> > Pekka Enberg <penberg@xxxxxxxxxxxxxx> wrote:
> >
> >  
> > > Hi Jesse,
> > >
> > > (I am cc'ing linux-kernel.)
> > >
> > > On Mon, 01 Jun 2009 22:31:13 +0300
> > >    
> > > > Pekka Enberg <penberg@xxxxxxxxxxxxxx> wrote:
> > > >
> > > >      
> > > > > Hi Jesse,
> > > > >
> > > > > I am seeing this on my Eee PC 701 which is running a Ubuntu 
> > > > > 2.6.28-11.42-generic stock kernel:
> > > > >
> > > > > [   76.826966] [drm:i915_gem_detect_bit_6_swizzle] *ERROR* Couldn't
> > > > > read from MCHBAR.  Disabling tiling.
> > > > > [   76.827215] [drm] Initialized i915 1.6.0 20080730 on minor 0
> > > > > [   76.834037] [drm:i915_setparam] *ERROR* unknown parameter 4
> > > > > [   76.834104] [drm:i915_getparam] *ERROR* Unknown parameter 6
> > > > >
> > > > > Google turned up this patch
> > > > >
> > > > > http://lists.freedesktop.org/archives/intel-gfx/2009-January/001186.html 
> > > > >
> > > > >
> > > > > but I don't seem to find it mainline kernel.
> > > > >
> > > > > Was the bug fixed in some other way? It seem that distributions have
> > > > > not yet picked up your patch and I am unsure if it's in any of the
> > > > > -stable kernels.
> > > > >         
> > > On Thu, 2009-06-04 at 11:27 +0100, Jesse Barnes wrote:
> > >    
> > > > I think Eric acked it but may not have pushed it to drm-intel-next
> > > > yet.  Should happen in the next week or two though as we prepare the
> > > > merge window series.
> > > >       
> > > OK, thanks for the info! FYI, I tested 2.6.30-rc8 with your patch
> > > applied and everything works smoothly on my EeePC 701. I did not test
> > > plain 2.6.30-rc8 as I expect it to show same kind of behavior as Ubuntu
> > > distro kernel. Is this correct or do you want me to test 2.6.30-rc8 too?
> > >
> > > I wonder why the patch hasn't received more attention as it's a pretty
> > > critical bug. The _default_ Ubuntu 9.04 netbook remix installation is
> > > completely broken for EeePC 701 (and probably others as well) without
> > > the patch. And it's not as if I'm the only one that suffers from it. A
> > > quick Google search reveals that a lot of people are hitting it.
> > >
> > > So I really do think we need to merge this patch to the upcoming 2.6.31
> > > ASAP and backport it to -stable after it has gotten some more testing.
> >
> > Wanna show us the patch?
> >
> > Because the world could certainly do with more i915 bugfixes :(
>   
> Here is Jesse's patch from 
> http://lists.freedesktop.org/archives/intel-gfx/2009-January/001186.html 
> adopted for 2.6.30-rc8... (needed to redo hunk #3 for i915_gem_tile.c)

Yup. I took the two patches from here:

https://bugs.launchpad.net/ubuntu/+source/linux/+bug/349314

and did the same rediffing on top of 2.6.30-rc8 for the first one.

Andrew, do you want me to forward you the (tested) patches or should we 
wait for Jesse et al to take care of it?

			Pekka
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/