On Tue, Mar 12, 2013 at 12:40:30AM +0100, Daniel Vetter wrote: > On Fri, Mar 08, 2013 at 10:45:52AM -0800, Jesse Barnes wrote: > > The Gunit has a separate reg for this, so allocate some stolen space for > > the power context and initialize the reg. > > Is it a requirement to use stolen space? The reason we're allocating rings > and all from stolen is just to test things a bit better and ensure we have > a decent chance that stolen handling for fastboot works ... Some comment > somewhere would be good to clarify this. I'm in favour of making these small permanent allocations from stolen whereever possible. But you are right to point out that we need to clearly mark anthing that may require physically contiguous memory, just in case. -Chris -- Chris Wilson, Intel Open Source Technology Centre