<div dir="ltr">On 4 December 2013 15:07, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br></div></div>
bugget, verticeally, and vestically! oh my!<br></blockquote><div><br></div><div>Wow, I wish I could say I did all those typos on purpose.  Those are hilarious!<br><br></div><div>They're all fixed now.<br></div><div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Patches 1-6 are<br>
Reviewed-by: Chad Versace <<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>><br>
<br>
But this patch 7... If the user specifies an ill-aligned clear rectangle,<br>
does this code clear a slightly larger, well-aligned rectangle? In other words,<br>
will this clear pixels outside the user-specified clear rectangle? Local<br>
inspection of the code suggests so to me. But my global understanding of<br>
these codepaths is vague.<br>
</blockquote></div><br></div><div class="gmail_extra">Ken is right in his response about this--partial fast clears never happen because of the !partial_clear check, so the extra pixels that get cleared are always outside the bounds of the buffer (and hence irrelevant).  Thanks to tiling, there's no worry about overflowing into memory owned by other regions--the extra pixels that get cleared always belong to tiles that contain in-bounds pixels.  In theory I believe we could do partial clears if we first verified that they were aligned, but (a) the bspec doesn't recommend it, and (b) I doubt there would be much benefit, since sufficiently aligned clears that don't clear the entire buffer are probably quite rare.<br>
<br></div><div class="gmail_extra">I'll follow up with a patch that improves the comments to clarify this.<br></div></div>