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.zhou@xxxxxxxxx > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Cc: Ben Widawsky <benjamin.widawsky@xxxxxxxxx> > > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> > > 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@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
