Murali Karicheri | 0e7623b | 2019-04-05 13:31:34 -0400 | [diff] [blame] | 1 | // SPDX-License-Identifier: GPL-2.0 |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 2 | /* Copyright 2011-2014 Autronica Fire and Security AS |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 3 | * |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 4 | * Author(s): |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 5 | * 2011-2014 Arvid Brodin, arvid.brodin@alten.se |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 6 | * |
| 7 | * This file contains device methods for creating, using and destroying |
| 8 | * virtual HSR devices. |
| 9 | */ |
| 10 | |
| 11 | #include <linux/netdevice.h> |
| 12 | #include <linux/skbuff.h> |
| 13 | #include <linux/etherdevice.h> |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 14 | #include <linux/rtnetlink.h> |
| 15 | #include <linux/pkt_sched.h> |
| 16 | #include "hsr_device.h" |
Arvid Brodin | 81ba6af | 2014-07-04 23:35:24 +0200 | [diff] [blame] | 17 | #include "hsr_slave.h" |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 18 | #include "hsr_framereg.h" |
| 19 | #include "hsr_main.h" |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 20 | #include "hsr_forward.h" |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 21 | |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 22 | static bool is_admin_up(struct net_device *dev) |
| 23 | { |
| 24 | return dev && (dev->flags & IFF_UP); |
| 25 | } |
| 26 | |
| 27 | static bool is_slave_up(struct net_device *dev) |
| 28 | { |
| 29 | return dev && is_admin_up(dev) && netif_oper_up(dev); |
| 30 | } |
| 31 | |
| 32 | static void __hsr_set_operstate(struct net_device *dev, int transition) |
| 33 | { |
| 34 | write_lock_bh(&dev_base_lock); |
| 35 | if (dev->operstate != transition) { |
| 36 | dev->operstate = transition; |
| 37 | write_unlock_bh(&dev_base_lock); |
| 38 | netdev_state_change(dev); |
| 39 | } else { |
| 40 | write_unlock_bh(&dev_base_lock); |
| 41 | } |
| 42 | } |
| 43 | |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 44 | static void hsr_set_operstate(struct hsr_port *master, bool has_carrier) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 45 | { |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 46 | if (!is_admin_up(master->dev)) { |
| 47 | __hsr_set_operstate(master->dev, IF_OPER_DOWN); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 48 | return; |
| 49 | } |
| 50 | |
Arvid Brodin | e9aae56 | 2014-07-04 23:36:40 +0200 | [diff] [blame] | 51 | if (has_carrier) |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 52 | __hsr_set_operstate(master->dev, IF_OPER_UP); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 53 | else |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 54 | __hsr_set_operstate(master->dev, IF_OPER_LOWERLAYERDOWN); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 55 | } |
| 56 | |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 57 | static bool hsr_check_carrier(struct hsr_port *master) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 58 | { |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 59 | struct hsr_port *port; |
Arvid Brodin | e9aae56 | 2014-07-04 23:36:40 +0200 | [diff] [blame] | 60 | |
Taehee Yoo | 81390d0 | 2020-02-28 18:01:56 +0000 | [diff] [blame] | 61 | ASSERT_RTNL(); |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 62 | |
Taehee Yoo | 81390d0 | 2020-02-28 18:01:56 +0000 | [diff] [blame] | 63 | hsr_for_each_port(master->hsr, port) { |
Murali Karicheri | 5670342 | 2019-04-05 13:31:25 -0400 | [diff] [blame] | 64 | if (port->type != HSR_PT_MASTER && is_slave_up(port->dev)) { |
Taehee Yoo | 81390d0 | 2020-02-28 18:01:56 +0000 | [diff] [blame] | 65 | netif_carrier_on(master->dev); |
| 66 | return true; |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 67 | } |
Taehee Yoo | 81390d0 | 2020-02-28 18:01:56 +0000 | [diff] [blame] | 68 | } |
Arvid Brodin | e9aae56 | 2014-07-04 23:36:40 +0200 | [diff] [blame] | 69 | |
Taehee Yoo | 81390d0 | 2020-02-28 18:01:56 +0000 | [diff] [blame] | 70 | netif_carrier_off(master->dev); |
Arvid Brodin | e9aae56 | 2014-07-04 23:36:40 +0200 | [diff] [blame] | 71 | |
Taehee Yoo | 81390d0 | 2020-02-28 18:01:56 +0000 | [diff] [blame] | 72 | return false; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 73 | } |
| 74 | |
Arvid Brodin | e9aae56 | 2014-07-04 23:36:40 +0200 | [diff] [blame] | 75 | static void hsr_check_announce(struct net_device *hsr_dev, |
| 76 | unsigned char old_operstate) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 77 | { |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 78 | struct hsr_priv *hsr; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 79 | |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 80 | hsr = netdev_priv(hsr_dev); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 81 | |
Murali Karicheri | 5670342 | 2019-04-05 13:31:25 -0400 | [diff] [blame] | 82 | if (hsr_dev->operstate == IF_OPER_UP && old_operstate != IF_OPER_UP) { |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 83 | /* Went up */ |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 84 | hsr->announce_count = 0; |
Eric Dumazet | 1e02796 | 2019-03-07 09:36:33 -0800 | [diff] [blame] | 85 | mod_timer(&hsr->announce_timer, |
| 86 | jiffies + msecs_to_jiffies(HSR_ANNOUNCE_INTERVAL)); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 87 | } |
| 88 | |
Murali Karicheri | 5670342 | 2019-04-05 13:31:25 -0400 | [diff] [blame] | 89 | if (hsr_dev->operstate != IF_OPER_UP && old_operstate == IF_OPER_UP) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 90 | /* Went down */ |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 91 | del_timer(&hsr->announce_timer); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 92 | } |
| 93 | |
Arvid Brodin | e9aae56 | 2014-07-04 23:36:40 +0200 | [diff] [blame] | 94 | void hsr_check_carrier_and_operstate(struct hsr_priv *hsr) |
| 95 | { |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 96 | struct hsr_port *master; |
Arvid Brodin | e9aae56 | 2014-07-04 23:36:40 +0200 | [diff] [blame] | 97 | unsigned char old_operstate; |
| 98 | bool has_carrier; |
| 99 | |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 100 | master = hsr_port_get_hsr(hsr, HSR_PT_MASTER); |
Arvid Brodin | e9aae56 | 2014-07-04 23:36:40 +0200 | [diff] [blame] | 101 | /* netif_stacked_transfer_operstate() cannot be used here since |
| 102 | * it doesn't set IF_OPER_LOWERLAYERDOWN (?) |
| 103 | */ |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 104 | old_operstate = master->dev->operstate; |
| 105 | has_carrier = hsr_check_carrier(master); |
| 106 | hsr_set_operstate(master, has_carrier); |
| 107 | hsr_check_announce(master->dev, old_operstate); |
Arvid Brodin | e9aae56 | 2014-07-04 23:36:40 +0200 | [diff] [blame] | 108 | } |
| 109 | |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 110 | int hsr_get_max_mtu(struct hsr_priv *hsr) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 111 | { |
Arvid Brodin | 51f3c60 | 2014-07-04 23:37:27 +0200 | [diff] [blame] | 112 | unsigned int mtu_max; |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 113 | struct hsr_port *port; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 114 | |
Arvid Brodin | 51f3c60 | 2014-07-04 23:37:27 +0200 | [diff] [blame] | 115 | mtu_max = ETH_DATA_LEN; |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 116 | hsr_for_each_port(hsr, port) |
| 117 | if (port->type != HSR_PT_MASTER) |
| 118 | mtu_max = min(port->dev->mtu, mtu_max); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 119 | |
Arvid Brodin | 51f3c60 | 2014-07-04 23:37:27 +0200 | [diff] [blame] | 120 | if (mtu_max < HSR_HLEN) |
| 121 | return 0; |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 122 | return mtu_max - HSR_HLEN; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 123 | } |
| 124 | |
| 125 | static int hsr_dev_change_mtu(struct net_device *dev, int new_mtu) |
| 126 | { |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 127 | struct hsr_priv *hsr; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 128 | |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 129 | hsr = netdev_priv(dev); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 130 | |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 131 | if (new_mtu > hsr_get_max_mtu(hsr)) { |
Taehee Yoo | f30e472 | 2020-04-24 12:43:09 +0000 | [diff] [blame] | 132 | netdev_info(dev, "A HSR master's MTU cannot be greater than the smallest MTU of its slaves minus the HSR Tag length (%d octets).\n", |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 133 | HSR_HLEN); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 134 | return -EINVAL; |
| 135 | } |
| 136 | |
| 137 | dev->mtu = new_mtu; |
| 138 | |
| 139 | return 0; |
| 140 | } |
| 141 | |
| 142 | static int hsr_dev_open(struct net_device *dev) |
| 143 | { |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 144 | struct hsr_priv *hsr; |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 145 | struct hsr_port *port; |
| 146 | char designation; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 147 | |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 148 | hsr = netdev_priv(dev); |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 149 | designation = '\0'; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 150 | |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 151 | hsr_for_each_port(hsr, port) { |
| 152 | if (port->type == HSR_PT_MASTER) |
| 153 | continue; |
| 154 | switch (port->type) { |
| 155 | case HSR_PT_SLAVE_A: |
| 156 | designation = 'A'; |
| 157 | break; |
| 158 | case HSR_PT_SLAVE_B: |
| 159 | designation = 'B'; |
| 160 | break; |
| 161 | default: |
| 162 | designation = '?'; |
| 163 | } |
| 164 | if (!is_slave_up(port->dev)) |
| 165 | netdev_warn(dev, "Slave %c (%s) is not up; please bring it up to get a fully working HSR network\n", |
| 166 | designation, port->dev->name); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 167 | } |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 168 | |
| 169 | if (designation == '\0') |
| 170 | netdev_warn(dev, "No slave devices configured\n"); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 171 | |
| 172 | return 0; |
| 173 | } |
| 174 | |
| 175 | static int hsr_dev_close(struct net_device *dev) |
| 176 | { |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 177 | /* Nothing to do here. */ |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 178 | return 0; |
| 179 | } |
| 180 | |
Arvid Brodin | 1cc1eb5 | 2014-07-04 23:38:57 +0200 | [diff] [blame] | 181 | static netdev_features_t hsr_features_recompute(struct hsr_priv *hsr, |
| 182 | netdev_features_t features) |
| 183 | { |
| 184 | netdev_features_t mask; |
| 185 | struct hsr_port *port; |
| 186 | |
| 187 | mask = features; |
| 188 | |
| 189 | /* Mask out all features that, if supported by one device, should be |
| 190 | * enabled for all devices (see NETIF_F_ONE_FOR_ALL). |
| 191 | * |
| 192 | * Anything that's off in mask will not be enabled - so only things |
| 193 | * that were in features originally, and also is in NETIF_F_ONE_FOR_ALL, |
| 194 | * may become enabled. |
| 195 | */ |
| 196 | features &= ~NETIF_F_ONE_FOR_ALL; |
| 197 | hsr_for_each_port(hsr, port) |
| 198 | features = netdev_increment_features(features, |
| 199 | port->dev->features, |
| 200 | mask); |
| 201 | |
| 202 | return features; |
| 203 | } |
| 204 | |
| 205 | static netdev_features_t hsr_fix_features(struct net_device *dev, |
| 206 | netdev_features_t features) |
| 207 | { |
| 208 | struct hsr_priv *hsr = netdev_priv(dev); |
| 209 | |
| 210 | return hsr_features_recompute(hsr, features); |
| 211 | } |
| 212 | |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 213 | static int hsr_dev_xmit(struct sk_buff *skb, struct net_device *dev) |
| 214 | { |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 215 | struct hsr_priv *hsr = netdev_priv(dev); |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 216 | struct hsr_port *master; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 217 | |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 218 | master = hsr_port_get_hsr(hsr, HSR_PT_MASTER); |
Taehee Yoo | df95467 | 2019-12-05 07:23:39 +0000 | [diff] [blame] | 219 | if (master) { |
| 220 | skb->dev = master->dev; |
| 221 | hsr_forward_skb(skb, master); |
| 222 | } else { |
| 223 | atomic_long_inc(&dev->tx_dropped); |
| 224 | dev_kfree_skb_any(skb); |
| 225 | } |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 226 | return NETDEV_TX_OK; |
| 227 | } |
| 228 | |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 229 | static const struct header_ops hsr_header_ops = { |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 230 | .create = eth_header, |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 231 | .parse = eth_header_parse, |
| 232 | }; |
| 233 | |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 234 | static void send_hsr_supervision_frame(struct hsr_port *master, |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 235 | u8 type, u8 hsr_ver) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 236 | { |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 237 | struct sk_buff *skb; |
| 238 | int hlen, tlen; |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 239 | struct hsr_tag *hsr_tag; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 240 | struct hsr_sup_tag *hsr_stag; |
| 241 | struct hsr_sup_payload *hsr_sp; |
| 242 | unsigned long irqflags; |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 243 | |
| 244 | hlen = LL_RESERVED_SPACE(master->dev); |
| 245 | tlen = master->dev->needed_tailroom; |
Murali Karicheri | 0525fc0 | 2019-04-05 13:31:27 -0400 | [diff] [blame] | 246 | skb = dev_alloc_skb(sizeof(struct hsr_tag) + |
| 247 | sizeof(struct hsr_sup_tag) + |
| 248 | sizeof(struct hsr_sup_payload) + hlen + tlen); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 249 | |
Murali Karicheri | 05ca6e6 | 2019-04-05 13:31:28 -0400 | [diff] [blame] | 250 | if (!skb) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 251 | return; |
| 252 | |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 253 | skb_reserve(skb, hlen); |
| 254 | |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 255 | skb->dev = master->dev; |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 256 | skb->protocol = htons(hsr_ver ? ETH_P_HSR : ETH_P_PRP); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 257 | skb->priority = TC_PRIO_CONTROL; |
| 258 | |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 259 | if (dev_hard_header(skb, skb->dev, (hsr_ver ? ETH_P_HSR : ETH_P_PRP), |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 260 | master->hsr->sup_multicast_addr, |
| 261 | skb->dev->dev_addr, skb->len) <= 0) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 262 | goto out; |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 263 | skb_reset_mac_header(skb); |
Taehee Yoo | 3ed0a1d | 2019-12-22 11:27:08 +0000 | [diff] [blame] | 264 | skb_reset_network_header(skb); |
| 265 | skb_reset_transport_header(skb); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 266 | |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 267 | if (hsr_ver > 0) { |
yuan linyu | b952f4d | 2017-06-18 22:52:04 +0800 | [diff] [blame] | 268 | hsr_tag = skb_put(skb, sizeof(struct hsr_tag)); |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 269 | hsr_tag->encap_proto = htons(ETH_P_PRP); |
| 270 | set_hsr_tag_LSDU_size(hsr_tag, HSR_V1_SUP_LSDUSIZE); |
| 271 | } |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 272 | |
yuan linyu | b952f4d | 2017-06-18 22:52:04 +0800 | [diff] [blame] | 273 | hsr_stag = skb_put(skb, sizeof(struct hsr_sup_tag)); |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 274 | set_hsr_stag_path(hsr_stag, (hsr_ver ? 0x0 : 0xf)); |
| 275 | set_hsr_stag_HSR_ver(hsr_stag, hsr_ver); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 276 | |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 277 | /* From HSRv1 on we have separate supervision sequence numbers. */ |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 278 | spin_lock_irqsave(&master->hsr->seqnr_lock, irqflags); |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 279 | if (hsr_ver > 0) { |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 280 | hsr_stag->sequence_nr = htons(master->hsr->sup_sequence_nr); |
| 281 | hsr_tag->sequence_nr = htons(master->hsr->sequence_nr); |
| 282 | master->hsr->sup_sequence_nr++; |
| 283 | master->hsr->sequence_nr++; |
| 284 | } else { |
| 285 | hsr_stag->sequence_nr = htons(master->hsr->sequence_nr); |
| 286 | master->hsr->sequence_nr++; |
| 287 | } |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 288 | spin_unlock_irqrestore(&master->hsr->seqnr_lock, irqflags); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 289 | |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 290 | hsr_stag->HSR_TLV_type = type; |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 291 | /* TODO: Why 12 in HSRv0? */ |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 292 | hsr_stag->HSR_TLV_length = |
| 293 | hsr_ver ? sizeof(struct hsr_sup_payload) : 12; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 294 | |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 295 | /* Payload: MacAddressA */ |
yuan linyu | b952f4d | 2017-06-18 22:52:04 +0800 | [diff] [blame] | 296 | hsr_sp = skb_put(skb, sizeof(struct hsr_sup_payload)); |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 297 | ether_addr_copy(hsr_sp->macaddress_A, master->dev->dev_addr); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 298 | |
Florian Fainelli | 414e7d7 | 2017-08-21 12:59:10 -0700 | [diff] [blame] | 299 | if (skb_put_padto(skb, ETH_ZLEN + HSR_HLEN)) |
| 300 | return; |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 301 | |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 302 | hsr_forward_skb(skb, master); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 303 | return; |
| 304 | |
| 305 | out: |
Dan Carpenter | 3d1a54e | 2015-11-21 13:34:12 +0300 | [diff] [blame] | 306 | WARN_ONCE(1, "HSR: Could not send supervision frame\n"); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 307 | kfree_skb(skb); |
| 308 | } |
| 309 | |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 310 | /* Announce (supervision frame) timer function |
| 311 | */ |
Kees Cook | dda436b | 2017-10-24 01:46:16 -0700 | [diff] [blame] | 312 | static void hsr_announce(struct timer_list *t) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 313 | { |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 314 | struct hsr_priv *hsr; |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 315 | struct hsr_port *master; |
Eric Dumazet | 1e02796 | 2019-03-07 09:36:33 -0800 | [diff] [blame] | 316 | unsigned long interval; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 317 | |
Kees Cook | dda436b | 2017-10-24 01:46:16 -0700 | [diff] [blame] | 318 | hsr = from_timer(hsr, t, announce_timer); |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 319 | |
| 320 | rcu_read_lock(); |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 321 | master = hsr_port_get_hsr(hsr, HSR_PT_MASTER); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 322 | |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 323 | if (hsr->announce_count < 3 && hsr->prot_version == 0) { |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 324 | send_hsr_supervision_frame(master, HSR_TLV_ANNOUNCE, |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 325 | hsr->prot_version); |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 326 | hsr->announce_count++; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 327 | |
Eric Dumazet | 1e02796 | 2019-03-07 09:36:33 -0800 | [diff] [blame] | 328 | interval = msecs_to_jiffies(HSR_ANNOUNCE_INTERVAL); |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 329 | } else { |
| 330 | send_hsr_supervision_frame(master, HSR_TLV_LIFE_CHECK, |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 331 | hsr->prot_version); |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 332 | |
Eric Dumazet | 1e02796 | 2019-03-07 09:36:33 -0800 | [diff] [blame] | 333 | interval = msecs_to_jiffies(HSR_LIFE_CHECK_INTERVAL); |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 334 | } |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 335 | |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 336 | if (is_admin_up(master->dev)) |
Eric Dumazet | 1e02796 | 2019-03-07 09:36:33 -0800 | [diff] [blame] | 337 | mod_timer(&hsr->announce_timer, jiffies + interval); |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 338 | |
| 339 | rcu_read_unlock(); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 340 | } |
| 341 | |
Taehee Yoo | e0a4b99 | 2020-02-28 18:02:10 +0000 | [diff] [blame] | 342 | static void hsr_del_ports(struct hsr_priv *hsr) |
| 343 | { |
| 344 | struct hsr_port *port; |
| 345 | |
| 346 | port = hsr_port_get_hsr(hsr, HSR_PT_SLAVE_A); |
| 347 | if (port) |
| 348 | hsr_del_port(port); |
| 349 | |
| 350 | port = hsr_port_get_hsr(hsr, HSR_PT_SLAVE_B); |
| 351 | if (port) |
| 352 | hsr_del_port(port); |
| 353 | |
| 354 | port = hsr_port_get_hsr(hsr, HSR_PT_MASTER); |
| 355 | if (port) |
| 356 | hsr_del_port(port); |
| 357 | } |
| 358 | |
Cong Wang | 311633b | 2019-07-09 23:24:54 -0700 | [diff] [blame] | 359 | /* This has to be called after all the readers are gone. |
| 360 | * Otherwise we would have to check the return value of |
| 361 | * hsr_port_get_hsr(). |
| 362 | */ |
| 363 | static void hsr_dev_destroy(struct net_device *hsr_dev) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 364 | { |
Taehee Yoo | e0a4b99 | 2020-02-28 18:02:10 +0000 | [diff] [blame] | 365 | struct hsr_priv *hsr = netdev_priv(hsr_dev); |
Arvid Brodin | 56b08fd | 2015-02-27 21:26:03 +0100 | [diff] [blame] | 366 | |
Murali Karicheri | 9c5f8a19 | 2019-04-15 11:36:01 -0400 | [diff] [blame] | 367 | hsr_debugfs_term(hsr); |
Taehee Yoo | e0a4b99 | 2020-02-28 18:02:10 +0000 | [diff] [blame] | 368 | hsr_del_ports(hsr); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 369 | |
Arvid Brodin | abff716 | 2014-07-04 23:35:47 +0200 | [diff] [blame] | 370 | del_timer_sync(&hsr->prune_timer); |
| 371 | del_timer_sync(&hsr->announce_timer); |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 372 | |
Taehee Yoo | 92a3567 | 2019-12-22 11:26:54 +0000 | [diff] [blame] | 373 | hsr_del_self_node(hsr); |
Cong Wang | b9a1e62 | 2019-07-03 17:21:13 -0700 | [diff] [blame] | 374 | hsr_del_nodes(&hsr->node_db); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 375 | } |
| 376 | |
| 377 | static const struct net_device_ops hsr_device_ops = { |
| 378 | .ndo_change_mtu = hsr_dev_change_mtu, |
| 379 | .ndo_open = hsr_dev_open, |
| 380 | .ndo_stop = hsr_dev_close, |
| 381 | .ndo_start_xmit = hsr_dev_xmit, |
Arvid Brodin | 1cc1eb5 | 2014-07-04 23:38:57 +0200 | [diff] [blame] | 382 | .ndo_fix_features = hsr_fix_features, |
Cong Wang | 311633b | 2019-07-09 23:24:54 -0700 | [diff] [blame] | 383 | .ndo_uninit = hsr_dev_destroy, |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 384 | }; |
| 385 | |
Arvid Brodin | 4c3477d | 2014-07-04 23:39:42 +0200 | [diff] [blame] | 386 | static struct device_type hsr_type = { |
| 387 | .name = "hsr", |
| 388 | }; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 389 | |
| 390 | void hsr_dev_setup(struct net_device *dev) |
| 391 | { |
Tobias Klauser | d4af4c0 | 2017-02-21 14:04:59 +0100 | [diff] [blame] | 392 | eth_hw_addr_random(dev); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 393 | |
| 394 | ether_setup(dev); |
Jarod Wilson | b3e3893 | 2016-10-20 13:55:22 -0400 | [diff] [blame] | 395 | dev->min_mtu = 0; |
Arvid Brodin | 4c3477d | 2014-07-04 23:39:42 +0200 | [diff] [blame] | 396 | dev->header_ops = &hsr_header_ops; |
| 397 | dev->netdev_ops = &hsr_device_ops; |
| 398 | SET_NETDEV_DEVTYPE(dev, &hsr_type); |
Phil Sutter | 9ad09c5 | 2015-08-18 10:30:45 +0200 | [diff] [blame] | 399 | dev->priv_flags |= IFF_NO_QUEUE; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 400 | |
David S. Miller | cf124db | 2017-05-08 12:52:56 -0400 | [diff] [blame] | 401 | dev->needs_free_netdev = true; |
Arvid Brodin | 1cc1eb5 | 2014-07-04 23:38:57 +0200 | [diff] [blame] | 402 | |
| 403 | dev->hw_features = NETIF_F_SG | NETIF_F_FRAGLIST | NETIF_F_HIGHDMA | |
| 404 | NETIF_F_GSO_MASK | NETIF_F_HW_CSUM | |
| 405 | NETIF_F_HW_VLAN_CTAG_TX; |
| 406 | |
| 407 | dev->features = dev->hw_features; |
| 408 | |
| 409 | /* Prevent recursive tx locking */ |
| 410 | dev->features |= NETIF_F_LLTX; |
| 411 | /* VLAN on top of HSR needs testing and probably some work on |
| 412 | * hsr_header_create() etc. |
| 413 | */ |
| 414 | dev->features |= NETIF_F_VLAN_CHALLENGED; |
Arvid Brodin | 4c3477d | 2014-07-04 23:39:42 +0200 | [diff] [blame] | 415 | /* Not sure about this. Taken from bridge code. netdev_features.h says |
| 416 | * it means "Does not change network namespaces". |
| 417 | */ |
| 418 | dev->features |= NETIF_F_NETNS_LOCAL; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 419 | } |
| 420 | |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 421 | /* Return true if dev is a HSR master; return false otherwise. |
| 422 | */ |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 423 | inline bool is_hsr_master(struct net_device *dev) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 424 | { |
| 425 | return (dev->netdev_ops->ndo_start_xmit == hsr_dev_xmit); |
| 426 | } |
| 427 | |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 428 | /* Default multicast address for HSR Supervision frames */ |
Joe Perches | e83abe3 | 2014-02-18 10:37:20 -0800 | [diff] [blame] | 429 | static const unsigned char def_multicast_addr[ETH_ALEN] __aligned(2) = { |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 430 | 0x01, 0x15, 0x4e, 0x00, 0x01, 0x00 |
| 431 | }; |
| 432 | |
| 433 | int hsr_dev_finalize(struct net_device *hsr_dev, struct net_device *slave[2], |
Taehee Yoo | 13eeb5f | 2020-02-28 18:01:35 +0000 | [diff] [blame] | 434 | unsigned char multicast_spec, u8 protocol_version, |
| 435 | struct netlink_ext_ack *extack) |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 436 | { |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 437 | struct hsr_priv *hsr; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 438 | int res; |
| 439 | |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 440 | hsr = netdev_priv(hsr_dev); |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 441 | INIT_LIST_HEAD(&hsr->ports); |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 442 | INIT_LIST_HEAD(&hsr->node_db); |
| 443 | INIT_LIST_HEAD(&hsr->self_node_db); |
Taehee Yoo | 92a3567 | 2019-12-22 11:26:54 +0000 | [diff] [blame] | 444 | spin_lock_init(&hsr->list_lock); |
Arvid Brodin | 51f3c60 | 2014-07-04 23:37:27 +0200 | [diff] [blame] | 445 | |
| 446 | ether_addr_copy(hsr_dev->dev_addr, slave[0]->dev_addr); |
| 447 | |
| 448 | /* Make sure we recognize frames from ourselves in hsr_rcv() */ |
Taehee Yoo | 92a3567 | 2019-12-22 11:26:54 +0000 | [diff] [blame] | 449 | res = hsr_create_self_node(hsr, hsr_dev->dev_addr, |
Arvid Brodin | 51f3c60 | 2014-07-04 23:37:27 +0200 | [diff] [blame] | 450 | slave[1]->dev_addr); |
| 451 | if (res < 0) |
| 452 | return res; |
| 453 | |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 454 | spin_lock_init(&hsr->seqnr_lock); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 455 | /* Overflow soon to find bugs easier: */ |
Arvid Brodin | f266a68 | 2014-07-04 23:41:03 +0200 | [diff] [blame] | 456 | hsr->sequence_nr = HSR_SEQNR_START; |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 457 | hsr->sup_sequence_nr = HSR_SUP_SEQNR_START; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 458 | |
Kees Cook | dda436b | 2017-10-24 01:46:16 -0700 | [diff] [blame] | 459 | timer_setup(&hsr->announce_timer, hsr_announce, 0); |
| 460 | timer_setup(&hsr->prune_timer, hsr_prune_nodes, 0); |
Arvid Brodin | abff716 | 2014-07-04 23:35:47 +0200 | [diff] [blame] | 461 | |
Arvid Brodin | 70ebe4a | 2014-07-04 23:34:38 +0200 | [diff] [blame] | 462 | ether_addr_copy(hsr->sup_multicast_addr, def_multicast_addr); |
| 463 | hsr->sup_multicast_addr[ETH_ALEN - 1] = multicast_spec; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 464 | |
Murali Karicheri | b1b4aa9 | 2019-04-05 13:31:32 -0400 | [diff] [blame] | 465 | hsr->prot_version = protocol_version; |
Peter Heise | ee1c279 | 2016-04-13 13:52:22 +0200 | [diff] [blame] | 466 | |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 467 | /* FIXME: should I modify the value of these? |
| 468 | * |
| 469 | * - hsr_dev->flags - i.e. |
| 470 | * IFF_MASTER/SLAVE? |
| 471 | * - hsr_dev->priv_flags - i.e. |
| 472 | * IFF_EBRIDGE? |
| 473 | * IFF_TX_SKB_SHARING? |
| 474 | * IFF_HSR_MASTER/SLAVE? |
| 475 | */ |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 476 | |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 477 | /* Make sure the 1st call to netif_carrier_on() gets through */ |
| 478 | netif_carrier_off(hsr_dev); |
| 479 | |
Taehee Yoo | 13eeb5f | 2020-02-28 18:01:35 +0000 | [diff] [blame] | 480 | res = hsr_add_port(hsr, hsr_dev, HSR_PT_MASTER, extack); |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 481 | if (res) |
Taehee Yoo | 1d19e2d | 2019-12-22 11:26:15 +0000 | [diff] [blame] | 482 | goto err_add_master; |
Arvid Brodin | 51f3c60 | 2014-07-04 23:37:27 +0200 | [diff] [blame] | 483 | |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 484 | res = register_netdevice(hsr_dev); |
Arvid Brodin | 51f3c60 | 2014-07-04 23:37:27 +0200 | [diff] [blame] | 485 | if (res) |
Taehee Yoo | 1d19e2d | 2019-12-22 11:26:15 +0000 | [diff] [blame] | 486 | goto err_unregister; |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 487 | |
Taehee Yoo | 13eeb5f | 2020-02-28 18:01:35 +0000 | [diff] [blame] | 488 | res = hsr_add_port(hsr, slave[0], HSR_PT_SLAVE_A, extack); |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 489 | if (res) |
Taehee Yoo | 1d19e2d | 2019-12-22 11:26:15 +0000 | [diff] [blame] | 490 | goto err_add_slaves; |
| 491 | |
Taehee Yoo | 13eeb5f | 2020-02-28 18:01:35 +0000 | [diff] [blame] | 492 | res = hsr_add_port(hsr, slave[1], HSR_PT_SLAVE_B, extack); |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 493 | if (res) |
Taehee Yoo | 1d19e2d | 2019-12-22 11:26:15 +0000 | [diff] [blame] | 494 | goto err_add_slaves; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 495 | |
Taehee Yoo | 1d19e2d | 2019-12-22 11:26:15 +0000 | [diff] [blame] | 496 | hsr_debugfs_init(hsr, hsr_dev); |
Muhammad Falak R Wani | 15db6e0 | 2016-05-15 19:37:44 +0530 | [diff] [blame] | 497 | mod_timer(&hsr->prune_timer, jiffies + msecs_to_jiffies(PRUNE_PERIOD)); |
Arvid Brodin | abff716 | 2014-07-04 23:35:47 +0200 | [diff] [blame] | 498 | |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 499 | return 0; |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 500 | |
Taehee Yoo | 1d19e2d | 2019-12-22 11:26:15 +0000 | [diff] [blame] | 501 | err_add_slaves: |
| 502 | unregister_netdevice(hsr_dev); |
| 503 | err_unregister: |
Taehee Yoo | e0a4b99 | 2020-02-28 18:02:10 +0000 | [diff] [blame] | 504 | hsr_del_ports(hsr); |
Taehee Yoo | 1d19e2d | 2019-12-22 11:26:15 +0000 | [diff] [blame] | 505 | err_add_master: |
Taehee Yoo | 92a3567 | 2019-12-22 11:26:54 +0000 | [diff] [blame] | 506 | hsr_del_self_node(hsr); |
Arvid Brodin | c5a7591 | 2014-07-04 23:38:05 +0200 | [diff] [blame] | 507 | |
| 508 | return res; |
Arvid Brodin | f421436 | 2013-10-30 21:10:47 +0100 | [diff] [blame] | 509 | } |