[Intel-gfx] [PATCH 03/14] drm/i915: Disable runtime-pm using lowlevel functions if !HAS_RC6

Chris Wilson chris at chris-wilson.co.uk
Tue Aug 14 12:15:53 UTC 2018


Quoting Mika Kuoppala (2018-08-14 13:06:45)
> Chris Wilson <chris at chris-wilson.co.uk> writes:
> 
> > If we cannot setup rc6, we cannot let the GPU suspend itself as it
> > cannot save its state (to a powercontext). As such, we must disable
> > runtime-pm, but we should do so using the low-level pm-runtime function
> > which leaves our own debugging functions intact (and continue to detect
> > errors in our runtime-pm handling should we ever be able to enable rc6).
> >
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > ---
> >  drivers/gpu/drm/i915/intel_pm.c | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
> > index 9a01560c5bd1..d99e5fabe93c 100644
> > --- a/drivers/gpu/drm/i915/intel_pm.c
> > +++ b/drivers/gpu/drm/i915/intel_pm.c
> > @@ -26,6 +26,7 @@
> >   */
> >  
> >  #include <linux/cpufreq.h>
> > +#include <linux/pm_runtime.h>
> >  #include <drm/drm_plane_helper.h>
> >  #include "i915_drv.h"
> >  #include "intel_drv.h"
> > @@ -8181,7 +8182,7 @@ void intel_init_gt_powersave(struct drm_i915_private *dev_priv)
> >        */
> >       if (!sanitize_rc6(dev_priv)) {
> >               DRM_INFO("RC6 disabled, disabling runtime PM support\n");
> > -             intel_runtime_pm_get(dev_priv);
> > +             pm_runtime_get(&dev_priv->drm.pdev->dev);
> 
> Should this be pm_runtime_get_sync()?

It can be. It's already active at this point so there's no difference in
practice.
-Chris


More information about the Intel-gfx mailing list