[Mesa-dev] [PATCH] i965/gen9: Don't change halign and valign to fit in fast copy blit

[Ben Widawsky]

On Wed, Jun 08, 2016 at 11:28:12AM -0700, Anuj Phogat wrote:
> An update in graphics specs has deleted the halign and valign fields
> from XY_FAST_COPY_BLT command. See mesa commit 97f0f91.
> 
> Cc: Ben Widawsky <ben at bwidawsk.net>
> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
Reviewed-by: Ben Widawsky <ben at bwidawsk.net>

> ---
>  src/mesa/drivers/dri/i965/brw_tex_layout.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_tex_layout.c b/src/mesa/drivers/dri/i965/brw_tex_layout.c
> index a294829..4618bc0 100644
> --- a/src/mesa/drivers/dri/i965/brw_tex_layout.c
> +++ b/src/mesa/drivers/dri/i965/brw_tex_layout.c
> @@ -763,10 +763,8 @@ intel_miptree_set_alignment(struct brw_context *brw,
>        mt->halign = 8;
>        mt->valign = brw->gen >= 7 ? 8 : 4;
>     } else if (brw->gen >= 9 && mt->tr_mode != INTEL_MIPTREE_TRMODE_NONE) {
> -      /* XY_FAST_COPY_BLT doesn't support horizontal alignment < 32 or
> -       * vertical alignment < 64. */
> -      mt->halign = MAX2(tr_mode_horizontal_texture_alignment(mt), 32);
> -      mt->valign = MAX2(tr_mode_vertical_texture_alignment(mt), 64);
> +      mt->halign = tr_mode_horizontal_texture_alignment(mt);
> +      mt->valign = tr_mode_vertical_texture_alignment(mt);
>     } else {
>        mt->halign =
>           intel_horizontal_texture_alignment_unit(brw, mt, layout_flags);
> -- 
> 2.5.5
>