diff options
author | Alex Xie <AlexBin.Xie@amd.com> | 2017-01-28 21:49:30 +0200 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2017-02-02 15:22:07 -0500 |
commit | 7a03cdf6a703911d2a8e8ab0781f1e6b88412329 (patch) | |
tree | e70375dee26f7fc5a22ba9841971ff5359de9121 | |
parent | 07edf5d905fffb908f48996a4fcc6eac63b9c3ae (diff) |
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 <funfunctor@folklore1984.net>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Signed-off-by: Alex Xie <AlexBin.Xie@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
-rw-r--r-- | amdgpu/amdgpu_device.c | 4 |
1 files changed, 2 insertions, 2 deletions
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); |