batman-adv: refactoring API: find generalized name for bat_ogm_update_mac callback
authorMarek Lindner <lindner_marek@yahoo.de>
Sat, 10 Mar 2012 22:17:50 +0000 (06:17 +0800)
committerAntonio Quartulli <ordex@autistici.org>
Fri, 11 May 2012 11:56:01 +0000 (13:56 +0200)
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>
Signed-off-by: Antonio Quartulli <ordex@autistici.org>
net/batman-adv/bat_iv_ogm.c
net/batman-adv/hard-interface.c
net/batman-adv/main.c
net/batman-adv/types.h

index 37e368d05937d0a412f0844a44449d97b9f9ccbf..9074e743ad6133c0d0d4d7e8d5193d22991b0700 100644 (file)
@@ -93,24 +93,24 @@ static void bat_iv_ogm_iface_disable(struct hard_iface *hard_iface)
        hard_iface->packet_buff = NULL;
 }
 
-static void bat_iv_ogm_primary_iface_set(struct hard_iface *hard_iface)
+static void bat_iv_ogm_iface_update_mac(struct hard_iface *hard_iface)
 {
        struct batman_ogm_packet *batman_ogm_packet;
 
        batman_ogm_packet = (struct batman_ogm_packet *)hard_iface->packet_buff;
-       batman_ogm_packet->flags = PRIMARIES_FIRST_HOP;
-       batman_ogm_packet->header.ttl = TTL;
+       memcpy(batman_ogm_packet->orig,
+              hard_iface->net_dev->dev_addr, ETH_ALEN);
+       memcpy(batman_ogm_packet->prev_sender,
+              hard_iface->net_dev->dev_addr, ETH_ALEN);
 }
 
-static void bat_iv_ogm_update_mac(struct hard_iface *hard_iface)
+static void bat_iv_ogm_primary_iface_set(struct hard_iface *hard_iface)
 {
        struct batman_ogm_packet *batman_ogm_packet;
 
        batman_ogm_packet = (struct batman_ogm_packet *)hard_iface->packet_buff;
-       memcpy(batman_ogm_packet->orig,
-              hard_iface->net_dev->dev_addr, ETH_ALEN);
-       memcpy(batman_ogm_packet->prev_sender,
-              hard_iface->net_dev->dev_addr, ETH_ALEN);
+       batman_ogm_packet->flags = PRIMARIES_FIRST_HOP;
+       batman_ogm_packet->header.ttl = TTL;
 }
 
 /* when do we schedule our own ogm to be sent */
@@ -1236,8 +1236,8 @@ static struct bat_algo_ops batman_iv __read_mostly = {
        .name = "BATMAN IV",
        .bat_iface_enable = bat_iv_ogm_iface_enable,
        .bat_iface_disable = bat_iv_ogm_iface_disable,
+       .bat_iface_update_mac = bat_iv_ogm_iface_update_mac,
        .bat_primary_iface_set = bat_iv_ogm_primary_iface_set,
-       .bat_ogm_update_mac = bat_iv_ogm_update_mac,
        .bat_ogm_schedule = bat_iv_ogm_schedule,
        .bat_ogm_emit = bat_iv_ogm_emit,
 };
index 95f869c7ca04965212f36be5753f8b756f18daeb..0b84bb1b62c4c0935a68679a9584346a446c94bc 100644 (file)
@@ -228,7 +228,7 @@ static void hardif_activate_interface(struct hard_iface *hard_iface)
 
        bat_priv = netdev_priv(hard_iface->soft_iface);
 
-       bat_priv->bat_algo_ops->bat_ogm_update_mac(hard_iface);
+       bat_priv->bat_algo_ops->bat_iface_update_mac(hard_iface);
        hard_iface->if_status = IF_TO_BE_ACTIVATED;
 
        /**
@@ -524,7 +524,7 @@ static int hard_if_event(struct notifier_block *this,
                check_known_mac_addr(hard_iface->net_dev);
 
                bat_priv = netdev_priv(hard_iface->soft_iface);
-               bat_priv->bat_algo_ops->bat_ogm_update_mac(hard_iface);
+               bat_priv->bat_algo_ops->bat_iface_update_mac(hard_iface);
 
                primary_if = primary_if_get_selected(bat_priv);
                if (!primary_if)
index f80c4474127cde3f903f1cd445dc0251c24360ea..083a2993efe43802c25d8be483e11470731227a5 100644 (file)
@@ -329,8 +329,8 @@ int bat_algo_register(struct bat_algo_ops *bat_algo_ops)
        /* all algorithms must implement all ops (for now) */
        if (!bat_algo_ops->bat_iface_enable ||
            !bat_algo_ops->bat_iface_disable ||
+           !bat_algo_ops->bat_iface_update_mac ||
            !bat_algo_ops->bat_primary_iface_set ||
-           !bat_algo_ops->bat_ogm_update_mac ||
            !bat_algo_ops->bat_ogm_schedule ||
            !bat_algo_ops->bat_ogm_emit) {
                pr_info("Routing algo '%s' does not implement required ops\n",
index 9fa8b73387ec66c215cee304935e6baa1324c28c..66a3750aa9e700a9aca8c131eafb60046a2f52c6 100644 (file)
@@ -381,10 +381,12 @@ struct bat_algo_ops {
        int (*bat_iface_enable)(struct hard_iface *hard_iface);
        /* de-init routing info when hard-interface is disabled */
        void (*bat_iface_disable)(struct hard_iface *hard_iface);
+       /* (re-)init mac addresses of the protocol information
+        * belonging to this hard-interface
+        */
+       void (*bat_iface_update_mac)(struct hard_iface *hard_iface);
        /* called when primary interface is selected / changed */
        void (*bat_primary_iface_set)(struct hard_iface *hard_iface);
-       /* init mac addresses of the OGM belonging to this hard-interface */
-       void (*bat_ogm_update_mac)(struct hard_iface *hard_iface);
        /* prepare a new outgoing OGM for the send queue */
        void (*bat_ogm_schedule)(struct hard_iface *hard_iface,
                                 int tt_num_changes);