[Mesa-dev] [PATCH 6/9] intel/blorp: Set needs_(dst|src)_offset for Gen4 cubemaps

[Jason Ekstrand]

I probably shouldn't review a patch that says it's from me, but why not

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

On Fri, Jun 16, 2017 at 2:01 PM, Ian Romanick <idr at freedesktop.org> wrote:

> From: Jason Ekstrand <jason.ekstrand at intel.com>
>
> We call convert_to_single_slice so they may end up with a non-trivial
> offset that needs to be taken into account.
>
> v2 (idr): Also set needs_src_offset.  Suggested by Jason.
>
> Fixes ES2-CTS.functional.texture.specification.basic_
> copyteximage2d.cube_rgba
> and ES2-CTS.functional.texture.specification.basic_
> copytexsubimage2d.cube_rgba
> on G45.
>
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101284
> ---
>  src/intel/blorp/blorp_blit.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/src/intel/blorp/blorp_blit.c b/src/intel/blorp/blorp_blit.c
> index d93cde2..e48e5da 100644
> --- a/src/intel/blorp/blorp_blit.c
> +++ b/src/intel/blorp/blorp_blit.c
> @@ -1672,11 +1672,15 @@ try_blorp_blit(struct blorp_batch *batch,
>        /* The MinLOD and MinimumArrayElement don't work properly for cube
> maps.
>         * Convert them to a single slice on gen4.
>         */
> -      if (params->dst.surf.usage & ISL_SURF_USAGE_CUBE_BIT)
> +      if (params->dst.surf.usage & ISL_SURF_USAGE_CUBE_BIT) {
>           blorp_surf_convert_to_single_slice(batch->blorp->isl_dev,
> &params->dst);
> +         wm_prog_key->need_dst_offset = true;
> +      }
>
> -      if (params->src.surf.usage & ISL_SURF_USAGE_CUBE_BIT)
> +      if (params->src.surf.usage & ISL_SURF_USAGE_CUBE_BIT) {
>           blorp_surf_convert_to_single_slice(batch->blorp->isl_dev,
> &params->src);
> +         wm_prog_key->need_src_offset = true;
> +      }
>     }
>
>     if (devinfo->gen > 6 &&
> --
> 2.9.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170616/356a6f4c/attachment-0001.html>