From 6ac44d2813f5ab1873eb5148f26746a8f9dbd4a8 Mon Sep 17 00:00:00 2001 From: Sachit Kadle Date: Mon, 27 Jun 2016 22:53:42 -0700 Subject: gpu: nvgpu: take platform power ref at power on Currently, host1x power refcount may decrement to 0, while GPU is still powered on and we're still servicing IRQs. So to prevent this situation,, take a ref while GPU is being powered on, and decrement it during power off. Since we are always holding one reference while GPU is powered on, we can remove this handling from gk20a_busy/idle() Bug 200187507 Change-Id: I249a4527178537c1dc53d769411f53c4451352c3 Signed-off-by: Sachit Kadle Reviewed-on: http://git-master/r/1172320 (cherry picked from commit 3e27e6a5820f5c1ad05596553d75e8979b71f1bd) Reviewed-on: http://git-master/r/1172607 (cherry picked from commit 1e01a49fdc139b8cdf5164b4a6767d22ef4ad1d3) Reviewed-on: http://git-master/r/1185175 GVS: Gerrit_Virtual_Submit Reviewed-by: Thomas Fleury Reviewed-by: Deepak Nibade Reviewed-by: Alex Waterman Reviewed-by: Vijayakumar Subbu --- drivers/gpu/nvgpu/gk20a/gk20a.c | 33 ++++++++++++++------------------- 1 file changed, 14 insertions(+), 19 deletions(-) (limited to 'drivers/gpu/nvgpu/gk20a/gk20a.c') diff --git a/drivers/gpu/nvgpu/gk20a/gk20a.c b/drivers/gpu/nvgpu/gk20a/gk20a.c index bb8cb33f..caa1583e 100644 --- a/drivers/gpu/nvgpu/gk20a/gk20a.c +++ b/drivers/gpu/nvgpu/gk20a/gk20a.c @@ -813,6 +813,10 @@ static int gk20a_pm_prepare_poweroff(struct device *dev) g->power_on = false; + /* Decrement platform power refcount */ + if (platform->idle) + platform->idle(dev); + /* Stop CPU from accessing the GPU registers. */ gk20a_lockout_registers(g); @@ -859,6 +863,16 @@ int gk20a_pm_finalize_poweron(struct device *dev) trace_gk20a_finalize_poweron(dev_name(dev)); + /* Increment platform power refcount */ + if (platform->busy) { + err = platform->busy(dev); + if (err < 0) { + dev_err(dev, "%s: failed to poweron platform dependency\n", + __func__); + goto done; + } + } + err = gk20a_restore_registers(g); if (err) return err; @@ -1787,25 +1801,12 @@ int gk20a_busy(struct device *dev) { int ret = 0; struct gk20a *g = get_gk20a(dev); - struct gk20a_platform *platform = gk20a_get_platform(dev); down_read(&g->busy_lock); - if (pm_runtime_enabled(dev)) { - if (platform->busy) { - ret = platform->busy(dev); - if (ret < 0) { - dev_err(dev, "%s: failed to poweron platform dependency\n", - __func__); - goto fail; - } - } - ret = pm_runtime_get_sync(dev); if (ret < 0) { pm_runtime_put_noidle(dev); - if (platform->idle) - platform->idle(dev); goto fail; } } else { @@ -1818,8 +1819,6 @@ int gk20a_busy(struct device *dev) } } - gk20a_scale_notify_busy(dev); - fail: up_read(&g->busy_lock); @@ -1828,8 +1827,6 @@ fail: void gk20a_idle(struct device *dev) { - struct gk20a_platform *platform = gk20a_get_platform(dev); - if (pm_runtime_enabled(dev)) { #ifdef CONFIG_PM if (atomic_read(&dev->power.usage_count) == 1) @@ -1839,8 +1836,6 @@ void gk20a_idle(struct device *dev) pm_runtime_mark_last_busy(dev); pm_runtime_put_sync_autosuspend(dev); - if (platform->idle) - platform->idle(dev); } else { gk20a_scale_notify_idle(dev); } -- cgit v1.2.2