diff options
author | Kamil Dudka <kdudka@redhat.com> | 2015-07-15 11:18:15 -0400 |
---|---|---|
committer | Ben Skeggs <bskeggs@redhat.com> | 2015-07-27 04:56:07 -0400 |
commit | ac8c79304280da6ef05c348a9da03ab04898b994 (patch) | |
tree | 2b682f542de7df64f0503095332d931b2b2365c2 | |
parent | cbfe8fa6cd672011c755c3cd85c9ffd4e2d10a6f (diff) |
drm/nouveau: hold mutex when calling nouveau_abi16_fini()
This was the only access to cli->abi16 without holding the mutex.
Signed-off-by: Kamil Dudka <kdudka@redhat.com>
-rw-r--r-- | drivers/gpu/drm/nouveau/nouveau_drm.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c b/drivers/gpu/drm/nouveau/nouveau_drm.c index 649024d4daf1..01c78a4a3efa 100644 --- a/drivers/gpu/drm/nouveau/nouveau_drm.c +++ b/drivers/gpu/drm/nouveau/nouveau_drm.c | |||
@@ -865,8 +865,10 @@ nouveau_drm_preclose(struct drm_device *dev, struct drm_file *fpriv) | |||
865 | 865 | ||
866 | pm_runtime_get_sync(dev->dev); | 866 | pm_runtime_get_sync(dev->dev); |
867 | 867 | ||
868 | mutex_lock(&cli->mutex); | ||
868 | if (cli->abi16) | 869 | if (cli->abi16) |
869 | nouveau_abi16_fini(cli->abi16); | 870 | nouveau_abi16_fini(cli->abi16); |
871 | mutex_unlock(&cli->mutex); | ||
870 | 872 | ||
871 | mutex_lock(&drm->client.mutex); | 873 | mutex_lock(&drm->client.mutex); |
872 | list_del(&cli->head); | 874 | list_del(&cli->head); |