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

2011-01-22 Thread Jiri Slaby
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 > Signed-off-by: Chris Wilson > --- > > Jiri, does this catch the invalid array access? For 3 rings, the

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

2011-01-22 Thread Jiri Slaby
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 > Signed-off-by: Chris Wilson > --- > > Jiri, does this catch the invalid array access? For 3 rings, the

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

2011-01-21 Thread Chris Wilson
There are I915_NUM_RINGS-1 inter-ring synchronisation counters, but we were clearing I915_NUM_RINGS of them. Oops. Reported-by: Jiri Slaby Signed-off-by: Chris Wilson --- Jiri, does this catch the invalid array access? For 3 rings, there should only be 2 sync counters... --- drivers/gpu/drm/i

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

2011-01-21 Thread Chris Wilson
There are I915_NUM_RINGS-1 inter-ring synchronisation counters, but we were clearing I915_NUM_RINGS of them. Oops. Reported-by: Jiri Slaby Signed-off-by: Chris Wilson --- Jiri, does this catch the invalid array access? For 3 rings, there should only be 2 sync counters... --- drivers/gpu/drm/i