UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

scsi: aha152x: Do not set message byte when calling scsi_done() 

The done() function is called with a host_byte indicating the actual error
when the message byte is set. As the host byte takes precedence during
error recovery we can drop setting the message byte if the host byte is
set, too.  The only other case is when the host byte is DID_OK, but in that
case the message byte is always COMMAND_COMPLETE (i.e. 0), so we can drop
it there, too.

Link: https://lore.kernel.org/r/20210427083046.31620-31-hare@suse.de
Signed-off-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
Hannes Reinecke 2021-04-27 10:30:36 +02:00 committed by Martin K. Petersen
parent aec166fde7
commit fdabe57d12
1 changed files with 15 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
drivers/scsi/aha152x.c
View File

@ -620,7 +620,7 @@ static irqreturn_t intr(int irq, void *dev_id);
static void reset_ports(struct Scsi_Host *shpnt);
static void aha152x_error(struct Scsi_Host *shpnt, char *msg);
static void done(struct Scsi_Host *shpnt, unsigned char status_byte,
unsigned char msg_byte, unsigned char host_byte);
unsigned char host_byte);
/* diagnostics */
static void show_command(struct scsi_cmnd * ptr);
@ -1273,7 +1273,7 @@ static int aha152x_biosparam(struct scsi_device *sdev, struct block_device *bdev
*
*/
static void done(struct Scsi_Host *shpnt, unsigned char status_byte,
unsigned char msg_byte, unsigned char host_byte)
unsigned char host_byte)
{
if (CURRENT_SC) {
if(DONE_SC)
@ -1283,8 +1283,7 @@ static void done(struct Scsi_Host *shpnt, unsigned char status_byte,
DONE_SC = CURRENT_SC;
CURRENT_SC = NULL;
DONE_SC->result = status_byte;
set_msg_byte(DONE_SC, msg_byte);
set_status_byte(DONE_SC, status_byte);
set_host_byte(DONE_SC, host_byte);
} else
printk(KERN_ERR "aha152x: done() called outside of command\n");
@ -1380,16 +1379,13 @@ static void busfree_run(struct Scsi_Host *shpnt)
if(CURRENT_SC->SCp.phase & completed) {
/* target sent COMMAND COMPLETE */
done(shpnt, CURRENT_SC->SCp.Status,
CURRENT_SC->SCp.Message, DID_OK);
done(shpnt, CURRENT_SC->SCp.Status, DID_OK);
} else if(CURRENT_SC->SCp.phase & aborted) {
done(shpnt, CURRENT_SC->SCp.Status,
CURRENT_SC->SCp.Message, DID_ABORT);
done(shpnt, CURRENT_SC->SCp.Status, DID_ABORT);
} else if(CURRENT_SC->SCp.phase & resetted) {
done(shpnt, CURRENT_SC->SCp.Status,
CURRENT_SC->SCp.Message, DID_RESET);
done(shpnt, CURRENT_SC->SCp.Status, DID_RESET);
} else if(CURRENT_SC->SCp.phase & disconnected) {
/* target sent DISCONNECT */
@ -1401,8 +1397,7 @@ static void busfree_run(struct Scsi_Host *shpnt)
CURRENT_SC = NULL;
} else {
done(shpnt, SAM_STAT_GOOD,
COMMAND_COMPLETE, DID_ERROR);
done(shpnt, SAM_STAT_GOOD, DID_ERROR);
}
#if defined(AHA152X_STAT)
} else {
@ -1523,8 +1518,7 @@ static void seldo_run(struct Scsi_Host *shpnt)
if (TESTLO(SSTAT0, SELDO)) {
scmd_printk(KERN_ERR, CURRENT_SC,
"aha152x: passing bus free condition\n");
done(shpnt, SAM_STAT_GOOD,
COMMAND_COMPLETE, DID_NO_CONNECT);
done(shpnt, SAM_STAT_GOOD, DID_NO_CONNECT);
return;
}
@ -1561,15 +1555,12 @@ static void selto_run(struct Scsi_Host *shpnt)
CURRENT_SC->SCp.phase &= ~selecting;
if (CURRENT_SC->SCp.phase & aborted)
done(shpnt, SAM_STAT_GOOD,
COMMAND_COMPLETE, DID_ABORT);
done(shpnt, SAM_STAT_GOOD, DID_ABORT);
else if (TESTLO(SSTAT0, SELINGO))
done(shpnt, SAM_STAT_GOOD,
COMMAND_COMPLETE, DID_BUS_BUSY);
done(shpnt, SAM_STAT_GOOD, DID_BUS_BUSY);
else
/* ARBITRATION won, but SELECTION failed */
done(shpnt, SAM_STAT_GOOD,
COMMAND_COMPLETE, DID_NO_CONNECT);
done(shpnt, SAM_STAT_GOOD, DID_NO_CONNECT);
}
/*
@ -1903,8 +1894,7 @@ static void cmd_init(struct Scsi_Host *shpnt)
if (CURRENT_SC->SCp.sent_command) {
scmd_printk(KERN_ERR, CURRENT_SC,
"command already sent\n");
done(shpnt, SAM_STAT_GOOD,
COMMAND_COMPLETE, DID_ERROR);
done(shpnt, SAM_STAT_GOOD, DID_ERROR);
return;
}
@ -2244,8 +2234,7 @@ static int update_state(struct Scsi_Host *shpnt)
static void parerr_run(struct Scsi_Host *shpnt)
{
scmd_printk(KERN_ERR, CURRENT_SC, "parity error\n");
done(shpnt, SAM_STAT_GOOD,
COMMAND_COMPLETE, DID_PARITY);
done(shpnt, SAM_STAT_GOOD, DID_PARITY);
}
/*
@ -2268,7 +2257,7 @@ static void rsti_run(struct Scsi_Host *shpnt)
kfree(ptr->host_scribble);
ptr->host_scribble=NULL;
ptr->result = DID_RESET << 16;
set_host_byte(ptr, DID_RESET);
ptr->scsi_done(ptr);
}
@ -2276,8 +2265,7 @@ static void rsti_run(struct Scsi_Host *shpnt)
}
if(CURRENT_SC && !CURRENT_SC->device->soft_reset)
done(shpnt, SAM_STAT_GOOD,
COMMAND_COMPLETE, DID_RESET);
done(shpnt, SAM_STAT_GOOD, DID_RESET);
}