<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Nov 3, 2016 at 7:26 AM, Pohjolainen, Topi <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"><span class="">On Fri, Oct 28, 2016 at 02:17:00AM -0700, Jason Ekstrand wrote:<br>
> There are some invariants such as number of samples on which we should<br>
> assert.  However, most other things should silently return false since<br>
> they're much easier for isl_surf_get_ccs to check than the caller.  We also<br>
> update the checking to be a bit more complete.<br>
><br>
> Signed-off-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
> ---<br>
>  src/intel/isl/isl.c | 9 +++++++--<br>
>  1 file changed, 7 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c<br>
> index ec53072..578c0e7 100644<br>
> --- a/src/intel/isl/isl.c<br>
> +++ b/src/intel/isl/isl.c<br>
> @@ -1448,9 +1448,14 @@ 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(ISL_DEV_GEN(dev) >= 8 || surf->dim == ISL_SURF_DIM_2D);<br>
> +   if (surf->usage & ISL_SURF_USAGE_DISABLE_AUX_<wbr>BIT)<br>
<br>
</span>Patch one stops setting the flag. Is this needed?<span class=""><br></span></blockquote><div><br></div><div>I don't know... As long as we have the flag, I figure we may as well respect it.  Once we get i965 converted over, we may decide it's useless and rop the flag.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> +      return false;<br>
><br>
> -   assert(surf->logical_level0_<wbr>px.depth == 1);<br>
> +   if (ISL_DEV_GEN(dev) <= 8 && surf->dim != ISL_SURF_DIM_2D)<br>
> +      return false;<br>
> +<br>
> +   if (isl_format_is_compressed(<wbr>surf->format))<br>
> +      return false;<br>
><br>
>     /* TODO: More conditions where it can fail. */<br>
><br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</span>> ______________________________<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>
</blockquote></div><br></div></div>