diff options
| author | Jonathan McDowell <[email protected]> | 2025-03-12 05:31:57 +0000 |
|---|---|---|
| committer | Jarkko Sakkinen <[email protected]> | 2025-03-27 13:34:05 +0000 |
| commit | 7146dffa875cd00e7a7f918e1fce79c7593ac1fa (patch) | |
| tree | 789fc2ae4d231b5f1eb1b7108d48a7d90902e27b /drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | |
| parent | tpm: Convert warn to dbg in tpm2_start_auth_session() (diff) | |
| download | kernel-7146dffa875cd00e7a7f918e1fce79c7593ac1fa.tar.gz kernel-7146dffa875cd00e7a7f918e1fce79c7593ac1fa.zip | |
tpm, tpm_tis: Fix timeout handling when waiting for TPM status
The change to only use interrupts to handle supported status changes
introduced an issue when it is necessary to poll for the status. Rather
than checking for the status after sleeping the code now sleeps after
the check. This means a correct, but slower, status change on the part
of the TPM can be missed, resulting in a spurious timeout error,
especially on a more loaded system. Switch back to sleeping *then*
checking. An up front check of the status has been done at the start of
the function, so this does not cause an additional delay when the status
is already what we're looking for.
Cc: [email protected] # v6.4+
Fixes: e87fcf0dc2b4 ("tpm, tpm_tis: Only handle supported interrupts")
Signed-off-by: Jonathan McDowell <[email protected]>
Reviewed-by: Michal Suchánek <[email protected]>
Reviewed-by: Lino Sanfilippo <[email protected]>
Reviewed-by: Jarkko Sakkinen <[email protected]>
Signed-off-by: Jarkko Sakkinen <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c')
0 files changed, 0 insertions, 0 deletions
