[Intel-gfx] [PATCH] drm/i915: Do a lightweight pm_get() from intel_mark_busy()
Daniel Vetter
daniel at ffwll.ch
Wed Jun 24 01:40:02 PDT 2015
On Wed, Jun 24, 2015 at 09:09:03AM +0100, Chris Wilson wrote:
> Akash noticed that we were recursing from the call to
> intel_runtime_pm_get() inside intel_mark_busy() when we were already
> waking the device (through another intel_runtime_pm_get()). In
> intel_mark_busy() we know the device is awake and purpose of the
> reference here is to simply keep the device awake until the GPU is idle
> again. As such we do not need the full resume, and can call the lighter
> intel_runtime_pm_get_noresume() instead.
How does that happen? I only see a mark_busy in add_request, and we
shouldn't call that from intel_rpm_get(). Or do we? Calltrace of how this
happens would be great.
-Daniel
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Akash Goel <akash.goel at intel.com>
> Cc: Imre Deak <imre.deak at intel.com>
> ---
> drivers/gpu/drm/i915/intel_display.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index a43abc38af90..da0d882cc71e 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -10668,7 +10668,7 @@ void intel_mark_busy(struct drm_device *dev)
> if (dev_priv->mm.busy)
> return;
>
> - intel_runtime_pm_get(dev_priv);
> + intel_runtime_pm_get_noresume(dev_priv);
> intel_rps_busy(dev_priv);
> dev_priv->mm.busy = true;
> }
> --
> 2.1.4
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
More information about the Intel-gfx
mailing list