drm/amdgpu: once more fix the call oder in amdgpu_ttm_move() v2
authorChristian König <christian.koenig@amd.com>
Thu, 21 Mar 2024 10:32:02 +0000 (11:32 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 1 May 2024 01:39:57 +0000 (21:39 -0400)
commitd3a9331a6591e9df64791e076f6591f440af51c3
tree68bc043ad57e5b9bf8e3173bf40f6de250db302f
parent46fe9cb1a9e62f4e6229f48ae303ef8e6c1fdc64
drm/amdgpu: once more fix the call oder in amdgpu_ttm_move() v2

This reverts drm/amdgpu: fix ftrace event amdgpu_bo_move always move
on same heap. The basic problem here is that after the move the old
location is simply not available any more.

Some fixes were suggested, but essentially we should call the move
notification before actually moving things because only this way we have
the correct order for DMA-buf and VM move notifications as well.

Also rework the statistic handling so that we don't update the eviction
counter before the move.

v2: add missing NULL check

Signed-off-by: Christian König <christian.koenig@amd.com>
Fixes: 94aeb4117343 ("drm/amdgpu: fix ftrace event amdgpu_bo_move always move on same heap")
Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/3171
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
CC: stable@vger.kernel.org
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
drivers/gpu/drm/amd/amdgpu/amdgpu_object.h
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c