From 7a03cdf6a703911d2a8e8ab0781f1e6b88412329 Mon Sep 17 00:00:00 2001 From: Alex Xie Date: Sat, 28 Jan 2017 21:49:30 +0200 Subject: amdgpu: Free/uninit vamgr_32 in theoretically correct order vamgr_32 is a region inside general VAM range. It is better to free and deinitialize it before general VAM range. Reviewed-by: Edward O'Callaghan Reviewed-by: Emil Velikov Signed-off-by: Alex Xie Reviewed-by: Christian König Signed-off-by: Alex Deucher --- amdgpu/amdgpu_device.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'amdgpu/amdgpu_device.c') diff --git a/amdgpu/amdgpu_device.c b/amdgpu/amdgpu_device.c index f4ede031..cad7133d 100644 --- a/amdgpu/amdgpu_device.c +++ b/amdgpu/amdgpu_device.c @@ -131,10 +131,10 @@ static int amdgpu_get_auth(int fd, int *auth) static void amdgpu_device_free_internal(amdgpu_device_handle dev) { - amdgpu_vamgr_deinit(dev->vamgr); - free(dev->vamgr); amdgpu_vamgr_deinit(dev->vamgr_32); free(dev->vamgr_32); + amdgpu_vamgr_deinit(dev->vamgr); + free(dev->vamgr); util_hash_table_destroy(dev->bo_flink_names); util_hash_table_destroy(dev->bo_handles); pthread_mutex_destroy(&dev->bo_table_mutex); -- cgit v1.2.3-54-g00ecf