On Sat, Sep 16, 2017 at 11:03:44AM +0800, Ming Lei wrote: > Hi, > > This patchset avoids to allocate driver tag beforehand for flush rq > in case of I/O scheduler, then flush rq isn't treated specially > wrt. get/put driver tag, code gets cleanup much, such as, > reorder_tags_to_front() is removed, and we needn't to worry > about request order in dispatch list for avoiding I/O deadlock. > > 'dbench -t 30 -s -F 64' has been run on different devices(shared tag, > multi-queue, singele queue, ...), and no issues are observed, > even very low queue depth(1) test are run, debench still works > well. > > Any comments are welcome! > > V2: > - release driver tag before requeue > - warning on inserting a req with driver tag Hi Jens, Did V2 pass your test? If yes and you are fine with this patchset, could you move on and make it land in your for-next? -- Ming
