From: Chen, Kenneth W Date: Tue, 12 Jul 2005 22:57:13 +0000 (-0700) Subject: [SCSI] Redundant this_count check in sd_init_command() X-Git-Url: http://git.mmlx.us/?a=commitdiff_plain;h=7fce2cf62e4bd9c24717009865ac00940cb664b8;p=linux-edison.git [SCSI] Redundant this_count check in sd_init_command() I was going over the scsi I/O submit path, when sd_init_command construct the scsi command, this_count is already checked in the previous else if clause. Why does it need to check it again in the last else block? Patch to delete the spurious check. Signed-off-by: Ken Chen Signed-off-by: James Bottomley --- diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c index bb823559878..0410e1bf109 100644 --- a/drivers/scsi/sd.c +++ b/drivers/scsi/sd.c @@ -373,9 +373,6 @@ static int sd_init_command(struct scsi_cmnd * SCpnt) SCpnt->cmnd[7] = (unsigned char) (this_count >> 8) & 0xff; SCpnt->cmnd[8] = (unsigned char) this_count & 0xff; } else { - if (this_count > 0xff) - this_count = 0xff; - SCpnt->cmnd[1] |= (unsigned char) ((block >> 16) & 0x1f); SCpnt->cmnd[2] = (unsigned char) ((block >> 8) & 0xff); SCpnt->cmnd[3] = (unsigned char) block & 0xff;