PSP_VMBX_POLLING_LIMIT too big
Mario Limonciello
mario.limonciello at amd.com
Tue Nov 28 16:21:04 UTC 2023
Hi,
In amd-staging-drm-next 46fe6312082c ("drm/amdgpu: update retry times
for psp BL wait") and upstream a11156ff6f41 ("drm/amdgpu: update retry
times for psp BL wait") the number of loops for
psp_v13_0_wait_for_bootloader() to try again increased significantly.
It went from 10 loops to 20k loops. Essentially this means that the
function can "effectively" no longer fail.
I've got an issue I'm looking at where runtime resume for a dGPU fails,
and because of this change the system gets stuck in a never ending busy
loop instead of cleanly returning an error code to the caller. The
outcome is the system appears hung while the 20k loops run instead of
just the dGPU failing to resume.
Is this 20k value really required? Or can we reduce it back to
something more manageable?
Thanks,
More information about the amd-gfx
mailing list