Skip to content

Unclaimed register reporting V2

Paulo Zanoni intel-gfx 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest7 patches 7 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/7drm/i915: create macros for the "unclaimed register" checksdrivers/gpu/drm/i915/i915_drv.c+16 -82 replies
2/7drm/i915: use FPGA_DBG for the "unclaimed register" checksdrivers/gpu/drm/i915/i915_drv.c drivers/gpu/drm/i915/i915_reg.h+7 -41 reply
3/7drm/i915: clear the FPGA_DBG_RM_NOCLAIM bit at driver initdrivers/gpu/drm/i915/i915_dma.c+4 -02 replies
4/7drm/i915: check for unclaimed registers on I915_READ toodrivers/gpu/drm/i915/i915_drv.c+8 -60 replies
5/7drm/i915: WARN on unclaimed registersdrivers/gpu/drm/i915/i915_drv.c+2 -21 reply
6/7drm/i915: only check for unclaimed registers if drm_debugdrivers/gpu/drm/i915/i915_drv.c+2 -20 replies
7/7drm/i915: print Gen 7 error interruptsdrivers/gpu/drm/i915/i915_irq.c drivers/gpu/drm/i915/i915_reg.h+28 -22 replies

Reviews trailers per patch

3 of 7 patches have a Reviewed-by or Acked-by.

PatchBen Widawsky
1/7R-b
2/7R-b
3/7
4/7
5/7R-b
6/7
7/7

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/7] Unclaimed register reporting V2

From: Paulo Zanoni <[email protected]>

Hi

This is new version of 2 patches I sent last week. First we change how unclaimed registers are currently reported by our driver and in the end we enable gen 7 error interrupts.

Here's what we do:
- Make the current code a little bit more organized
- Use FPGA_DBG instead of GEN7_ERR_INT for the polling code
- Initialize FPGA_DBG to clear one "unclaimed register" message
- Check unclaimed registers on I915_READ
- Replace the DRM_ERROR with WARNs
- Only print uncnlaimed registers if drm_debug is set
- Finally, print Gen 7 error interrupts

We had some discussion regarding these patches, and there is still no consensus on what exactly we should do, so feel free to nack patches. Still, the patches here are to some extent based on the comments I received (mainly from Ben, Daniel and Chris), so I hope they're not too bad. Also, I really think that at least the first 3 patches should be merged, and also some equivalent form of the last patch should be merged.

Another thing I have to add: after patch 7 you may see a few dozen "unclaimed register" messages when booting if you have the power well disabled. All these messages go away if you blacklist snd_hda_intel. Still, I like to have messages reminding me that something is wrong in some driver. And you'll only see those messages if you have drm_debug, because of patch 0006.

Thanks,
Paulo

Paulo Zanoni (7):
drm/i915: create macros for the "unclaimed register" checks drm/i915: use FPGA_DBG for the "unclaimed register" checks drm/i915: clear the FPGA_DBG_RM_NOCLAIM bit at driver init drm/i915: check for unclaimed registers on I915_READ too drm/i915: WARN on unclaimed registers
drm/i915: only check for unclaimed registers if drm_debug drm/i915: print Gen 7 error interrupts

 drivers/gpu/drm/i915/i915_dma.c |    4 ++++
 drivers/gpu/drm/i915/i915_drv.c |   26 ++++++++++++++++++--------
 drivers/gpu/drm/i915/i915_irq.c |   28 +++++++++++++++++++++++++++-
 drivers/gpu/drm/i915/i915_reg.h |    5 ++++-
 4 files changed, 53 insertions(+), 10 deletions(-)
Signature · 1 line
1.7.10.4