[Intel-gfx] [PATCH 10/10] drm/i915: remove "unclaimed register" checks from I915_WRITE

Ben Widawsky ben at bwidawsk.net
Fri Jan 18 21:49:13 CET 2013


On Fri, Jan 18, 2013 at 06:29:12PM -0200, Paulo Zanoni wrote:
> From: Paulo Zanoni <paulo.r.zanoni at intel.com>
> 
> Some developers don't really like this code polluting I915_WRITE, and
> we've never really measured its negative impacts. So now that we
> properly print ERR_INT interrupts, let's remove the I915_WRITE code
> and promote the interrupt error message to DRM_ERROR.
> 
> The downside of this change is that we lose the ability to check the
> register and print nice backtraces, but at this point most of the
> errors have already been fixed and we're investigating the few
> remaining cases.
> 
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com>

I'm really sad to see this go. Especially since our time between new
platform bring-up is decreasing so much. If I were to guess right, every
developer working on a new platform would want this. So while HSW may be
in the clear, HSW+1 suffers.

Could I request that you measure the negative impacts?



More information about the Intel-gfx mailing list