<div dir="auto"><div><div class="gmail_extra"><div class="gmail_quote">On Feb 20, 2017 4:44 AM, "Lionel Landwerlin" <<a href="mailto:lionel.g.landwerlin@intel.com">lionel.g.landwerlin@intel.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Applies on top of Jason's patch :<br>
<br>
   <a href="https://patchwork.freedesktop.org/patch/139603/" rel="noreferrer" target="_blank">https://patchwork.freedesktop.<wbr>org/patch/139603/</a><br>
<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>
 src/intel/isl/isl.c | 3 +++<br>
 1 file changed, 3 insertions(+)<br>
<br>
diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c<br>
index 1a47da5257..3b4579b343 100644<br>
--- a/src/intel/isl/isl.c<br>
+++ b/src/intel/isl/isl.c<br>
@@ -1417,6 +1417,9 @@ isl_surf_get_mcs_surf(const struct isl_device *dev,<br>
    assert(surf->levels == 1);<br>
    assert(surf->logical_level0_<wbr>px.depth == 1);<br>
<br>
+   if (ISL_DEV_GEN(dev) == 9 && surf->samples == 16 && surf->width > 8192)<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">The gen check isn't needed as 16x MSAA isn't supported an any earlier hardware.  Also, a PRM citation would be good.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+      return false;<br>
+<br>
    enum isl_format mcs_format;<br>
    switch (surf->samples) {<br>
    case 2:  mcs_format = ISL_FORMAT_MCS_2X;  break;<br>
<font color="#888888">--<br>
2.11.0<br>
<br>
</font></blockquote></div><br></div></div></div>