netdev - Re: [PATCH net-next] bridge: set is_local and is_static before fdb entry is added to the fdb hashtable

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <562F68BB.1040201@cumulusnetworks.com>
Date:	Tue, 27 Oct 2015 05:06:19 -0700
From:	roopa <roopa@...ulusnetworks.com>
To:	kbuild test robot <lkp@...el.com>
CC:	kbuild-all@...org, davem@...emloft.net, stephen@...workplumber.org,
	nikolay@...ulusnetworks.com, netdev@...r.kernel.org
Subject: Re: [PATCH net-next] bridge: set is_local and is_static before fdb
 entry is added to the fdb hashtable

On 10/26/15, 9:16 PM, kbuild test robot wrote:
> Hi Roopa,
>
> [auto build test ERROR on net-next/master -- if it's inappropriate base, please suggest rules for selecting the more suitable base]
>
> url:    https://github.com/0day-ci/linux/commits/Roopa-Prabhu/bridge-set-is_local-and-is_static-before-fdb-entry-is-added-to-the-fdb-hashtable/20151027-120635
> config: i386-randconfig-x009-201543 (attached as .config)
> reproduce:
>         # save the attached .config to linux build tree
>         make ARCH=i386 
>
> All errors (new ones prefixed by >>):
>
>    net/bridge/br_fdb.c: In function 'br_fdb_external_learn_add':
>>> net/bridge/br_fdb.c:1103:9: error: too few arguments to function 'fdb_create'
>       fdb = fdb_create(head, p, addr, vid);
>             ^
>    net/bridge/br_fdb.c:495:37: note: declared here
>     static struct net_bridge_fdb_entry *fdb_create(struct hlist_head *head,
>                                         ^
>
> vim +/fdb_create +1103 net/bridge/br_fdb.c
>
> 3aeb6617 Jiri Pirko    2015-01-15  1097  	ASSERT_RTNL();
> cf6b8e1e Scott Feldman 2014-11-28  1098  	spin_lock_bh(&br->hash_lock);
> cf6b8e1e Scott Feldman 2014-11-28  1099  
> cf6b8e1e Scott Feldman 2014-11-28  1100  	head = &br->hash[br_mac_hash(addr, vid)];
> cf6b8e1e Scott Feldman 2014-11-28  1101  	fdb = fdb_find(head, addr, vid);
> cf6b8e1e Scott Feldman 2014-11-28  1102  	if (!fdb) {
> cf6b8e1e Scott Feldman 2014-11-28 @1103  		fdb = fdb_create(head, p, addr, vid);
> cf6b8e1e Scott Feldman 2014-11-28  1104  		if (!fdb) {
> cf6b8e1e Scott Feldman 2014-11-28  1105  			err = -ENOMEM;
> cf6b8e1e Scott Feldman 2014-11-28  1106  			goto err_unlock;
>
> :::::: The code at line 1103 was first introduced by commit
> :::::: cf6b8e1eedffd9ef9a22c0c9453d752b07daf89a bridge: add API to notify bridge driver of learned FBD on offloaded device
>
> :::::: TO: Scott Feldman <sfeldma@...il.com>
> :::::: CC: David S. Miller <davem@...emloft.net>
>
sorry, looks like i posted an older version. Re-posting. Thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html