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?