diff options
author | Mayuresh Kulkarni <mkulkarni@nvidia.com> | 2012-07-17 16:03:21 +0530 |
---|---|---|
committer | Simone Willett <swillett@nvidia.com> | 2012-07-24 11:42:08 -0700 |
commit | 519b5100bc17b177c2ba7f33c250646a5dacfee3 (patch) | |
tree | 6ba46f83fcc6388c05ea5c11eaee64efcc12191c /drivers/video/tegra/host | |
parent | 7e2377b9f79b5f8bc4b519444dc5e6441b2d8671 (diff) |
video: tegra: host: fix memory leak in ctx save
- host1x_save_context() explicitly updated the ref count on context
- with the re-factoring of nvhost_job code, nvhost_job_alloc()
takes a ref count on context
- this caused the explicit ref count in host1x_save_context()
redundant and lead to memory leak. hence remove it
Bug 1015924
Change-Id: Id18c74412e8659b60288700972690d1b895de4c1
Signed-off-by: Mayuresh Kulkarni <mkulkarni@nvidia.com>
Reviewed-on: http://git-master/r/116409
(cherry picked from commit c98d0cfbbdacdda8e540b5d9e0d513e279b3b2f5)
Reviewed-on: http://git-master/r/117326
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>
Diffstat (limited to 'drivers/video/tegra/host')
-rw-r--r-- | drivers/video/tegra/host/host1x/host1x_channel.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/video/tegra/host/host1x/host1x_channel.c b/drivers/video/tegra/host/host1x/host1x_channel.c index 9e9fc25dc966..1d02537f64b5 100644 --- a/drivers/video/tegra/host/host1x/host1x_channel.c +++ b/drivers/video/tegra/host/host1x/host1x_channel.c @@ -580,7 +580,6 @@ static int host1x_save_context(struct nvhost_channel *ch) } hwctx_to_save->valid = true; - ch->ctxhandler->get(hwctx_to_save); ch->cur_ctx = NULL; syncpt_id = to_host1x_hwctx_handler(hwctx_to_save->h)->syncpt; |