[Bridge] [PATCH] bridge: only provide proxy ARP when CONFIG_INET is enabled
arnd at arndb.de
Tue Jan 13 21:33:48 UTC 2015
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
> static inline void br_do_proxy_arp(...args...)
> That covers both arp_tbl and arp_send.
The effect is very similar to my patch (probably same object code), the
only difference should be that it would add an ugly #ifdef instead of
the preferred IS_ENABLED() check, so you don't get any compile-time
coverage of the function. It's not really important because everybody
has CONFIG_INET enabled in practice and it does get more than enough
More information about the Bridge