On 10/30/18 6:37 PM, Sagi Grimberg wrote: > >> static inline struct blk_mq_hw_ctx *blk_mq_map_queue(struct request_queue *q, >> + unsigned int flags, >> unsigned int cpu) >> { >> struct blk_mq_tag_set *set = q->tag_set; >> @@ -84,7 +85,7 @@ static inline struct blk_mq_hw_ctx *blk_mq_map_queue_type(struct request_queue * >> unsigned int hctx_type, >> unsigned int cpu) >> { >> - return blk_mq_map_queue(q, cpu); >> + return blk_mq_map_queue(q, hctx_type, cpu); >> } > > Haven't read through the rest of the patch set, but it looks a bit > strange that a type enumeration is being passed on as flags field. It's impossible to get it 100% right at this point, which is why it looks a bit odd. If you read on, it'll make sense. -- Jens Axboe