On Tuesday, February 13, 2018 11:15:04 AM PST Anuj Phogat wrote: > Signed-off-by: Anuj Phogat <anuj.pho...@gmail.com> > --- > src/intel/isl/isl.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c > index 59f512fc05..c9873d96df 100644 > --- a/src/intel/isl/isl.c > +++ b/src/intel/isl/isl.c > @@ -1483,7 +1483,7 @@ isl_surf_init_s(const struct isl_device *dev, > */ > if (size > (uint64_t) 1 << 31) > return false; > - } else { > + } else if (ISL_DEV_GEN(dev) == 9 || ISL_DEV_GEN(dev) == 10) {
I'd just do: } else if (ISL_DEV_GEN(dev) < 11) { Reviewed-by: Kenneth Graunke <kenn...@whitecape.org> > /* From the Skylake PRM Vol 5, Maximum Surface Size in Bytes: > * "In addition to restrictions on maximum height, width, and depth, > * surfaces are also restricted to a maximum size of 2^38 bytes. > @@ -1492,6 +1492,10 @@ isl_surf_init_s(const struct isl_device *dev, > */ > if (size > (uint64_t) 1 << 38) > return false; > + } else { > + /* gen11+ platforms raised this limit to 2^44 bytes. */ > + if (size > (uint64_t) 1 << 44) > + return false; > } > > *surf = (struct isl_surf) { >
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev