On Fri, 2016-06-10 at 16:58 -0400, Ewan D. Milne wrote:
> I'm not sure if this is the problem, but the tagging changes to
> scsi_tcq.h may have altered the 53c700 driver's assumptions.
> In one case it sets sdev->current_cmnd and then some of the
> tagging calls would return it if the tag was SCSI_NO_TAG.
> 
> NCR_700_queuecommand_lck() does:
> 
>         if ((hostdata->tag_negotiated & (1<<scmd_id(SCp))) &&
>             SCp->device->simple_tags) {
>                 slot->tag = SCp->request->tag;
>                 CDEBUG(KERN_DEBUG, SCp, "sending out tag %d, slot
> %p\n",
>                        slot->tag, slot);
>         } else {
>                 slot->tag = SCSI_NO_TAG;
>                 /* must populate current_cmnd for scsi_host_find_tag
> to
> work */
>                 SCp->device->current_cmnd = SCp;
>         }

Thanks ... I was just about to look for something this.  I'd got to
interpreting the script as reselected with tag information present and
then trying to look the command up with no tag present, hence the
BUG().

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