[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20141130182046.GA16296@kria>
Date: Sun, 30 Nov 2014 19:20:46 +0100
From: Sabrina Dubroca <sd@...asysnail.net>
To: Stephen Hemminger <stephen@...workplumber.org>
Cc: David Miller <davem@...emloft.net>,
Kyeyoon Park <kyeyoonp@...eaurora.org>, netdev@...r.kernel.org
Subject: Re: [PATCH net-next v2] bridge: make proxy arp configurable
Hello Stephen,
2014-11-30, 09:20:01 -0800, Stephen Hemminger wrote:
>
> This solves kernel build without INET enabled, and allows user
> to control enabling this feature.
>
> Signed-off-by: Stephen Hemminger <stephen@...workplumber.org>
>
> ---
> net/bridge/Kconfig | 15 +++++++++++++++
> net/bridge/br_input.c | 4 ++++
> net/bridge/br_sysfs_if.c | 4 ++++
> 3 files changed, 23 insertions(+)
>
> --- a/net/bridge/Kconfig 2014-11-25 09:42:34.687891662 -0800
> +++ b/net/bridge/Kconfig 2014-11-25 09:42:34.683891643 -0800
> @@ -60,3 +60,18 @@ config BRIDGE_VLAN_FILTERING
> Say N to exclude this support and reduce the binary size.
>
> If unsure, say Y.
> +
> +config BRIDGE_ARP_PROXY
Shouldn't this be BRIDGE_PROXY_ARP to match all the
#ifdef CONFIG_BRIDGE_PROXY_ARP
below?
> --- a/net/bridge/br_input.c 2014-11-25 09:42:34.687891662 -0800
> +++ b/net/bridge/br_input.c 2014-11-25 09:42:34.683891643 -0800
> @@ -59,6 +59,7 @@ static int br_pass_frame_up(struct sk_bu
> netif_receive_skb);
> }
>
> +#ifdef CONFIG_BRIDGE_PROXY_ARP
> static void br_do_proxy_arp(struct sk_buff *skb, struct net_bridge *br,
> u16 vid)
> {
> @@ -112,6 +113,7 @@ static void br_do_proxy_arp(struct sk_bu
> neigh_release(n);
> }
> }
> +#endif
--
Sabrina
--
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
Powered by blists - more mailing lists