On Tue, Jan 07, 2014 at 08:43:28AM +0100, Daniel Vetter wrote: > On Wed, Jan 01, 2014 at 02:00:54PM +0000, Chris Wilson wrote: > > One side-effect of the introduction of ppgtt was that we needed to > > rebind the object into the appropriate vm (and global gtt in some > > peculiar cases). For simplicity this was done twice for every object on > > every call to execbuffer. However, that adds a tremendous amount of CPU > > overhead (rewriting all the PTE for all objects into WC memory) per > > draw. The fix is to push all the decision about which vm to bind into > > and when down into the low-level bind routines through hints rather than > > inside the execbuffer routine. > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=72906 > > Tested-by: jianx.z...@intel.com > > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk> > > Cc: Ben Widawsky <benjamin.widaw...@intel.com> > > Cc: Daniel Vetter <daniel.vet...@ffwll.ch> > > Can you please split out the code cleanups into a separate patch? I like > them, but as is they're hiding the actual bugfix in the diff quite badly > imo.
Which part of this is cleanup? -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx