<div dir="ltr"><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br><br></div>And pushed<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 24, 2016 at 9:52 AM, Lionel Landwerlin <span dir="ltr"><<a href="mailto:llandwerlin@gmail.com" target="_blank">llandwerlin@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We need to compute detiling coordinates using the physical size of W tiling<br>
(128x32) rather than the logical size (64x64).<br>
<br>
v2: Correct comment (Jason)<br>
<span class=""><br>
Fixes dEQP-VK.api.copy_and_blit.<wbr>image_to_image_stencil<br>
<br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=97448" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=97448</a><br>
Signed-off-by: Lionel Landwerlin <<a href="mailto:lionel.g.landwerlin@intel.com">lionel.g.landwerlin@intel.com</a><wbr>><br>
Cc: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
---<br>
</span> src/intel/vulkan/anv_meta_<wbr>blit2d.c | 4 +++-<br>
 1 file changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/intel/vulkan/anv_meta_<wbr>blit2d.c b/src/intel/vulkan/anv_meta_<wbr>blit2d.c<br>
index 7085f2d..6b2222c 100644<br>
--- a/src/intel/vulkan/anv_meta_<wbr>blit2d.c<br>
+++ b/src/intel/vulkan/anv_meta_<wbr>blit2d.c<br>
@@ -914,9 +914,11 @@ build_nir_w_tiled_fetch(struct nir_builder *b, struct anv_device *device,<br>
<span class="">    /* First, compute the block-aligned offset */<br>
    nir_ssa_def *x_major = nir_ushr(b, x, nir_imm_int(b, 6));<br>
    nir_ssa_def *y_major = nir_ushr(b, y, nir_imm_int(b, 6));<br>
</span>+   /* W tiles have physical size of 128x32 and logical size of 64x64, hence<br>
+    * the multiplication by 32 (instead of 64). */<br>
<div class="HOEnZb"><div class="h5">    nir_ssa_def *offset =<br>
       nir_iadd(b, nir_imul(b, y_major,<br>
-                              nir_imul(b, tex_pitch, nir_imm_int(b, 64))),<br>
+                              nir_imul(b, tex_pitch, nir_imm_int(b, 32))),<br>
                   nir_imul(b, x_major, nir_imm_int(b, 4096)));<br>
<br>
    /* Compute the bottom 12 bits of the offset */<br>
--<br>
2.9.3<br>
<br>
</div></div></blockquote></div><br></div>