On Tue, 2005-02-22 at 16:11 +0900, Yokota Hiroshi wrote:
> +     if (data->CurrentSC     != NULL) {
> +             nsp_msg(KERN_WARNING, "CurrentSC!=NULL this can't be happen");
>               SCpnt->result   = DID_BAD_TARGET << 16;
>               nsp_scsi_done(SCpnt);
> -             return 0;
> +             return SCSI_MLQUEUE_HOST_BUSY;
>       }


This looks wrong.  Either you call done on the command or you return
busy.  If you do both, the command will be completed twice.  There seem
to be a lot of other places where the driver will call done then return
BUSY too ...

James


-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to