[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: properly sanity check batch_start_offset (rev2)

Chris Wilson chris at chris-wilson.co.uk
Fri Mar 6 09:08:11 UTC 2020


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


More information about the Intel-gfx mailing list