On Fri, Jul 22, 2016 at 10:39 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote:
> intel_mipmap_tree::logical_depth0 is now in 2-D slices so there is no need
> for us to multiply by 6 when we go to fill out a blorp surface state.
>
> Signed-off-by: Jason Ekstrand <ja...@jlekstrand.net>
> ---
> src/mesa/drivers/dri/i965/gen8_blorp.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/gen8_blorp.c
> b/src/mesa/drivers/dri/i965/gen8_blorp.c
> index 870b67f..ab9b747 100644
> --- a/src/mesa/drivers/dri/i965/gen8_blorp.c
> +++ b/src/mesa/drivers/dri/i965/gen8_blorp.c
> @@ -526,9 +526,6 @@ gen8_blorp_emit_surface_states(struct brw_context *brw,
> mt->msaa_layout == INTEL_MSAA_LAYOUT_CMS) ?
> MAX2(mt->num_samples, 1) : 1;
>
> - const bool is_cube = mt->target == GL_TEXTURE_CUBE_MAP_ARRAY ||
> - mt->target == GL_TEXTURE_CUBE_MAP;
> - const unsigned depth = (is_cube ? 6 : 1) * mt->logical_depth0;
> const unsigned layer = mt->target != GL_TEXTURE_3D ?
> surface->layer / layer_divider : 0;
>
> @@ -537,7 +534,7 @@ gen8_blorp_emit_surface_states(struct brw_context *brw,
> .base_level = surface->level,
> .levels = mt->last_level - surface->level + 1,
> .base_array_layer = layer,
> - .array_len = depth - layer,
> + .array_len = mt->logical_depth0 - layer,
> .channel_select = {
> swizzle_to_scs(GET_SWZ(surface->swizzle, 0)),
> swizzle_to_scs(GET_SWZ(surface->swizzle, 1)),
> --
> 2.5.0.400.gff86faf
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
LGTM. Series is:
Reviewed-by: Anuj Phogat <anuj.pho...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
