diff options
author | Felix Kuehling <Felix.Kuehling@amd.com> | 2016-02-16 15:31:30 -0500 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-02-17 16:10:56 -0500 |
commit | 390be2824fa4211c2e973c69b72e04000559bba3 (patch) | |
tree | e3b04955a2882e185839afee3093b39c8c40d4c5 | |
parent | b8ea3783467586246d0b2b08f2e3f42853219d94 (diff) |
drm/amdgpu: Don't call interval_tree_remove in amdgpu_mn_destroy
rbtree_postorder_for_each_entry_safe can skip over some entries if
the tree is rebalanced in interval_tree_remove. interval_tree_remove
is also redundant when the tree is just about to be freed.
Signed-off-by: Felix Kuehling <Felix.Kuehling@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c index d12dff92f3ca..d7ec9bd6755f 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c | |||
@@ -76,7 +76,6 @@ static void amdgpu_mn_destroy(struct work_struct *work) | |||
76 | hash_del(&rmn->node); | 76 | hash_del(&rmn->node); |
77 | rbtree_postorder_for_each_entry_safe(node, next_node, &rmn->objects, | 77 | rbtree_postorder_for_each_entry_safe(node, next_node, &rmn->objects, |
78 | it.rb) { | 78 | it.rb) { |
79 | interval_tree_remove(&node->it, &rmn->objects); | ||
80 | list_for_each_entry_safe(bo, next_bo, &node->bos, mn_list) { | 79 | list_for_each_entry_safe(bo, next_bo, &node->bos, mn_list) { |
81 | bo->mn = NULL; | 80 | bo->mn = NULL; |
82 | list_del_init(&bo->mn_list); | 81 | list_del_init(&bo->mn_list); |