<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 27, 2016 at 9:53 AM, Nanley Chery <span dir="ltr"><<a href="mailto:nanleychery@gmail.com" target="_blank">nanleychery@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Apr 26, 2016 at 06:16:12PM -0700, Jason Ekstrand wrote:<br>
> Good catch!<br>
><br>
> Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
><br>
> Why does it cause an assert?<br>
><br>
<br>
</span>Thanks! The test does math on the returned properties struct if the<br>
format query is successful. One of the functions, deLog2Ceil32(),<br>
asserts that properties.maxExtent.width is greater than 0. Since we<br>
returned a zeroed struct for unsupported formats coupled with a<br>
success status, we've been managing to trip up the test.<br></blockquote><div><br></div><div>Ah.  Makes sense.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
- Nanley<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> On Tue, Apr 26, 2016 at 3:35 PM, Nanley Chery <<a href="mailto:nanleychery@gmail.com">nanleychery@gmail.com</a>> wrote:<br>
><br>
> > From: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
> ><br>
> > Fixes some failures in dEQP-VK.api.info.image_format_properties.* and<br>
> > enables the test group to execute without assert failing.<br>
> ><br>
> > Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=94896" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=94896</a><br>
> > Signed-off-by: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
> > ---<br>
> >  src/intel/vulkan/anv_formats.c | 2 +-<br>
> >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/src/intel/vulkan/anv_formats.c<br>
> > b/src/intel/vulkan/anv_formats.c<br>
> > index 750af79..395f917 100644<br>
> > --- a/src/intel/vulkan/anv_formats.c<br>
> > +++ b/src/intel/vulkan/anv_formats.c<br>
> > @@ -583,7 +583,7 @@ unsupported:<br>
> >        .maxResourceSize = 0,<br>
> >     };<br>
> ><br>
> > -   return VK_SUCCESS;<br>
> > +   return VK_ERROR_FORMAT_NOT_SUPPORTED;<br>
> >  }<br>
> ><br>
> >  void anv_GetPhysicalDeviceSparseImageFormatProperties(<br>
> > --<br>
> > 2.8.0<br>
> ><br>
> > _______________________________________________<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/mailman/listinfo/mesa-dev</a><br>
> ><br>
</div></div></blockquote></div><br></div></div>