On Mon, Nov 26, 2012 at 01:27:29AM +0100, Antonio Quartulli wrote:

When the Bridge Loop Avoidance component is not compiled-in, its boolean switch should be not compiled as well. This patch surrounds the switch with a proper ifdef.

This behaviour was introduced by 9fd6b0615b5499b270d39a92b8790e206cf75833 ("batman-adv: add bridge loop avoidance compile option")

Signed-off-by: Antonio Quartulli ordex@autistici.org

Acked-by: Simon Wunderlich siwu@hrz.tu-chemnitz.de

---

soft-interface.c | 2 ++ types.h | 2 ++ 2 files changed, 4 insertions(+)

diff --git a/soft-interface.c b/soft-interface.c index 8000639..aa34267 100644 --- a/soft-interface.c +++ b/soft-interface.c @@ -480,7 +480,9 @@ struct net_device *batadv_softif_create(const char *name)

atomic_set(&bat_priv->aggregated_ogms, 1); atomic_set(&bat_priv->bonding, 0); +#ifdef CONFIG_BATMAN_ADV_BLA atomic_set(&bat_priv->bridge_loop_avoidance, 0); +#endif #ifdef CONFIG_BATMAN_ADV_DAT atomic_set(&bat_priv->distributed_arp_table, 1); #endif diff --git a/types.h b/types.h index ae9ac9a..030ce41 100644 --- a/types.h +++ b/types.h @@ -273,7 +273,9 @@ struct batadv_priv { atomic_t bonding; /* boolean */ atomic_t fragmentation; /* boolean */ atomic_t ap_isolation; /* boolean */ +#ifdef CONFIG_BATMAN_ADV_BLA atomic_t bridge_loop_avoidance; /* boolean */ +#endif #ifdef CONFIG_BATMAN_ADV_DAT atomic_t distributed_arp_table; /* boolean */

#endif

1.8.0