Commit 62e2ff5d authored by Stanley Chu's avatar Stanley Chu Committed by Gerrit - the friendly Code Review server
Browse files

scsi: ufs: fix broken hba->outstanding_tasks



Currently bits in hba->outstanding_tasks are cleared only after their
corresponding task management commands are successfully done by
__ufshcd_issue_tm_cmd().

If timeout happens in a task management command, its corresponding bit in
hba->outstanding_tasks will not be cleared until next task management
command with the same tag used successfully finishes.

This is wrong and can lead to some issues, like power issue.  For example,
ufshcd_release() and ufshcd_gate_work() will do nothing if
hba->outstanding_tasks is not zero even if both UFS host and devices are
actually idle.

Solution is referred from error handling of device commands: bits in
hba->outstanding_tasks shall be cleared regardless of their execution
results.

Change-Id: I92269b69da38f65b23355aa928869ff2f98c9120
Signed-off-by: default avatarStanley Chu <stanley.chu@mediatek.com>
Signed-off-by: default avatarChun-Hung Wu <chun-hung.wu@mediatek.com>
Reviewed-by: default avatarAvri Altman <avri.altman@wdc.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
Git-commit: b557217c
Git-repo: https://android.googlesource.com/kernel/common/


[nitirawa@codeaurora.org: Ported to 4.19 kernel]
Signed-off-by: default avatarNitin Rawat <nitirawa@codeaurora.org>
parent 4625959c
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment