> } else { > + if (blk_mq_unique_tag_to_hwq(rsp->tag) != ch - target->ch) > + pr_err("Channel idx mismatch: tag %#llx <> ch %#lx\n", > + rsp->tag, ch - target->ch); > scmnd = scsi_host_find_tag(target->scsi_host, rsp->tag); Shouldn't we do this validity check inside scsi_host_find_tag, so that all callers get it? That means adding an argument to it, but there are very few callers at the moment. -- 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