Re: [PATCH 06/10] blk-mq-tag: cleanup the normal/reserved tag allocation

2017-01-13 Thread Jens Axboe
On Fri, Jan 13 2017, Christoph Hellwig wrote: > On Wed, Jan 11, 2017 at 02:39:59PM -0700, Jens Axboe wrote: > > This is in preparation for having another tag set available. Cleanup > > the parameters, and allow passing in of tags fo blk_mq_put_tag(). > > I've been playing around with this are a

Re: [PATCH 06/10] blk-mq-tag: cleanup the normal/reserved tag allocation

2017-01-13 Thread Christoph Hellwig
On Wed, Jan 11, 2017 at 02:39:59PM -0700, Jens Axboe wrote: > This is in preparation for having another tag set available. Cleanup > the parameters, and allow passing in of tags fo blk_mq_put_tag(). I've been playing around with this are a bit but never submitted anything. Below is an untested

Re: [PATCH 06/10] blk-mq-tag: cleanup the normal/reserved tag allocation

2017-01-12 Thread Jens Axboe
On Thu, Jan 12 2017, Bart Van Assche wrote: > On Wed, 2017-01-11 at 14:39 -0700, Jens Axboe wrote: > > This is in preparation for having another tag set available. Cleanup > > the parameters, and allow passing in of tags fo blk_mq_put_tag(). > > It seems like an 'r' is missing from the

Re: [PATCH 06/10] blk-mq-tag: cleanup the normal/reserved tag allocation

2017-01-12 Thread Bart Van Assche
On Wed, 2017-01-11 at 14:39 -0700, Jens Axboe wrote: > This is in preparation for having another tag set available. Cleanup > the parameters, and allow passing in of tags fo blk_mq_put_tag(). It seems like an 'r' is missing from the description ("tags fo")? Anyway: Reviewed-by: Bart Van Assche