<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 31, 2016 at 8:17 AM, Topi Pohjolainen <span dir="ltr"><<a href="mailto:topi.pohjolainen@intel.com" target="_blank">topi.pohjolainen@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Otherwise once mcs buffer gets allocated without delay for<br>
lossless compression (same as we do for msaa), assert starts<br>
to fire in piglit case: tex3d. The test uses depth of one<br>
which is in fact supported even now.<br>
<br>
Signed-off-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><br>
---<br>
 src/intel/isl/isl.c | 7 ++++++-<br>
 1 file changed, 6 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c<br>
index c4989dd..b5c4a8c 100644<br>
--- a/src/intel/isl/isl.c<br>
+++ b/src/intel/isl/isl.c<br>
@@ -1328,7 +1328,12 @@ isl_surf_get_ccs_surf(const struct isl_device *dev,<br>
    assert(surf->samples == 1 && surf->msaa_layout == ISL_MSAA_LAYOUT_NONE);<br>
    assert(ISL_DEV_GEN(dev) >= 7);<br>
<br>
-   assert(surf->dim == ISL_SURF_DIM_2D);<br>
+   if (ISL_DEV_GEN(dev) < 8) {<br>
+      assert(surf->dim == ISL_SURF_DIM_2D);<br>
+   } else {<br>
+      assert(surf->dim == ISL_SURF_DIM_2D || surf->dim == ISL_SURF_DIM_3D);<br>
+   }<br></blockquote><div><br></div><div>What about 1D?  Maybe we just never try to allocate 1D Y-tiled.  Also, if it's 3D, should we assert depth == 1?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+<br>
    assert(surf->logical_level0_<wbr>px.depth == 1);<br>
<br>
    /* TODO: More conditions where it can fail. */<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.5.5<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>