[Mesa-dev] [PATCH 02/13] i965: Keep track of whether LRI is allowed in the context struct.

Chris Wilson chris at chris-wilson.co.uk
Sun Sep 6 09:40:12 PDT 2015


On Sun, Sep 06, 2015 at 07:28:12PM +0300, Francisco Jerez wrote:
> Chris Wilson <chris at chris-wilson.co.uk> writes:
> 
> > On Sun, Sep 06, 2015 at 06:12:40PM +0200, Francisco Jerez wrote:
> >> This stores the result of can_do_pipelined_register_writes() in the
> >> context struct so we can find out later whether LRI can be used to
> >> program the L3 configuration.
> >
> > LRI are whitelisted by their register. To be generic you must explicitly
> > test access to the register you want to modify.
> > -Chris
> >
> AFAIK except for the chicken bits used to enable L3 atomics on HSW
> (which is tested explicitly elsewhere, see PATCH 07), all other
> registers written to are whitelisted since command parser revision 1 --
> I don't think that any released kernel version had the command parser
> enabled on Gen7 but any of the required registers blacklisted.

At the moment you are not even checking that param. The test is written
to be independent of knowledge of  cmdparser internals and by virtue a
much stronger test. Use it to your advantage.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the mesa-dev mailing list