diff options
| author | Xiaojie Yuan <xiaojie.yuan@amd.com> | 2020-07-14 15:47:31 +0800 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2020-07-14 14:38:48 -0400 |
| commit | 87d6883b327eef30b56e0a6bca9d5c9800ab6462 (patch) | |
| tree | 5ccfddbb52bce2d8954b2f0485c9421087d6e7bc /drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h | |
| parent | 9987d70d10eea0d685d4bfbf2101eaad64d36c37 (diff) | |
drm/amdgpu/sdma5: fix wptr overwritten in ->get_wptr()
"u64 *wptr" points to the the wptr value in write back buffer and
"*wptr = (*wptr) >> 2;" results in the value being overwritten each time
when ->get_wptr() is called.
umr uses /sys/kernel/debug/dri/0/amdgpu_ring_sdma0 to get rptr/wptr and
decode ring content and it is affected by this issue.
fix and simplify the logic similar as sdma_v4_0_ring_get_wptr().
v2: fix for sdma5.2 as well
Suggested-by: Le Ma <le.ma@amd.com>
Signed-off-by: Xiaojie Yuan <xiaojie.yuan@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h')
0 files changed, 0 insertions, 0 deletions