Merge tag 'dm-3.17-changes' of git://git.kernel.org/pub/scm/linux/kernel/git/device...
[cascardo/linux.git] / net / hsr / hsr_framereg.c
index 83e5844..bace124 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright 2011-2013 Autronica Fire and Security AS
+/* Copyright 2011-2014 Autronica Fire and Security AS
  *
  * This program is free software; you can redistribute it and/or modify it
  * under the terms of the GNU General Public License as published by the Free
@@ -6,7 +6,7 @@
  * any later version.
  *
  * Author(s):
- *     2011-2013 Arvid Brodin, arvid.brodin@xdin.com
+ *     2011-2014 Arvid Brodin, arvid.brodin@alten.se
  *
  * The HSR spec says never to forward the same frame twice on the same
  * interface. A frame is identified by its source MAC address and its HSR
 #include "hsr_netlink.h"
 
 
-struct node_entry {
-       struct list_head mac_list;
-       unsigned char   MacAddressA[ETH_ALEN];
-       unsigned char   MacAddressB[ETH_ALEN];
-       enum hsr_dev_idx   AddrB_if;    /* The local slave through which AddrB
-                                        * frames are received from this node
-                                        */
-       unsigned long   time_in[HSR_MAX_SLAVE];
-       bool            time_in_stale[HSR_MAX_SLAVE];
-       u16             seq_out[HSR_MAX_DEV];
-       struct rcu_head rcu_head;
+struct hsr_node {
+       struct list_head        mac_list;
+       unsigned char           MacAddressA[ETH_ALEN];
+       unsigned char           MacAddressB[ETH_ALEN];
+       /* Local slave through which AddrB frames are received from this node */
+       enum hsr_port_type      AddrB_port;
+       unsigned long           time_in[HSR_PT_PORTS];
+       bool                    time_in_stale[HSR_PT_PORTS];
+       u16                     seq_out[HSR_PT_PORTS];
+       struct rcu_head         rcu_head;
 };
 
-/*     TODO: use hash lists for mac addresses (linux/jhash.h)?    */
 
+/*     TODO: use hash lists for mac addresses (linux/jhash.h)?    */
 
 
-/* Search for mac entry. Caller must hold rcu read lock.
+/* seq_nr_after(a, b) - return true if a is after (higher in sequence than) b,
+ * false otherwise.
  */
-static struct node_entry *find_node_by_AddrA(struct list_head *node_db,
-                                            const unsigned char addr[ETH_ALEN])
+static bool seq_nr_after(u16 a, u16 b)
 {
-       struct node_entry *node;
-
-       list_for_each_entry_rcu(node, node_db, mac_list) {
-               if (ether_addr_equal(node->MacAddressA, addr))
-                       return node;
-       }
+       /* Remove inconsistency where
+        * seq_nr_after(a, b) == seq_nr_before(a, b)
+        */
+       if ((int) b - a == 32768)
+               return false;
 
-       return NULL;
+       return (((s16) (b - a)) < 0);
 }
+#define seq_nr_before(a, b)            seq_nr_after((b), (a))
+#define seq_nr_after_or_eq(a, b)       (!seq_nr_before((a), (b)))
+#define seq_nr_before_or_eq(a, b)      (!seq_nr_after((a), (b)))
 
 
-/* Search for mac entry. Caller must hold rcu read lock.
- */
-static struct node_entry *find_node_by_AddrB(struct list_head *node_db,
-                                            const unsigned char addr[ETH_ALEN])
+bool hsr_addr_is_self(struct hsr_priv *hsr, unsigned char *addr)
 {
-       struct node_entry *node;
+       struct hsr_node *node;
 
-       list_for_each_entry_rcu(node, node_db, mac_list) {
-               if (ether_addr_equal(node->MacAddressB, addr))
-                       return node;
+       node = list_first_or_null_rcu(&hsr->self_node_db, struct hsr_node,
+                                     mac_list);
+       if (!node) {
+               WARN_ONCE(1, "HSR: No self node\n");
+               return false;
        }
 
-       return NULL;
-}
+       if (ether_addr_equal(addr, node->MacAddressA))
+               return true;
+       if (ether_addr_equal(addr, node->MacAddressB))
+               return true;
 
+       return false;
+}
 
 /* Search for mac entry. Caller must hold rcu read lock.
  */
