--=_courier-22694-1047075768-0001-2
Content-Type: text/plain; charset=iso-8859-1
Content-Transfer-Encoding: 7bit
On Fri, 2003-03-07 at 15:17, Mike Anderson wrote:
> I believe James is still pursuing a solution also.
The attached is the fix that works for me (plus fixing a few other
problems I noticed along the way).
This is against vanilla 2.5.64 (you have to take Andries work around out
to see the fix in the BK version).
James
--=_courier-22694-1047075768-0001-2
Content-Type: text/plain; name="tmp.diff"; charset=iso-8859-1
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline; filename=tmp.diff
=3D=3D=3D=3D=3D drivers/scsi/scsi_error.c 1.38 vs edited =3D=3D=3D=3D=3D
--- 1.38/drivers/scsi/scsi_error.c Sat Feb 22 10:17:01 2003
+++ edited/drivers/scsi/scsi_error.c Fri Mar 7 15:39:51 2003
@@ -515,7 +515,7 @@
* actually did complete normally.
*/
if (rtn =3D=3D SUCCESS) {
- int rtn =3D scsi_eh_completed_normally(scmd);
+ rtn =3D scsi_eh_completed_normally(scmd);
SCSI_LOG_ERROR_RECOVERY(3,
printk("%s: scsi_eh_completed_normally %x\n",
__FUNCTION__, rtn));
@@ -545,20 +545,20 @@
static int scsi_request_sense(struct scsi_cmnd *scmd)
{
static unsigned char generic_sense[6] =3D
- {REQUEST_SENSE, 0, 0, 0, 255, 0};
- unsigned char scsi_result0[256], *scsi_result =3D &scsi_result0[0];
+ {REQUEST_SENSE, 0, 0, 0, 254, 0};
+ unsigned char *scsi_result;
int saved_result;
int rtn;
=20
memcpy(scmd->cmnd, generic_sense, sizeof(generic_sense));
=20
- if (scmd->device->host->hostt->unchecked_isa_dma) {
- scsi_result =3D kmalloc(512, GFP_ATOMIC | __GFP_DMA);
- if (unlikely(!scsi_result)) {
- printk(KERN_ERR "%s: cannot allocate scsi_result.\n",
- __FUNCTION__);
- return FAILED;
- }
+ scsi_result =3D kmalloc(254, GFP_ATOMIC | (scmd->device->host->hostt->unc=
hecked_isa_dma) ? __GFP_DMA : 0);
+
+
+ if (unlikely(!scsi_result)) {
+ printk(KERN_ERR "%s: cannot allocate scsi_result.\n",
+ __FUNCTION__);
+ return FAILED;
}
=20
/*
@@ -568,11 +568,11 @@
* address (db). 0 is not a valid sense code.=20
*/
memset(scmd->sense_buffer, 0, sizeof(scmd->sense_buffer));
- memset(scsi_result, 0, 256);
+ memset(scsi_result, 0, 254);
=20
saved_result =3D scmd->result;
scmd->request_buffer =3D scsi_result;
- scmd->request_bufflen =3D 256;
+ scmd->request_bufflen =3D 254;
scmd->use_sg =3D 0;
scmd->cmd_len =3D COMMAND_SIZE(scmd->cmnd[0]);
scmd->sc_data_direction =3D SCSI_DATA_READ;
@@ -581,13 +581,12 @@
rtn =3D scsi_send_eh_cmnd(scmd, SENSE_TIMEOUT);
=20
/* last chance to have valid sense data */
- if (!SCSI_SENSE_VALID(scmd)) {
+ if(!SCSI_SENSE_VALID(scmd)) {
memcpy(scmd->sense_buffer, scmd->request_buffer,
- sizeof(scmd->sense_buffer));
+ sizeof(scmd->sense_buffer));
}
=20
- if (scsi_result !=3D &scsi_result0[0])
- kfree(scsi_result);
+ kfree(scsi_result);
=20
/*
* when we eventually call scsi_finish, we really wish to complete
@@ -702,8 +701,14 @@
* if the result was normal, then just pass it along to the
* upper level.
*/
- if (rtn =3D=3D SUCCESS)
+ if (rtn =3D=3D SUCCESS) {
+ /* we don't want this command reissued, just
+ * finished with the sense data, so set
+ * retries to the max allowed to ensure it
+ * won't get reissued */
+ scmd->retries =3D scmd->allowed;
scsi_eh_finish_cmd(scmd, done_q);
+ }
if (rtn !=3D NEEDS_RETRY)
continue;
=20
=3D=3D=3D=3D=3D drivers/scsi/scsi_lib.c 1.73 vs edited =3D=3D=3D=3D=3D
--- 1.73/drivers/scsi/scsi_lib.c Sat Feb 22 14:35:33 2003
+++ edited/drivers/scsi/scsi_lib.c Fri Mar 7 15:19:46 2003
@@ -265,7 +265,6 @@
cmd->serial_number =3D 0;
cmd->serial_number_at_timeout =3D 0;
cmd->flags =3D 0;
- cmd->retries =3D 0;
cmd->abort_reason =3D 0;
=20
memset(cmd->sense_buffer, 0, sizeof cmd->sense_buffer);
--=_courier-22694-1047075768-0001-2--