diff options
| author | Terje Bergstrom <tbergstrom@nvidia.com> | 2012-06-07 08:09:43 +0300 |
|---|---|---|
| committer | Ryan Wong <ryanw@nvidia.com> | 2012-06-07 14:03:57 -0700 |
| commit | 886f0c357c2799085f9745b9c9ca99a4db8413ed (patch) | |
| tree | 33f40b5c6faa90e2b61343dc0585a233233e0dbf | |
| parent | 22a1bf6884e9fcb495f22090f561c1a9a5758775 (diff) | |
video: tegra: host: Do not allocate job at channelopen
Do not allocate a nvhost_job at channel open time. That instance is
never used, and is not freed, which causes a memory leak.
Bug 994844
Change-Id: I4e6aa4fe34a110600b80d559239422941677dd5e
Signed-off-by: Terje Bergstrom <tbergstrom@nvidia.com>
Reviewed-on: http://git-master/r/106943
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Ryan Wong <ryanw@nvidia.com>
Tested-by: Ryan Wong <ryanw@nvidia.com>
| -rw-r--r-- | drivers/video/tegra/host/bus_client.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/video/tegra/host/bus_client.c b/drivers/video/tegra/host/bus_client.c index e238844b1ad9..2590b3aef3ee 100644 --- a/drivers/video/tegra/host/bus_client.c +++ b/drivers/video/tegra/host/bus_client.c @@ -142,11 +142,6 @@ static int nvhost_channelopen(struct inode *inode, struct file *filp) &nvhost_get_host(ch->dev)->clientid); priv->timeout = MAX_STUCK_CHECK_COUNT * SYNCPT_CHECK_PERIOD; - priv->job = nvhost_job_alloc(ch, priv->hwctx, &priv->hdr, - NULL, priv->priority, priv->clientid); - if (!priv->job) - goto fail; - return 0; fail: nvhost_channelrelease(inode, filp); |