Hello,
On Fri, Mar 03, 2017 at 03:01:10PM +0800, Hou Tao wrote:
> bio_associate_current() invokes get_io_context_active() to tell
> CFQ scheduler that the current io_context is still issuing IOs
> by increasing active_ref. When the bio is done, we also need
> to invoke put_io_context_active() to decrease active_ref else
> the associated io_context will always be active.
>
> Signed-off-by: Hou Tao <[email protected]>
Can you please explain how you noticed issue and verified the patch?
Also, the patch will need the following.
Fixes: 852c788f8365 ("block: implement bio_associate_current()")
Cc: [email protected] # v3.5+
I think the failure mode isn't severe. We'll be carrying around ioc's
longer than necessary but that's about it. I wonder whether the logic
can be simplified in general. Anyways, please feel free to add
Acked-by: Tejun Heo <[email protected]>
Thanks.
--
tejun
