<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jul 28, 2017 at 8:17 PM, Tomasz Figa <span dir="ltr"><<a href="mailto:tfiga@chromium.org" target="_blank">tfiga@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Topi, Jason,<br>
<span class=""><br>
On Sat, Jul 22, 2017 at 12:00 AM, Topi Pohjolainen<br>
<<a href="mailto:topi.pohjolainen@gmail.com">topi.pohjolainen@gmail.com</a>> wrote:<br>
> First patch actually should have been included already when<br>
> gen6 stencil got transitioned - it has been giving warning ever<br>
> since.<br>
><br>
> Most of the work actually got already done for depth surfaces (which<br>
> is y-tiled such as color surfaces). What is left are color surface<br>
> specifics, mostly preparing for corner cases.<br>
><br>
> This is now all green in ci-system. For snb and older i965 wasn't<br>
> checking hardware incapabilities as hard as isl does. Certain<br>
> format/size/msaa combinations were allowed that shouldn't have.<br>
> Moving to isl exposed code paths that didn't report surface creation<br>
> failures resulting in asserts firing later on. Patches 10 and 11<br>
> now properly tell the client if the surface type can't be supported<br>
> allowing piglit tests to skip them.<br>
<br>
</span>I think it might be related to previously merged patches, but the<br>
topic is still ISL, so let me ask my question here. Is there any<br>
possibility to add some diagnostic information to the validation code?<br>
We've been seeing EGL image import failures on new Mesa as a result of<br>
ISL catching issues in our allocator (cros_gralloc on top of ChromeOS<br>
minigbm), but it's close to impossible to identify the cause without<br>
manually inserting some printfs and recompiling the code. I think<br>
having some error messages printed in case of a buffer validation<br>
failure would be a great benefit.<br></blockquote><div><br></div><div>What kind of prints are you looking for exactly?  I've run into some issues myself but if you could provide a concrete example, that would help.<br><br></div><div>--Jason<br></div></div></div></div>