Hi Felix, [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Linus-L-ssing/bridge-multicast-to-unicast/20170118-120345 config: x86_64-rhel-7.2 (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the attached .config to linux build tree make ARCH=x86_64 Note: it may well be a FALSE warning. FWIW you are at least aware of it now. http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings All warnings (new ones prefixed by >>): net/bridge/br_forward.c: In function 'br_multicast_flood': >> net/bridge/br_forward.c:261:27: warning: 'port' may be used uninitialized in this function [-Wmaybe-uninitialized] struct net_bridge_port *port, *lport, *rport; ^~~~ vim +/port +261 net/bridge/br_forward.c 5cb5e947 Herbert Xu 2010-02-27 245 #ifdef CONFIG_BRIDGE_IGMP_SNOOPING 5cb5e947 Herbert Xu 2010-02-27 246 /* called with rcu_read_lock */ 37b090e6 Nikolay Aleksandrov 2016-07-14 247 void br_multicast_flood(struct net_bridge_mdb_entry *mdst, b35c5f63 Nikolay Aleksandrov 2016-07-14 248 struct sk_buff *skb, 37b090e6 Nikolay Aleksandrov 2016-07-14 249 bool local_rcv, bool local_orig) 5cb5e947 Herbert Xu 2010-02-27 250 { 5cb5e947 Herbert Xu 2010-02-27 251 struct net_device *dev = BR_INPUT_SKB_CB(skb)->brdev; 1080ab95 Nikolay Aleksandrov 2016-06-28 252 u8 igmp_type = br_multicast_igmp_type(skb); 5cb5e947 Herbert Xu 2010-02-27 253 struct net_bridge *br = netdev_priv(dev); afe0159d stephen hemminger 2010-04-27 254 struct net_bridge_port *prev = NULL; 5cb5e947 Herbert Xu 2010-02-27 255 struct net_bridge_port_group *p; 5cb5e947 Herbert Xu 2010-02-27 256 struct hlist_node *rp; 5cb5e947 Herbert Xu 2010-02-27 257 e8051688 Eric Dumazet 2010-11-15 258 rp = rcu_dereference(hlist_first_rcu(&br->router_list)); 83f6a740 stephen hemminger 2010-04-27 259 p = mdst ? rcu_dereference(mdst->ports) : NULL; 5cb5e947 Herbert Xu 2010-02-27 260 while (p || rp) { afe0159d stephen hemminger 2010-04-27 @261 struct net_bridge_port *port, *lport, *rport; afe0159d stephen hemminger 2010-04-27 262 5cb5e947 Herbert Xu 2010-02-27 263 lport = p ? p->port : NULL; 5cb5e947 Herbert Xu 2010-02-27 264 rport = rp ? hlist_entry(rp, struct net_bridge_port, rlist) : 5cb5e947 Herbert Xu 2010-02-27 265 NULL; 5cb5e947 Herbert Xu 2010-02-27 266 507962cd Felix Fietkau 2017-01-17 267 if ((unsigned long)lport > (unsigned long)rport) { 507962cd Felix Fietkau 2017-01-17 268 if (p->flags & MDB_PG_FLAGS_MCAST_TO_UCAST) { 507962cd Felix Fietkau 2017-01-17 269 maybe_deliver_addr(lport, skb, p->eth_addr, :::::: The code at line 261 was first introduced by commit :::::: afe0159d935ab731c682e811356914bb2be9470c bridge: multicast_flood cleanup :::::: TO: stephen hemminger <shemminger@xxxxxxxxxx> :::::: CC: David S. Miller <davem@xxxxxxxxxxxxx> --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
Attachment:
.config.gz
Description: application/gzip
