On Mon, Nov 27, 2017 at 07:06:18PM -0800, Jason Ekstrand wrote:
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 31 ++-
> 1 file changed, 14 insertions(+), 17 deletions(-)
Reviewed-by: Topi Pohjolainen
>
> diff --git a/src/intel/vulkan/genX_cmd_buffer.c
> b/src/intel/vulkan/genX_cmd_buffer.c
> index 9e584c1..dcd5a8f 100644
> --- a/src/intel/vulkan/genX_cmd_buffer.c
> +++ b/src/intel/vulkan/genX_cmd_buffer.c
> @@ -823,29 +823,26 @@ transition_color_buffer(struct anv_cmd_buffer
> *cmd_buffer,
> * We don't have any data to show that this is a problem, but we want
> to
> * avoid causing difficult-to-debug problems.
> */
> - if ((GEN_GEN >= 9 && image->samples == 1) || image->samples > 1) {
> + if (GEN_GEN >= 9 && image->samples == 1) {
> + for (uint32_t l = 0; l < level_count; l++) {
> +const uint32_t level = base_level + l;
> +const uint32_t level_layer_count =
> + MIN2(layer_count, anv_image_aux_layers(image, aspect, level));
> +anv_image_ccs_op(cmd_buffer, image, aspect, level,
> + base_layer, level_layer_count,
> + ISL_AUX_OP_FAST_CLEAR, false);
> + }
> + } else if (image->samples > 1) {
> if (image->samples == 4 || image->samples == 16) {
> anv_perf_warn(cmd_buffer->device->instance, image,
>"Doing a potentially unnecessary fast-clear to "
>"define an MCS buffer.");
> }
>
> - if (image->samples == 1) {
> -for (uint32_t l = 0; l < level_count; l++) {
> - const uint32_t level = base_level + l;
> - const uint32_t level_layer_count =
> - MIN2(layer_count, anv_image_aux_layers(image, aspect,
> level));
> - anv_image_ccs_op(cmd_buffer, image, aspect, level,
> -base_layer, level_layer_count,
> -ISL_AUX_OP_FAST_CLEAR, false);
> -}
> - } else {
> -assert(image->samples > 1);
> -assert(base_level == 0 && level_count == 1);
> -anv_image_mcs_op(cmd_buffer, image, aspect,
> - base_layer, layer_count,
> - ISL_AUX_OP_FAST_CLEAR, false);
> - }
> + assert(base_level == 0 && level_count == 1);
> + anv_image_mcs_op(cmd_buffer, image, aspect,
> + base_layer, layer_count,
> + ISL_AUX_OP_FAST_CLEAR, false);
>}
>/* At this point, some elements of the CCS buffer may have the
> fast-clear
> * bit-arrangement. As the user writes to a subresource, we need to
> have
> --
> 2.5.0.400.gff86faf
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
