Re: [PATCH V5 1/6] blk-mq: make __blk_mq_issue_directly be able to accept NULL cookie pointer

From: jianchao.wang
Date: Mon Nov 12 2018 - 21:41:58 EST


Hi Jens

On 11/13/18 1:10 AM, Jens Axboe wrote:
> On 11/12/18 2:23 AM, Jianchao Wang wrote:
>> Make __blk_mq_issue_directly be able to accept a NULL cookie pointer
>> and remove the dummy unused_cookie in blk_mq_request_issue_directly.
>
> What's the reasoning behind this? Seems faster to store the cookie
> always, instead of introducing a branch to the hot path.
>

It is to make the interface more usable then we needn't input a dummy cookie
pointer but just a NULL if we don't use the return cookie.

If the introduced branch in hot path is concerned, I could drop this one.

Thanks
Jianchao