On Thu, 8 Jul 2010 09:01:44 -0700
Jesse Barnes <[email protected]> wrote:

> On Thu, 8 Jul 2010 10:58:21 +0100
> Simon Farnsworth <[email protected]> wrote:
> 
> > On Wednesday 7 July 2010, Jesse Barnes <[email protected]> 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
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to