Add more detailed SCSI error printing by Matthew Jacob. Enabled by

default.  Use "option SCSIVERBOSE=0" to turn off (only added 8Kb to my kernel).
From the NetBSD commit message:
    Add in more detailed SCSI error printing, based off of final SCSI-2 spec.
    This will also print out things like progress indicator errors, and will
    also dump out additional sense bytes that haven't been decoded.

1 files changed, 454 insertions, 4 deletions

diff --git a/sys/scsi/scsi_base.c b/sys/scsi/scsi_base.c
index 52eede4ff8f..af28d7afe21 100644
--- a/sys/scsi/scsi_base.c
+++ b/sys/scsi/scsi_base.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: scsi_base.c,v 1.12 1997/07/03 17:47:29 deraadt Exp $	*/
+/*	$OpenBSD: scsi_base.c,v 1.13 1997/08/22 04:43:25 millert Exp $	*/
 /*	$NetBSD: scsi_base.c,v 1.43 1997/04/02 02:29:36 mycroft Exp $	*/
 
 /*
@@ -31,6 +31,11 @@
  */
 
 /*
+ * Additions for detail SCSI error printing are
+ * Copyright (c) 1997 by Matthew Jacob.
+ */
+
+/*
  * Originally written by Julian Elischer (julian@dialix.oz.au)
  */
 
@@ -63,6 +68,9 @@ static __inline struct scsi_xfer *scsi_make_xs __P((struct scsi_link *,
 
 int sc_err1 __P((struct scsi_xfer *, int));
 int scsi_interpret_sense __P((struct scsi_xfer *));
+#if	SCSIVERBOSE
+char *scsi_decode_sense __P((void *, int));
+#endif
 
 /*
  * Get a scsi transfer structure for the caller. Charge the structure
@@ -590,7 +598,7 @@ scsi_interpret_sense(xs)
 	u_int8_t key;
 	u_int32_t info;
 	int error;
-
+#if	!SCSIVERBOSE
 	static char *error_mes[] = {
 		"soft error (corrected)",
 		"not ready", "medium error",
@@ -601,6 +609,7 @@ scsi_interpret_sense(xs)
 		"search returned equal", "volume overflow",
 		"verify miscompare", "unknown error key"
 	};
+#endif
 
 	sense = &xs->sense;
 #ifdef	SCSIDEBUG
@@ -626,7 +635,7 @@ scsi_interpret_sense(xs)
 			printf("%x ", sense->cmd_spec_info[count]);
 		printf("\n");
 	}
-#endif	/*SCSIDEBUG */
+#endif	/* SCSIDEBUG */
 	/*
 	 * If the device has it's own error handler, call it first.
 	 * If it returns a legit error value, return that, otherwise
@@ -646,7 +655,8 @@ scsi_interpret_sense(xs)
 	case 0x71:		/* delayed error */
 		sc_print_addr(sc_link);
 		key = sense->flags & SSD_KEY;
-		printf(" DELAYED ERROR, key = 0x%x\n", key);
+		printf(" DEFERRED ERROR, key = 0x%x\n", key);
+		/* FALLTHROUGH */
 	case 0x70:
 		if ((sense->error_code & SSD_ERRCODE_VALID) != 0)
 			info = _4btol(sense->info);
@@ -710,6 +720,10 @@ scsi_interpret_sense(xs)
 			break;
 		}
 
+
+#if	SCSIVERBOSE
+		scsi_print_sense(xs, 0);
+#else
 		if (key) {
 			sc_print_addr(sc_link);
 			printf("%s", error_mes[key - 1]);
@@ -742,6 +756,7 @@ scsi_interpret_sense(xs)
 			}
 			printf("\n");
 		}
