From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Widawsky Subject: Re: [PATCH] drm/i915: Update gen8_ppgtt_info to keep working in full ppgtt Date: Fri, 8 Aug 2014 17:14:32 -0700 Message-ID: <20140809001432.GA21621@bwidawsk.net> References: <1407503432-14037-1-git-send-email-michel.thierry@intel.com> <20140808134754.GR8727@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail.bwidawsk.net (bwidawsk.net [166.78.191.112]) by gabe.freedesktop.org (Postfix) with ESMTP id 7C8F96E8C3 for ; Fri, 8 Aug 2014 17:12:46 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140808134754.GR8727@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Fri, Aug 08, 2014 at 03:47:54PM +0200, Daniel Vetter wrote: > On Fri, Aug 08, 2014 at 02:10:32PM +0100, Michel Thierry wrote: > > The driver will no longer initialize the aliasing ppgtt if we have > > full ppgtt enabled. > > > > gen8_ppgtt_info uses the aliasing ppgtt or the ppgtt from the > > default context. This patch makes it clear. > > > > Cc: Daniel Vetter > > Signed-off-by: Michel Thierry > > I actually looked at this one here and decided that ppgtt_info for gen8 is > sufficiently broken so that we don't care. Imo we should refactor it a bit > so that it does the same as the gen6 version, i.e. dump the aliasing ppgtt > or the contexts. > > Trying to dump the default ctx ppgtt when full ppgtt is enabled is fairly > usesless since nothing at all will ever use that one. > > Best approach is probably to remove the gen8 version and push the GenX > checks down into the actual ppgtt dump functions or something like that. > -Daniel I did start down this path here: http://patchwork.freedesktop.org/patch/25728/ (originally here: http://patchwork.freedesktop.org/patch/22348/) It also helped with the later addition for dynamic page tables. I would ask that someone look at fixing that patch to their needs should the dynamic page table patches ever get merged. > > > --- > > drivers/gpu/drm/i915/i915_debugfs.c | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c > > index 8dc82c3..a62ac66 100644 > > --- a/drivers/gpu/drm/i915/i915_debugfs.c > > +++ b/drivers/gpu/drm/i915/i915_debugfs.c > > @@ -1995,8 +1995,14 @@ static void gen8_ppgtt_info(struct seq_file *m, struct drm_device *dev) > > struct drm_i915_private *dev_priv = dev->dev_private; > > struct intel_engine_cs *ring; > > struct i915_hw_ppgtt *ppgtt = dev_priv->mm.aliasing_ppgtt; > > + struct intel_context *ctx = dev_priv->ring[RCS].default_context; > > int unused, i; > > > > + if (USES_FULL_PPGTT(dev)) > > + ppgtt = ctx->ppgtt; > > + else > > + ppgtt = dev_priv->mm.aliasing_ppgtt; > > + > > if (!ppgtt) > > return; > > > > -- > > 2.0.3 > > > > -- > Daniel Vetter > Software Engineer, Intel Corporation > +41 (0) 79 365 57 48 - http://blog.ffwll.ch > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ben Widawsky, Intel Open Source Technology Center