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 15:42:17 -0400 |
commit | 05051496b2622e4d12e2036b35165969aa502f89 (patch) | |
tree | 772b65efe21dc07bf473e1ce65527acabb433bc7 /drivers/visorbus | |
parent | 98a34cf931e848f8489d3fb15a8f5fc03802ad65 (diff) | |
download | linux-05051496b2622e4d12e2036b35165969aa502f89.tar.bz2 |
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
v3: drop sdma 5.2 changes for 5.8 and stable
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>
Cc: stable@vger.kernel.org
Diffstat (limited to 'drivers/visorbus')
0 files changed, 0 insertions, 0 deletions