<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, May 3, 2017 at 2:22 AM, Topi Pohjolainen <span dir="ltr"><<a href="mailto:topi.pohjolainen@gmail.com" target="_blank">topi.pohjolainen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Signed-off-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><br>
---<br>
 src/intel/isl/isl_gen7.c | 6 +++++-<br>
 1 file changed, 5 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/intel/isl/isl_gen7.c b/src/intel/isl/isl_gen7.c<br>
index 18687b5..cf5b377 100644<br>
--- a/src/intel/isl/isl_gen7.c<br>
+++ b/src/intel/isl/isl_gen7.c<br>
@@ -375,7 +375,11 @@ gen7_choose_valign_el(const struct isl_device *dev,<br>
        * FINISHME(chadv): Decide to set valign=4 or valign=8 after isl's API<br>
        * is more polished.<br>
        */<br>
-      require_valign4 = true;<br>
+<br>
+      /* Using valign 4 upsets all depthstencil-render-miplevels on IVB and<br>
+       * HSW. Use alignment 8 instead.<br>
+       */<br>
+      return 8;<br></blockquote><div><br></div><div>I'm confused.  This seems to contradict the documentation which says that valign for depth is hard-coded to 4 in the hardware.<br></div></div><br></div></div>