Re: RFC: default group_isolation to 1, remove option

[Jens Axboe <axboe@kernel.dk>]

On 2011-03-01 09:20, Vivek Goyal wrote:
> I think creating per group request pool will complicate the
> implementation further. (we have done that once in the past). Jens
> once mentioned that he liked number of requests per iocontext limit
> better than overall queue limit. So if we implement per iocontext
> limit, it will get rid of need of doing anything extra for group
> infrastructure.
> 
> Jens, do you think per iocontext per queue limit on request
> descriptors make sense and we can get rid of per queue overall limit? 

Since we practically don't need a limit anymore to begin with (or so is
the theory), then yes we can move to per-ioc limits instead and get rid
of that queue state. We'd have to hold on to the ioc for the duration of
the IO explicitly from the request then.

I primarily like that implementation since it means we can make the IO
completion lockless, at least on the block layer side. We still have
state to complete in the schedulers that require that, but it's a good
step at least.

-- 
Jens Axboe