[Mesa-dev] [PATCH] intel/blorp: Adjust intra-tile x when faking rgb with red-only

[Jason Ekstrand]

On August 19, 2017 7:24:55 AM Topi Pohjolainen <topi.pohjolainen at gmail.com> 
wrote:

> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101910
>
> CC: Jason Ekstrand <jason at jlekstrand.net>
> CC: Mark Janes <mark.a.janes at intel.com>
> CC: mesa-stable at lists.freedesktop.org
>
> Signed-off-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
> ---
>  src/intel/blorp/blorp_blit.c | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/src/intel/blorp/blorp_blit.c b/src/intel/blorp/blorp_blit.c
> index ed00516373..db93d0f585 100644
> --- a/src/intel/blorp/blorp_blit.c
> +++ b/src/intel/blorp/blorp_blit.c
> @@ -1839,6 +1839,12 @@ try_blorp_blit(struct blorp_batch *batch,
>        surf_get_intratile_offset_px(&params->dst,
>                                     &params->wm_inputs.dst_offset.x,
>                                     &params->wm_inputs.dst_offset.y);
> +
> +      if (wm_prog_key->dst_rgb) {
> +         /* See surf_fake_rgb_with_red() */
> +         params->wm_inputs.dst_offset.x *= 3;

It would be better to correct this in surf_fake_rgb_with_red by adjusting 
surf->tile_x_sa.

> +      }
> +
>        params->x0 += params->wm_inputs.dst_offset.x;
>        params->y0 += params->wm_inputs.dst_offset.y;
>        params->x1 += params->wm_inputs.dst_offset.x;
> --
> 2.11.0
>