On Wed, Oct 01, 2014 at 10:17:41AM +0200, Hans de Goede wrote: > The problematic part here, which I believe is caused by scsi_mod.blk_mq=Y, > is the tag number 33. uas.c does the following in slave_configure: > > scsi_activate_tcq(sdev, devinfo->qdepth - 2); > > Where qdepth is 32, so 30 gets passed in. uas.c stranslates scsi tags > to uas stream ids, which means it adds 2 (stream ids start at 1 not 0, > and 1 is reserved for untagged commands). > > So the tag 33 above, means that the scsi subsys has called uas.c with > a tagged command with a tag of 31, which should not happen when using > scsi_activate_tcq(sdev, 30). > > So should the uas.c code do something different with blk-mq to tell > it to only use tags 0-29, or is this a blk-mq bug ? This is a mismatch between the (undocumented) existing behavior and what blk-mq implements. In the old code unless you use host-shared maps you would never see a tag number greater than the queue depth when using block layer tags. With blk-mq we also use host-wide maps, so you can easily see tag numbers bigger than the queue depth. So far it seems uas is the only driver with this expectation. Given how it maps tags to a non-scsi concept it might be better to just use a separate bitmaps for the streams inside uas than reusing the tags. Is this mapping an implementation detail of the Linux uas driver or part of the spec? -- 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