-struct node_entry *hsr_find_node(struct list_head *node_db, struct sk_buff *skb)
+static struct hsr_node *find_node_by_AddrA(struct list_head *node_db,
+                                          const unsigned char addr[ETH_ALEN])
 {
-       struct node_entry *node;
-       struct ethhdr *ethhdr;
-
-       if (!skb_mac_header_was_set(skb))
-               return NULL;
-
-       ethhdr = (struct ethhdr *) skb_mac_header(skb);
+       struct hsr_node *node;
 
        list_for_each_entry_rcu(node, node_db, mac_list) {
-               if (ether_addr_equal(node->MacAddressA, ethhdr->h_source))
-                       return node;
-               if (ether_addr_equal(node->MacAddressB, ethhdr->h_source))
+               if (ether_addr_equal(node->MacAddressA, addr))
                        return node;
        }
 
@@ -102,7 +99,7 @@ int hsr_create_self_node(struct list_head *self_node_db,
                         unsigned char addr_a[ETH_ALEN],
                         unsigned char addr_b[ETH_ALEN])
 {
-       struct node_entry *node, *oldnode;
+       struct hsr_node *node, *oldnode;
 
        node = kmalloc(sizeof(*node), GFP_KERNEL);
        if (!node)
@@ -113,7 +110,7 @@ int hsr_create_self_node(struct list_head *self_node_db,
 
        rcu_read_lock();
        oldnode = list_first_or_null_rcu(self_node_db,
-                                               struct node_entry, mac_list);
+                                               struct hsr_node, mac_list);
        if (oldnode) {
                list_replace_rcu(&oldnode->mac_list, &node->mac_list);
                rcu_read_unlock();
@@ -128,135 +125,144 @@ int hsr_create_self_node(struct list_head *self_node_db,
 }
 
 
-/* Add/merge node to the database of nodes. 'skb' must contain an HSR
- * supervision frame.
- * - If the supervision header's MacAddressA field is not yet in the database,
- * this frame is from an hitherto unknown node - add it to the database.
- * - If the sender's MAC address is not the same as its MacAddressA address,
- * the node is using PICS_SUBS (address substitution). Record the sender's
- * address as the node's MacAddressB.
- *
- * This function needs to work even if the sender node has changed one of its
- * slaves' MAC addresses. In this case, there are four different cases described
- * by (Addr-changed, received-from) pairs as follows. Note that changing the
- * SlaveA address is equal to changing the node's own address:
- *
- * - (AddrB, SlaveB): The new AddrB will be recorded by PICS_SUBS code since
- *                   node == NULL.
- * - (AddrB, SlaveA): Will work as usual (the AddrB change won't be detected
- *                   from this frame).
- *
- * - (AddrA, SlaveB): The old node will be found. We need to detect this and
- *                   remove the node.
- * - (AddrA, SlaveA): A new node will be registered (non-PICS_SUBS at first).
- *                   The old one will be pruned after HSR_NODE_FORGET_TIME.
- *
- * We also need to detect if the sender's SlaveA and SlaveB cables have been
- * swapped.
+/* Allocate an hsr_node and add it to node_db. 'addr' is the node's AddressA;
+ * seq_out is used to initialize filtering of outgoing duplicate frames
+ * originating from the newly added node.
  */
-struct node_entry *hsr_merge_node(struct hsr_priv *hsr_priv,
-                                 struct node_entry *node,
-                                 struct sk_buff *skb,
-                                 enum hsr_dev_idx dev_idx)
+struct hsr_node *hsr_add_node(struct list_head *node_db, unsigned char addr[],
+                             u16 seq_out)
 {
-       struct hsr_sup_payload *hsr_sp;
-       struct hsr_ethhdr_sp *hsr_ethsup;
-       int i;
+       struct hsr_node *node;
        unsigned long now;
-
-       hsr_ethsup = (struct hsr_ethhdr_sp *) skb_mac_header(skb);
-       hsr_sp = (struct hsr_sup_payload *) skb->data;
-
-       if (node && !ether_addr_equal(node->MacAddressA, hsr_sp->MacAddressA)) {
-               /* Node has changed its AddrA, frame was received from SlaveB */
-               list_del_rcu(&node->mac_list);
-               kfree_rcu(node, rcu_head);
-               node = NULL;
-       }
-
-       if (node && (dev_idx == node->AddrB_if) &&
-           !ether_addr_equal(node->MacAddressB, hsr_ethsup->ethhdr.h_source)) {
-               /* Cables have been swapped */
-               list_del_rcu(&node->mac_list);
-               kfree_rcu(node, rcu_head);
-               node = NULL;
-       }
-
-       if (node && (dev_idx != node->AddrB_if) &&
-           (node->AddrB_if != HSR_DEV_NONE) &&
-           !ether_addr_equal(node->MacAddressA, hsr_ethsup->ethhdr.h_source)) {
-               /* Cables have been swapped */
-               list_del_rcu(&node->mac_list);
-               kfree_rcu(node, rcu_head);
-               node = NULL;
-       }
-
-       if (node)
-               return node;
-
-       node = find_node_by_AddrA(&hsr_priv->node_db, hsr_sp->MacAddressA);
-       if (node) {
-               /* Node is known, but frame was received from an unknown
-                * address. Node is PICS_SUBS capable; merge its AddrB.
-                */
-               ether_addr_copy(node->MacAddressB, hsr_ethsup->ethhdr.h_source);
-               node->AddrB_if = dev_idx;
-               return node;
-       }
+       int i;
 
        node = kzalloc(sizeof(*node), GFP_ATOMIC);
        if (!node)
                return NULL;
 
-       ether_addr_copy(node->MacAddressA, hsr_sp->MacAddressA);
-       ether_addr_copy(node->MacAddressB, hsr_ethsup->ethhdr.h_source);
-       if (!ether_addr_equal(hsr_sp->MacAddressA, hsr_ethsup->ethhdr.h_source))
-               node->AddrB_if = dev_idx;
-       else
-               node->AddrB_if = HSR_DEV_NONE;
+       ether_addr_copy(node->MacAddressA, addr);
 
        /* We are only interested in time diffs here, so use current jiffies
         * as initialization. (0 could trigger an spurious ring error warning).
         */
        now = jiffies;
-       for (i = 0; i < HSR_MAX_SLAVE; i++)
+       for (i = 0; i < HSR_PT_PORTS; i++)
                node->time_in[i] = now;
-       for (i = 0; i < HSR_MAX_DEV; i++)
-               node->seq_out[i] = ntohs(hsr_ethsup->hsr_sup.sequence_nr) - 1;
+       for (i = 0; i < HSR_PT_PORTS; i++)
+               node->seq_out[i] = seq_out;
 
-       list_add_tail_rcu(&node->mac_list, &hsr_priv->node_db);
+       list_add_tail_rcu(&node->mac_list, node_db);
 
        return node;
 }
 
+/* Get the hsr_node from which 'skb' was sent.
+ */
+struct hsr_node *hsr_get_node(struct list_head *node_db, struct sk_buff *skb,
+                             bool is_sup)
+{
+       struct hsr_node *node;
+       struct ethhdr *ethhdr;
+       u16 seq_out;
+
+       if (!skb_mac_header_was_set(skb))
+               return NULL;
+
+       ethhdr = (struct ethhdr *) skb_mac_header(skb);
+
+       list_for_each_entry_rcu(node, node_db, mac_list) {
+               if (ether_addr_equal(node->MacAddressA, ethhdr->h_source))
+                       return node;
+               if (ether_addr_equal(node->MacAddressB, ethhdr->h_source))
+                       return node;
+       }
+
+       if (!is_sup)
+               return NULL; /* Only supervision frame may create node entry */
+
+       if (ethhdr->h_proto == htons(ETH_P_PRP)) {
+               /* Use the existing sequence_nr from the tag as starting point
+                * for filtering duplicate frames.
+                */
+               seq_out = hsr_get_skb_sequence_nr(skb) - 1;
+       } else {
+               WARN_ONCE(1, "%s: Non-HSR frame\n", __func__);
+               seq_out = 0;
+       }
+
+       return hsr_add_node(node_db, ethhdr->h_source, seq_out);
+}
+
+/* Use the Supervision frame's info about an eventual MacAddressB for merging
+ * nodes that has previously had their MacAddressB registered as a separate
+ * node.
+ */
+void hsr_handle_sup_frame(struct sk_buff *skb, struct hsr_node *node_curr,
+                         struct hsr_port *port_rcv)
+{
+       struct hsr_node *node_real;
+       struct hsr_sup_payload *hsr_sp;
+       struct list_head *node_db;
+       int i;
+
+       skb_pull(skb, sizeof(struct hsr_ethhdr_sp));
+       hsr_sp = (struct hsr_sup_payload *) skb->data;
+
+       if (ether_addr_equal(eth_hdr(skb)->h_source, hsr_sp->MacAddressA))
+               /* Not sent from MacAddressB of a PICS_SUBS capable node */
+               goto done;
+
+       /* Merge node_curr (registered on MacAddressB) into node_real */
+       node_db = &port_rcv->hsr->node_db;
+       node_real = find_node_by_AddrA(node_db, hsr_sp->MacAddressA);
+       if (!node_real)
+               /* No frame received from AddrA of this node yet */
+               node_real = hsr_add_node(node_db, hsr_sp->MacAddressA,
+                                        HSR_SEQNR_START - 1);
+       if (!node_real)
+               goto done; /* No mem */
+       if (node_real == node_curr)
+               /* Node has already been merged */
+               goto done;
+
+       ether_addr_copy(node_real->MacAddressB, eth_hdr(skb)->h_source);
+       for (i = 0; i < HSR_PT_PORTS; i++) {
+               if (!node_curr->time_in_stale[i] &&
+                   time_after(node_curr->time_in[i], node_real->time_in[i])) {
+                       node_real->time_in[i] = node_curr->time_in[i];
+                       node_real->time_in_stale[i] = node_curr->time_in_stale[i];
+               }
+               if (seq_nr_after(node_curr->seq_out[i], node_real->seq_out[i]))
+                       node_real->seq_out[i] = node_curr->seq_out[i];
+       }
+       node_real->AddrB_port = port_rcv->type;
+
+       list_del_rcu(&node_curr->mac_list);
+       kfree_rcu(node_curr, rcu_head);
+
+done:
+       skb_push(skb, sizeof(struct hsr_ethhdr_sp));
+}
+
 
 /* 'skb' is a frame meant for this host, that is to be passed to upper layers.
  *
- * If the frame was sent by a node's B interface, replace the sender
+ * If the frame was sent by a node's B interface, replace the source
  * address with that node's "official" address (MacAddressA) so that upper
  * layers recognize where it came from.
  */
-void hsr_addr_subst_source(struct hsr_priv *hsr_priv, struct sk_buff *skb)
+void hsr_addr_subst_source(struct hsr_node *node, struct sk_buff *skb)
 {
-       struct ethhdr *ethhdr;
-       struct node_entry *node;
-
        if (!skb_mac_header_was_set(skb)) {
                WARN_ONCE(1, "%s: Mac header not set\n", __func__);
                return;
        }
-       ethhdr = (struct ethhdr *) skb_mac_header(skb);
 
-       rcu_read_lock();
-       node = find_node_by_AddrB(&hsr_priv->node_db, ethhdr->h_source);
-       if (node)
-               ether_addr_copy(ethhdr->h_source, node->MacAddressA);
-       rcu_read_unlock();
+       memcpy(&eth_hdr(skb)->h_source, node->MacAddressA, ETH_ALEN);
 }
 
-
 /* 'skb' is a frame meant for another host.
- * 'hsr_dev_idx' is the HSR index of the outgoing device
+ * 'port' is the outgoing interface
  *
  * Substitute the target (dest) MAC address if necessary, so the it matches the
  * recipient interface MAC address, regardless of whether that is the
@@ -264,47 +270,44 @@ void hsr_addr_subst_source(struct hsr_priv *hsr_priv, struct sk_buff *skb)
  * This is needed to keep the packets flowing through switches that learn on
  * which "side" the different interfaces are.
  */
-void hsr_addr_subst_dest(struct hsr_priv *hsr_priv, struct ethhdr *ethhdr,
-                        enum hsr_dev_idx dev_idx)
+void hsr_addr_subst_dest(struct hsr_node *node_src, struct sk_buff *skb,
+                        struct hsr_port *port)
 {
-       struct node_entry *node;
+       struct hsr_node *node_dst;
 
-       rcu_read_lock();
-       node = find_node_by_AddrA(&hsr_priv->node_db, ethhdr->h_dest);
-       if (node && (node->AddrB_if == dev_idx))
-               ether_addr_copy(ethhdr->h_dest, node->MacAddressB);
-       rcu_read_unlock();
-}
+       if (!skb_mac_header_was_set(skb)) {
+               WARN_ONCE(1, "%s: Mac header not set\n", __func__);
+               return;
+       }
 
+       if (!is_unicast_ether_addr(eth_hdr(skb)->h_dest))
+               return;
 
-/* seq_nr_after(a, b) - return true if a is after (higher in sequence than) b,
- * false otherwise.
- */
-static bool seq_nr_after(u16 a, u16 b)
-{
-       /* Remove inconsistency where
-        * seq_nr_after(a, b) == seq_nr_before(a, b)
-        */
-       if ((int) b - a == 32768)
-               return false;
+       node_dst = find_node_by_AddrA(&port->hsr->node_db, eth_hdr(skb)->h_dest);
+       if (!node_dst) {
+               WARN_ONCE(1, "%s: Unknown node\n", __func__);
+               return;
+       }
+       if (port->type != node_dst->AddrB_port)
+               return;
 
-       return (((s16) (b - a)) < 0);
+       ether_addr_copy(eth_hdr(skb)->h_dest, node_dst->MacAddressB);
 }
-#define seq_nr_before(a, b)            seq_nr_after((b), (a))
-#define seq_nr_after_or_eq(a, b)       (!seq_nr_before((a), (b)))
-#define seq_nr_before_or_eq(a, b)      (!seq_nr_after((a), (b)))
 
 
-void hsr_register_frame_in(struct node_entry *node, enum hsr_dev_idx dev_idx)
+void hsr_register_frame_in(struct hsr_node *node, struct hsr_port *port,
+                          u16 sequence_nr)
 {
-       if ((dev_idx < 0) || (dev_idx >= HSR_MAX_SLAVE)) {
-               WARN_ONCE(1, "%s: Invalid dev_idx (%d)\n", __func__, dev_idx);
+       /* Don't register incoming frames without a valid sequence number. This
+        * ensures entries of restarted nodes gets pruned so that they can
+        * re-register and resume communications.
+        */
+       if (seq_nr_before(sequence_nr, node->seq_out[port->type]))
                return;
-       }
-       node->time_in[dev_idx] = jiffies;
-       node->time_in_stale[dev_idx] = false;
-}
 
+       node->time_in[port->type] = jiffies;
+       node->time_in_stale[port->type] = false;
+}
 
 /* 'skb' is a HSR Ethernet frame (with a HSR tag inserted), with a valid
  * ethhdr->h_source address and skb->mac_header set.
@@ -314,102 +317,87 @@ void hsr_register_frame_in(struct node_entry *node, enum hsr_dev_idx dev_idx)
  *      0 otherwise, or
  *      negative error code on error
  */
-int hsr_register_frame_out(struct node_entry *node, enum hsr_dev_idx dev_idx,
-                          struct sk_buff *skb)
+int hsr_register_frame_out(struct hsr_port *port, struct hsr_node *node,
+                          u16 sequence_nr)
 {
-       struct hsr_ethhdr *hsr_ethhdr;
-       u16 sequence_nr;
-
-       if ((dev_idx < 0) || (dev_idx >= HSR_MAX_DEV)) {
-               WARN_ONCE(1, "%s: Invalid dev_idx (%d)\n", __func__, dev_idx);
-               return -EINVAL;
-       }
-       if (!skb_mac_header_was_set(skb)) {
-               WARN_ONCE(1, "%s: Mac header not set\n", __func__);
-               return -EINVAL;
-       }
-       hsr_ethhdr = (struct hsr_ethhdr *) skb_mac_header(skb);
-
-       sequence_nr = ntohs(hsr_ethhdr->hsr_tag.sequence_nr);
-       if (seq_nr_before_or_eq(sequence_nr, node->seq_out[dev_idx]))
+       if (seq_nr_before_or_eq(sequence_nr, node->seq_out[port->type]))
                return 1;
 
-       node->seq_out[dev_idx] = sequence_nr;
+       node->seq_out[port->type] = sequence_nr;
        return 0;
 }
 
 
-
-static bool is_late(struct node_entry *node, enum hsr_dev_idx dev_idx)
+static struct hsr_port *get_late_port(struct hsr_priv *hsr,
+                                     struct hsr_node *node)
 {
-       enum hsr_dev_idx other;
-
-       if (node->time_in_stale[dev_idx])
-               return true;
-
-       if (dev_idx == HSR_DEV_SLAVE_A)
-               other = HSR_DEV_SLAVE_B;
-       else
-               other = HSR_DEV_SLAVE_A;
-
-       if (node->time_in_stale[other])
-               return false;
+       if (node->time_in_stale[HSR_PT_SLAVE_A])
+               return hsr_port_get_hsr(hsr, HSR_PT_SLAVE_A);
+       if (node->time_in_stale[HSR_PT_SLAVE_B])
+               return hsr_port_get_hsr(hsr, HSR_PT_SLAVE_B);
+
+       if (time_after(node->time_in[HSR_PT_SLAVE_B],
+                      node->time_in[HSR_PT_SLAVE_A] +
+                                       msecs_to_jiffies(MAX_SLAVE_DIFF)))
+               return hsr_port_get_hsr(hsr, HSR_PT_SLAVE_A);
+       if (time_after(node->time_in[HSR_PT_SLAVE_A],
+                      node->time_in[HSR_PT_SLAVE_B] +
+                                       msecs_to_jiffies(MAX_SLAVE_DIFF)))
+               return hsr_port_get_hsr(hsr, HSR_PT_SLAVE_B);
 
-       if (time_after(node->time_in[other], node->time_in[dev_idx] +
-                      msecs_to_jiffies(MAX_SLAVE_DIFF)))
-               return true;
-
-       return false;
+       return NULL;
 }
 
 
 /* Remove stale sequence_nr records. Called by timer every
  * HSR_LIFE_CHECK_INTERVAL (two seconds or so).
  */
-void hsr_prune_nodes(struct hsr_priv *hsr_priv)
+void hsr_prune_nodes(unsigned long data)
 {
-       struct node_entry *node;
+       struct hsr_priv *hsr;
+       struct hsr_node *node;
+       struct hsr_port *port;
        unsigned long timestamp;
        unsigned long time_a, time_b;
 
+       hsr = (struct hsr_priv *) data;
+
        rcu_read_lock();
-       list_for_each_entry_rcu(node, &hsr_priv->node_db, mac_list) {
+       list_for_each_entry_rcu(node, &hsr->node_db, mac_list) {
                /* Shorthand */
-               time_a = node->time_in[HSR_DEV_SLAVE_A];
-               time_b = node->time_in[HSR_DEV_SLAVE_B];
+               time_a = node->time_in[HSR_PT_SLAVE_A];
+               time_b = node->time_in[HSR_PT_SLAVE_B];
 
                /* Check for timestamps old enough to risk wrap-around */
                if (time_after(jiffies, time_a + MAX_JIFFY_OFFSET/2))
-                       node->time_in_stale[HSR_DEV_SLAVE_A] = true;
+                       node->time_in_stale[HSR_PT_SLAVE_A] = true;
                if (time_after(jiffies, time_b + MAX_JIFFY_OFFSET/2))
-                       node->time_in_stale[HSR_DEV_SLAVE_B] = true;
+                       node->time_in_stale[HSR_PT_SLAVE_B] = true;
 
                /* Get age of newest frame from node.
                 * At least one time_in is OK here; nodes get pruned long
                 * before both time_ins can get stale
                 */
                timestamp = time_a;
-               if (node->time_in_stale[HSR_DEV_SLAVE_A] ||
-                   (!node->time_in_stale[HSR_DEV_SLAVE_B] &&
+               if (node->time_in_stale[HSR_PT_SLAVE_A] ||
+                   (!node->time_in_stale[HSR_PT_SLAVE_B] &&
                    time_after(time_b, time_a)))
                        timestamp = time_b;
 
                /* Warn of ring error only as long as we get frames at all */
                if (time_is_after_jiffies(timestamp +
                                        msecs_to_jiffies(1.5*MAX_SLAVE_DIFF))) {
-
-                       if (is_late(node, HSR_DEV_SLAVE_A))
-                               hsr_nl_ringerror(hsr_priv, node->MacAddressA,
-                                                HSR_DEV_SLAVE_A);
-                       else if (is_late(node, HSR_DEV_SLAVE_B))
-                               hsr_nl_ringerror(hsr_priv, node->MacAddressA,
-                                                HSR_DEV_SLAVE_B);
+                       rcu_read_lock();
+                       port = get_late_port(hsr, node);
+                       if (port != NULL)
+                               hsr_nl_ringerror(hsr, node->MacAddressA, port);
+                       rcu_read_unlock();
                }
 
                /* Prune old entries */
                if (time_is_before_jiffies(timestamp +
                                        msecs_to_jiffies(HSR_NODE_FORGET_TIME))) {
-                       hsr_nl_nodedown(hsr_priv, node->MacAddressA);
+                       hsr_nl_nodedown(hsr, node->MacAddressA);
                        list_del_rcu(&node->mac_list);
                        /* Note that we need to free this entry later: */
                        kfree_rcu(node, rcu_head);
@@ -419,21 +407,21 @@ void hsr_prune_nodes(struct hsr_priv *hsr_priv)
 }
 
 
-void *hsr_get_next_node(struct hsr_priv *hsr_priv, void *_pos,
+void *hsr_get_next_node(struct hsr_priv *hsr, void *_pos,
                        unsigned char addr[ETH_ALEN])
 {
-       struct node_entry *node;
+       struct hsr_node *node;
 
        if (!_pos) {
-               node = list_first_or_null_rcu(&hsr_priv->node_db,
-                                               struct node_entry, mac_list);
+               node = list_first_or_null_rcu(&hsr->node_db,
+                                             struct hsr_node, mac_list);
                if (node)
                        ether_addr_copy(addr, node->MacAddressA);
                return node;
        }
 
        node = _pos;
-       list_for_each_entry_continue_rcu(node, &hsr_priv->node_db, mac_list) {
+       list_for_each_entry_continue_rcu(node, &hsr->node_db, mac_list) {
                ether_addr_copy(addr, node->MacAddressA);
                return node;
        }
@@ -442,7 +430,7 @@ void *hsr_get_next_node(struct hsr_priv *hsr_priv, void *_pos,
 }
 
 
-int hsr_get_node_data(struct hsr_priv *hsr_priv,
+int hsr_get_node_data(struct hsr_priv *hsr,
                      const unsigned char *addr,
                      unsigned char addr_b[ETH_ALEN],
                      unsigned int *addr_b_ifindex,
@@ -451,12 +439,13 @@ int hsr_get_node_data(struct hsr_priv *hsr_priv,
                      int *if2_age,
                      u16 *if2_seq)
 {
-       struct node_entry *node;
+       struct hsr_node *node;
+       struct hsr_port *port;
        unsigned long tdiff;
 
 
        rcu_read_lock();
-       node = find_node_by_AddrA(&hsr_priv->node_db, addr);
+       node = find_node_by_AddrA(&hsr->node_db, addr);
        if (!node) {
                rcu_read_unlock();
                return -ENOENT; /* No such entry */
@@ -464,8 +453,8 @@ int hsr_get_node_data(struct hsr_priv *hsr_priv,
 
        ether_addr_copy(addr_b, node->MacAddressB);
 
-       tdiff = jiffies - node->time_in[HSR_DEV_SLAVE_A];
-       if (node->time_in_stale[HSR_DEV_SLAVE_A])
+       tdiff = jiffies - node->time_in[HSR_PT_SLAVE_A];
+       if (node->time_in_stale[HSR_PT_SLAVE_A])
                *if1_age = INT_MAX;
 #if HZ <= MSEC_PER_SEC
        else if (tdiff > msecs_to_jiffies(INT_MAX))
@@ -474,8 +463,8 @@ int hsr_get_node_data(struct hsr_priv *hsr_priv,
        else
                *if1_age = jiffies_to_msecs(tdiff);
 
-       tdiff = jiffies - node->time_in[HSR_DEV_SLAVE_B];
-       if (node->time_in_stale[HSR_DEV_SLAVE_B])
+       tdiff = jiffies - node->time_in[HSR_PT_SLAVE_B];
+       if (node->time_in_stale[HSR_PT_SLAVE_B])
                *if2_age = INT_MAX;
 #if HZ <= MSEC_PER_SEC
        else if (tdiff > msecs_to_jiffies(INT_MAX))
@@ -485,13 +474,15 @@ int hsr_get_node_data(struct hsr_priv *hsr_priv,
                *if2_age = jiffies_to_msecs(tdiff);
 
        /* Present sequence numbers as if they were incoming on interface */
-       *if1_seq = node->seq_out[HSR_DEV_SLAVE_B];
-       *if2_seq = node->seq_out[HSR_DEV_SLAVE_A];
+       *if1_seq = node->seq_out[HSR_PT_SLAVE_B];
+       *if2_seq = node->seq_out[HSR_PT_SLAVE_A];
 
-       if ((node->AddrB_if != HSR_DEV_NONE) && hsr_priv->slave[node->AddrB_if])
-               *addr_b_ifindex = hsr_priv->slave[node->AddrB_if]->ifindex;
-       else
+       if (node->AddrB_port != HSR_PT_NONE) {
+               port = hsr_port_get_hsr(hsr, node->AddrB_port);
+               *addr_b_ifindex = port->dev->ifindex;
+       } else {
                *addr_b_ifindex = -1;
+       }
 
        rcu_read_unlock();