gen8+: Allow 3D auxiliary surfaces

Jason Ekstrand Thu, 01 Sep 2016 12:24:49 -0700

On Wed, Aug 31, 2016 at 9:56 PM, Pohjolainen, Topi <
[email protected]> wrote:


> On Wed, Aug 31, 2016 at 05:41:24PM -0700, Jason Ekstrand wrote:
> >    On Wed, Aug 31, 2016 at 8:17 AM, Topi Pohjolainen
> >    <[1][email protected]> wrote:
> >
> >      Otherwise once mcs buffer gets allocated without delay for
> >      lossless compression (same as we do for msaa), assert starts
> >      to fire in piglit case: tex3d. The test uses depth of one
> >      which is in fact supported even now.
> >      Signed-off-by: Topi Pohjolainen <[2][email protected]>
> >      ---
> >      Â src/intel/isl/isl.c | 7 ++++++-
> >      Â 1 file changed, 6 insertions(+), 1 deletion(-)
> >      diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c
> >      index c4989dd..b5c4a8c 100644
> >      --- a/src/intel/isl/isl.c
> >      +++ b/src/intel/isl/isl.c
> >      @@ -1328,7 +1328,12 @@ isl_surf_get_ccs_surf(const struct isl_device
> >      *dev,
> >      Â  Â  assert(surf->samples == 1 && surf->msaa_layout ==
> >      ISL_MSAA_LAYOUT_NONE);
> >      Â  Â  assert(ISL_DEV_GEN(dev) >= 7);
> >      -Â  Â assert(surf->dim == ISL_SURF_DIM_2D);
> >      +Â  Â if (ISL_DEV_GEN(dev) < 8) {
> >      +Â  Â  Â  assert(surf->dim == ISL_SURF_DIM_2D);
> >      +Â  Â } else {
> >      +Â  Â  Â  assert(surf->dim == ISL_SURF_DIM_2D || surf->dim ==
> >      ISL_SURF_DIM_3D);
> >      +Â  Â }
> >
> >    What about 1D?Â  Maybe we just never try to allocate 1D Y-tiled.Â
>
> I guess so, I can't think of any restrictions why compression or fast clear
> itself wouldn't be supported.
>

In that case, let's just get rid of the assert entirely.


> >    Also, if it's 3D, should we assert depth == 1?
>
> Check for the depth is just below.
>

So it is...


> >    Â
> >
> >      +
> >      Â  Â  assert(surf->logical_level0_px.depth == 1);
> >      Â  Â  /* TODO: More conditions where it can fail. */
> >      --
> >      2.5.5
> >      _______________________________________________
> >      mesa-dev mailing list
> >      [3][email protected]
> >      [4]https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> >
> > References
> >
> >    1. mailto:[email protected]
> >    2. mailto:[email protected]
> >    3. mailto:[email protected]
> >    4. https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
> > _______________________________________________
> > mesa-dev mailing list
> > [email protected]
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
>

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 03/12] isl/gen8+: Allow 3D auxiliary surfaces

Reply via email to