Revert "drm/amdgpu: move PD/PT bos on LRU again"
authorMichel Dänzer <[email protected]>
Wed, 29 Aug 2018 09:59:38 +0000 (11:59 +0200)
committerAlex Deucher <[email protected]>
Wed, 29 Aug 2018 17:35:43 +0000 (12:35 -0500)
This reverts commit 31625ccae4464b61ec8cdb9740df848bbc857a5b.

It triggered various badness on my development machine when running the
piglit gpu profile with radeonsi on Bonaire, looks like memory
corruption due to insufficiently protected list manipulations.

Signed-off-by: Michel Dänzer <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c

index 23c78af850c61a7f5a62830e8824aff18e102239..521ddb358ec17555e4f3f3fe27d951872928105c 100644 (file)
@@ -1127,7 +1127,7 @@ restart:
                                           struct amdgpu_vm_bo_base,
                                           vm_status);
                bo_base->moved = false;
-               list_move(&bo_base->vm_status, &vm->idle);
+               list_del_init(&bo_base->vm_status);
 
                bo = bo_base->bo->parent;
                if (!bo)