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