On Wednesday, June 27, 2012 10:23:06 Antonio Quartulli wrote:
@@ -43,6 +43,10 @@ #define BATADV_PURGE_TIMEOUT 200000 /* 200 seconds */ #define BATADV_TT_LOCAL_TIMEOUT 3600000 /* in miliseconds */ #define BATADV_TT_CLIENT_ROAM_TIMEOUT 600000 /* in miliseconds */ +/* Used to compute the timeout value for the entries marked with TT_CLIENT_TEMP: + * temp_timeout_value = temp_timeout_fact * orig_int
- */
+#define BATADV_TT_CLIENT_TEMP_TIMEOUT_FACT 10UL
You implictely assume that all nodes have the originator interval which might not be the case. As soon as they are different this will horribly fail.
Why not using the originator sequence number instead ? It will give you a time independent timeout. You could say if we haven't received a TT ADD 3 sequence numbers after we added the temporary client is gets merged.
Regards, Marek