From: Can Guo <cang@codeaurora.org>
To: asutoshd@codeaurora.org, nguyenb@codeaurora.org,
hongwus@codeaurora.org, rnayak@codeaurora.org,
linux-scsi@vger.kernel.org, kernel-team@android.com,
saravanak@google.com, salyzyn@google.com, cang@codeaurora.org
Subject: [PATCH V7 0/3] Minor fixes to UFS error handling
Date: Wed, 2 Dec 2020 04:04:00 -0800 [thread overview]
Message-ID: <1606910644-21185-1-git-send-email-cang@codeaurora.org> (raw)
This series mainly fixes below two things which come along with UFS error
handling in some corner cases.
[1] Concurrency problems btw err_handler and paths like system suspend/resume/shutdown and async scan.
[2] Race condition btw UFS error recovery and task abort which happens to W-LU during suspend/resume/shutdown.
The 1st change is tested with error/fault injections to power mode change
operations during system PM operations and async scan. The 2nd change is
tested by mimicing SSU cmd timeout during suspend/resume/shutdown. The 3rd
one is just a minor change to a check condition in IRQ handler such that
the driver can dump host regs when AH8 error happens.
Change since v6:
- Updated the check condition of updating clk_scaling.active_reqs in __ufshcd_transfer_req_compl()
Change since v5:
- Incorporated Stanley's comment
Change since v4:
- Fixed a typo found by Stanley.
Change since v3:
- Slightly updated some commit msg lines, no code changes.
Change since v2:
- Added one more minor change into this series.
Change since v1:
- Removed Change-Id from commit msg
Can Guo (3):
scsi: ufs: Serialize eh_work with system PM events and async scan
scsi: ufs: Fix a race condition between ufshcd_abort and eh_work
scsi: ufs: Print host regs in IRQ handler when AH8 error happens
drivers/scsi/ufs/ufshcd.c | 127 ++++++++++++++++++++++++++++++++--------------
drivers/scsi/ufs/ufshcd.h | 3 ++
2 files changed, 92 insertions(+), 38 deletions(-)
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.
next reply other threads:[~2020-12-02 12:04 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-02 12:04 Can Guo [this message]
2020-12-02 12:04 ` [PATCH V7 1/3] scsi: ufs: Serialize eh_work with system PM events and async scan Can Guo
2020-12-02 12:04 ` Can Guo
2020-12-02 12:04 ` Can Guo
2020-12-02 12:04 ` [PATCH V7 2/3] scsi: ufs: Fix a race condition between ufshcd_abort and eh_work Can Guo
2020-12-03 2:21 ` Stanley Chu
2020-12-03 4:01 ` Can Guo
2020-12-03 4:51 ` Stanley Chu
2020-12-03 5:11 ` Can Guo
2020-12-03 5:19 ` Stanley Chu
2020-12-03 5:21 ` Can Guo
2020-12-02 12:04 ` [PATCH V7 3/3] scsi: ufs: Print host regs in IRQ handler when AH8 error happens Can Guo
2020-12-02 12:04 ` Can Guo
2020-12-02 12:04 ` Can Guo
2020-12-07 23:31 ` [PATCH V7 0/3] Minor fixes to UFS error handling Martin K. Petersen
2020-12-08 5:04 ` Stanley Chu
2020-12-09 17:23 ` Martin K. Petersen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1606910644-21185-1-git-send-email-cang@codeaurora.org \
--to=cang@codeaurora.org \
--cc=asutoshd@codeaurora.org \
--cc=hongwus@codeaurora.org \
--cc=kernel-team@android.com \
--cc=linux-scsi@vger.kernel.org \
--cc=nguyenb@codeaurora.org \
--cc=rnayak@codeaurora.org \
--cc=salyzyn@google.com \
--cc=saravanak@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.