linux-toradex.git/net/batman-adv, branch v3.2.13

batman-adv: delete global entry in case of roaming

2011-12-06T19:12:59+00:00

When receiving a DEL change for a client due to a roaming event (change is
marked with TT_CLIENT_ROAM), each node  has to check if the client roamed
to itself or somewhere else.

In the latter case the global entry is kept to avoid having no route at all
otherwise we can safely delete the global entry

Signed-off-by: Antonio Quartulli

batman-adv: in case of roaming mark the client with TT_CLIENT_ROAM

2011-12-06T19:12:55+00:00

In case of a client roaming from node A to node B, the latter have to mark the
corresponding global entry with TT_CLIENT_ROAM (instead of TT_CLIENT_PENDING).

Marking a global entry with TT_CLIENT_PENDING will end up in keeping such entry
forever (because this flag is only meant to be used with local entries and it is
never checked on global ones).

In the worst case (all the clients roaming to the same node A) the local and the
global table will contain exactly the same clients. Batman-adv will continue to
work, but the memory usage is duplicated.

Signed-off-by: Antonio Quartulli

Merge branch 'batman-adv/maint' of git://git.open-mesh.org/linux-merge

2011-10-30T07:05:07+00:00

batman-adv: unify hash_entry field position in tt_local/global_entry

2011-10-25T12:23:05+00:00

Function tt_response_fill_table() actually uses a tt_local_entry pointer to
iterate either over the local or the global table entries (it depends on the
what hash table is passed as argument). To iterate over such entries the
hlist_for_each_entry_rcu() macro has to access their "hash_entry" field which
MUST be at the same position in both the tt_global/local_entry structures.

Reported-by: Simon Wunderlich 
Signed-off-by: Antonio Quartulli 
Signed-off-by: Marek Lindner

batman-adv: add sanity check when removing global tts

2011-10-25T12:23:04+00:00

After removing the batman-adv module, the hash may be already gone
when tt_global_del_orig() tries to clean the hash. This patch adds
a sanity check to avoid this.

Signed-off-by: Simon Wunderlich 
Tested-by: Alexey Fisher 
Signed-off-by: Marek Lindner

batman-adv: remove references for global tt entries

2011-10-25T12:23:04+00:00

struct tt_global_entry holds a reference to an orig_node which must be
decremented before deallocating the structure.

Signed-off-by: Simon Wunderlich 
Tested-by: Alexey Fisher 
Signed-off-by: Marek Lindner

Merge branch 'master' of ra.kernel.org:/pub/scm/linux/kernel/git/davem/net

2011-10-24T22:18:09+00:00

batman-adv: correctly set the data field in the TT_REPONSE packet

2011-10-18T20:45:10+00:00

In the TT_RESPONSE packet, the number of carried entries is not correctly set.
This leads to a wrong interpretation of the packet payload on the receiver side
causing random entries to be added to the global translation table. Therefore
the latter gets always corrupted, triggering a table recovery all the time.

Signed-off-by: Antonio Quartulli 
Signed-off-by: Marek Lindner

batman-adv: fix tt_local_reset_flags() function

2011-10-18T20:45:03+00:00

Currently the counter of tt_local_entry structures (tt_local_num) is incremented
each time the tt_local_reset_flags() is invoked causing the node to send wrong
TT_REPONSE packets containing a copy of non-initialised memory thus corrupting
other nodes global translation table and making higher level communication
impossible.

Reported-by: Junkeun Song 
Signed-off-by: Antonio Quartulli 
Acked-by: Junkeun Song 
Signed-off-by: Marek Lindner

Merge branch 'master' of github.com:davem330/net

2011-10-07T17:38:43+00:00

Conflicts:
	net/batman-adv/soft-interface.c