+#endif
 		return error;
 
 	/*
@@ -782,6 +797,441 @@ sc_print_addr(sc_link)
 	    sc_link->target, sc_link->lun);		
 }
 
+#if	SCSIVERBOSE
+static const char *sense_keys[16] = {
+	"No Additional Sense",
+	"Soft Error",
+	"Not Ready",
+	"Media Error",
+	"Hardware Error",
+	"Illegal Request",
+	"Unit Attention",
+	"Write Protected",
+	"Blank Check",
+	"Vendor Unique",
+	"Copy Aborted",
+	"Aborted Command",
+	"Equal Error",
+	"Volume Overflow",
+	"Miscompare Error",
+	"Reserved"
+};
+static const struct {
+	unsigned char asc;
+	unsigned char ascq;
+	char *description;
+} adesc[] = {
+{ 0x00, 0x00, "No Additional Sense Information" },
+{ 0x00, 0x01, "Filemark Detected" },
+{ 0x00, 0x02, "End-Of-Partition/Medium Detected" },
+{ 0x00, 0x03, "Setmark Detected" },
+{ 0x00, 0x04, "Beginning-Of-Partition/Medium Detected" },
+{ 0x00, 0x05, "End-Of-Data Detected" },
+{ 0x00, 0x06, "I/O Process Terminated" },
+{ 0x00, 0x11, "Audio Play Operation In Progress" },
+{ 0x00, 0x12, "Audio Play Operation Paused" },
+{ 0x00, 0x13, "Audio Play Operation Successfully Completed" },
+{ 0x00, 0x14, "Audio Play Operation Stopped Due to Error" },
+{ 0x00, 0x15, "No Current Audio Status To Return" },
+{ 0x01, 0x00, "No Index/Sector Signal" },
+{ 0x02, 0x00, "No Seek Complete" },
+{ 0x03, 0x00, "Peripheral Device Write Fault" },
+{ 0x03, 0x01, "No Write Current" },
+{ 0x03, 0x02, "Excessive Write Errors" },
+{ 0x04, 0x00, "Logical Unit Not Ready, Cause Not Reportable" },
+{ 0x04, 0x01, "Logical Unit Is in Process Of Becoming Ready" },
+{ 0x04, 0x02, "Logical Unit Not Ready, Initialization Command Required" },
+{ 0x04, 0x03, "Logical Unit Not Ready, Manual Intervention Required" },
+{ 0x04, 0x04, "Logical Unit Not Ready, Format In Progress" },
+{ 0x05, 0x00, "Logical Unit Does Not Respond To Selection" },
+{ 0x06, 0x00, "No Reference Position Found" },
+{ 0x07, 0x00, "Multiple Peripheral Devices Selected" },
+{ 0x08, 0x00, "Logical Unit Communication Failure" },
+{ 0x08, 0x01, "Logical Unit Communication Timeout" },
+{ 0x08, 0x02, "Logical Unit Communication Parity Error" },
+{ 0x09, 0x00, "Track Following Error" },
+{ 0x09, 0x01, "Tracking Servo Failure" },
+{ 0x09, 0x02, "Focus Servo Failure" },
+{ 0x09, 0x03, "Spindle Servo Failure" },
+{ 0x0A, 0x00, "Error Log Overflow" },
+{ 0x0C, 0x00, "Write Error" },
+{ 0x0C, 0x01, "Write Error Recovered with Auto Reallocation" },
+{ 0x0C, 0x02, "Write Error - Auto Reallocate Failed" },
+{ 0x10, 0x00, "ID CRC Or ECC Error" },
+{ 0x11, 0x00, "Unrecovered Read Error" },
+{ 0x11, 0x01, "Read Retried Exhausted" },
+{ 0x11, 0x02, "Error Too Long To Correct" },
+{ 0x11, 0x03, "Multiple Read Errors" },
+{ 0x11, 0x04, "Unrecovered Read Error - Auto Reallocate Failed" },
+{ 0x11, 0x05, "L-EC Uncorrectable Error" },
+{ 0x11, 0x06, "CIRC Unrecovered Error" },
+{ 0x11, 0x07, "Data Resynchronization Error" },
+{ 0x11, 0x08, "Incomplete Block Found" },
+{ 0x11, 0x09, "No Gap Found" },
+{ 0x11, 0x0A, "Miscorrected Error" },
+{ 0x11, 0x0B, "Uncorrected Read Error - Recommend Reassignment" },
+{ 0x11, 0x0C, "Uncorrected Read Error - Recommend Rewrite the Data" },
+{ 0x12, 0x00, "Address Mark Not Found for ID Field" },
+{ 0x13, 0x00, "Address Mark Not Found for Data Field" },
+{ 0x14, 0x00, "Recorded Entity Not Found" },
+{ 0x14, 0x01, "Record Not Found" },
+{ 0x14, 0x02, "Filemark or Setmark Not Found" },
+{ 0x14, 0x03, "End-Of-Data Not Found" },
+{ 0x14, 0x04, "Block Sequence Error" },
+{ 0x15, 0x00, "Random Positioning Error" },
+{ 0x15, 0x01, "Mechanical Positioning Error" },
+{ 0x15, 0x02, "Positioning Error Detected By Read of Medium" },
+{ 0x16, 0x00, "Data Synchronization Mark Error" },
+{ 0x17, 0x00, "Recovered Data With No Error Correction Applied" },
+{ 0x17, 0x01, "Recovered Data With Retries" },
+{ 0x17, 0x02, "Recovered Data With Positive Head Offset" },
+{ 0x17, 0x03, "Recovered Data With Negative Head Offset" },
+{ 0x17, 0x04, "Recovered Data With Retries and/or CIRC Applied" },
+{ 0x17, 0x05, "Recovered Data Using Previous Sector ID" },
+{ 0x17, 0x06, "Recovered Data Without ECC - Data Auto-Reallocated" },
+{ 0x17, 0x07, "Recovered Data Without ECC - Recommend Reassignment" },
+{ 0x17, 0x08, "Recovered Data Without ECC - Recommend Rewrite" },
+{ 0x18, 0x00, "Recovered Data With Error Correction Applied" },
+{ 0x18, 0x01, "Recovered Data With Error Correction & Retries Applied" },
+{ 0x18, 0x02, "Recovered Data - Data Auto-Reallocated" },
+{ 0x18, 0x03, "Recovered Data With CIRC" },
+{ 0x18, 0x04, "Recovered Data With LEC" },
+{ 0x18, 0x05, "Recovered Data - Recommend Reassignment" },
+{ 0x18, 0x06, "Recovered Data - Recommend Rewrite" },
+{ 0x19, 0x00, "Defect List Error" },
+{ 0x19, 0x01, "Defect List Not Available" },
+{ 0x19, 0x02, "Defect List Error in Primary List" },
+{ 0x19, 0x03, "Defect List Error in Grown List" },
+{ 0x1A, 0x00, "Parameter List Length Error" },
+{ 0x1B, 0x00, "Synchronous Data Transfer Error" },
+{ 0x1C, 0x00, "Defect List Not Found" },
+{ 0x1C, 0x01, "Primary Defect List Not Found" },
+{ 0x1C, 0x02, "Grown Defect List Not Found" },
+{ 0x1D, 0x00, "Miscompare During Verify Operation" },
+{ 0x1E, 0x00, "Recovered ID with ECC" },
+{ 0x20, 0x00, "Invalid Command Operation Code" },
+{ 0x21, 0x00, "Logical Block Address Out of Range" },
+{ 0x21, 0x01, "Invalid Element Address" },
+{ 0x22, 0x00, "Illegal Function (Should 20 00, 24 00, or 26 00)" },
+{ 0x24, 0x00, "Illegal Field in CDB" },
+{ 0x25, 0x00, "Logical Unit Not Supported" },
+{ 0x26, 0x00, "Invalid Field In Parameter List" },
+{ 0x26, 0x01, "Parameter Not Supported" },
+{ 0x26, 0x02, "Parameter Value Invalid" },
+{ 0x26, 0x03, "Threshold Parameters Not Supported" },
+{ 0x27, 0x00, "Write Protected" },
+{ 0x28, 0x00, "Not Ready To Ready Transition (Medium May Have Changed)" },
+{ 0x28, 0x01, "Import Or Export Element Accessed" },
+{ 0x29, 0x00, "Power On, Reset, or Bus Device Reset Occurred" },
+{ 0x2A, 0x00, "Parameters Changed" },
+{ 0x2A, 0x01, "Mode Parameters Changed" },
+{ 0x2A, 0x02, "Log Parameters Changed" },
+{ 0x2B, 0x00, "Copy Cannot Execute Since Host Cannot Disconnect" },
+{ 0x2C, 0x00, "Command Sequence Error" },
+{ 0x2C, 0x01, "Too Many Windows Specified" },
+{ 0x2C, 0x02, "Invalid Combination of Windows Specified" },
+{ 0x2D, 0x00, "Overwrite Error On Update In Place" },
+{ 0x2F, 0x00, "Commands Cleared By Another Initiator" },
+{ 0x30, 0x00, "Incompatible Medium Installed" },
+{ 0x30, 0x01, "Cannot Read Medium - Unknown Format" },
+{ 0x30, 0x02, "Cannot Read Medium - Incompatible Format" },
+{ 0x30, 0x03, "Cleaning Cartridge Installed" },
+{ 0x31, 0x00, "Medium Format Corrupted" },
+{ 0x31, 0x01, "Format Command Failed" },
+{ 0x32, 0x00, "No Defect Spare Location Available" },
+{ 0x32, 0x01, "Defect List Update Failure" },
+{ 0x33, 0x00, "Tape Length Error" },
+{ 0x36, 0x00, "Ribbon, Ink, or Toner Failure" },
+{ 0x37, 0x00, "Rounded Parameter" },
+{ 0x39, 0x00, "Saving Parameters Not Supported" },
+{ 0x3A, 0x00, "Medium Not Present" },
+{ 0x3B, 0x00, "Positioning Error" },
+{ 0x3B, 0x01, "Tape Position Error At Beginning-of-Medium" },
+{ 0x3B, 0x02, "Tape Position Error At End-of-Medium" },
+{ 0x3B, 0x03, "Tape or Electronic Vertical Forms Unit Not Ready" },
+{ 0x3B, 0x04, "Slew Failure" },
+{ 0x3B, 0x05, "Paper Jam" },
+{ 0x3B, 0x06, "Failed To Sense Top-Of-Form" },
+{ 0x3B, 0x07, "Failed To Sense Bottom-Of-Form" },
+{ 0x3B, 0x08, "Reposition Error" },
+{ 0x3B, 0x09, "Read Past End Of Medium" },
+{ 0x3B, 0x0A, "Read Past Begining Of Medium" },
+{ 0x3B, 0x0B, "Position Past End Of Medium" },
+{ 0x3B, 0x0C, "Position Past Beginning Of Medium" },
+{ 0x3B, 0x0D, "Medium Destination Element Full" },
+{ 0x3B, 0x0E, "Medium Source Element Empty" },
+{ 0x3D, 0x00, "Invalid Bits In IDENTFY Message" },
+{ 0x3E, 0x00, "Logical Unit Has Not Self-Configured Yet" },
+{ 0x3F, 0x00, "Target Operating Conditions Have Changed" },
+{ 0x3F, 0x01, "Microcode Has Changed" },
+{ 0x3F, 0x02, "Changed Operating Definition" },
+{ 0x3F, 0x03, "INQUIRY Data Has Changed" },
+{ 0x40, 0x00, "RAM FAILURE (Should Use 40 NN)" },
+{ 0x41, 0x00, "Data Path FAILURE (Should Use 40 NN)" },
+{ 0x42, 0x00, "Power-On or Self-Test FAILURE (Should Use 40 NN)" },
+{ 0x43, 0x00, "Message Error" },
+{ 0x44, 0x00, "Internal Target Failure" },
+{ 0x45, 0x00, "Select Or Reselect Failure" },
+{ 0x46, 0x00, "Unsuccessful Soft Reset" },
+{ 0x47, 0x00, "SCSI Parity Error" },
+{ 0x48, 0x00, "INITIATOR DETECTED ERROR Message Received" },
+{ 0x49, 0x00, "Invalid Message Error" },
+{ 0x4A, 0x00, "Command Phase Error" },
+{ 0x4B, 0x00, "Data Phase Error" },
+{ 0x4C, 0x00, "Logical Unit Failed Self-Configuration" },
+{ 0x4E, 0x00, "Overlapped Commands Attempted" },
+{ 0x50, 0x00, "Write Append Error" },
+{ 0x50, 0x01, "Write Append Position Error" },
+{ 0x50, 0x02, "Position Error Related To Timing" },
+{ 0x51, 0x00, "Erase Failure" },
+{ 0x52, 0x00, "Cartridge Fault" },
+{ 0x53, 0x00, "Media Load or Eject Failed" },
+{ 0x53, 0x01, "Unload Tape Failure" },
+{ 0x53, 0x02, "Medium Removal Prevented" },
+{ 0x54, 0x00, "SCSI To Host System Interface Failure" },
+{ 0x55, 0x00, "System Resource Failure" },
+{ 0x57, 0x00, "Unable To Recover Table-Of-Contents" },
+{ 0x58, 0x00, "Generation Does Not Exist" },
+{ 0x59, 0x00, "Updated Block Read" },
+{ 0x5A, 0x00, "Operator Request or State Change Input (Unspecified)" },
+{ 0x5A, 0x01, "Operator Medium Removal Requested" },
+{ 0x5A, 0x02, "Operator Selected Write Protect" },
+{ 0x5A, 0x03, "Operator Selected Write Permit" },
+{ 0x5B, 0x00, "Log Exception" },
+{ 0x5B, 0x01, "Threshold Condition Met" },
+{ 0x5B, 0x02, "Log Counter At Maximum" },
+{ 0x5B, 0x03, "Log List Codes Exhausted" },
+{ 0x5C, 0x00, "RPL Status Change" },
+{ 0x5C, 0x01, "Spindles Synchronized" },
+{ 0x5C, 0x02, "Spindles Not Synchronized" },
+{ 0x60, 0x00, "Lamp Failure" },
+{ 0x61, 0x00, "Video Acquisition Error" },
+{ 0x61, 0x01, "Unable To Acquire Video" },
+{ 0x61, 0x02, "Out Of Focus" },
+{ 0x62, 0x00, "Scan Head Positioning Error" },
+{ 0x63, 0x00, "End Of User Area Encountered On This Track" },
+{ 0x64, 0x00, "Illegal Mode For This Track" },
+{ 0x00, 0x00, (char *) 0 }
+};
+
+static inline void
+asc2ascii(unsigned char asc, unsigned char ascq, char *result)
+{
+	register int i = 0;
+
+	while (adesc[i].description != (char *) 0) {
+		if (adesc[i].asc == asc && adesc[i].ascq == ascq) {
+			break;
+		}
+		i++;
+	}
+	if (adesc[i].description == (char *) 0) {
+		if (asc == 0x40 && ascq != 0) {
+			(void) sprintf(result,
+			    "Diagnostic Failure on Component 0x%02x",
+			    ascq & 0xff);
+		} else {
+			(void) sprintf(result, "ASC 0x%02x ASCQ 0x%02x",
+			    asc & 0xff, ascq & 0xff);
+		}
+	} else {
+		(void) strcpy(result, adesc[i].description);
+	}
+}
+
+void
+scsi_print_sense(xs, verbosity)
+	struct scsi_xfer *xs;
+	int verbosity;
+{
+	int32_t info;
+	register int i, j, k;
+	char *sbs, *s;
+
+	sc_print_addr(xs->sc_link);
+	s = (char *) &xs->sense;
+	printf(" Check Condition on opcode %x\n", xs->cmd->opcode);
+
+	/*
+	 * Basics- print out SENSE KEY
+	 */
+	printf("    SENSE KEY:  %s", scsi_decode_sense(s, 0));
+
+	/*
+ 	 * Print out, unqualified but aligned, FMK, EOM and ILI status.
+	 */
+	if (s[2] & 0xe0) {
+		char pad;
+		printf("\n              ");
+		pad = ' ';
+		if (s[2] & SSD_FILEMARK) {
+			printf("%c Filemark Detected", pad);
+			pad = ',';
+		}
+		if (s[2] & SSD_EOM) {
+			printf("%c EOM Detected", pad);
+			pad = ',';
+		}
+		if (s[2] & SSD_ILI) {
+			printf("%c Incorrect Length Indicator Set", pad);
+		}
+	}
+
+	/*
+	 * Now we should figure out, based upon device type, how
+	 * to format the information field. Unfortunately, that's
+	 * not convenient here, so we'll print it as a signed
+	 * 32 bit integer.
+	 */
+	info = _4btol(&s[3]);
+	if (info) {
+		printf("\n   INFO FIELD:  %d", info);
+	}
+
+	/*
+	 * Now we check additional length to see whether there is
+	 * more information to extract.
+	 */
+
+	/* enough for command specific information? */
+	if (s[7] < 4) {
+		printf("\n");
+		return;
+	}
+	info = _4btol(&s[8]);
+	if (info) {
+		printf("\n COMMAND INFO:  %d (0x%x)", info, info);
+	}
+
+	/*
+	 * Decode ASC && ASCQ info, plus FRU, plus the rest...
+	 */
+
+	sbs = scsi_decode_sense(s, 1);
+	if (sbs) {
+		printf("\n     ASC/ASCQ:  %s", sbs);
+	}
+	if (s[14] != 0) {
+		printf("\n     FRU CODE:  0x%x\n", s[14] & 0xff);
+	}
+	sbs = scsi_decode_sense(s, 3);
+	if (sbs) {
+		printf("\n         SKSV:  %s", sbs);
+	}
+	printf("\n");
+	if (verbosity == 0) {
+		printf("\n");
+		return;
+	}
+
+	/*
+	 * Now figure whether we should print any additional informtion.
+	 *
+	 * Where should we start from? If we had SKSV data,
+	 * start from offset 18, else from offset 15.
+	 *
+	 * From that point until the end of the buffer, check for any
+	 * nonzero data. If we have some, go back and print the lot,
+	 * otherwise we're done.
+	 */
+	if (sbs) {
+		i = 18;
+	} else {
+		i = 15;
+	}
+	for (j = i; j < sizeof (xs->sense); j++) {
+		if (s[j])
+			break;
+	}
+	if (j == sizeof (xs->sense))
+		return;
+
+	printf("\n Additional Sense Information (byte %d out...):\n", i);
+	if (i == 15) {
+		printf("\n\t%2d:", i);
+		k = 7;
+	} else {
+		printf("\n\t%2d:", i);
+		k = 2;
+		j -= 2;
+	}
+	while (j > 0) {
+		if (i >= sizeof (xs->sense))
+			break;
+		if (k == 8) {
+			k = 0;
+			printf("\n\t%2d:", i);
+		}
+		printf(" 0x%02x", s[i] & 0xff);
+		k++;
+		j--;
+		i++;
+	}
+	printf("\n\n");
+}
+
+char *
+scsi_decode_sense(void *sinfo, int flag)
+{
+	unsigned char *snsbuf;
+	unsigned char skey;
+	static char rqsbuf[132];
+
+	skey = 0;
+
+	snsbuf = (unsigned char *) sinfo;
+	if (flag == 0 || flag == 2 || flag == 3) {
+		skey = snsbuf[2] & 0xf;
+	}
+	if (flag == 0) {		/* Sense Key Only */
+		(void) strcpy(rqsbuf, sense_keys[skey]);
+		return (rqsbuf);
+	} else if (flag == 1) {		/* ASC/ASCQ Only */
+		asc2ascii(snsbuf[12], snsbuf[13], rqsbuf);
+		return (rqsbuf);
+	} else  if (flag == 2) {	/* Sense Key && ASC/ASCQ */
+		auto char localbuf[64];
+		asc2ascii(snsbuf[12], snsbuf[13], localbuf);
+		(void) sprintf(rqsbuf, "%s, %s", sense_keys[skey], localbuf);
+		return (rqsbuf);
+	} else if (flag == 3  && snsbuf[7] >= 9 && (snsbuf[15] & 0x80)) {
+		/*
+		 * SKSV Data
+		 */
+		switch (skey) {
+		case 0x5:	/* Illegal Request */
+			if (snsbuf[15] & 0x8) {
+				(void) sprintf(rqsbuf,
+				    "Error in %s, Offset %d, bit %d",
+				    (snsbuf[15] & 0x40)? "CDB" : "Parameters",
+				    (snsbuf[16] & 0xff) << 8 |
+				    (snsbuf[17] & 0xff), snsbuf[15] & 0xf);
+			} else {
+				(void) sprintf(rqsbuf,
+				    "Error in %s, Offset %d",
+				    (snsbuf[15] & 0x40)? "CDB" : "Parameters",
+				    (snsbuf[16] & 0xff) << 8 |
+				    (snsbuf[17] & 0xff));
+			}
+			return (rqsbuf);
+		case 0x1:
+		case 0x3:
+		case 0x4:
+			(void) sprintf(rqsbuf, "Actual Retry Count: %d",
+			    (snsbuf[16] & 0xff) << 8 | (snsbuf[17] & 0xff));
+			return (rqsbuf);
+		case 0x2:
+			(void) sprintf(rqsbuf, "Progress Indicator: %d",
+			    (snsbuf[16] & 0xff) << 8 | (snsbuf[17] & 0xff));
+			return (rqsbuf);
+		default:
+			break;
+		}
+	}
+	return ((char *) 0);
+}
+
+#endif
 #ifdef	SCSIDEBUG
 /*
  * Given a scsi_xfer, dump the request, in all it's glory