On Mon, 2018-07-02 at 15:27 +0200, Johannes Thumshirn wrote: > On Mon, Jul 02, 2018 at 10:20:03PM +0900, Minwoo Im wrote: > [...] > > > > > > > > You could still evaluate if it isn't worth to make it: > > > +static void blk_mq_clear_mq_map(struct blk_mq_tag_set *set) > > > +{ > > > + int cpu; > > > + > > > + for_each_possible_cpu(cpu) > > > + set->mq_map[cpu] = 0; > > > +} > > > > > > and put it into 'include/linux/blk-mq.h'. > > Do you mean this whole bunch of function should be moved into blk-mq.h with > > 'inline' format? > yes Johannes, I was wondering you did mean 'inline' or not because the function in the previous mail only with 'static' without 'inline' keyword :) That's why I asked it one more time to figure out what you really mean. I really appreciate your kindly directives. I'll make it up for v2 patch. Thanks, really. Minwoo Im