[Mesa-dev] [PATCH] anv: fix multi level clears with VK_REMAINING_MIP_LEVELS

Nanley Chery nanleychery at gmail.com
Mon Nov 14 17:40:54 UTC 2016


On Mon, Nov 14, 2016 at 05:26:09PM +0000, Lionel Landwerlin wrote:
> A commit from the CTS suite on the 1.0-dev branch started using
> VK_REMAINING_MIP_LEVELS, we're not dealing with it properly for clears.
> 
> Fixes:
>    dEQP-VK.api.image_clearing.clear_color_image.*
> 
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> ---
>  src/intel/vulkan/anv_blorp.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

This patch is
Reviewed-by: Nanley Chery <nanley.g.chery at intel.com>

> 
> diff --git a/src/intel/vulkan/anv_blorp.c b/src/intel/vulkan/anv_blorp.c
> index b78c21d..d59c1a7 100644
> --- a/src/intel/vulkan/anv_blorp.c
> +++ b/src/intel/vulkan/anv_blorp.c
> @@ -803,7 +803,7 @@ void anv_CmdClearColorImage(
>        unsigned base_layer = pRanges[r].baseArrayLayer;
>        unsigned layer_count = pRanges[r].layerCount;
>  
> -      for (unsigned i = 0; i < pRanges[r].levelCount; i++) {
> +      for (unsigned i = 0; i < anv_get_levelCount(image, &pRanges[r]); i++) {
>           const unsigned level = pRanges[r].baseMipLevel + i;
>           const unsigned level_width = anv_minify(image->extent.width, level);
>           const unsigned level_height = anv_minify(image->extent.height, level);
> @@ -863,7 +863,7 @@ void anv_CmdClearDepthStencilImage(
>        unsigned base_layer = pRanges[r].baseArrayLayer;
>        unsigned layer_count = pRanges[r].layerCount;
>  
> -      for (unsigned i = 0; i < pRanges[r].levelCount; i++) {
> +      for (unsigned i = 0; i < anv_get_levelCount(image, &pRanges[r]); i++) {
>           const unsigned level = pRanges[r].baseMipLevel + i;
>           const unsigned level_width = anv_minify(image->extent.width, level);
>           const unsigned level_height = anv_minify(image->extent.height, level);
> -- 
> 2.10.2
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list