diff options
author | Debarshi Dutta <ddutta@nvidia.com> | 2018-09-03 01:21:02 -0400 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2018-09-04 10:25:45 -0400 |
commit | bac38f52cc6f2ddc435a748dd9c85f3c5d88153f (patch) | |
tree | b53198fc14aa9d470d41b5d36caee934ef8c5bbc /drivers/gpu/nvgpu/gk20a/gk20a.h | |
parent | 16ad9f537979c5f3717fc5781b1c2fad22a76f96 (diff) |
gpu: nvgpu: protect clk_arb init with mutex.
g->clk_arb is currently initialized as a part of gk20a_finalize_poweron().
Any subsequent call to gk20a_finalize_poweron reinitializes the clk_arb
and leading to memory leaks. This is resolved by protecting the
g->clk_arb initialization with a mutex clk_arb_enable_lock in struct
gk20a. We skip initializing the g->clk_arb if its not NULL.
Bug 2061372
Change-Id: I59158e0a5e4c827fdbd6d9ea2d04c78d0986347a
Signed-off-by: Debarshi Dutta <ddutta@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1811650
Reviewed-by: Deepak Nibade <dnibade@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Vijayakumar Subbu <vsubbu@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Diffstat (limited to 'drivers/gpu/nvgpu/gk20a/gk20a.h')
-rw-r--r-- | drivers/gpu/nvgpu/gk20a/gk20a.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpu/nvgpu/gk20a/gk20a.h b/drivers/gpu/nvgpu/gk20a/gk20a.h index bdf3a168..f62dfb94 100644 --- a/drivers/gpu/nvgpu/gk20a/gk20a.h +++ b/drivers/gpu/nvgpu/gk20a/gk20a.h | |||
@@ -1588,6 +1588,8 @@ struct gk20a { | |||
1588 | 1588 | ||
1589 | struct nvgpu_clk_arb *clk_arb; | 1589 | struct nvgpu_clk_arb *clk_arb; |
1590 | 1590 | ||
1591 | struct nvgpu_mutex clk_arb_enable_lock; | ||
1592 | |||
1591 | struct gk20a_ce_app ce_app; | 1593 | struct gk20a_ce_app ce_app; |
1592 | 1594 | ||
1593 | bool ltc_intr_en_illegal_compstat; | 1595 | bool ltc_intr_en_illegal_compstat; |