Re: [PATCH] drm/i915: Invalidate TLBs for the rings after a reset

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

 



On Fri, Aug 16, 2013 at 08:31:35AM +0100, Chris Wilson wrote:
> On Tue, Aug 06, 2013 at 07:01:14PM +0100, Chris Wilson wrote:
> > After any "soft gfx reset" we must manually invalidate the TLBs
> > associated with each ring. Empirically, it seems that a
> > suspend/resume or D3-D0 cycle count as a "soft reset". The symptom is
> > that the hardware would fail to note the new address for its status
> > page, and so it would continue to write the shadow registers and
> > breadcrumbs into the old physical address (now used by something
> > completely different, scary). Whereas the driver would read the new
> > status page and never see any progress, it would appear that the GPU
> > hung immediately upon resume.
> > 
> > Based on a patch by naresh kumar kachhi <naresh.kumar.kacchi@xxxxxxxxx>
> > 
> > Reported-by: Thiago Macieira <thiago@xxxxxxx>
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=64725
> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> 
> Thiago reports that early testing indicates success.
> 
> Anyone fancy acking this and sending this onto to stable@?

Picked up for -fixes, thanks for the patch. I'll let it hang there a bit
though before forwarding, so I don't plan to update the -fixes pull
request I've just recently sent out.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux