Series is,

Reviewed-by: Samuel Iglesias Gonsálvez <sigles...@igalia.com>

Sam

On 06/02/18 00:13, Jason Ekstrand wrote:
> This little series adds support for enabling CCS_E for images which may
> have VK_IMAGE_USAGE_STORAGE_BIT set.  Previously, we just bailed on these
> images and disabled CCS.  However, so long as we do a full resolve when
> entering VK_IMAGE_LAYOUT_GENERAL, we can support CCS_E for all the other
> layouts just fine.
>
> The primary motivation of this series is actually by trying to get better
> test coverage of our resolve code.  By doing resolves when going into
> VK_IMAGE_LAYOUT_GENERAL, the full resolve paths now get tested on many more
> image types.  I have no idea what the perf impact of this will be.
>
> The first 4 patches just make use use layout_to_* more often.
>
> Jason Ekstrand (5):
>   anv/cmd_buffer: Simplify transition_depth_buffer
>   anv/cmd_buffer: Use layout_to_* helpers in compute_aux_usage
>   anv/cmd_buffer: Delete some assert-only variables
>   anv/blorp: Use layout_to_aux_usage when a layout is provided
>   anv/image: Support CCS_E for images which may be used for storage
>
>  src/intel/vulkan/anv_blorp.c       | 71 
> ++++++++++++++++++++++++--------------
>  src/intel/vulkan/anv_image.c       | 18 ++++++++--
>  src/intel/vulkan/genX_cmd_buffer.c | 70 +++++++++++++++++++------------------
>  3 files changed, 97 insertions(+), 62 deletions(-)
>

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to