amdgpu: fix double mutex_unlock in amdgpu_bo_import
authorMarek Olšák <marek.olsak@amd.com>
Thu, 4 Jun 2015 16:57:57 +0000 (18:57 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 5 Aug 2015 17:47:51 +0000 (13:47 -0400)
The handles array is used below, which is followed by another unlock,
so remove the first one.

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
amdgpu/amdgpu_bo.c

index eca2c6f..66f1b9f 100644 (file)
@@ -452,10 +452,10 @@ int amdgpu_bo_import(amdgpu_device_handle dev,
        atomic_set(&bo->refcount, 1);
        bo->dev = dev;
        pthread_mutex_init(&bo->cpu_access_mutex, NULL);
-       pthread_mutex_unlock(&dev->bo_table_mutex);
 
        r = amdgpu_bo_map(bo, 1 << 20);
        if (r) {
+               pthread_mutex_unlock(&dev->bo_table_mutex);
                amdgpu_bo_reference(&bo, NULL);
                return r;
        }