drm/ttm: Always and only destroy bo->ttm_resv in ttm_bo_release_list
authorMichel Dänzer <michel.daenzer@amd.com>
Fri, 3 Nov 2017 15:00:35 +0000 (16:00 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Sat, 4 Nov 2017 13:48:01 +0000 (09:48 -0400)
commite1fc12c5d9ad06a2a74e97a91f1b0c5f4c723b50
treea138da480d215bdc6608d6c275bdc6874d4bbb92
parent37c5f2c99adf63adf13f1ca309a1ffce25c5589a
drm/ttm: Always and only destroy bo->ttm_resv in ttm_bo_release_list

Fixes a use-after-free due to a race condition in
ttm_bo_cleanup_refs_and_unlock, which allows one task to reserve a BO
and destroy its ttm_resv while another task is waiting for it to signal
in reservation_object_wait_timeout_rcu.

v2:
* Always initialize bo->ttm_resv in ttm_bo_init_reserved
 (Christian König)

Fixes: 0d2bd2ae045d "drm/ttm: fix memory leak while individualizing BOs"
Reviewed-by: Chunming Zhou <david1.zhou@amd.com> # v1
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/ttm/ttm_bo.c