On Mon, Apr 18, 2016 at 10:04:19AM +0300, Imre Deak wrote:
> If system resume fails, this may lead to a runtime PM wake reference
> underflow used for runtime PM state checking.
If a later resume succeeds!
> Fixes: 1f814daca43a ("drm/i915: add support for checking if we hold an RPM
> reference")
> Signed-off-by: Imre Deak <imre.d...@intel.com>
Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk>
> ---
>
> As this only fixes a debugging feature, it's not for -fixes or stable.
Following a catastrophic device failure.
> ---
> drivers/gpu/drm/i915/i915_drv.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
> index 50bc05f..d550ae2 100644
> --- a/drivers/gpu/drm/i915/i915_drv.c
> +++ b/drivers/gpu/drm/i915/i915_drv.c
> @@ -842,11 +842,11 @@ static int i915_drm_resume_early(struct drm_device *dev)
> !(dev_priv->suspended_to_idle && dev_priv->csr.dmc_payload))
> intel_power_domains_init_hw(dev_priv, true);
>
> + enable_rpm_wakeref_asserts(dev_priv);
> +
> out:
> dev_priv->suspended_to_idle = false;
I'm not even sure about twiddling anything here, an early return if
pci_enable_device() fails would be clearer.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
