[Intel-gfx] [PATCH 0/4] FIFO underrun reporting v2

Paulo Zanoni przanoni at gmail.com
Fri Feb 22 21:05:27 CET 2013


From: Paulo Zanoni <paulo.r.zanoni at intel.com>

Hi

Here's a new series that tries to report FIFO underruns. The new implementation
is completely different from the old one, due to the reviews I received. Now we
don't just "ignore" the FIFO underrun interrupts when we receive them, we
effectively disable the interrupts (the downsides of this approach are
documented in the commit message of patch 2).

We will still see at most one of each error FIFO underrun message per mode set,
so this is not really going to flood dmesg. I tested this series on ILK, SNB and
HSW, and I am only seeing FIFO underruns on ILK when using 2 monitors
(LVDS+HDMI). I'll work on a patch to fix this message later (at least now we
know we have problems!).

It seems not everybody is in favor of adding the "poison" error message, so the
"print poison interrupts" patch comes _after_ the "print FIFO underruns". We can
discard the patch if we conclude we don't want it. I haven't seen the poison
message on any of my tests.

Thanks,
Paulo

Paulo Zanoni (4):
  drm/i915: also disable south interrupts when handling them
  drm/i915: report Gen5+ CPU and PCH FIFO underruns
  drm/i915: print Gen5+ CPU/PCH poison interrupts
  drm/i915: remove "inline" keyword from ironlake_disable_display_irq

 drivers/gpu/drm/i915/i915_irq.c      |  345 ++++++++++++++++++++++++++++++++--
 drivers/gpu/drm/i915/i915_reg.h      |   15 +-
 drivers/gpu/drm/i915/intel_display.c |   17 +-
 drivers/gpu/drm/i915/intel_drv.h     |   10 +
 4 files changed, 371 insertions(+), 16 deletions(-)

-- 
1.7.10.4




More information about the Intel-gfx mailing list