On Jul 28, 2016 8:02 AM, "Pohjolainen, Topi" <topi.pohjolai...@intel.com>
wrote:
>
> On Tue, Jul 26, 2016 at 03:02:21PM -0700, Jason Ekstrand wrote:
> > ---
> > src/intel/isl/isl_surface_state.c | 10 ++++++++++
> > 1 file changed, 10 insertions(+)
> >
> > diff --git a/src/intel/isl/isl_surface_state.c
b/src/intel/isl/isl_surface_state.c
> > index 6febcbf..fb23414 100644
> > --- a/src/intel/isl/isl_surface_state.c
> > +++ b/src/intel/isl/isl_surface_state.c
> > @@ -414,6 +414,16 @@ isl_genX(surf_fill_state_s)(const struct
isl_device *dev, void *state,
> > assert(info->surf->levels == 1);
> > assert(info->surf->logical_level0_px.array_len == 1);
> > assert(info->aux_usage == ISL_AUX_USAGE_NONE);
> > +
> > + if (GEN_GEN >= 8) {
> > + /* Broadwell added more rules. */
> > + assert(info->surf->samples == 1);
> > + if (isl_format_get_layout(info->view->format)->bpb == 8)
> > + assert(info->x_offset_sa % 16 == 0);
>
> Should this go below and the one there up here?
Feel free to double-check the PRM (I just did) but I believe this is
correct.
> > + if (isl_format_get_layout(info->view->format)->bpb == 16)
> > + assert(info->x_offset_sa % 8 == 0);
> > + }
> > +
> > #if GEN_GEN >= 7
> > s.SurfaceArray = false;
> > #endif
> > --
> > 2.5.0.400.gff86faf
> >
> > _______________________________________________
> > 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
