[PATCH 12/17] batman-adv: Use rcu_dereference_protected by update-side

    Usually rcu_dereference isn't necessary in situations were the
    RCU-protected data structure cannot change, but sparse and lockdep still
    need a similar functionality for analysis. rcu_dereference_protected
    implements the reduced version which should be used to support the
    dynamic and static analysis.

    Signed-off-by: Sven Eckelmann <sven@narfation.org>
[PATCH 13/17] batman-adv: move smallest_signed_int(), seq_before() and seq_after() into main.h

    smallest_signed_int(), seq_before() and seq_after() are very useful
    functions that help to handle comparisons between sequence numbers.
    However they were only defined in vis.c. With this patch every
    batman-adv function will be able to use them.

    Signed-off-by: Antonio Quartulli <ordex@autistici.org>
    Signed-off-by: Sven Eckelmann <sven@narfation.org>
[PATCH 14/17] batman-adv: Check type of x and y in seq_(before|after)

    seq_before and seq_after depend on the fact that both sequence numbers
    have the same type and thus the same bitwidth. We can ensure that by
    compile time checking using a compare between the pointer to the
    temporary buffers which were created using the typeof of both
    parameters. For example gcc would create a warning like
    "warning: comparison of distinct pointer types lacks a cast".
[PATCH 15/17] batman-adv: use is_broadcast_ether_addr() instead of compare_eth(.., brd_addr)

    Instead of comparing mac addresses with the broadcast address by means
    of compare_eth(), the is_broadcast_ether_addr() kernel function has to be
    used.

    Signed-off-by: Antonio Quartulli <ordex@autistici.org>
    Acked-by: Sven Eckelmann <sven@narfation.org>
    Signed-off-by: Sven Eckelmann <sven@narfation.org>
[PATCH 16/17] batman-adv: a multiline comment should precede the variable it is describing

    This comment has been wrongly put after the variable it refers to and was also bad indented

    Signed-off-by: Antonio Quartulli <ordex@autistici.org>
    Acked-by: Sven Eckelmann <sven@narfation.org>
    Signed-off-by: Sven Eckelmann <sven@narfation.org>

commit 44e92bc8d6366ebe477cab23199b9f50328d326a
Author: Antonio Quartulli <ordex@autistici.org>
Date:   Sat May 21 01:33:07 2011 +0200
[PATCH 17/17] batman-adv: Ensure that we really have route changes in update_route

    The debug output of update_route has tests for "route deleted" and "route
    added". All other situations are handled as "route changed". This is not
    true because neigh_node and curr_router could be both NULL.

    The function is not called in this situation, but the code might be
    interpreted wrong when reading it without this test.

    Signed-off-by: Sven Eckelmann <sven@narfation.org>