On Thu, 2013-08-29 at 11:54 +0300, Raviv Shvili wrote: > The patch solves the request completion report order. At the current > implementation, when multiple requests end at the same interrupt call, > the requests reported as completed according to a bitmap scan from the > lowest tags to the highest, regardless the requests priority. That cause > to a priority unfairness and starvation of requests with a high tags.
It does? Why? What seems to happen is that you loop over all the pending requests and call done for them. The way SCSI handles done commands is that it queues them to the softirq, so there doesn't look to be any real unfairness problem here. > SCSI Architecture Model 5 defines 3 task-attributes that are part of each > SCSI command, and integrated into each Command UPIU. The task-attribute is > for the device usage, it determines the order in which the device > prioritizes the requests. > The task-attributes according to their priority are (from high to low): > HEAD OF QUEUE, ORDERED and SIMPLE. There is a queue per task-attribute. > Each request is assigned to one of the above sw queues > according to its task attribute field. > Requests which are not SCSI commands (native UFS) will be assigned to > the lowest priority queue, since there is no much difference between > completing it first or last.. > > When request is completed, we go over the queues (from > the queue's highest priority to the lowest) and report > the completion. > > Requests are removed from the queue in case of command completion > or when aborting pending command. Since we never use anything other than SIMPLE attributes, this rather looks like a solution in search of a problem. James -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
