nv-tegra-mirror/linux-nvgpu
1
0
Fork 0
mirror of git://nv-tegra.nvidia.com/linux-nvgpu.git synced 2025-12-24 02:22:34 +03:00
Code Issues Packages Projects Releases Wiki Activity

gpu: nvgpu: take platform power ref at power on

Browse Source 
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 <skadle@nvidia.com>
Reviewed-on: http://git-master/r/1172320
(cherry picked from commit 3e27e6a5820f5c1ad05596553d75e8979b71f1bd)
Reviewed-on: http://git-master/r/1172607
(cherry picked from commit 1e01a49fdc)
Reviewed-on: http://git-master/r/1185175
GVS: Gerrit_Virtual_Submit
Reviewed-by: Thomas Fleury <tfleury@nvidia.com>
Reviewed-by: Deepak Nibade <dnibade@nvidia.com>
Reviewed-by: Alex Waterman <alexw@nvidia.com>
Reviewed-by: Vijayakumar Subbu <vsubbu@nvidia.com>
This commit is contained in:
Sachit Kadle
2016-06-27 22:53:42 -07:00
committed by Vijayakumar Subbu
parent 823f00d484
commit 6ac44d2813
1 changed files with 14 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
drivers/gpu/nvgpu/gk20a/gk20a.c
View File

@@ -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);
}