On Tue, 10 Jan 2012 13:20:06 +0100, Daniel Vetter <dan...@ffwll.ch> wrote: > On Wed, Jan 04, 2012 at 07:40:45PM +0100, Daniel Vetter wrote: > > Two things seem to do the trick on my ivb machine here: > > - prevent the gt from powering down while waiting for seqno > > notification interrupts by grabbing the force_wake in get_irq (and > > dropping it in put_irq again). > > - ordering writes from the ring's CS by reading a CS register, ACTHD > > seems to work. > > > > Only the blt&bsd ring on ivb seem to be massively affected by this, > > but for paranoia do this dance also on the render ring and on snb > > (i.e. all gpus with forcewake). > > > > Tested with Eric's glCopyPixels loop which without this patch scores a > > missed irq every few seconds. > > > > This patch needs my forcewake rework to use a spinlock instead of > > dev->struct_mutex. > > > > v2: Improve the comment per Eugeni Dodonov's suggestion. > > > > Cc: sta...@kernel.org > > Cc: Eric Anholt <e...@anholt.net> > > Cc: Kenneth Graunke <kenn...@whitecape.org> > > Cc: Eugeni Dodonov <eugeni.dodo...@intel.com> > > Tested-by: Eugeni Dodonov <eugeni.dodo...@intel.com> > > Reviewed-by: Eugeni Dodonov <eugeni.dodo...@intel.com> > > Signed-Off-by: Daniel Vetter <daniel.vet...@ffwll.ch> > > From the internal doc "SNB GT PM Programming Guide", Section 4.3.1: > > "GT does not generate interrupts while in RC6 (by design)"
So they've gone out of their way to build broken stuff. Awesome. I'd say you've found the clue here -- I'm a lot happier with going with your patches now (and I was pretty happy with the gen7 side before). I'd just like to not mess with gen6 unless we've got missed irq bugs there to fix.
pgpycXmgxonl1.pgp
Description: PGP signature
_______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx