On Fri, Jan 9, 2015 at 2:12 PM, Shaohua Li <shli@xxxxxx> wrote: > On Fri, Jan 09, 2015 at 01:59:19PM -0800, Shaohua Li wrote: >> On Fri, Jan 09, 2015 at 04:43:07PM -0500, Tejun Heo wrote: >> > On Fri, Jan 09, 2015 at 10:15:29AM -0800, Shaohua Li wrote: >> > > Ping! >> > >> > I like the idea but it bothers me that we end up with two separate >> > ways of allocating command tags. Any chance the old one can be >> > removed? > > Oh, sorry, my reply truncated. > > So I checked with IPR driver guys, the ipr doesn't use ncq, so we can > always use sata tag 0, but not sure for libsas. > > Maybe Dan can answer if there is a way we can map SCSI tag to SATA tag. > For libsas or for ata drivers? For libsas, iirc, the internal libata tag is ignored and we use the scsi tag for the sas task For ata some drivers want round robin, but others appear to care about using the lowest available: https://bugzilla.kernel.org/show_bug.cgi?id=87101. Is ata using it's own legacy tag ordering scheme getting in the way of other improvements? I'd just as soon recommend letting legacy dogs lie. In other words what do we gain by switching? I need to follow up on bz87101, seems I never reworked the patch as Tejun asked. However, I'm glad a fix like the one proposed in that report can be self-contained to libata and need not worry about supporting ata specific quirks in the block layer tag ordering scheme. -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html