diff options
author | FUJITA Tomonori <tomof@acm.org> | 2008-01-13 01:46:12 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-01-23 12:29:26 -0500 |
commit | 94aa5e5f6251ca0e1d77e083f8c2f9f40ee548c5 (patch) | |
tree | c925a5a49f17331f069be6815b72983b03514702 /drivers/scsi/ips.c | |
parent | 382db811a8cc1427efabdc959cd1aeef8685ffa6 (diff) |
[SCSI] aic7xxx_old, eata_pio, ips, libsas: don't zero out sense_buffer in queuecommand
LLDs don't need to zero out scsi_cmnd::sense_buffer in queuecommand
since scsi-ml does. This is a preparation of the future changes to
allocate the sense_buffer only when necessary.
Many LLDs zero out the sense_buffer before touching it on the error
case. This patch lets them alone for now because new APIs for them
would be added later on.
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Acked-by: "Salyzyn, Mark" <Mark_Salyzyn@adaptec.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/scsi/ips.c')
-rw-r--r-- | drivers/scsi/ips.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/scsi/ips.c b/drivers/scsi/ips.c index e54d30c6a342..b1b229506113 100644 --- a/drivers/scsi/ips.c +++ b/drivers/scsi/ips.c | |||
@@ -2736,8 +2736,6 @@ ips_next(ips_ha_t * ha, int intr) | |||
2736 | SC->result = DID_OK; | 2736 | SC->result = DID_OK; |
2737 | SC->host_scribble = NULL; | 2737 | SC->host_scribble = NULL; |
2738 | 2738 | ||
2739 | memset(SC->sense_buffer, 0, sizeof (SC->sense_buffer)); | ||
2740 | |||
2741 | scb->target_id = SC->device->id; | 2739 | scb->target_id = SC->device->id; |
2742 | scb->lun = SC->device->lun; | 2740 | scb->lun = SC->device->lun; |
2743 | scb->bus = SC->device->channel; | 2741 | scb->bus = SC->device->channel; |
@@ -3821,7 +3819,6 @@ ips_send_cmd(ips_ha_t * ha, ips_scb_t * scb) | |||
3821 | /* attempted, a Check Condition occurred, and Sense */ | 3819 | /* attempted, a Check Condition occurred, and Sense */ |
3822 | /* Data indicating an Invalid CDB OpCode is returned. */ | 3820 | /* Data indicating an Invalid CDB OpCode is returned. */ |
3823 | sp = (char *) scb->scsi_cmd->sense_buffer; | 3821 | sp = (char *) scb->scsi_cmd->sense_buffer; |
3824 | memset(sp, 0, sizeof (scb->scsi_cmd->sense_buffer)); | ||
3825 | 3822 | ||
3826 | sp[0] = 0x70; /* Error Code */ | 3823 | sp[0] = 0x70; /* Error Code */ |
3827 | sp[2] = ILLEGAL_REQUEST; /* Sense Key 5 Illegal Req. */ | 3824 | sp[2] = ILLEGAL_REQUEST; /* Sense Key 5 Illegal Req. */ |