Yes, without your badassness this would have taken me much longer to debug (especially since I was sure I had already tried avoiding stolen).? But I have a better patch to replace this that just makes a preallocated object from the BIOS gunk and lets us use the rest. Seems to be working here, so I'll clean it up and send it tomorrow. Just don't ask about the BIOS gunk; I can't talk about it.? -- Jesse Barnes, Intel Open Source Technology Center -------- Original message -------- From: Ben Widawsky <ben at bwidawsk.net> Date: 05/01/2013 6:00 PM (GMT-08:00) To: Jesse Barnes <jbarnes at virtuousgeek.org> Cc: intel-gfx at lists.freedesktop.org Subject: Re: [PATCH 1/3] drm/i915: don't use stolen space on VLV On Wed, May 01, 2013 at 04:09:20PM -0700, Jesse Barnes wrote: > BIOS stuffs things here we can't clobber. > > Signed-off-by: Jesse Barnes <jbarnes at virtuosugeek.org> As I said in private email, I think the quirk is a bit better suited in i915_stolen_to_physical(), and the commit message *could* explain things a bit clearer, oh, and you're welcome. Reviewed-by: Ben Widawsky <ben at bwidawsk.net> -- Ben Widawsky, Intel Open Source Technology Center -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20130501/c9d72fbb/attachment.html>