Skip to content

Commit 92d0a28

Browse files
alexdeuchergregkh
authored andcommitted
drm/amdgpu/hdp7: use memcfg register to post the write for HDP flush
commit 5a11a27 upstream. Reading back the remapped HDP flush register seems to cause problems on some platforms. All we need is a read, so read back the memcfg register. Fixes: 6892751 ("drm/amdgpu/hdp7.0: do a posting read when flushing HDP") Reported-by: Alexey Klimov <[email protected]> Link: https://lists.freedesktop.org/archives/amd-gfx/2025-April/123150.html Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4119 Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/3908 Reviewed-by: Felix Kuehling <[email protected]> Signed-off-by: Alex Deucher <[email protected]> (cherry picked from commit dbc064a) Cc: [email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]>
1 parent df04418 commit 92d0a28

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

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

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,12 @@ static void hdp_v7_0_flush_hdp(struct amdgpu_device *adev,
3333
{
3434
if (!ring || !ring->funcs->emit_wreg) {
3535
WREG32((adev->rmmio_remap.reg_offset + KFD_MMIO_REMAP_HDP_MEM_FLUSH_CNTL) >> 2, 0);
36-
RREG32((adev->rmmio_remap.reg_offset + KFD_MMIO_REMAP_HDP_MEM_FLUSH_CNTL) >> 2);
36+
/* We just need to read back a register to post the write.
37+
* Reading back the remapped register causes problems on
38+
* some platforms so just read back the memory size register.
39+
*/
40+
if (adev->nbio.funcs->get_memsize)
41+
adev->nbio.funcs->get_memsize(adev);
3742
} else {
3843
amdgpu_ring_emit_wreg(ring, (adev->rmmio_remap.reg_offset + KFD_MMIO_REMAP_HDP_MEM_FLUSH_CNTL) >> 2, 0);
3944
}

0 commit comments

Comments
 (0)