Quoting Imre Deak (2018-02-08 11:23:31) > FORCEWAKE_ACK is depricated by BSpec at least starting from BDW, > referring to the multi-threaded version of it instead. Accessing > FORCEWAKE_ACK triggers an unclaimed register access error - at > least on GLK - see the Reference: below. > > The correct registers to use would be FORCEWAKE_MT_ACK on IVB+ and > FORCEWAKE_ACK_RENDER_GEN9 on SKL+ like it's done elsewhere in the > driver. > > The forcewake check itself is inconsistent and redundant, since there > could be other forcewake requesters besides the kernel (being the > multithreaded version of the register) and the kernel's per-domain > forcewake counters are shown anyway at the end of the file. So let's > just remove the check. > > Suggested-by: Chris Wilson <ch...@chris-wilson.co.uk> > Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> > Cc: Chris Wilson <ch...@chris-wilson.co.uk> > Cc: Mika Kuoppala <mika.kuopp...@linux.intel.com> > Reference: https://bugs.freedesktop.org/show_bug.cgi?id=103337 > Signed-off-by: Imre Deak <imre.d...@intel.com>
Works for me, Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
