[Intel-gfx] [PATCH] drm/agp/i915: trim stolen space to 32M

[Jesse Barnes]

On Thu, 8 Jul 2010 09:01:44 -0700
Jesse Barnes <jbarnes at virtuousgeek.org> wrote:

> On Thu, 8 Jul 2010 10:58:21 +0100
> Simon Farnsworth <simon.farnsworth at onelan.com> wrote:
> 
> > On Wednesday 7 July 2010, Jesse Barnes <jbarnes at virtuousgeek.org> wrote:
> > > Some BIOSes will claim a large chunk of stolen space.  Unless we
> > > reclaim it, our aperture for remapping buffer objects will be
> > > constrained.  So clamp the stolen space to 32M and ignore the rest.
> > > 
> > I'm not sure that this changelog fits the patch - if I'm understanding the code 
> > correctly, you're clamping to 16M, not 32M.
> > 
> > Apart from that, the code looks sensible.
> 
> Oops updated to 32M for Ironlake but didn't update the changelog or
> comments.

Oh and I see you were trying to tell me that I failed to update the
code too.  Sigh.

I'll post an updated version with the code and comments fixed.

-- 
Jesse Barnes, Intel Open Source Technology Center