On 7/10/19 11:32 AM, Bart Van Assche wrote:
> On 7/9/19 2:02 AM, Damien Le Moal wrote:
>> +static inline struct blk_plug *blk_mq_plug(struct request_queue *q,
>> + struct bio *bio)
>> +{
>> + struct blk_plug *plug = current->plug;
>> +
>> + if (!blk_queue_is_zoned(q) || !op_is_write(bio_op(bio)))
>> + return plug;
>> +
>> + /* Zoned block device write case: do not plug the BIO */
>> + return NULL;
>> +}
>
> Can the 'plug' variable be left out from this function and can 'return
> plug' be changed into 'return current->plug'? Anyway:
Sure, that would be cleaner. Will Send a V2.
>
> Reviewed-by: Bart Van Assche <[email protected]>
Thanks. Can I add this to the V2 or would you prefer to see the revised
patch first ?
Best regards.
--
Damien Le Moal
Western Digital Research
