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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx