[Bridge] [PATCH] bridge: only provide proxy ARP when CONFIG_INET is enabled

2015-01-13 Thread Arnd Bergmann
When IPV4 support is disabled, we cannot call arp_send from the bridge code, which would result in a kernel link error: net/built-in.o: In function `br_handle_frame_finish': :(.text+0x59914): undefined reference to `arp_send' :(.text+0x59a50): undefined reference to `arp_tbl' This makes the newly

Re: [Bridge] [PATCH] bridge: only provide proxy ARP when CONFIG_INET is enabled

2015-01-13 Thread Cong Wang
On Tue, Jan 13, 2015 at 6:10 AM, Arnd Bergmann wrote: > When IPV4 support is disabled, we cannot call arp_send from > the bridge code, which would result in a kernel link error: > > net/built-in.o: In function `br_handle_frame_finish': > :(.text+0x59914): undefined reference to `arp_send' > :(.tex

Re: [Bridge] [PATCH] bridge: only provide proxy ARP when CONFIG_INET is enabled

2015-01-13 Thread David Miller
From: Cong Wang Date: Tue, 13 Jan 2015 11:25:45 -0800 > On Tue, Jan 13, 2015 at 6:10 AM, Arnd Bergmann wrote: >> When IPV4 support is disabled, we cannot call arp_send from >> the bridge code, which would result in a kernel link error: >> >> net/built-in.o: In function `br_handle_frame_finish':

Re: [Bridge] [PATCH] bridge: only provide proxy ARP when CONFIG_INET is enabled

2015-01-13 Thread Arnd Bergmann
On Tuesday 13 January 2015 14:14:20 David Ahern wrote: > > Rather than connect CONFIG_BRIDGE to CONFIG_INET, why not make > br_do_proxy_arp (and setting BR_PROXYARP flag) a no-op if CONFIG_INET is > not set? > > #ifdef CONFIG_INET > #else > static inline void br_do_proxy_arp(...args...) > { > }

Re: [Bridge] [PATCH] bridge: only provide proxy ARP when CONFIG_INET is enabled

2015-01-13 Thread David Ahern
On 1/13/15 1:57 PM, David Miller wrote: From: Cong Wang Date: Tue, 13 Jan 2015 11:25:45 -0800 On Tue, Jan 13, 2015 at 6:10 AM, Arnd Bergmann wrote: When IPV4 support is disabled, we cannot call arp_send from the bridge code, which would result in a kernel link error: net/built-in.o: In func