On Tue, Jul 18, 2017 at 02:11:59PM -0700, Jason Ekstrand wrote:
> On Tue, Jul 18, 2017 at 1:46 AM, Topi Pohjolainen <
> topi.pohjolai...@gmail.com> wrote:
>
> > Signed-off-by: Topi Pohjolainen <topi.pohjolai...@intel.com>
> > ---
> > src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 21 ++++++++++++++++++---
> > 1 file changed, 18 insertions(+), 3 deletions(-)
> >
> > diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> > b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> > index d96f5c7938..fafd0c1e59 100644
> > --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> > +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> > @@ -696,8 +696,14 @@ create_aux_state_map(struct intel_mipmap_tree *mt,
> > const uint32_t levels = mt->last_level + 1;
> >
> > uint32_t total_slices = 0;
> > - for (uint32_t level = 0; level < levels; level++)
> > - total_slices += mt->level[level].depth;
> > + for (uint32_t level = 0; level < levels; level++) {
> > + if (mt->surf.size > 0)
> > + total_slices += (mt->surf.dim == ISL_SURF_DIM_3D ?
> > + minify(mt->surf.phys_level0_sa.depth,
> > level) :
> > + mt->surf.phys_level0_sa.array_len);
> > + else
> > + total_slices += mt->level[level].depth;
> >
>
> The intention was always for the state map to use logical layers, not
> physical. There's no point in tracking fast-clear state for an individual
> sample. That said, that may not actually be quite correct with the current
> implementation.
Right, I was thinking the same. However, there are already quite a few things
on the move and I thought leaving this for later. How would you feel
addressing this after the transition to ISL?
>
> --Jason
>
>
> > + }
> >
> > const size_t per_level_array_size = levels * sizeof(enum isl_aux_state
> > *);
> >
> > @@ -715,7 +721,16 @@ create_aux_state_map(struct intel_mipmap_tree *mt,
> > enum isl_aux_state *s = data + per_level_array_size;
> > for (uint32_t level = 0; level < levels; level++) {
> > per_level_arr[level] = s;
> > - for (uint32_t a = 0; a < mt->level[level].depth; a++)
> > +
> > + unsigned level_depth;
> > + if (mt->surf.size > 0)
> > + level_depth = mt->surf.dim == ISL_SURF_DIM_3D ?
> > + minify(mt->surf.phys_level0_sa.depth, level) :
> > + mt->surf.phys_level0_sa.array_len;
> > + else
> > + level_depth = mt->level[level].depth;
> > +
> > + for (uint32_t a = 0; a < level_depth; a++)
> > *(s++) = initial;
> > }
> > assert((void *)s == data + total_size);
> > --
> > 2.11.0
> >
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> >
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
