From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] drm/i915: Wrap external callers to IPS state with appropriate locks Date: Wed, 26 Sep 2012 13:47:23 +0200 Message-ID: <20120926114723.GA1980@bremse> References: <1348564572-20046-1-git-send-email-chris@chris-wilson.co.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-lb0-f177.google.com (mail-lb0-f177.google.com [209.85.217.177]) by gabe.freedesktop.org (Postfix) with ESMTP id 5810B9E776 for ; Wed, 26 Sep 2012 04:47:33 -0700 (PDT) Received: by lbbgi11 with SMTP id gi11so833752lbb.36 for ; Wed, 26 Sep 2012 04:47:32 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1348564572-20046-1-git-send-email-chris@chris-wilson.co.uk> 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: Chris Wilson Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Tue, Sep 25, 2012 at 10:16:12AM +0100, Chris Wilson wrote: > Finishes commit 02d719562ef40483648b2cc46899d4a2ff5953bb > Author: Daniel Vetter > Date: Thu Aug 9 16:44:54 2012 +0200 > > drm/i915: properly guard ilk ips state > > The core functions were annotated with their locking requirements, but > we overlooked that they were exported, without any control over the > locking, to debugfs. So in order to enable debugfs to read the registers > without triggering sanity checks, we change the exported entry points to > properly take the required locks before calling the core routines. > > Reported-by: yangguang > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=55304 > Signed-off-by: Chris Wilson Merged, thanks for the patch. -Daniel > --- > drivers/gpu/drm/i915/intel_pm.c | 40 +++++++++++++++++++++++++++++++++++---- > 1 file changed, 36 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c > index 4e16cc1..f459e30 100644 > --- a/drivers/gpu/drm/i915/intel_pm.c > +++ b/drivers/gpu/drm/i915/intel_pm.c > @@ -2735,7 +2735,7 @@ static const struct cparams { > { 0, 800, 231, 23784 }, > }; > > -unsigned long i915_chipset_val(struct drm_i915_private *dev_priv) > +static unsigned long __i915_chipset_val(struct drm_i915_private *dev_priv) > { > u64 total_count, diff, ret; > u32 count1, count2, count3, m = 0, c = 0; > @@ -2789,6 +2789,22 @@ unsigned long i915_chipset_val(struct drm_i915_private *dev_priv) > return ret; > } > > +unsigned long i915_chipset_val(struct drm_i915_private *dev_priv) > +{ > + unsigned long val; > + > + if (dev_priv->info->gen != 5) > + return 0; > + > + spin_lock_irq(&mchdev_lock); > + > + val = __i915_chipset_val(dev_priv); > + > + spin_unlock_irq(&mchdev_lock); > + > + return val; > +} > + > unsigned long i915_mch_val(struct drm_i915_private *dev_priv) > { > unsigned long m, x, b; > @@ -2992,7 +3008,7 @@ void i915_update_gfx_val(struct drm_i915_private *dev_priv) > spin_unlock_irq(&mchdev_lock); > } > > -unsigned long i915_gfx_val(struct drm_i915_private *dev_priv) > +static unsigned long __i915_gfx_val(struct drm_i915_private *dev_priv) > { > unsigned long t, corr, state1, corr2, state2; > u32 pxvid, ext_v; > @@ -3029,6 +3045,22 @@ unsigned long i915_gfx_val(struct drm_i915_private *dev_priv) > return dev_priv->ips.gfx_power + state2; > } > > +unsigned long i915_gfx_val(struct drm_i915_private *dev_priv) > +{ > + unsigned long val; > + > + if (dev_priv->info->gen != 5) > + return 0; > + > + spin_lock_irq(&mchdev_lock); > + > + val = __i915_gfx_val(dev_priv); > + > + spin_unlock_irq(&mchdev_lock); > + > + return val; > +} > + > /** > * i915_read_mch_val - return value for IPS use > * > @@ -3045,8 +3077,8 @@ unsigned long i915_read_mch_val(void) > goto out_unlock; > dev_priv = i915_mch_dev; > > - chipset_val = i915_chipset_val(dev_priv); > - graphics_val = i915_gfx_val(dev_priv); > + chipset_val = __i915_chipset_val(dev_priv); > + graphics_val = __i915_gfx_val(dev_priv); > > ret = chipset_val + graphics_val; > > -- > 1.7.10.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