diff options
| author |   Rob Clark <robdclark@chromium.org> | 2020-08-17 15:01:26 -0700 |
|---|---|---|
| committer | Rob Clark <robdclark@chromium.org> | 2020-09-12 10:45:55 -0700 |
| commit | a3367f5ffe9a49d1184815be7b841c9d11ec7bce (patch) | |
| tree | 8357ed2d957ae17299b1e17cb42916c04527a73b /drivers/gpu/drm/msm | |
| parent | drm/msm/a6xx: fix a potential overflow issue (diff) | |
| download | linux-dev-a3367f5ffe9a49d1184815be7b841c9d11ec7bce.tar.xz linux-dev-a3367f5ffe9a49d1184815be7b841c9d11ec7bce.zip | |
drm/msm: Remove dangling submitqueue references
Currently it doesn't matter, since we free the ctx immediately. But
when we start refcnt'ing the ctx, we don't want old dangling list
entries to hang around.
Signed-off-by: Rob Clark <robdclark@chromium.org>
Reviewed-by: Jordan Crouse <jcrouse@codeaurora.org>
Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Diffstat (limited to 'drivers/gpu/drm/msm')
| -rw-r--r-- | drivers/gpu/drm/msm/msm_submitqueue.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/gpu/drm/msm/msm_submitqueue.c b/drivers/gpu/drm/msm/msm_submitqueue.c index a1d94be7883a..90c9d84e6155 100644 --- a/drivers/gpu/drm/msm/msm_submitqueue.c +++ b/drivers/gpu/drm/msm/msm_submitqueue.c @@ -49,8 +49,10 @@ void msm_submitqueue_close(struct msm_file_private *ctx) * No lock needed in close and there won't * be any more user ioctls coming our way */ - list_for_each_entry_safe(entry, tmp, &ctx->submitqueues, node) + list_for_each_entry_safe(entry, tmp, &ctx->submitqueues, node) { + list_del(&entry->node); msm_submitqueue_put(entry); + } } int msm_submitqueue_create(struct drm_device *drm, struct msm_file_private *ctx, |