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

gpu: nvgpu: cde: do not rearm deleter on failure

Browse Source 
Rescheduling the temp context deleter when it is not immediately
possible is not necessary, and complicates things. Don't do it.

The context would anyway be used later when its time comes in the free
list, and the deletion would then be retried.

This simplifies canceling the works when shutting down or going into
suspend, since re-canceling the possibly rescheduled work is not needed.
Releasing the app mutex is still necessary when deleting the whole cde.

Bug 200052943

Change-Id: I06afe1766097a78d7bcb93f3140855799ac903ca
Signed-off-by: Konsta Holtta <kholtta@nvidia.com>
Reviewed-on: http://git-master/r/601035
GVS: Gerrit_Virtual_Submit
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>
This commit is contained in:
Konsta Holtta
2014-11-11 11:23:45 +02:00
committed by Dan Willemsen
parent 6a4597d5cd
commit 0e112b6b4b
1 changed files with 19 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

41
drivers/gpu/nvgpu/gk20a/cde_gk20a.c
View File

@@ -91,7 +91,8 @@ __must_hold(&cde_app->mutex)
kfree(cde_ctx);
}
static void gk20a_cde_cancel_deleter(struct gk20a_cde_ctx *cde_ctx)
static void gk20a_cde_cancel_deleter(struct gk20a_cde_ctx *cde_ctx,
bool wait_finish)
__releases(&cde_app->mutex)
__acquires(&cde_app->mutex)
{
@@ -101,14 +102,13 @@ __acquires(&cde_app->mutex)
if (!cde_ctx->is_temporary)
return;
mutex_unlock(&cde_app->mutex);
/* the deleter can rearm itself */
while (delayed_work_pending(&cde_ctx->ctx_deleter_work)) {
if (wait_finish) {
mutex_unlock(&cde_app->mutex);
cancel_delayed_work_sync(&cde_ctx->ctx_deleter_work);
mutex_lock(&cde_app->mutex);
} else {
cancel_delayed_work(&cde_ctx->ctx_deleter_work);
}
mutex_lock(&cde_app->mutex);
}
static void gk20a_cde_remove_contexts(struct gk20a *g)
@@ -123,13 +123,13 @@ __must_hold(&cde_app->mutex)
list_for_each_entry_safe(cde_ctx, cde_ctx_save,
&cde_app->free_contexts, list) {
gk20a_cde_cancel_deleter(cde_ctx);
gk20a_cde_cancel_deleter(cde_ctx, true);
gk20a_cde_remove_ctx(cde_ctx);
}
list_for_each_entry_safe(cde_ctx, cde_ctx_save,
&cde_app->used_contexts, list) {
gk20a_cde_cancel_deleter(cde_ctx);
gk20a_cde_cancel_deleter(cde_ctx, true);
gk20a_cde_remove_ctx(cde_ctx);
}
}
@@ -173,14 +173,12 @@ __releases(&cde_app->mutex)
list_for_each_entry_safe(cde_ctx, cde_ctx_save,
&cde_app->free_contexts, list) {
if (cde_ctx->is_temporary)
cancel_delayed_work(&cde_ctx->ctx_deleter_work);
gk20a_cde_cancel_deleter(cde_ctx, false);
}
list_for_each_entry_safe(cde_ctx, cde_ctx_save,
&cde_app->used_contexts, list) {
if (cde_ctx->is_temporary)
cancel_delayed_work(&cde_ctx->ctx_deleter_work);
gk20a_cde_cancel_deleter(cde_ctx, false);
}
mutex_unlock(&cde_app->mutex);
@@ -720,7 +718,7 @@ static int gk20a_cde_execute_buffer(struct gk20a_cde_ctx *cde_ctx,
num_entries, flags, fence, fence_out);
}
static void gk20a_ctx_release(struct gk20a_cde_ctx *cde_ctx)
static void gk20a_cde_ctx_release(struct gk20a_cde_ctx *cde_ctx)
__acquires(&cde_app->mutex)
__releases(&cde_app->mutex)
{
@@ -755,16 +753,15 @@ __releases(&cde_app->mutex)
gk20a_dbg(gpu_dbg_fn | gpu_dbg_cde_ctx,
"cde: attempting to delete temporary %p", cde_ctx);
/* this should fail only when shutting down the whole device */
err = gk20a_busy(pdev);
if (WARN(err, "gk20a cde: cannot set gk20a on, not freeing channel yet."
" rescheduling...")) {
schedule_delayed_work(&cde_ctx->ctx_deleter_work,
msecs_to_jiffies(CTX_DELETE_TIME));
if (err) {
/* this context would find new use anyway later, so not freeing
* here does not leak anything */
gk20a_warn(&pdev->dev, "cde: cannot set gk20a on, postponing"
" temp ctx deletion");
return;
}
/* mark so that nobody else assumes it's free to take */
mutex_lock(&cde_app->mutex);
if (cde_ctx->in_use || !cde_app->initialised) {
gk20a_dbg(gpu_dbg_cde_ctx,
@@ -809,7 +806,7 @@ __must_hold(&cde_app->mutex)
/* cancel any deletions now that ctx is in use */
if (delayed_work_pending(&cde_ctx->ctx_deleter_work))
gk20a_cde_cancel_deleter(cde_ctx);
gk20a_cde_cancel_deleter(cde_ctx, false);
return cde_ctx;
}
@@ -1035,7 +1032,7 @@ __releases(&cde_app->mutex)
mutex_unlock(&cde_app->mutex);
}
} else {
gk20a_ctx_release(cde_ctx);
gk20a_cde_ctx_release(cde_ctx);
}
/* delete temporary contexts later */