netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [net-next:master 622/639] net/dsa/port.c:255: undefined reference to `br_vlan_enabled'
@ 2017-11-11  8:57 kbuild test robot
  2017-11-11  9:42 ` David Miller
  0 siblings, 1 reply; 3+ messages in thread
From: kbuild test robot @ 2017-11-11  8:57 UTC (permalink / raw)
  To: Andrew Lunn; +Cc: kbuild-all, netdev

[-- Attachment #1: Type: text/plain, Size: 1475 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git master
head:   bee955cd3ab4f1a1eb8fc16e7ed69364143df8d7
commit: 2ea7a679ca2abd251c1ec03f20508619707e1749 [622/639] net: dsa: Don't add vlans when vlan filtering is disabled
config: x86_64-randconfig-s2-11111208 (attached as .config)
compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
reproduce:
        git checkout 2ea7a679ca2abd251c1ec03f20508619707e1749
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

   net/dsa/port.o: In function `dsa_port_vlan_add':
>> net/dsa/port.c:255: undefined reference to `br_vlan_enabled'
   net/dsa/port.o: In function `dsa_port_vlan_del':
   net/dsa/port.c:270: undefined reference to `br_vlan_enabled'

vim +255 net/dsa/port.c

   243	
   244	int dsa_port_vlan_add(struct dsa_port *dp,
   245			      const struct switchdev_obj_port_vlan *vlan,
   246			      struct switchdev_trans *trans)
   247	{
   248		struct dsa_notifier_vlan_info info = {
   249			.sw_index = dp->ds->index,
   250			.port = dp->index,
   251			.trans = trans,
   252			.vlan = vlan,
   253		};
   254	
 > 255		if (br_vlan_enabled(dp->bridge_dev))
   256			return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_ADD, &info);
   257	
   258		return 0;
   259	}
   260	

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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 32967 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [net-next:master 622/639] net/dsa/port.c:255: undefined reference to `br_vlan_enabled'
  2017-11-11  8:57 [net-next:master 622/639] net/dsa/port.c:255: undefined reference to `br_vlan_enabled' kbuild test robot
@ 2017-11-11  9:42 ` David Miller
  2017-11-11 15:05   ` Andrew Lunn
  0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2017-11-11  9:42 UTC (permalink / raw)
  To: fengguang.wu; +Cc: andrew, kbuild-all, netdev

From: kbuild test robot <fengguang.wu@intel.com>
Date: Sat, 11 Nov 2017 16:57:08 +0800

> All errors (new ones prefixed by >>):
> 
>    net/dsa/port.o: In function `dsa_port_vlan_add':
>>> net/dsa/port.c:255: undefined reference to `br_vlan_enabled'
>    net/dsa/port.o: In function `dsa_port_vlan_del':
>    net/dsa/port.c:270: undefined reference to `br_vlan_enabled'

Problem is NET_DSA=y and BRIDGE_VLAN_FILTERING=m

We need some Kconfig dependency foo to prevent this.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [net-next:master 622/639] net/dsa/port.c:255: undefined reference to `br_vlan_enabled'
  2017-11-11  9:42 ` David Miller
@ 2017-11-11 15:05   ` Andrew Lunn
  0 siblings, 0 replies; 3+ messages in thread
From: Andrew Lunn @ 2017-11-11 15:05 UTC (permalink / raw)
  To: David Miller; +Cc: fengguang.wu, kbuild-all, netdev

On Sat, Nov 11, 2017 at 06:42:21PM +0900, David Miller wrote:
> From: kbuild test robot <fengguang.wu@intel.com>
> Date: Sat, 11 Nov 2017 16:57:08 +0800
> 
> > All errors (new ones prefixed by >>):
> > 
> >    net/dsa/port.o: In function `dsa_port_vlan_add':
> >>> net/dsa/port.c:255: undefined reference to `br_vlan_enabled'
> >    net/dsa/port.o: In function `dsa_port_vlan_del':
> >    net/dsa/port.c:270: undefined reference to `br_vlan_enabled'
> 
> Problem is NET_DSA=y and BRIDGE_VLAN_FILTERING=m
> 
> We need some Kconfig dependency foo to prevent this.

Yes. Lets see if i can put together a patch before Arnd!

     Andrew

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-11-11 15:05 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-11-11  8:57 [net-next:master 622/639] net/dsa/port.c:255: undefined reference to `br_vlan_enabled' kbuild test robot
2017-11-11  9:42 ` David Miller
2017-11-11 15:05   ` Andrew Lunn

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).