[Mesa-dev] [PATCH v2 25/35] i965/blorp: Map 1-D render targets with DIM_LAYOUT_GEN4_2D as 2D on gen9

[Pohjolainen, Topi]

On Tue, Jul 26, 2016 at 03:02:16PM -0700, Jason Ekstrand wrote:
> The sampling hardware can handle them ok.  It just looks at the tiling to
> determine whether it's the new gen9 1-D layout or the old one.  The render
> hardware isn't so smart.

To clarify, this is not needed at this point but prepares for the upcoming
patches (i.e, prevents regression there)?

> ---
>  src/mesa/drivers/dri/i965/brw_blorp.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c b/src/mesa/drivers/dri/i965/brw_blorp.c
> index d9b5554..2cf0f99 100644
> --- a/src/mesa/drivers/dri/i965/brw_blorp.c
> +++ b/src/mesa/drivers/dri/i965/brw_blorp.c
> @@ -371,6 +371,12 @@ brw_blorp_emit_surface_state(struct brw_context *brw,
>  
>     struct isl_surf surf = surface->surf;
>  
> +   if (surf.dim == ISL_SURF_DIM_1D &&
> +       surf.dim_layout == ISL_DIM_LAYOUT_GEN4_2D) {
> +      assert(surf.logical_level0_px.height == 1);
> +      surf.dim = ISL_SURF_DIM_2D;
> +   }
> +
>     union isl_color_value clear_color = { .u32 = { 0, 0, 0, 0 } };
>  
>     const struct isl_surf *aux_surf = NULL;
> -- 
> 2.5.0.400.gff86faf
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev