[net-next:master 53/61] net/sched/sch_tbf.c:399:3: note: in expansion of macro 'if'

[kbuild test robot]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 
master
head:   429a372e3ec020534bd314dca94e5281dc453d29
commit: 49b499718fa1b0d639663cfd03085b9bfd23cdc8 [53/61] net: sched: make 
default fifo qdiscs appear in the dump
config: i386-randconfig-x077-201711 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        git checkout 49b499718fa1b0d639663cfd03085b9bfd23cdc8
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/linux/list.h:4,
                    from include/linux/module.h:9,
                    from net/sched/sch_tbf.c:15:
   net/sched/sch_tbf.c: In function 'tbf_change':
   include/linux/compiler.h:160:2: warning: this 'if' clause does not guard... 
[-Wmisleading-indentation]
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
     ^
   include/linux/compiler.h:158:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
>> net/sched/sch_tbf.c:399:3: note: in expansion of macro 'if'
      if (child != &noop_qdisc);
      ^~
   net/sched/sch_tbf.c:400:4: note: ...this statement, but the latter is 
misleadingly indented as if it is guarded by the 'if'
       qdisc_hash_add(child, true);
       ^~~~~~~~~~~~~~

vim +/if +399 net/sched/sch_tbf.c

   383                  if (err)
   384                          goto done;
   385          } else if (qopt->limit > 0) {
   386                  child = fifo_create_dflt(sch, &bfifo_qdisc_ops, 
qopt->limit);
   387                  if (IS_ERR(child)) {
   388                          err = PTR_ERR(child);
   389                          goto done;
   390                  }
   391          }
   392  
   393          sch_tree_lock(sch);
   394          if (child) {
   395                  qdisc_tree_reduce_backlog(q->qdisc, q->qdisc->q.qlen,
   396                                            q->qdisc->qstats.backlog);
   397                  qdisc_destroy(q->qdisc);
   398                  q->qdisc = child;
 > 399                  if (child != &noop_qdisc);
   400                          qdisc_hash_add(child, true);
   401          }
   402          q->limit = qopt->limit;
   403          if (tb[TCA_TBF_PBURST])
   404                  q->mtu = mtu;
   405          else
   406                  q->mtu = PSCHED_TICKS2NS(qopt->mtu);
   407          q->max_size = max_size;

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

.config.gz
Description: application/gzip