]> www.infradead.org Git - users/jedix/linux-maple.git/commit
x86/platform/amd: move final timeout check to after final sleep
authorJake Hillion <jake@hillion.co.uk>
Thu, 5 Jun 2025 18:09:26 +0000 (19:09 +0100)
committerIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Mon, 9 Jun 2025 07:50:40 +0000 (10:50 +0300)
commitf8afb12a2d7503de6558c23cacd7acbf6e9fe678
tree2a5037152824220a89bb79417b0c07fdefc14615
parent4dbd11796f3a8eb95647507befc41995458a4023
x86/platform/amd: move final timeout check to after final sleep

__hsmp_send_message sleeps between result read attempts and has a
timeout of 100ms. Under extreme load it's possible for these sleeps to
take a long time, exceeding the 100ms. In this case the current code
does not check the register and fails with ETIMEDOUT.

Refactor the loop to ensure there is at least one read of the register
after a sleep of any duration. This removes instances of ETIMEDOUT with
a single caller, even with a misbehaving scheduler. Tested on AMD
Bergamo machines.

Suggested-by: Blaise Sanouillet <linux@blaise.sanouillet.com>
Reviewed-by: Suma Hegde <suma.hegde@amd.com>
Tested-by: Suma Hegde <suma.hegde@amd.com>
Signed-off-by: Jake Hillion <jake@hillion.co.uk>
Link: https://lore.kernel.org/r/20250605-amd-hsmp-v2-1-a811bc3dd74a@hillion.co.uk
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
drivers/platform/x86/amd/hsmp/hsmp.c