[Intel-gfx] [PATCH] igt/gem_workarounds: igt to test workaround registers

[Chris Wilson]

On Wed, Aug 27, 2014 at 05:17:11PM +0100, Siluvery, Arun wrote:
> On 27/08/2014 16:59, Chris Wilson wrote:
> >On Wed, Aug 27, 2014 at 05:50:16PM +0200, Daniel Vetter wrote:
> >>On Tue, Aug 26, 2014 at 02:50:28PM +0100, Arun Siluvery wrote:
> >>>Some of the workarounds are lost followed by a gpu reset, suspend/resume;
> >>>this patch adds a test which compares register state before and after
> >>>the test scenario.
> >>>
> >>>This test currently verifies only bdw workarounds.
> >
> >The existing tool didn't need kernel help (other than forcewake). Why
> >was that not used as a starting point?
> >-Chris
> >
> Do you mean intel_reg_checker()?
> This new test uses kernel help to get the initial state of
> workarounds which are exported to debugfs. We could add this known
> state to the test itself but Daniel is not ok with that. debugfs
> part is only added to support the test.

I disagree vehemently with Daniel here then. The kernel lies.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre