On Mon, Aug 04, 2014 at 06:20:00PM -0700, Ben Widawsky wrote: > On Mon, Aug 04, 2014 at 11:15:13AM -0700, Rodrigo Vivi wrote: > > From: Ben Widawsky <benjamin.widawsky@xxxxxxxxx> > > > > The PDPs seem to get screwed up otherwise, specifically PDP0. I am not > > really clear why this is required, it just works with full PPGTT. > > > > v2: Only do it for gen8, to limit regression potential > > > > v3: Fix the bugzilla links > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78891 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78935 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78936 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78937 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78938 > > > > Signed-off-by: Ben Widawsky <ben@xxxxxxxxxxxx> > > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > > Hi Rodrigo. This patch was superseded by my [partially merged] series > here: > http://lists.freedesktop.org/archives/intel-gfx/2014-July/048311.html > > In that series I went through very thoroughly with design why we need to > do this, and verified those patches do address this issue. > > So yeah, NAK from me - and please get those reworked or merged or > whatever. Iirc I've stopped merging on that series since it mixes up gen8 specific fixes with generic ppgtt fixes. Specifically for the ppgtt_release issues we now have a patch (plus fixups) which is almost ready (just stalling for review). Can you pls go through that series with Rodrigo so that he knows which parts are the gen8 fixes? Thanks, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
