[Mesa-dev] [PATCH v2 10/11] anv: enable multiple planes per image/imageView

Lionel Landwerlin lionel.g.landwerlin at intel.com
Wed Oct 4 02:41:37 UTC 2017


On 04/10/17 02:16, Jason Ekstrand wrote:
>
> Ok, maybe I'm being a bit dense, but doesn't the aspect imply the 
> plane?  Why can't we just use anv_image_aspect_to_plane here?  That 
> would reduce the churn by a good bit.
>
>                                   enum isl_aux_usage aux_usage,
>                                   struct blorp_surf *blorp_surf)
>      {
>     +   /* For the stencil surface aux_usage is always NONE. */
>         if (aspect == VK_IMAGE_ASPECT_STENCIL_BIT ||
>             aux_usage == ISL_AUX_USAGE_HIZ)
>            aux_usage = ISL_AUX_USAGE_NONE;
>
It was for that bit right here.
I need to check again, maybe this can go. I remember this triggered some 
failures.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20171004/15870c3d/attachment.html>


More information about the mesa-dev mailing list