Author: mav
Date: Tue Apr 16 18:31:40 2019
New Revision: 346290
URL: https://svnweb.freebsd.org/changeset/base/346290
Log:
MFC r345805: Unify SCSI_STATUS_BUSY retry handling with other cases.
- Do not retry if periph was invalidated.
- Do not decrement retry_count if already zero.
- Report action_string when applicable.
Modified:
stable/12/sys/cam/cam_periph.c
Directory Properties:
stable/12/ (props changed)
Modified: stable/12/sys/cam/cam_periph.c
==============================================================================
--- stable/12/sys/cam/cam_periph.c Tue Apr 16 17:56:53 2019
(r346289)
+++ stable/12/sys/cam/cam_periph.c Tue Apr 16 18:31:40 2019
(r346290)
@@ -1448,6 +1448,7 @@ camperiphscsistatuserror(union ccb *ccb, union ccb **o
int *openings, u_int32_t *relsim_flags,
u_int32_t *timeout, u_int32_t *action, const char **action_string)
{
+ struct cam_periph *periph;
int error;
switch (ccb->csio.scsi_status) {
@@ -1530,14 +1531,21 @@ camperiphscsistatuserror(union ccb *ccb, union ccb **o
* Restart the queue after either another
* command completes or a 1 second timeout.
*/
- if ((sense_flags & SF_RETRY_BUSY) != 0 ||
- (ccb->ccb_h.retry_count--) > 0) {
+ periph = xpt_path_periph(ccb->ccb_h.path);
+ if (periph->flags & CAM_PERIPH_INVALID) {
+ error = EIO;
+ *action_string = "Periph was invalidated";
+ } else if ((sense_flags & SF_RETRY_BUSY) != 0 ||
+ ccb->ccb_h.retry_count > 0) {
+ if ((sense_flags & SF_RETRY_BUSY) == 0)
+ ccb->ccb_h.retry_count--;
error = ERESTART;
*relsim_flags = RELSIM_RELEASE_AFTER_TIMEOUT
| RELSIM_RELEASE_AFTER_CMDCMPLT;
*timeout = 1000;
} else {
error = EIO;
+ *action_string = "Retries exhausted";
}
break;
case SCSI_STATUS_RESERV_CONFLICT:
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
