scsi: Allow scsi_execute users to control retries

Mike Christie
2023-09-05T23:23:39Z

The following patches were made over Martin's 6.6 staging branch.
They allow scsi_execute_cmd users to control exactly which errors are
retried, so we can reduce the sense/sshd handling they have to do and
have it one place.

The patches allow scsi_execute_cmd users to pass in an array of failures
which they want retried/failed and also specify how many times they want
them retried. If we hit an error that the user did not specify then we drop
down to the default behavior. This allows us to remove almost all the
retry logic from scsi_execute_cmd users. We just have the special cases
where we want to retry with a difference size command or sleep between
retries.

v11:
- Document scsi_failure.result special values
- Fix sshdr fix git commit message where there was a missing word
- Use designated initializers for cdb setup
- Fix up various coding style comments from John like redoing if/else
error/success checks.
- Add patch to fix rdac issue where stale SCSH_DH values were returned
- Remove old comment from:
"[PATCH v10 16/33] scsi: spi: Have scsi-ml retry spi_execute errors"
- Drop EOPNOTSUPP use from:
"[PATCH v10 17/33] scsi: sd: Fix sshdr use in sd_suspend_common"
and instead have sd_sync_cache handle ILLEGAL_REQUEST
- Init errno to 0 when declared in:
"[PATCH v10 20/33] scsi: ch: Have scsi-ml retry ch_do_scsi errors"
- Add diffstat below

v10:
- Drop "," after {}.
- Hopefully fix outlook issues.

v9:
- Drop spi_execute changes from [PATCH] scsi: spi: Fix sshdr use
- Change git commit message for sshdr use fixes.

v8:
- Rebase.
- Saw the discussion about the possible bug where callers are
accessing the sshdr when it's not setup, so I added some patches
for that since I was going over the same code.

v7:
- Rebase against scsi_execute_cmd patchset.

v6:
- Fix kunit build when it's built as a module but scsi is not.
- Drop [PATCH 17/35] scsi: ufshcd: Convert to scsi_exec_req because
that driver no longer uses scsi_execute.
- Convert ufshcd to use the scsi_failures struct because it now just does
direct retries and does not do it's own deadline timing.
- Add back memset in read_capacity_16.
- Remove memset in get_sectorsize and replace with { } to init buf.

v5:
- Fix spelling (made sure I ran checkpatch strict)
- Drop SCMD_FAILURE_NONE
- Rename SCMD_FAILURE_ANY
- Fix media_not_present handling where it was being retried instead of
failed.
- Fix ILLEGAL_REQUEST handling in read_capacity_16 so it was not retried.
- Fix coding style, spelling and and naming convention in kunit and added
more tests to handle cases like the media_not_present one where we want
to force failures instead of retries.
- Drop cxlflash patch because it actually checked it's internal state before
performing a retry which we currently do not support.

v4:
- Redefine cmd definitions if the cmd is touched.
- Fix up coding style issues.
- Use sam_status enum.
- Move failures initialization to scsi_initialize_rq
(also fixes KASAN error).
- Add kunit test.
- Add function comments.

v3:
- Use a for loop in scsi_check_passthrough
- Fix result handling/testing.
- Fix scsi_status_is_good handling.
- make __scsi_exec_req take a const arg
- Fix formatting in patch 24

v2:
- Rename scsi_prep_sense
- Change scsi_check_passthrough's loop and added some fixes
- Modified scsi_execute* so it uses a struct to pass in args

drivers/scsi/Kconfig | 9 +
drivers/scsi/ch.c | 24 +-
drivers/scsi/device_handler/scsi_dh_hp_sw.c | 106 ++++----
drivers/scsi/device_handler/scsi_dh_rdac.c | 100 ++++----
drivers/scsi/scsi.c | 2 +-
drivers/scsi/scsi_error.c | 112 ++++++++-
drivers/scsi/scsi_error_test.c | 170 +++++++++++++
drivers/scsi/scsi_lib.c | 34 ++-
drivers/scsi/scsi_scan.c | 102 ++++----
drivers/scsi/scsi_transport_spi.c | 36 ++-
drivers/scsi/sd.c | 366 +++++++++++++++++-----------
drivers/scsi/ses.c | 60 +++--
drivers/scsi/sr.c | 38 +--
drivers/ufs/core/ufshcd.c | 19 +-
include/scsi/scsi_cmnd.h | 35 +++
include/scsi/scsi_device.h | 2 +
16 files changed, 842 insertions(+), 373 deletions(-)