Re: [Intel-gfx] [PATCH] drm/i915: Read the hardware state for the transcoder link upon error

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jun 24, 2013 at 09:59:33AM +0200, Daniel Vetter wrote:
> On Fri, Jun 21, 2013 at 03:40:04PM +0100, Chris Wilson wrote:
> > Do not trust our bookkeeping when reporting errors, and instead dump the
> > register contents. In particular, this solves one particular issue when
> > an error is reported before we finish setting up the outputs and have a
> > complete mapping (i.e. during initialisation we set garbage state). If
> > an error occurs at that early stage, it is vital that we get an accurate
> > report of the hardware state and not conflated with our own inaccurate
> > opinions.
> > 
> > This fixes a panic for a large number of pre-Haswell machines that
> > currently trigger an error during KMS takeover.
> > 
> > Reported-by: Dustin King <daking@xxxxxxxxxxxxxxxxxxxx>
> > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=60021
> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > Cc: stable@xxxxxxxxxxxxxxx
> 
> Shouldn't we just dump all transcoder registers on Haswell instead of
> potentially fragile dances trying to reconstruct state?

You still need to reconstruct the pipe->transcoder link as they are
separate banks of registers. This patch is just a smaller part of the
patch to dump everything.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]