diff options
author | Danilo Krummrich <dakr@redhat.com> | 2023-09-16 19:28:31 +0300 |
---|---|---|
committer | Danilo Krummrich <dakr@redhat.com> | 2023-09-20 01:15:50 +0300 |
commit | 31499b0192cea06bbfe2782f288ac5cfe3dc9167 (patch) | |
tree | 849beb5c09475a433f9a67821a92c274b24b2cdb /drivers/gpu/drm/nouveau/nouveau_exec.c | |
parent | 7ece3fc9b76b2d4596607fd8751f36c4e5f1f072 (diff) | |
download | linux-31499b0192cea06bbfe2782f288ac5cfe3dc9167.tar.xz |
drm/nouveau: sched: fix leaking memory of timedout job
Always stop and re-start the scheduler in order to let the scheduler
free up the timedout job in case it got signaled. In case of exec jobs
the job type specific callback will take care to signal all fences and
tear down the channel.
Fixes: b88baab82871 ("drm/nouveau: implement new VM_BIND uAPI")
Signed-off-by: Danilo Krummrich <dakr@redhat.com>
Reviewed-by: Dave Airlie <airlied@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230916162835.5719-1-dakr@redhat.com
Diffstat (limited to 'drivers/gpu/drm/nouveau/nouveau_exec.c')
-rw-r--r-- | drivers/gpu/drm/nouveau/nouveau_exec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/nouveau/nouveau_exec.c b/drivers/gpu/drm/nouveau/nouveau_exec.c index 19024ce21fbb..5dda94e1318c 100644 --- a/drivers/gpu/drm/nouveau/nouveau_exec.c +++ b/drivers/gpu/drm/nouveau/nouveau_exec.c @@ -213,7 +213,7 @@ nouveau_exec_job_timeout(struct nouveau_job *job) nouveau_sched_entity_fini(job->entity); - return DRM_GPU_SCHED_STAT_ENODEV; + return DRM_GPU_SCHED_STAT_NOMINAL; } static struct nouveau_job_ops nouveau_exec_job_ops = { |