On Thu, Mar 14, 2013 at 05:52:07PM +0200, Mika Kuoppala wrote: > index d66208c..9599c56 100644 > --- a/drivers/gpu/drm/i915/intel_ringbuffer.h > +++ b/drivers/gpu/drm/i915/intel_ringbuffer.h > @@ -137,6 +137,8 @@ struct intel_ring_buffer { > struct i915_hw_context *default_context; > struct drm_i915_gem_object *last_context_obj; > > + u32 hangcheck_seqno; We end up with quite a few hangheck_* variables. An indication that we should sort those out into their own struct. -Chris -- Chris Wilson, Intel Open Source Technology Centre