On Fri, Jan 19, 2018 at 03:47:28PM -0800, Jason Ekstrand wrote:
> ---
>  src/intel/vulkan/anv_blorp.c | 14 ++++++--------
>  1 file changed, 6 insertions(+), 8 deletions(-)
> 

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

> diff --git a/src/intel/vulkan/anv_blorp.c b/src/intel/vulkan/anv_blorp.c
> index ccb9d18..e4e4135 100644
> --- a/src/intel/vulkan/anv_blorp.c
> +++ b/src/intel/vulkan/anv_blorp.c
> @@ -247,6 +247,7 @@ void anv_CmdCopyImage(
>        VkExtent3D extent =
>           anv_sanitize_image_extent(src_image->type, pRegions[r].extent);
>  
> +      const uint32_t dst_level = pRegions[r].dstSubresource.mipLevel;
>        unsigned dst_base_layer, layer_count;
>        if (dst_image->type == VK_IMAGE_TYPE_3D) {
>           dst_base_layer = pRegions[r].dstOffset.z;
> @@ -257,6 +258,7 @@ void anv_CmdCopyImage(
>              anv_get_layerCount(dst_image, &pRegions[r].dstSubresource);
>        }
>  
> +      const uint32_t src_level = pRegions[r].srcSubresource.mipLevel;
>        unsigned src_base_layer;
>        if (src_image->type == VK_IMAGE_TYPE_3D) {
>           src_base_layer = pRegions[r].srcOffset.z;
> @@ -283,10 +285,8 @@ void anv_CmdCopyImage(
>                                           ANV_AUX_USAGE_DEFAULT, &dst_surf);
>  
>              for (unsigned i = 0; i < layer_count; i++) {
> -               blorp_copy(&batch, &src_surf, 
> pRegions[r].srcSubresource.mipLevel,
> -                          src_base_layer + i,
> -                          &dst_surf, pRegions[r].dstSubresource.mipLevel,
> -                          dst_base_layer + i,
> +               blorp_copy(&batch, &src_surf, src_level, src_base_layer + i,
> +                          &dst_surf, dst_level, dst_base_layer + i,
>                            srcOffset.x, srcOffset.y,
>                            dstOffset.x, dstOffset.y,
>                            extent.width, extent.height);
> @@ -300,10 +300,8 @@ void anv_CmdCopyImage(
>                                        ANV_AUX_USAGE_DEFAULT, &dst_surf);
>  
>           for (unsigned i = 0; i < layer_count; i++) {
> -            blorp_copy(&batch, &src_surf, 
> pRegions[r].srcSubresource.mipLevel,
> -                       src_base_layer + i,
> -                       &dst_surf, pRegions[r].dstSubresource.mipLevel,
> -                       dst_base_layer + i,
> +            blorp_copy(&batch, &src_surf, src_level, src_base_layer + i,
> +                       &dst_surf, dst_level, dst_base_layer + i,
>                         srcOffset.x, srcOffset.y,
>                         dstOffset.x, dstOffset.y,
>                         extent.width, extent.height);
> -- 
> 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

Reply via email to