<html><head><meta http-equiv="Content-Type" content="text/html; charset=UTF-8"></head><body ><div>Yes, without your badassness this would have taken me much longer to debug (especially since I was sure I had already tried avoiding stolen). </div><div><br></div><div>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. </div><div><br></div><div><br></div>--<div>Jesse Barnes, Intel Open Source Technology Center</div> <br><br><br>-------- Original message --------<br>From: Ben Widawsky <ben@bwidawsk.net> <br>Date: 05/01/2013 6:00 PM (GMT-08:00) <br>To: Jesse Barnes <jbarnes@virtuousgeek.org> <br>Cc: intel-gfx@lists.freedesktop.org <br>Subject: Re: [Intel-gfx] [PATCH 1/3] drm/i915: don't use stolen space on VLV <br> <br><br>On Wed, May 01, 2013 at 04:09:20PM -0700, Jesse Barnes wrote:<br>> BIOS stuffs things here we can't clobber.<br>> <br>> Signed-off-by: Jesse Barnes <jbarnes@virtuosugeek.org><br>As I said in private email, I think the quirk is a bit better suited in<br>i915_stolen_to_physical(), and the commit message *could* explain things<br>a bit clearer, oh, and you're welcome.<br>Reviewed-by: Ben Widawsky <ben@bwidawsk.net><br><br>-- <br>Ben Widawsky, Intel Open Source Technology Center<br><br></body>