drm/amdgpu: once more fix the call oder in amdgpu_ttm_move() v2

Browse source

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: 94aeb41173 ("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

...

This commit is contained in:

Christian König 2024-03-21 11:32:02 +01:00 • committed by Alex Deucher

parent 4d4d3ff16d

commit ffda708148

3 changed files with 38 additions and 28 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

14

drivers/gpu/drm/amd/amdgpu/amdgpu_object.c

View file

4

drivers/gpu/drm/amd/amdgpu/amdgpu_object.h

View file

52

drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c

View file