The genxml packing structs will assert this for you. I'm guessing it did and you found it rather obtuse. :-)
Rb On Sep 21, 2016 9:16 PM, "Anuj Phogat" <anuj.pho...@gmail.com> wrote: > Signed-off-by: Anuj Phogat <anuj.pho...@gmail.com> > --- > src/intel/isl/isl_surface_state.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/intel/isl/isl_surface_state.c > b/src/intel/isl/isl_surface_state.c > index 1a1a819..b20f767 100644 > --- a/src/intel/isl/isl_surface_state.c > +++ b/src/intel/isl/isl_surface_state.c > @@ -249,6 +249,9 @@ isl_genX(surf_fill_state_s)(const struct isl_device > *dev, void *state, > s.IntegerSurfaceFormat = isl_format_has_int_channel(s.SurfaceFormat); > #endif > > + assert(info->surf->logical_level0_px.width > 0 && > + info->surf->logical_level0_px.height > 0); > + > s.Width = info->surf->logical_level0_px.width - 1; > s.Height = info->surf->logical_level0_px.height - 1; > > -- > 2.5.5 > > _______________________________________________ > 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