On Mon, 2017-03-13 at 09:48 -0600, Christoph Hellwig wrote: > This flag was never used since it was introduced. > > Signed-off-by: Christoph Hellwig <[email protected]> > --- > block/blk-mq.c | 8 +------- > include/linux/blk-mq.h | 1 - > 2 files changed, 1 insertion(+), 8 deletions(-) > > diff --git a/block/blk-mq.c b/block/blk-mq.c > index 159187a28d66..acf0ddf4af52 100644 > --- a/block/blk-mq.c > +++ b/block/blk-mq.c > @@ -1534,13 +1534,7 @@ static blk_qc_t blk_mq_make_request(struct > request_queue *q, struct bio *bio) > } > > plug = current->plug; > - /* > - * If the driver supports defer issued based on 'last', then > - * queue it up like normal since we can potentially save some > - * CPU this way. > - */ > - 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. Thanks, Bart.
