From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 14/29] drm/i915: Restore PDEs on gtt restore Date: Tue, 6 Aug 2013 20:14:13 +0200 Message-ID: <20130806181413.GA22035@phenom.ffwll.local> References: <1375315222-4785-1-git-send-email-ben@bwidawsk.net> <1375315222-4785-15-git-send-email-ben@bwidawsk.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-we0-f174.google.com (mail-we0-f174.google.com [74.125.82.174]) by gabe.freedesktop.org (Postfix) with ESMTP id 76153E5D24 for ; Tue, 6 Aug 2013 11:14:07 -0700 (PDT) Received: by mail-we0-f174.google.com with SMTP id q54so668641wes.5 for ; Tue, 06 Aug 2013 11:14:06 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1375315222-4785-15-git-send-email-ben@bwidawsk.net> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Ben Widawsky Cc: Intel GFX List-Id: intel-gfx@lists.freedesktop.org On Wed, Jul 31, 2013 at 05:00:07PM -0700, Ben Widawsky wrote: > I can't remember why I added this initially. > > TODO: Throw out if not necessary > > Signed-off-by: Ben Widawsky Oops. I vaguely remember that there was some shuffling around with where the ppgtt pdes are written once we have real ppgtt. Since the aliasing ppgtt isn't a real ppgtt it could be that the write_pdes call in gen6_ppgtt_enable isn't executed any more and hence we need to do this manually. Otoh it's looks like we should just call gen6_ppgtt_enable in such a case. So I can't really help with a reason why we need this. I'm pretty sure that we don't need it right now, so I'll punt for now. But please keep it in mind when rebasing the real ppgtt stuff on top, once the VMA patches are merged to dinq. -Daniel > --- > drivers/gpu/drm/i915/i915_gem_gtt.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c > index 1ed9acb..e9b269f 100644 > --- a/drivers/gpu/drm/i915/i915_gem_gtt.c > +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c > @@ -470,6 +470,9 @@ void i915_gem_restore_gtt_mappings(struct drm_device *dev) > dev_priv->gtt.base.start / PAGE_SIZE, > dev_priv->gtt.base.total / PAGE_SIZE); > > + if (dev_priv->mm.aliasing_ppgtt) > + gen6_write_pdes(dev_priv->mm.aliasing_ppgtt); > + > list_for_each_entry(obj, &dev_priv->mm.bound_list, global_list) { > i915_gem_clflush_object(obj); > i915_gem_gtt_bind_object(obj, obj->cache_level); > -- > 1.8.3.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch