From: Sven Eckelmann s...@narfation.org
The hard_if_event is called by the notifier with rtnl_lock and tries to
remove sysfs entries when a NETDEV_UNREGISTER event is received. This
will automatically take the s_active lock.
The s_active lock is also used when a new interface is added to a
Introduce an *_dec_not_zero operation. Make this a special case of
*_add_unless because batman-adv uses atomic_dec_not_zero in different
places like re-broadcast queue or aggregation queue management. There
are other non-final patches which may also want to use this macro.
Reported-by: David S.
From: Sven Eckelmann s...@narfation.org
hardif_list_lock is unneccessary because we already ensure that no
multiple admin operations can take place through rtnl_lock.
hardif_list_lock only adds additional overhead and complexity.
Critical functions now check whether they are called with
With that tiny change, it now all works as expected and the patch 1/2 can also
be used and verified without having to add patch 2/2 as well. Hope I'm not
getting too picky :D.
Thanks for your effort again, Sven!
Cheers, Linus
On Tuesday 03 May 2011 11:51:37 Linus Lüssing wrote:
With that tiny change, it now all works as expected and the patch 1/2 can
also be used and verified without having to add patch 2/2 as well. Hope
I'm not getting too picky :D.
Thanks for your effort again, Sven!
No, I am very happy with
From: Sven Eckelmann s...@narfation.org
The hard_if_event is called by the notifier with rtnl_lock and tries to
remove sysfs entries when a NETDEV_UNREGISTER event is received. This
will automatically take the s_active lock.
The s_active lock is also used when a new interface is added to a
In this new patchset, patch 2/4 has been revised following Andrew's
suggestions.
In particular all the changes are new comments, except one:
send_tt_response() has been slightly modified to avoid it to return
NET_RX_SUCCESS/NET_RX_DROP so avoiding to create confusion.
Regards,
Antonio
To be coherent, all the functions/variables/constats have been renamed
to the TranslationTable style
Signed-off-by: Antonio Quartulli or...@autistici.org
---
README |8 +-
aggregation.c | 16 +-
aggregation.h |4 +-
bat_debugfs.c |4 +-
Exploting the new announcement implementation, it has been possible to
improve the roaming mechanism and reduce the number of packet drops.
For details, please visit:
http://www.open-mesh.org/wiki/batman-adv/Roaming-improvements
Signed-off-by: Antonio Quartulli or...@autistici.org
---
The local and the global translation-tables are now lock free and rcu
protected.
Signed-off-by: Antonio Quartulli or...@autistici.org
---
main.c |2 -
translation-table.c | 266 +--
types.h |6 +-
vis.c
On Tuesday 03 May 2011 17:50:07 Antonio Quartulli wrote:
I think this is the information bisect uses to look for routing loops
etc. Do you plan to extend bisect to look for TT problems? Does it
make sense to add a new DBG_TT which dumps the adds and removes in the
OGM?
I don't
On Tue, May 03, 2011 at 05:56:45PM +0200, Marek Lindner wrote:
On Tuesday 03 May 2011 17:50:07 Antonio Quartulli wrote:
I think this is the information bisect uses to look for routing loops
etc. Do you plan to extend bisect to look for TT problems? Does it
make sense to add a new DBG_TT
Signed-off-by: Sven Eckelmann s...@narfation.org
---
This is pure speculation and only to show how ugly it is... It is currently
not used in Linus' tree
DONT APPLY THIS PATCH
Makefile.kbuild |1 +
compat.c| 41
13 matches
Mail list logo