summaryrefslogtreecommitdiffstats
path: root/drivers/gpu/nvgpu/common/linux/module.c
diff options
context:
space:
mode:
authorskadamati <skadamati@nvidia.com>2017-06-30 04:47:25 -0400
committermobile promotions <svcmobile_promotions@nvidia.com>2017-07-05 12:36:03 -0400
commite768e74df8be8abe8442d958bde7a4a2886694ba (patch)
tree9dc9b2adb04e9b2b674cd29267752ed4d3e3807f /drivers/gpu/nvgpu/common/linux/module.c
parent8b36c45b39d06e273d00cd76180b589727dcde45 (diff)
gpu: nvgpu: Fix race condition during poweron
When two or more apps ran simultaneously First app context sets power_on flag & starts init Other app context check the power_on flag and try to use GPU without init completed Which makes aother apps to assert Added mutex to synchronize poweron access Bug 200297265 Change-Id: Ie138f7f43bb0dd3304ed91ae3649a6a4947bee91 Signed-off-by: skadamati <skadamati@nvidia.com> Reviewed-on: https://git-master/r/1511436 Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Diffstat (limited to 'drivers/gpu/nvgpu/common/linux/module.c')
-rw-r--r--drivers/gpu/nvgpu/common/linux/module.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/gpu/nvgpu/common/linux/module.c b/drivers/gpu/nvgpu/common/linux/module.c
index b107a721..0b5dfec8 100644
--- a/drivers/gpu/nvgpu/common/linux/module.c
+++ b/drivers/gpu/nvgpu/common/linux/module.c
@@ -88,15 +88,18 @@ int gk20a_busy(struct gk20a *g)
88 goto fail; 88 goto fail;
89 } 89 }
90 } else { 90 } else {
91 nvgpu_mutex_acquire(&g->poweron_lock);
91 if (!g->power_on) { 92 if (!g->power_on) {
92 ret = gk20a_gpu_is_virtual(dev) ? 93 ret = gk20a_gpu_is_virtual(dev) ?
93 vgpu_pm_finalize_poweron(dev) 94 vgpu_pm_finalize_poweron(dev)
94 : gk20a_pm_finalize_poweron(dev); 95 : gk20a_pm_finalize_poweron(dev);
95 if (ret) { 96 if (ret) {
96 atomic_dec(&g->usage_count); 97 atomic_dec(&g->usage_count);
98 nvgpu_mutex_release(&g->poweron_lock);
97 goto fail; 99 goto fail;
98 } 100 }
99 } 101 }
102 nvgpu_mutex_release(&g->poweron_lock);
100 } 103 }
101 104
102fail: 105fail: