[Intel-gfx] [PATCH 05/43] drm/i915: collect more per ring error state

Eugeni Dodonov eugeni at dodonov.net
Wed Dec 14 19:43:40 CET 2011


On Wed, Dec 14, 2011 at 10:57, Daniel Vetter <daniel.vetter at ffwll.ch> wrote:

> Based on a patch by Ben Widawsky, but with different colors
> for the bikeshed.
>
> In contrast to Ben's patch this one doesn't add the fault regs.
> Afaics they're for the optional page fault support which
> - we're not enabling
> - and which seems to be unsupported by the hw team. Recent bspec
>  lacks tons of information about this that the public docs released
>  half a year back still contain.
>
> Also dump ring HEAD/TAIL registers - I've recently seen a few
> error_state where just guessing these is not good enough.
>
> v2: Also dump INSTPM for every ring.
>
> v3: Fix a few really silly goof-ups spotted by Chris Wilson.
>
> Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
> Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
>

Reviewed-by: Eugeni Dodonov <eugeni.dodonov at intel.com>


-- 
Eugeni Dodonov
<http://eugeni.dodonov.net/>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20111214/b015ba5f/attachment.html>


More information about the Intel-gfx mailing list