On Thu, 08 Jul 2010 17:42:19 +0100 Chris Wilson <[email protected]> wrote:
> On Thu, 8 Jul 2010 17:37:10 +0100, Simon Farnsworth > <[email protected]> wrote: > > On Thursday 8 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. > > > > > > Fixes https://bugzilla.kernel.org/show_bug.cgi?id=15469 among others. > > > > > > Adding the ignored stolen memory back into the general pool using the > > > memory hotplug code is left as an exercise for the reader. > > > > > > Signed-off-by: Jesse Barnes <[email protected]> > > > > FWIW, given how simple the code actually is: > > Reviewed-by: Simon Farnsworth <[email protected]> > > Heh, you were meant to say: > > "What? Export a magic parameter from intel-gtt.c to indicate how much > space to remove from the aperture guestimate, but not replace the *broken* > code in i915_dma.c with an interface from intel-gtt.c to query and control > the populated GTT more precisely?" Yeah, that's what this patch does. It'll all be killed by Daniel's AGP rework though! -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
