[Intel-gfx] [PATCH 1/2] drm/i915: Split out !RCS legacy context switching

Chris Wilson chris at chris-wilson.co.uk
Wed Apr 13 15:04:13 UTC 2016


On Wed, Apr 13, 2016 at 04:56:05PM +0200, Daniel Vetter wrote:
> On Wed, Apr 13, 2016 at 03:16:30PM +0100, Chris Wilson wrote:
> > +		if (needs_pd_load_pre(engine, to)) {
> 
> Hm, I'd inline this condition now since it's a bit confusing if there's no
> POST. Assuming I read code correctly it seems to boil down to to->ppgtt
> (which reads a lot cleaner) for !RCS.

I did it this way because it makes a later patch trivial ;)

Yes, it boils down to just a couple of checks. I plan a third in
hopefully not the too distant future.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list