On Tue, 2009-01-27 at 06:36 +0800, ed.lin wrote: > The interrupt routine is good for normal cases. However, if the firmware is > abnormal and returns an invalid response, the driver may reuse a ccb > structure that has already been handled. This may cause problem. Fix this by > setting the req member to NULL. Next time we know the response is invalid and > handle accordingly if req is NULL. > > Signed-off-by: Ed Lin <ed.lin@xxxxxxxxxxx> > --- > > diff --git a/drivers/scsi/stex.c b/drivers/scsi/stex.c > index a3a18ad..6129db4 100644 > --- a/drivers/scsi/stex.c > +++ b/drivers/scsi/stex.c > @@ -746,6 +746,7 @@ static void stex_mu_intr(struct st_hba *hba, u32 doorbell) > stex_copy_data(ccb, resp, size); > } > > + ccb->req = NULL; ^^^^^^^^ I'm afraid all the tabs have been converted to single spaces in this one. Just mime attach the patches ... we can play the tune the email client game later. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html