Patch: SAUCE: Disable acceleration on GeForce 6100

[Christopher James Halse Rogers]

On Tue, 2010-04-13 at 13:16 +0100, Andy Whitcroft wrote: 
> On Tue, Apr 13, 2010 at 09:31:54PM +1000, Christopher James Halse Rogers wrote:
> > This extra quirk applies on the top of the previous 3 patches.
> > 
> > Subject: [PATCH] UBUNTU: SAUCE: Nouveau: Disable acceleration on 6100
> > cards
> > 
> > Upstream obviously wants to fix acceleration rather than disable it.
> > For Lucid however we need these cards to boot more than we need them to
> > be fast.
> > 
> > With acceleration enabled these cards hang during boot.
> > 
> > BugLink: http://bugs.launchpad.net/bugs/542950
> > 
> > Signed-off-by: Christopher James Halse Rogers
> > <christopher.halse.rogers at canonical.com>
> > 
> > 
> 
> > From 9532aca5caedaf43086b68c1352422c727373f46 Mon Sep 17 00:00:00 2001
> > From: Christopher James Halse Rogers <raof at ubuntu.com>
> > Date: Tue, 13 Apr 2010 19:36:39 +1000
> > Subject: [PATCH] UBUNTU: SAUCE: Nouveau: Disable acceleration on 6100 cards
> > 
> > Upstream obviously wants to fix acceleration rather than disable it.
> > For Lucid however we need these cards to boot more than we need them to
> > be fast.
> > 
> > With acceleration enabled these cards hang during boot.
> > 
> > BugLink: http://bugs.launchpad.net/bugs/542950
> > 
> > Signed-off-by: Christopher James Halse Rogers <christopher.halse.rogers at canonical.com>
> > ---
> >  drivers/gpu/drm/nouveau/nouveau_state.c |    7 +++++++
> >  1 files changed, 7 insertions(+), 0 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/nouveau/nouveau_state.c b/drivers/gpu/drm/nouveau/nouveau_state.c
> > index 63014f4..a112998 100644
> > --- a/drivers/gpu/drm/nouveau/nouveau_state.c
> > +++ b/drivers/gpu/drm/nouveau/nouveau_state.c
> > @@ -621,6 +621,13 @@ static void nouveau_apply_noaccel_quirks (struct drm_device *dev)
> >  			NV_INFO(dev, "Detected NV20 (GeForce 3) chip. "
> >  				 "Disabling acceleration\n");
> >  		}
> > +		/* GeForce 6100 cards also hang with acceleration */
> > +		/* See https://bugs.launchpad.net/bugs/542950 */
> > +		if (dev->pdev->device == 0x0242) {
> > +			nouveau_noaccel = 1;
> > +			NV_INFO(dev, "Detected GeForce 6100 chip. "
> > +				 "Disabling acceleration\n");
> > +		}
> >  	}
> >  }
> 
> This list is getting comprehensive enough that we should look at making
> this into some kind of list if we are going to get any more of these.

I was thinking of that when I wrote the initial patch, but decided
against it because the first two quirks relied on different data (macs
are quirked against pciid, geforce 3 is a broader quirk against chipset
family).

If this list grows any longer I think it'll become obvious what data
needs to be stored for the quirk list. 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: This is a digitally signed message part
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20100413/93cba8bd/attachment.sig>