On Mon, Mar 13, 2017 at 08:52:54PM +0000, Bart Van Assche wrote: > > - if (((plug && !blk_queue_nomerges(q)) || is_sync) && > > - !(data.hctx->flags & BLK_MQ_F_DEFER_ISSUE)) { > > + if (((plug && !blk_queue_nomerges(q)) || is_sync)) { > > A minor comment: due to this change the outer pair of parentheses > became superfluous. Please consider removing these. The last patch in the series removes the statement in this form. But if I have to respin the series for some reason I'll make sure it gets removed here already.