Quoting Patchwork (2020-03-06 04:28:26) > == Series Details == > > Series: drm/i915: properly sanity check batch_start_offset (rev2) > URL : https://patchwork.freedesktop.org/series/74287/ > State : success Apologies, SPF vs intel.com again. So, I think the _end variant should be for the inclusive endpoint which is the less common form (there's only 3 where we use U32_MAX and the other 24 all use size or total, i.e. exclusive). So I would convert those 3 to use _end or _excl, and convert the main overflow check to use >= -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
