On Wed, 2017-10-18 at 23:23 -0700, Kenneth Graunke wrote: > The restriction is supposed to apply if the width *field* is >= 8192, > meaning the actual width *value* is >= 8193. > > The code also incorrectly used == for some reason.
Reviewed-by: Juan A. Suarez Romero <[email protected]> > --- > src/intel/isl/isl_gen7.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/intel/isl/isl_gen7.c b/src/intel/isl/isl_gen7.c > index 24d411f51e9..c42428cba7a 100644 > --- a/src/intel/isl/isl_gen7.c > +++ b/src/intel/isl/isl_gen7.c > @@ -132,7 +132,7 @@ isl_gen7_choose_msaa_layout(const struct isl_device *dev, > * is >= 8192 (meaning the actual surface width is >= 8193 pixels), > this > * field must be set to MSFMT_MSS. > */ > - if (info->samples == 8 && info->width == 8192) > + if (info->samples == 8 && info->width > 8192) > require_array = true; > > /* From the Ivybridge PRM, Volume 4 Part 1 p72, SURFACE_STATE, > Multisampled _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
