<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 10, 2015 at 11:17 AM, Anuj Phogat <span dir="ltr"><<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Many piglit tests (if using fast copy blit in Mesa) failed earlier<br>
because I missed adding this condition. Fast copy blit is currently<br>
enabled for use only with Yf/Ys tiling.<br>
<br>
Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>><br>
---<br>
src/mesa/drivers/dri/i965/intel_blit.c | 6 ++++--<br>
1 file changed, 4 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/src/mesa/drivers/dri/i965/intel_blit.c b/src/mesa/drivers/dri/i965/intel_blit.c<br>
index 6d92580..bab7d90 100644<br>
--- a/src/mesa/drivers/dri/i965/intel_blit.c<br>
+++ b/src/mesa/drivers/dri/i965/intel_blit.c<br>
@@ -431,8 +431,10 @@ can_fast_copy_blit(struct brw_context *brw,<br>
if ((dst_offset | src_offset) & 63)<br>
return false;<br>
<br>
- /* Color depth greater than 128 bits not supported. */<br>
- if (cpp > 16)<br>
+ /* Color depths which are not power of 2 or greater than 128 bits are<br>
+ * not supported.<br>
+ */<br>
+ if (!is_power_of_two(cpp) || cpp > 16)<br></blockquote><div> This function got deleted on master. I'll change it to _mesa_is_pow_two().</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
return false;<br>
<br>
/* For Fast Copy Blits the pitch cannot be a negative number. So, bit 15<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.4.3<br>
<br>
</font></span></blockquote></div><br></div></div>