On Friday, February 10, 2012 07:41:40 Antonio Quartulli wrote:
@@ -23,6 +23,8 @@ export CONFIG_BATMAN_ADV_DEBUG=n # B.A.T.M.A.N. bridge loop avoidance: export CONFIG_BATMAN_ADV_BLA=y +# B.A.T.M.A.N. distributed ARP table: +export CONFIG_BATMAN_ADV_DAT=n
Any particular reason why you wish to disable it by default ?
--- a/distributed-arp-table.c +++ b/distributed-arp-table.c @@ -30,10 +30,13 @@ #include "hard-interface.h" #include "originator.h" #include "send.h" +#include "soft-interface.h" #include "types.h" #include "translation-table.h" #include "unicast.h"
Why changing an include in this patch ?
--- a/distributed-arp-table.h +++ b/distributed-arp-table.h @@ -22,9 +22,12 @@ #ifndef _NET_BATMAN_ADV_ARP_H_ #define _NET_BATMAN_ADV_ARP_H_
+#ifdef CONFIG_BATMAN_ADV_DAT
#include "main.h"
#include <linux/if_arp.h> +#include <linux/netdevice.h>
Another include change ?
--- a/hard-interface.c +++ b/hard-interface.c @@ -119,9 +119,11 @@ static void primary_if_update_addr(struct bat_priv *bat_priv, if (!primary_if) goto out;
+#ifdef CONFIG_BATMAN_ADV_DAT bat_priv->dht_hash = (dat_addr_t) choose_orig(primary_if->net_dev->dev_addr, DAT_ADDR_MAX); +#endif
A general dat_init()/dat_free() structure would be better than adding defines everywhere.
--- a/send.c +++ b/send.c @@ -30,8 +30,6 @@ #include "gateway_common.h" #include "originator.h"
-#include <net/arp.h>
static void send_outstanding_bcast_packet(struct work_struct *work);
More changing includes ..
The README update is missing in this patch.
Cheers, Marek