[PATCH] drm/i915: Fix use of invalid array size for ring->sync_seqno

Jiri Slaby jirislaby at gmail.com
Sat Jan 22 14:24:21 PST 2011


On 01/21/2011 11:11 AM, Chris Wilson wrote:
> There are I915_NUM_RINGS-1 inter-ring synchronisation counters, but we
> were clearing I915_NUM_RINGS of them. Oops.
> 
> Reported-by: Jiri Slaby <jirislaby at gmail.com>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---
> 
> Jiri, does this catch the invalid array access? For 3 rings, there should
> only be 2 sync counters...

Yup, it's gone.

thanks,
--
js


More information about the dri-devel mailing list