netdev: Remove monitors and notifiers.
[openvswitch] / lib / bond.c
1 /*
2  * Copyright (c) 2008, 2009, 2010, 2011 Nicira Networks.
3  *
4  * Licensed under the Apache License, Version 2.0 (the "License");
5  * you may not use this file except in compliance with the License.
6  * You may obtain a copy of the License at:
7  *
8  *     http://www.apache.org/licenses/LICENSE-2.0
9  *
10  * Unless required by applicable law or agreed to in writing, software
11  * distributed under the License is distributed on an "AS IS" BASIS,
12  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13  * See the License for the specific language governing permissions and
14  * limitations under the License.
15  */
16
17 #include <config.h>
18
19 #include "bond.h"
20
21 #include <limits.h>
22 #include <stdint.h>
23 #include <stdlib.h>
24
25 #include "coverage.h"
26 #include "dynamic-string.h"
27 #include "flow.h"
28 #include "hmap.h"
29 #include "list.h"
30 #include "netdev.h"
31 #include "odp-util.h"
32 #include "ofpbuf.h"
33 #include "packets.h"
34 #include "poll-loop.h"
35 #include "tag.h"
36 #include "timeval.h"
37 #include "unixctl.h"
38 #include "vlog.h"
39
40 VLOG_DEFINE_THIS_MODULE(bond);
41
42 /* Bit-mask for hashing a flow down to a bucket.
43  * There are (BOND_MASK + 1) buckets. */
44 #define BOND_MASK 0xff
45
46 /* A hash bucket for mapping a flow to a slave.
47  * "struct bond" has an array of (BOND_MASK + 1) of these. */
48 struct bond_entry {
49     struct bond_slave *slave;   /* Assigned slave, NULL if unassigned. */
50     uint64_t tx_bytes;          /* Count of bytes recently transmitted. */
51     tag_type tag;               /* Tag for entry<->slave association. */
52     struct list list_node;      /* In bond_slave's 'entries' list. */
53 };
54
55 /* A bond slave, that is, one of the links comprising a bond. */
56 struct bond_slave {
57     struct hmap_node hmap_node; /* In struct bond's slaves hmap. */
58     struct bond *bond;          /* The bond that contains this slave. */
59     void *aux;                  /* Client-provided handle for this slave. */
60
61     struct netdev *netdev;      /* Network device, owned by the client. */
62     unsigned int change_seq;    /* Tracks changes in 'netdev'. */
63     char *name;                 /* Name (a copy of netdev_get_name(netdev)). */
64
65     /* Link status. */
66     long long delay_expires;    /* Time after which 'enabled' may change. */
67     bool enabled;               /* May be chosen for flows? */
68     bool may_enable;            /* Client considers this slave bondable. */
69     tag_type tag;               /* Tag associated with this slave. */
70
71     /* Rebalancing info.  Used only by bond_rebalance(). */
72     struct list bal_node;       /* In bond_rebalance()'s 'bals' list. */
73     struct list entries;        /* 'struct bond_entry's assigned here. */
74     uint64_t tx_bytes;          /* Sum across 'tx_bytes' of entries. */
75
76     /* BM_STABLE specific bonding info. */
77     uint32_t stb_id;            /* ID used for 'stb_slaves' ordering. */
78 };
79
80 /* A bond, that is, a set of network devices grouped to improve performance or
81  * robustness.  */
82 struct bond {
83     struct hmap_node hmap_node; /* In 'all_bonds' hmap. */
84     char *name;                 /* Name provided by client. */
85
86     /* Slaves. */
87     struct hmap slaves;
88
89     /* Bonding info. */
90     enum bond_mode balance;     /* Balancing mode, one of BM_*. */
91     struct bond_slave *active_slave;
92     tag_type no_slaves_tag;     /* Tag for flows when all slaves disabled. */
93     int updelay, downdelay;     /* Delay before slave goes up/down, in ms. */
94     bool lacp_negotiated;       /* LACP negotiations were successful. */
95     bool bond_revalidate;       /* True if flows need revalidation. */
96     uint32_t basis;             /* Basis for flow hash function. */
97
98     /* SLB specific bonding info. */
99     struct bond_entry *hash;     /* An array of (BOND_MASK + 1) elements. */
100     int rebalance_interval;      /* Interval between rebalances, in ms. */
101     long long int next_rebalance; /* Next rebalancing time. */
102     bool send_learning_packets;
103
104     /* BM_STABLE specific bonding info. */
105     tag_type stb_tag;               /* Tag associated with this bond. */
106
107     /* Legacy compatibility. */
108     long long int next_fake_iface_update; /* LLONG_MAX if disabled. */
109
110     /* Tag set saved for next bond_run().  This tag set is a kluge for cases
111      * where we can't otherwise provide revalidation feedback to the client.
112      * That's only unixctl commands now; I hope no other cases will arise. */
113     struct tag_set unixctl_tags;
114 };
115
116 static struct hmap all_bonds = HMAP_INITIALIZER(&all_bonds);
117
118 static void bond_entry_reset(struct bond *);
119 static struct bond_slave *bond_slave_lookup(struct bond *, const void *slave_);
120 static void bond_enable_slave(struct bond_slave *, bool enable,
121                               struct tag_set *);
122 static void bond_link_status_update(struct bond_slave *, struct tag_set *);
123 static void bond_choose_active_slave(struct bond *, struct tag_set *);
124 static bool bond_is_tcp_hash(const struct bond *);
125 static unsigned int bond_hash_src(const uint8_t mac[ETH_ADDR_LEN],
126                                   uint16_t vlan, uint32_t basis);
127 static unsigned int bond_hash_tcp(const struct flow *, uint16_t vlan,
128                                   uint32_t basis);
129 static struct bond_entry *lookup_bond_entry(const struct bond *,
130                                             const struct flow *,
131                                             uint16_t vlan);
132 static tag_type bond_get_active_slave_tag(const struct bond *);
133 static struct bond_slave *choose_output_slave(const struct bond *,
134                                               const struct flow *,
135                                               uint16_t vlan);
136 static void bond_update_fake_slave_stats(struct bond *);
137
138 /* Attempts to parse 's' as the name of a bond balancing mode.  If successful,
139  * stores the mode in '*balance' and returns true.  Otherwise returns false
140  * without modifying '*balance'. */
141 bool
142 bond_mode_from_string(enum bond_mode *balance, const char *s)
143 {
144     if (!strcmp(s, bond_mode_to_string(BM_TCP))) {
145         *balance = BM_TCP;
146     } else if (!strcmp(s, bond_mode_to_string(BM_SLB))) {
147         *balance = BM_SLB;
148     } else if (!strcmp(s, bond_mode_to_string(BM_STABLE))) {
149         *balance = BM_STABLE;
150     } else if (!strcmp(s, bond_mode_to_string(BM_AB))) {
151         *balance = BM_AB;
152     } else {
153         return false;
154     }
155     return true;
156 }
157
158 /* Returns a string representing 'balance'. */
159 const char *
160 bond_mode_to_string(enum bond_mode balance) {
161     switch (balance) {
162     case BM_TCP:
163         return "balance-tcp";
164     case BM_SLB:
165         return "balance-slb";
166     case BM_STABLE:
167         return "stable";
168     case BM_AB:
169         return "active-backup";
170     }
171     NOT_REACHED();
172 }
173
174 \f
175 /* Creates and returns a new bond whose configuration is initially taken from
176  * 's'.
177  *
178  * The caller should register each slave on the new bond by calling
179  * bond_slave_register().  */
180 struct bond *
181 bond_create(const struct bond_settings *s)
182 {
183     struct bond *bond;
184
185     bond = xzalloc(sizeof *bond);
186     hmap_init(&bond->slaves);
187     bond->no_slaves_tag = tag_create_random();
188     bond->stb_tag = tag_create_random();
189     bond->next_fake_iface_update = LLONG_MAX;
190
191     bond_reconfigure(bond, s);
192
193     tag_set_init(&bond->unixctl_tags);
194
195     return bond;
196 }
197
198 /* Frees 'bond'. */
199 void
200 bond_destroy(struct bond *bond)
201 {
202     struct bond_slave *slave, *next_slave;
203
204     if (!bond) {
205         return;
206     }
207
208     hmap_remove(&all_bonds, &bond->hmap_node);
209
210     HMAP_FOR_EACH_SAFE (slave, next_slave, hmap_node, &bond->slaves) {
211         hmap_remove(&bond->slaves, &slave->hmap_node);
212         /* Client owns 'slave->netdev'. */
213         free(slave->name);
214         free(slave);
215     }
216     hmap_destroy(&bond->slaves);
217
218     free(bond->hash);
219     free(bond->name);
220     free(bond);
221 }
222
223 /* Updates 'bond''s overall configuration to 's'.
224  *
225  * The caller should register each slave on 'bond' by calling
226  * bond_slave_register().  This is optional if none of the slaves'
227  * configuration has changed.  In any case it can't hurt.
228  *
229  * Returns true if the configuration has changed in such a way that requires
230  * flow revalidation.
231  * */
232 bool
233 bond_reconfigure(struct bond *bond, const struct bond_settings *s)
234 {
235     bool revalidate = false;
236
237     if (!bond->name || strcmp(bond->name, s->name)) {
238         if (bond->name) {
239             hmap_remove(&all_bonds, &bond->hmap_node);
240             free(bond->name);
241         }
242         bond->name = xstrdup(s->name);
243         hmap_insert(&all_bonds, &bond->hmap_node, hash_string(bond->name, 0));
244     }
245
246     bond->updelay = s->up_delay;
247     bond->downdelay = s->down_delay;
248     bond->rebalance_interval = s->rebalance_interval;
249
250     if (bond->balance != s->balance) {
251         bond->balance = s->balance;
252         revalidate = true;
253     }
254
255     if (bond->basis != s->basis) {
256         bond->basis = s->basis;
257         revalidate = true;
258     }
259
260     if (s->fake_iface) {
261         if (bond->next_fake_iface_update == LLONG_MAX) {
262             bond->next_fake_iface_update = time_msec();
263         }
264     } else {
265         bond->next_fake_iface_update = LLONG_MAX;
266     }
267
268     if (bond->bond_revalidate) {
269         revalidate = true;
270         bond->bond_revalidate = false;
271     }
272
273     if (bond->balance == BM_AB || !bond->hash || revalidate) {
274         bond_entry_reset(bond);
275     }
276
277     return revalidate;
278 }
279
280 static void
281 bond_slave_set_netdev__(struct bond_slave *slave, struct netdev *netdev)
282 {
283     if (slave->netdev != netdev) {
284         slave->netdev = netdev;
285         slave->change_seq = 0;
286     }
287 }
288
289 /* Registers 'slave_' as a slave of 'bond'.  The 'slave_' pointer is an
290  * arbitrary client-provided pointer that uniquely identifies a slave within a
291  * bond.  If 'slave_' already exists within 'bond' then this function
292  * reconfigures the existing slave.
293  *
294  * 'stb_id' is used in BM_STABLE bonds to guarantee consistent slave choices
295  * across restarts and distributed vswitch instances.  It should be unique per
296  * slave, and preferably consistent across restarts and reconfigurations.
297  *
298  * 'netdev' must be the network device that 'slave_' represents.  It is owned
299  * by the client, so the client must not close it before either unregistering
300  * 'slave_' or destroying 'bond'.
301  */
302 void
303 bond_slave_register(struct bond *bond, void *slave_, uint32_t stb_id,
304                     struct netdev *netdev)
305 {
306     struct bond_slave *slave = bond_slave_lookup(bond, slave_);
307
308     if (!slave) {
309         slave = xzalloc(sizeof *slave);
310
311         hmap_insert(&bond->slaves, &slave->hmap_node, hash_pointer(slave_, 0));
312         slave->bond = bond;
313         slave->aux = slave_;
314         slave->delay_expires = LLONG_MAX;
315         slave->name = xstrdup(netdev_get_name(netdev));
316         bond->bond_revalidate = true;
317
318         slave->enabled = false;
319         bond_enable_slave(slave, netdev_get_carrier(netdev), NULL);
320     }
321
322     if (slave->stb_id != stb_id) {
323         slave->stb_id = stb_id;
324         bond->bond_revalidate = true;
325     }
326
327     bond_slave_set_netdev__(slave, netdev);
328
329     free(slave->name);
330     slave->name = xstrdup(netdev_get_name(netdev));
331 }
332
333 /* Updates the network device to be used with 'slave_' to 'netdev'.
334  *
335  * This is useful if the caller closes and re-opens the network device
336  * registered with bond_slave_register() but doesn't need to change anything
337  * else. */
338 void
339 bond_slave_set_netdev(struct bond *bond, void *slave_, struct netdev *netdev)
340 {
341     struct bond_slave *slave = bond_slave_lookup(bond, slave_);
342     if (slave) {
343         bond_slave_set_netdev__(slave, netdev);
344     }
345 }
346
347 /* Unregisters 'slave_' from 'bond'.  If 'bond' does not contain such a slave
348  * then this function has no effect.
349  *
350  * Unregistering a slave invalidates all flows. */
351 void
352 bond_slave_unregister(struct bond *bond, const void *slave_)
353 {
354     struct bond_slave *slave = bond_slave_lookup(bond, slave_);
355     bool del_active;
356
357     if (!slave) {
358         return;
359     }
360
361     bond_enable_slave(slave, false, NULL);
362
363     del_active = bond->active_slave == slave;
364     if (bond->hash) {
365         struct bond_entry *e;
366         for (e = bond->hash; e <= &bond->hash[BOND_MASK]; e++) {
367             if (e->slave == slave) {
368                 e->slave = NULL;
369             }
370         }
371     }
372
373     free(slave->name);
374
375     hmap_remove(&bond->slaves, &slave->hmap_node);
376     /* Client owns 'slave->netdev'. */
377     free(slave);
378
379     if (del_active) {
380         struct tag_set tags;
381
382         tag_set_init(&tags);
383         bond_choose_active_slave(bond, &tags);
384         bond->send_learning_packets = true;
385     }
386 }
387
388 /* Should be called on each slave in 'bond' before bond_run() to indicate
389  * whether or not 'slave_' may be enabled. This function is intended to allow
390  * other protocols to have some impact on bonding decisions.  For example LACP
391  * or high level link monitoring protocols may decide that a given slave should
392  * not be able to send traffic. */
393 void
394 bond_slave_set_may_enable(struct bond *bond, void *slave_, bool may_enable)
395 {
396     bond_slave_lookup(bond, slave_)->may_enable = may_enable;
397 }
398
399 /* Performs periodic maintenance on 'bond'.  The caller must provide 'tags' to
400  * allow tagged flows to be invalidated.
401  *
402  * The caller should check bond_should_send_learning_packets() afterward. */
403 void
404 bond_run(struct bond *bond, struct tag_set *tags, bool lacp_negotiated)
405 {
406     struct bond_slave *slave;
407     bool is_tcp_hash = bond_is_tcp_hash(bond);
408
409     bond->lacp_negotiated = lacp_negotiated;
410
411     /* Enable slaves based on link status and LACP feedback. */
412     HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
413         bond_link_status_update(slave, tags);
414         slave->change_seq = netdev_change_seq(slave->netdev);
415     }
416     if (!bond->active_slave || !bond->active_slave->enabled) {
417         bond_choose_active_slave(bond, tags);
418     }
419
420     /* Update fake bond interface stats. */
421     if (time_msec() >= bond->next_fake_iface_update) {
422         bond_update_fake_slave_stats(bond);
423         bond->next_fake_iface_update = time_msec() + 1000;
424     }
425
426     if (is_tcp_hash != bond_is_tcp_hash(bond)) {
427         bond->bond_revalidate = true;
428     }
429
430     if (bond->bond_revalidate) {
431         bond->bond_revalidate = false;
432
433         bond_entry_reset(bond);
434         if (bond->balance != BM_STABLE) {
435             struct bond_slave *slave;
436
437             HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
438                 tag_set_add(tags, slave->tag);
439             }
440         } else {
441             tag_set_add(tags, bond->stb_tag);
442         }
443         tag_set_add(tags, bond->no_slaves_tag);
444     }
445
446     /* Invalidate any tags required by  */
447     tag_set_union(tags, &bond->unixctl_tags);
448     tag_set_init(&bond->unixctl_tags);
449 }
450
451 /* Causes poll_block() to wake up when 'bond' needs something to be done. */
452 void
453 bond_wait(struct bond *bond)
454 {
455     struct bond_slave *slave;
456
457     HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
458         if (slave->delay_expires != LLONG_MAX) {
459             poll_timer_wait_until(slave->delay_expires);
460         }
461
462         if (slave->change_seq != netdev_change_seq(slave->netdev)) {
463             poll_immediate_wake();
464         }
465     }
466
467     if (bond->next_fake_iface_update != LLONG_MAX) {
468         poll_timer_wait_until(bond->next_fake_iface_update);
469     }
470
471     /* Ensure that any saved tags get revalidated right away. */
472     if (!tag_set_is_empty(&bond->unixctl_tags)) {
473         poll_immediate_wake();
474     }
475
476     /* We don't wait for bond->next_rebalance because rebalancing can only run
477      * at a flow account checkpoint.  ofproto does checkpointing on its own
478      * schedule and bond_rebalance() gets called afterward, so we'd just be
479      * waking up for no purpose. */
480 }
481 \f
482 /* MAC learning table interaction. */
483
484 static bool
485 may_send_learning_packets(const struct bond *bond)
486 {
487     return !bond->lacp_negotiated && bond->balance != BM_AB;
488 }
489
490 /* Returns true if 'bond' needs the client to send out packets to assist with
491  * MAC learning on 'bond'.  If this function returns true, then the client
492  * should iterate through its MAC learning table for the bridge on which 'bond'
493  * is located.  For each MAC that has been learned on a port other than 'bond',
494  * it should call bond_send_learning_packet().
495  *
496  * This function will only return true if 'bond' is in SLB mode and LACP is not
497  * negotiated.  Otherwise sending learning packets isn't necessary.
498  *
499  * Calling this function resets the state that it checks. */
500 bool
501 bond_should_send_learning_packets(struct bond *bond)
502 {
503     bool send = bond->send_learning_packets && may_send_learning_packets(bond);
504     bond->send_learning_packets = false;
505     return send;
506 }
507
508 /* Sends a gratuitous learning packet on 'bond' from 'eth_src' on 'vlan'.
509  *
510  * See bond_should_send_learning_packets() for description of usage. */
511 int
512 bond_send_learning_packet(struct bond *bond,
513                           const uint8_t eth_src[ETH_ADDR_LEN],
514                           uint16_t vlan)
515 {
516     struct bond_slave *slave;
517     struct ofpbuf packet;
518     struct flow flow;
519     int error;
520
521     assert(may_send_learning_packets(bond));
522     if (!bond->active_slave) {
523         /* Nowhere to send the learning packet. */
524         return 0;
525     }
526
527     memset(&flow, 0, sizeof flow);
528     memcpy(flow.dl_src, eth_src, ETH_ADDR_LEN);
529     slave = choose_output_slave(bond, &flow, vlan);
530
531     ofpbuf_init(&packet, 0);
532     compose_benign_packet(&packet, "Open vSwitch Bond Failover", 0xf177,
533                           eth_src);
534     if (vlan) {
535         eth_set_vlan_tci(&packet, htons(vlan));
536     }
537     error = netdev_send(slave->netdev, &packet);
538     ofpbuf_uninit(&packet);
539
540     return error;
541 }
542 \f
543 /* Checks whether a packet that arrived on 'slave_' within 'bond', with an
544  * Ethernet destination address of 'eth_dst', should be admitted.
545  *
546  * The return value is one of the following:
547  *
548  *    - BV_ACCEPT: Admit the packet.
549  *
550  *    - BV_DROP: Drop the packet.
551  *
552  *    - BV_DROP_IF_MOVED: Consult the MAC learning table for the packet's
553  *      Ethernet source address and VLAN.  If there is none, or if the packet
554  *      is on the learned port, then admit the packet.  If a different port has
555  *      been learned, however, drop the packet (and do not use it for MAC
556  *      learning).
557  */
558 enum bond_verdict
559 bond_check_admissibility(struct bond *bond, const void *slave_,
560                          const uint8_t eth_dst[ETH_ADDR_LEN], tag_type *tags)
561 {
562     /* Admit all packets if LACP has been negotiated, because that means that
563      * the remote switch is aware of the bond and will "do the right thing". */
564     if (bond->lacp_negotiated) {
565         return BV_ACCEPT;
566     }
567
568     /* Drop all multicast packets on inactive slaves. */
569     if (eth_addr_is_multicast(eth_dst)) {
570         *tags |= bond_get_active_slave_tag(bond);
571         if (bond->active_slave != bond_slave_lookup(bond, slave_)) {
572             return BV_DROP;
573         }
574     }
575
576     /* Drop all packets for which we have learned a different input port,
577      * because we probably sent the packet on one slave and got it back on the
578      * other.  Gratuitous ARP packets are an exception to this rule: the host
579      * has moved to another switch.  The exception to the exception is if we
580      * locked the learning table to avoid reflections on bond slaves. */
581     return BV_DROP_IF_MOVED;
582 }
583
584 /* Returns the slave (registered on 'bond' by bond_slave_register()) to which
585  * a packet with the given 'flow' and 'vlan' should be forwarded.  Returns
586  * NULL if the packet should be dropped because no slaves are enabled.
587  *
588  * 'vlan' is not necessarily the same as 'flow->vlan_tci'.  First, 'vlan'
589  * should be a VID only (i.e. excluding the PCP bits).  Second,
590  * 'flow->vlan_tci' is the VLAN TCI that appeared on the packet (so it will be
591  * nonzero only for trunk ports), whereas 'vlan' is the logical VLAN that the
592  * packet belongs to (so for an access port it will be the access port's VLAN).
593  *
594  * Adds a tag to '*tags' that associates the flow with the returned slave.
595  */
596 void *
597 bond_choose_output_slave(struct bond *bond, const struct flow *flow,
598                          uint16_t vlan, tag_type *tags)
599 {
600     struct bond_slave *slave = choose_output_slave(bond, flow, vlan);
601     if (slave) {
602         *tags |= bond->balance == BM_STABLE ? bond->stb_tag : slave->tag;
603         return slave->aux;
604     } else {
605         *tags |= bond->no_slaves_tag;
606         return NULL;
607     }
608 }
609 \f
610 /* Rebalancing. */
611
612 static bool
613 bond_is_balanced(const struct bond *bond)
614 {
615     return bond->balance == BM_SLB || bond->balance == BM_TCP;
616 }
617
618 /* Notifies 'bond' that 'n_bytes' bytes were sent in 'flow' within 'vlan'. */
619 void
620 bond_account(struct bond *bond, const struct flow *flow, uint16_t vlan,
621              uint64_t n_bytes)
622 {
623
624     if (bond_is_balanced(bond)) {
625         lookup_bond_entry(bond, flow, vlan)->tx_bytes += n_bytes;
626     }
627 }
628
629 static struct bond_slave *
630 bond_slave_from_bal_node(struct list *bal)
631 {
632     return CONTAINER_OF(bal, struct bond_slave, bal_node);
633 }
634
635 static void
636 log_bals(struct bond *bond, const struct list *bals)
637 {
638     if (VLOG_IS_DBG_ENABLED()) {
639         struct ds ds = DS_EMPTY_INITIALIZER;
640         const struct bond_slave *slave;
641
642         LIST_FOR_EACH (slave, bal_node, bals) {
643             if (ds.length) {
644                 ds_put_char(&ds, ',');
645             }
646             ds_put_format(&ds, " %s %"PRIu64"kB",
647                           slave->name, slave->tx_bytes / 1024);
648
649             if (!slave->enabled) {
650                 ds_put_cstr(&ds, " (disabled)");
651             }
652             if (!list_is_empty(&slave->entries)) {
653                 struct bond_entry *e;
654
655                 ds_put_cstr(&ds, " (");
656                 LIST_FOR_EACH (e, list_node, &slave->entries) {
657                     if (&e->list_node != list_front(&slave->entries)) {
658                         ds_put_cstr(&ds, " + ");
659                     }
660                     ds_put_format(&ds, "h%td: %"PRIu64"kB",
661                                   e - bond->hash, e->tx_bytes / 1024);
662                 }
663                 ds_put_cstr(&ds, ")");
664             }
665         }
666         VLOG_DBG("bond %s:%s", bond->name, ds_cstr(&ds));
667         ds_destroy(&ds);
668     }
669 }
670
671 /* Shifts 'hash' from its current slave to 'to'. */
672 static void
673 bond_shift_load(struct bond_entry *hash, struct bond_slave *to,
674                 struct tag_set *set)
675 {
676     struct bond_slave *from = hash->slave;
677     struct bond *bond = from->bond;
678     uint64_t delta = hash->tx_bytes;
679
680     VLOG_INFO("bond %s: shift %"PRIu64"kB of load (with hash %td) "
681               "from %s to %s (now carrying %"PRIu64"kB and "
682               "%"PRIu64"kB load, respectively)",
683               bond->name, delta / 1024, hash - bond->hash,
684               from->name, to->name,
685               (from->tx_bytes - delta) / 1024,
686               (to->tx_bytes + delta) / 1024);
687
688     /* Shift load away from 'from' to 'to'. */
689     from->tx_bytes -= delta;
690     to->tx_bytes += delta;
691
692     /* Arrange for flows to be revalidated. */
693     tag_set_add(set, hash->tag);
694     hash->slave = to;
695     hash->tag = tag_create_random();
696 }
697
698 /* Pick and returns a bond_entry to migrate to 'to' (the least-loaded slave),
699  * given that doing so must decrease the ratio of the load on the two slaves by
700  * at least 0.1.  Returns NULL if there is no appropriate entry.
701  *
702  * The list of entries isn't sorted.  I don't know of a reason to prefer to
703  * shift away small hashes or large hashes. */
704 static struct bond_entry *
705 choose_entry_to_migrate(const struct bond_slave *from, uint64_t to_tx_bytes)
706 {
707     struct bond_entry *e;
708
709     if (list_is_short(&from->entries)) {
710         /* 'from' carries no more than one MAC hash, so shifting load away from
711          * it would be pointless. */
712         return NULL;
713     }
714
715     LIST_FOR_EACH (e, list_node, &from->entries) {
716         double old_ratio, new_ratio;
717         uint64_t delta;
718
719         if (to_tx_bytes == 0) {
720             /* Nothing on the new slave, move it. */
721             return e;
722         }
723
724         delta = e->tx_bytes;
725         old_ratio = (double)from->tx_bytes / to_tx_bytes;
726         new_ratio = (double)(from->tx_bytes - delta) / (to_tx_bytes + delta);
727         if (old_ratio - new_ratio > 0.1) {
728             /* Would decrease the ratio, move it. */
729             return e;
730         }
731     }
732
733     return NULL;
734 }
735
736 /* Inserts 'slave' into 'bals' so that descending order of 'tx_bytes' is
737  * maintained. */
738 static void
739 insert_bal(struct list *bals, struct bond_slave *slave)
740 {
741     struct bond_slave *pos;
742
743     LIST_FOR_EACH (pos, bal_node, bals) {
744         if (slave->tx_bytes > pos->tx_bytes) {
745             break;
746         }
747     }
748     list_insert(&pos->bal_node, &slave->bal_node);
749 }
750
751 /* Removes 'slave' from its current list and then inserts it into 'bals' so
752  * that descending order of 'tx_bytes' is maintained. */
753 static void
754 reinsert_bal(struct list *bals, struct bond_slave *slave)
755 {
756     list_remove(&slave->bal_node);
757     insert_bal(bals, slave);
758 }
759
760 /* If 'bond' needs rebalancing, does so.
761  *
762  * The caller should have called bond_account() for each active flow, to ensure
763  * that flow data is consistently accounted at this point. */
764 void
765 bond_rebalance(struct bond *bond, struct tag_set *tags)
766 {
767     struct bond_slave *slave;
768     struct bond_entry *e;
769     struct list bals;
770
771     if (!bond_is_balanced(bond) || time_msec() < bond->next_rebalance) {
772         return;
773     }
774     bond->next_rebalance = time_msec() + bond->rebalance_interval;
775
776     /* Add each bond_entry to its slave's 'entries' list.
777      * Compute each slave's tx_bytes as the sum of its entries' tx_bytes. */
778     HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
779         slave->tx_bytes = 0;
780         list_init(&slave->entries);
781     }
782     for (e = &bond->hash[0]; e <= &bond->hash[BOND_MASK]; e++) {
783         if (e->slave && e->tx_bytes) {
784             e->slave->tx_bytes += e->tx_bytes;
785             list_push_back(&e->slave->entries, &e->list_node);
786         }
787     }
788
789     /* Add enabled slaves to 'bals' in descending order of tx_bytes.
790      *
791      * XXX This is O(n**2) in the number of slaves but it could be O(n lg n)
792      * with a proper list sort algorithm. */
793     list_init(&bals);
794     HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
795         if (slave->enabled) {
796             insert_bal(&bals, slave);
797         }
798     }
799     log_bals(bond, &bals);
800
801     /* Shift load from the most-loaded slaves to the least-loaded slaves. */
802     while (!list_is_short(&bals)) {
803         struct bond_slave *from = bond_slave_from_bal_node(list_front(&bals));
804         struct bond_slave *to = bond_slave_from_bal_node(list_back(&bals));
805         uint64_t overload;
806
807         overload = from->tx_bytes - to->tx_bytes;
808         if (overload < to->tx_bytes >> 5 || overload < 100000) {
809             /* The extra load on 'from' (and all less-loaded slaves), compared
810              * to that of 'to' (the least-loaded slave), is less than ~3%, or
811              * it is less than ~1Mbps.  No point in rebalancing. */
812             break;
813         }
814
815         /* 'from' is carrying significantly more load than 'to', and that load
816          * is split across at least two different hashes. */
817         e = choose_entry_to_migrate(from, to->tx_bytes);
818         if (e) {
819             bond_shift_load(e, to, tags);
820
821             /* Delete element from from->entries.
822              *
823              * We don't add the element to to->hashes.  That would only allow
824              * 'e' to be migrated to another slave in this rebalancing run, and
825              * there is no point in doing that. */
826             list_remove(&e->list_node);
827
828             /* Re-sort 'bals'. */
829             reinsert_bal(&bals, from);
830             reinsert_bal(&bals, to);
831         } else {
832             /* Can't usefully migrate anything away from 'from'.
833              * Don't reconsider it. */
834             list_remove(&from->bal_node);
835         }
836     }
837
838     /* Implement exponentially weighted moving average.  A weight of 1/2 causes
839      * historical data to decay to <1% in 7 rebalancing runs.  1,000,000 bytes
840      * take 20 rebalancing runs to decay to 0 and get deleted entirely. */
841     for (e = &bond->hash[0]; e <= &bond->hash[BOND_MASK]; e++) {
842         e->tx_bytes /= 2;
843         if (!e->tx_bytes) {
844             e->slave = NULL;
845         }
846     }
847 }
848 \f
849 /* Bonding unixctl user interface functions. */
850
851 static struct bond *
852 bond_find(const char *name)
853 {
854     struct bond *bond;
855
856     HMAP_FOR_EACH_WITH_HASH (bond, hmap_node, hash_string(name, 0),
857                              &all_bonds) {
858         if (!strcmp(bond->name, name)) {
859             return bond;
860         }
861     }
862     return NULL;
863 }
864
865 static struct bond_slave *
866 bond_lookup_slave(struct bond *bond, const char *slave_name)
867 {
868     struct bond_slave *slave;
869
870     HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
871         if (!strcmp(slave->name, slave_name)) {
872             return slave;
873         }
874     }
875     return NULL;
876 }
877
878 static void
879 bond_unixctl_list(struct unixctl_conn *conn,
880                   const char *args OVS_UNUSED, void *aux OVS_UNUSED)
881 {
882     struct ds ds = DS_EMPTY_INITIALIZER;
883     const struct bond *bond;
884
885     ds_put_cstr(&ds, "bond\ttype\tslaves\n");
886
887     HMAP_FOR_EACH (bond, hmap_node, &all_bonds) {
888         const struct bond_slave *slave;
889         size_t i;
890
891         ds_put_format(&ds, "%s\t%s\t",
892                       bond->name, bond_mode_to_string(bond->balance));
893
894         i = 0;
895         HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
896             if (i++ > 0) {
897                 ds_put_cstr(&ds, ", ");
898             }
899             ds_put_cstr(&ds, slave->name);
900         }
901         ds_put_char(&ds, '\n');
902     }
903     unixctl_command_reply(conn, 200, ds_cstr(&ds));
904     ds_destroy(&ds);
905 }
906
907 static void
908 bond_unixctl_show(struct unixctl_conn *conn,
909                   const char *args, void *aux OVS_UNUSED)
910 {
911     struct ds ds = DS_EMPTY_INITIALIZER;
912     const struct bond_slave *slave;
913     const struct bond *bond;
914
915     bond = bond_find(args);
916     if (!bond) {
917         unixctl_command_reply(conn, 501, "no such bond");
918         return;
919     }
920
921     ds_put_format(&ds, "bond_mode: %s\n",
922                   bond_mode_to_string(bond->balance));
923
924     if (bond->balance != BM_AB) {
925         ds_put_format(&ds, "bond-hash-algorithm: %s\n",
926                       bond_is_tcp_hash(bond) ? "balance-tcp" : "balance-slb");
927     }
928
929     ds_put_format(&ds, "bond-hash-basis: %"PRIu32"\n", bond->basis);
930
931     ds_put_format(&ds, "updelay: %d ms\n", bond->updelay);
932     ds_put_format(&ds, "downdelay: %d ms\n", bond->downdelay);
933
934     if (bond_is_balanced(bond)) {
935         ds_put_format(&ds, "next rebalance: %lld ms\n",
936                       bond->next_rebalance - time_msec());
937     }
938
939     ds_put_format(&ds, "lacp_negotiated: %s\n",
940                   bond->lacp_negotiated ? "true" : "false");
941
942     HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
943         struct bond_entry *be;
944
945         /* Basic info. */
946         ds_put_format(&ds, "\nslave %s: %s\n",
947                       slave->name, slave->enabled ? "enabled" : "disabled");
948         if (slave == bond->active_slave) {
949             ds_put_cstr(&ds, "\tactive slave\n");
950         }
951         if (slave->delay_expires != LLONG_MAX) {
952             ds_put_format(&ds, "\t%s expires in %lld ms\n",
953                           slave->enabled ? "downdelay" : "updelay",
954                           slave->delay_expires - time_msec());
955         }
956
957         ds_put_format(&ds, "\tmay_enable: %s\n",
958                       slave->may_enable ? "true" : "false");
959
960         if (!bond_is_balanced(bond)) {
961             continue;
962         }
963
964         /* Hashes. */
965         for (be = bond->hash; be <= &bond->hash[BOND_MASK]; be++) {
966             int hash = be - bond->hash;
967
968             if (be->slave != slave) {
969                 continue;
970             }
971
972             ds_put_format(&ds, "\thash %d: %"PRIu64" kB load\n",
973                           hash, be->tx_bytes / 1024);
974
975             if (bond->balance != BM_SLB) {
976                 continue;
977             }
978
979             /* XXX How can we list the MACs assigned to hashes? */
980         }
981     }
982     unixctl_command_reply(conn, 200, ds_cstr(&ds));
983     ds_destroy(&ds);
984 }
985
986 static void
987 bond_unixctl_migrate(struct unixctl_conn *conn, const char *args_,
988                      void *aux OVS_UNUSED)
989 {
990     char *args = (char *) args_;
991     char *save_ptr = NULL;
992     char *bond_s, *hash_s, *slave_s;
993     struct bond *bond;
994     struct bond_slave *slave;
995     struct bond_entry *entry;
996     int hash;
997
998     bond_s = strtok_r(args, " ", &save_ptr);
999     hash_s = strtok_r(NULL, " ", &save_ptr);
1000     slave_s = strtok_r(NULL, " ", &save_ptr);
1001     if (!slave_s) {
1002         unixctl_command_reply(conn, 501,
1003                               "usage: bond/migrate BOND HASH SLAVE");
1004         return;
1005     }
1006
1007     bond = bond_find(bond_s);
1008     if (!bond) {
1009         unixctl_command_reply(conn, 501, "no such bond");
1010         return;
1011     }
1012
1013     if (bond->balance != BM_SLB) {
1014         unixctl_command_reply(conn, 501, "not an SLB bond");
1015         return;
1016     }
1017
1018     if (strspn(hash_s, "0123456789") == strlen(hash_s)) {
1019         hash = atoi(hash_s) & BOND_MASK;
1020     } else {
1021         unixctl_command_reply(conn, 501, "bad hash");
1022         return;
1023     }
1024
1025     slave = bond_lookup_slave(bond, slave_s);
1026     if (!slave) {
1027         unixctl_command_reply(conn, 501, "no such slave");
1028         return;
1029     }
1030
1031     if (!slave->enabled) {
1032         unixctl_command_reply(conn, 501, "cannot migrate to disabled slave");
1033         return;
1034     }
1035
1036     entry = &bond->hash[hash];
1037     tag_set_add(&bond->unixctl_tags, entry->tag);
1038     entry->slave = slave;
1039     entry->tag = tag_create_random();
1040     unixctl_command_reply(conn, 200, "migrated");
1041 }
1042
1043 static void
1044 bond_unixctl_set_active_slave(struct unixctl_conn *conn, const char *args_,
1045                               void *aux OVS_UNUSED)
1046 {
1047     char *args = (char *) args_;
1048     char *save_ptr = NULL;
1049     char *bond_s, *slave_s;
1050     struct bond *bond;
1051     struct bond_slave *slave;
1052
1053     bond_s = strtok_r(args, " ", &save_ptr);
1054     slave_s = strtok_r(NULL, " ", &save_ptr);
1055     if (!slave_s) {
1056         unixctl_command_reply(conn, 501,
1057                               "usage: bond/set-active-slave BOND SLAVE");
1058         return;
1059     }
1060
1061     bond = bond_find(bond_s);
1062     if (!bond) {
1063         unixctl_command_reply(conn, 501, "no such bond");
1064         return;
1065     }
1066
1067     slave = bond_lookup_slave(bond, slave_s);
1068     if (!slave) {
1069         unixctl_command_reply(conn, 501, "no such slave");
1070         return;
1071     }
1072
1073     if (!slave->enabled) {
1074         unixctl_command_reply(conn, 501, "cannot make disabled slave active");
1075         return;
1076     }
1077
1078     if (bond->active_slave != slave) {
1079         tag_set_add(&bond->unixctl_tags, bond_get_active_slave_tag(bond));
1080         bond->active_slave = slave;
1081         bond->active_slave->tag = tag_create_random();
1082         VLOG_INFO("bond %s: active interface is now %s",
1083                   bond->name, slave->name);
1084         bond->send_learning_packets = true;
1085         unixctl_command_reply(conn, 200, "done");
1086     } else {
1087         unixctl_command_reply(conn, 200, "no change");
1088     }
1089 }
1090
1091 static void
1092 enable_slave(struct unixctl_conn *conn, const char *args_, bool enable)
1093 {
1094     char *args = (char *) args_;
1095     char *save_ptr = NULL;
1096     char *bond_s, *slave_s;
1097     struct bond *bond;
1098     struct bond_slave *slave;
1099
1100     bond_s = strtok_r(args, " ", &save_ptr);
1101     slave_s = strtok_r(NULL, " ", &save_ptr);
1102     if (!slave_s) {
1103         char *usage = xasprintf("usage: bond/%s-slave BOND SLAVE",
1104                                 enable ? "enable" : "disable");
1105         unixctl_command_reply(conn, 501, usage);
1106         free(usage);
1107         return;
1108     }
1109
1110     bond = bond_find(bond_s);
1111     if (!bond) {
1112         unixctl_command_reply(conn, 501, "no such bond");
1113         return;
1114     }
1115
1116     slave = bond_lookup_slave(bond, slave_s);
1117     if (!slave) {
1118         unixctl_command_reply(conn, 501, "no such slave");
1119         return;
1120     }
1121
1122     bond_enable_slave(slave, enable, &bond->unixctl_tags);
1123     unixctl_command_reply(conn, 501, enable ? "enabled" : "disabled");
1124 }
1125
1126 static void
1127 bond_unixctl_enable_slave(struct unixctl_conn *conn, const char *args,
1128                           void *aux OVS_UNUSED)
1129 {
1130     enable_slave(conn, args, true);
1131 }
1132
1133 static void
1134 bond_unixctl_disable_slave(struct unixctl_conn *conn, const char *args,
1135                            void *aux OVS_UNUSED)
1136 {
1137     enable_slave(conn, args, false);
1138 }
1139
1140 static void
1141 bond_unixctl_hash(struct unixctl_conn *conn, const char *args_,
1142                   void *aux OVS_UNUSED)
1143 {
1144     char *args = (char *) args_;
1145     uint8_t mac[ETH_ADDR_LEN];
1146     uint8_t hash;
1147     char *hash_cstr;
1148     unsigned int vlan;
1149     uint32_t basis;
1150     char *mac_s, *vlan_s, *basis_s;
1151     char *save_ptr = NULL;
1152
1153     mac_s  = strtok_r(args, " ", &save_ptr);
1154     vlan_s = strtok_r(NULL, " ", &save_ptr);
1155     basis_s = strtok_r(NULL, " ", &save_ptr);
1156
1157     if (vlan_s) {
1158         if (sscanf(vlan_s, "%u", &vlan) != 1) {
1159             unixctl_command_reply(conn, 501, "invalid vlan");
1160             return;
1161         }
1162     } else {
1163         vlan = OFP_VLAN_NONE;
1164     }
1165
1166     if (basis_s) {
1167         if (sscanf(basis_s, "%"PRIu32, &basis) != 1) {
1168             unixctl_command_reply(conn, 501, "invalid basis");
1169             return;
1170         }
1171     } else {
1172         basis = 0;
1173     }
1174
1175     if (sscanf(mac_s, ETH_ADDR_SCAN_FMT, ETH_ADDR_SCAN_ARGS(mac))
1176         == ETH_ADDR_SCAN_COUNT) {
1177         hash = bond_hash_src(mac, vlan, basis) & BOND_MASK;
1178
1179         hash_cstr = xasprintf("%u", hash);
1180         unixctl_command_reply(conn, 200, hash_cstr);
1181         free(hash_cstr);
1182     } else {
1183         unixctl_command_reply(conn, 501, "invalid mac");
1184     }
1185 }
1186
1187 void
1188 bond_init(void)
1189 {
1190     unixctl_command_register("bond/list", bond_unixctl_list, NULL);
1191     unixctl_command_register("bond/show", bond_unixctl_show, NULL);
1192     unixctl_command_register("bond/migrate", bond_unixctl_migrate, NULL);
1193     unixctl_command_register("bond/set-active-slave",
1194                              bond_unixctl_set_active_slave, NULL);
1195     unixctl_command_register("bond/enable-slave", bond_unixctl_enable_slave,
1196                              NULL);
1197     unixctl_command_register("bond/disable-slave", bond_unixctl_disable_slave,
1198                              NULL);
1199     unixctl_command_register("bond/hash", bond_unixctl_hash, NULL);
1200 }
1201 \f
1202 static void
1203 bond_entry_reset(struct bond *bond)
1204 {
1205     if (bond->balance != BM_AB) {
1206         size_t hash_len = (BOND_MASK + 1) * sizeof *bond->hash;
1207
1208         if (!bond->hash) {
1209             bond->hash = xmalloc(hash_len);
1210         }
1211         memset(bond->hash, 0, hash_len);
1212
1213         bond->next_rebalance = time_msec() + bond->rebalance_interval;
1214     } else {
1215         free(bond->hash);
1216         bond->hash = NULL;
1217     }
1218 }
1219
1220 static struct bond_slave *
1221 bond_slave_lookup(struct bond *bond, const void *slave_)
1222 {
1223     struct bond_slave *slave;
1224
1225     HMAP_FOR_EACH_IN_BUCKET (slave, hmap_node, hash_pointer(slave_, 0),
1226                              &bond->slaves) {
1227         if (slave->aux == slave_) {
1228             return slave;
1229         }
1230     }
1231
1232     return NULL;
1233 }
1234
1235 static void
1236 bond_enable_slave(struct bond_slave *slave, bool enable, struct tag_set *tags)
1237 {
1238     struct bond *bond = slave->bond;
1239     slave->delay_expires = LLONG_MAX;
1240     if (enable != slave->enabled) {
1241         slave->enabled = enable;
1242         if (!slave->enabled) {
1243             VLOG_WARN("interface %s: disabled", slave->name);
1244             if (tags) {
1245                 tag_set_add(tags, slave->tag);
1246             }
1247         } else {
1248             VLOG_WARN("interface %s: enabled", slave->name);
1249             slave->tag = tag_create_random();
1250         }
1251
1252         if (bond->balance == BM_STABLE) {
1253             bond->bond_revalidate = true;
1254         }
1255     }
1256 }
1257
1258 static void
1259 bond_link_status_update(struct bond_slave *slave, struct tag_set *tags)
1260 {
1261     struct bond *bond = slave->bond;
1262     bool up;
1263
1264     up = netdev_get_carrier(slave->netdev) && slave->may_enable;
1265     if ((up == slave->enabled) != (slave->delay_expires == LLONG_MAX)) {
1266         static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(5, 20);
1267         VLOG_INFO_RL(&rl, "interface %s: link state %s",
1268                      slave->name, up ? "up" : "down");
1269         if (up == slave->enabled) {
1270             slave->delay_expires = LLONG_MAX;
1271             VLOG_INFO_RL(&rl, "interface %s: will not be %s",
1272                          slave->name, up ? "disabled" : "enabled");
1273         } else {
1274             int delay = (bond->lacp_negotiated ? 0
1275                          : up ? bond->updelay : bond->downdelay);
1276             slave->delay_expires = time_msec() + delay;
1277             if (delay) {
1278                 VLOG_INFO_RL(&rl, "interface %s: will be %s if it stays %s "
1279                              "for %d ms",
1280                              slave->name,
1281                              up ? "enabled" : "disabled",
1282                              up ? "up" : "down",
1283                              delay);
1284             }
1285         }
1286     }
1287
1288     if (time_msec() >= slave->delay_expires) {
1289         bond_enable_slave(slave, up, tags);
1290     }
1291 }
1292
1293 static bool
1294 bond_is_tcp_hash(const struct bond *bond)
1295 {
1296     return (bond->balance == BM_TCP && bond->lacp_negotiated)
1297         || bond->balance == BM_STABLE;
1298 }
1299
1300 static unsigned int
1301 bond_hash_src(const uint8_t mac[ETH_ADDR_LEN], uint16_t vlan, uint32_t basis)
1302 {
1303     return hash_3words(hash_bytes(mac, ETH_ADDR_LEN, 0), vlan, basis);
1304 }
1305
1306 static unsigned int
1307 bond_hash_tcp(const struct flow *flow, uint16_t vlan, uint32_t basis)
1308 {
1309     struct flow hash_flow = *flow;
1310     hash_flow.vlan_tci = htons(vlan);
1311
1312     /* The symmetric quality of this hash function is not required, but
1313      * flow_hash_symmetric_l4 already exists, and is sufficient for our
1314      * purposes, so we use it out of convenience. */
1315     return flow_hash_symmetric_l4(&hash_flow, basis);
1316 }
1317
1318 static unsigned int
1319 bond_hash(const struct bond *bond, const struct flow *flow, uint16_t vlan)
1320 {
1321     assert(bond->balance != BM_AB);
1322
1323     return (bond_is_tcp_hash(bond)
1324             ? bond_hash_tcp(flow, vlan, bond->basis)
1325             : bond_hash_src(flow->dl_src, vlan, bond->basis));
1326 }
1327
1328 static struct bond_entry *
1329 lookup_bond_entry(const struct bond *bond, const struct flow *flow,
1330                   uint16_t vlan)
1331 {
1332     return &bond->hash[bond_hash(bond, flow, vlan) & BOND_MASK];
1333 }
1334
1335 /* This function uses Highest Random Weight hashing to choose an output slave.
1336  * This approach only reassigns a minimal number of flows when slaves are
1337  * enabled or disabled.  Unfortunately, it has O(n) performance against the
1338  * number of slaves.  There exist algorithms which are O(1), but have slightly
1339  * more complex implementations and require the use of memory.  This may need
1340  * to be reimplemented if it becomes a performance bottleneck. */
1341 static struct bond_slave *
1342 choose_stb_slave(const struct bond *bond, const struct flow *flow,
1343                  uint16_t vlan)
1344 {
1345     struct bond_slave *best, *slave;
1346     uint32_t best_hash, flow_hash;
1347
1348     best = NULL;
1349     best_hash = 0;
1350     flow_hash = bond_hash(bond, flow, vlan);
1351     HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
1352         if (slave->enabled) {
1353             uint32_t hash;
1354
1355             hash = hash_2words(flow_hash, slave->stb_id);
1356             if (!best || hash > best_hash) {
1357                 best = slave;
1358                 best_hash = hash;
1359             }
1360         }
1361     }
1362
1363     return best;
1364 }
1365
1366 static struct bond_slave *
1367 choose_output_slave(const struct bond *bond, const struct flow *flow,
1368                     uint16_t vlan)
1369 {
1370     struct bond_entry *e;
1371
1372     switch (bond->balance) {
1373     case BM_AB:
1374         return bond->active_slave;
1375
1376     case BM_STABLE:
1377         return choose_stb_slave(bond, flow, vlan);
1378     case BM_SLB:
1379     case BM_TCP:
1380         e = lookup_bond_entry(bond, flow, vlan);
1381         if (!e->slave || !e->slave->enabled) {
1382             e->slave = CONTAINER_OF(hmap_random_node(&bond->slaves),
1383                                     struct bond_slave, hmap_node);
1384             if (!e->slave->enabled) {
1385                 e->slave = bond->active_slave;
1386             }
1387             e->tag = tag_create_random();
1388         }
1389         return e->slave;
1390
1391     default:
1392         NOT_REACHED();
1393     }
1394 }
1395
1396 static struct bond_slave *
1397 bond_choose_slave(const struct bond *bond)
1398 {
1399     struct bond_slave *slave, *best;
1400
1401     /* Find an enabled slave. */
1402     HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
1403         if (slave->enabled) {
1404             return slave;
1405         }
1406     }
1407
1408     /* All interfaces are disabled.  Find an interface that will be enabled
1409      * after its updelay expires.  */
1410     best = NULL;
1411     HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
1412         if (slave->delay_expires != LLONG_MAX
1413             && slave->may_enable
1414             && (!best || slave->delay_expires < best->delay_expires)) {
1415             best = slave;
1416         }
1417     }
1418     return best;
1419 }
1420
1421 static void
1422 bond_choose_active_slave(struct bond *bond, struct tag_set *tags)
1423 {
1424     static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(5, 20);
1425     struct bond_slave *old_active_slave = bond->active_slave;
1426
1427     bond->active_slave = bond_choose_slave(bond);
1428     if (bond->active_slave) {
1429         if (bond->active_slave->enabled) {
1430             VLOG_INFO_RL(&rl, "bond %s: active interface is now %s",
1431                          bond->name, bond->active_slave->name);
1432         } else {
1433             VLOG_INFO_RL(&rl, "bond %s: active interface is now %s, skipping "
1434                          "remaining %lld ms updelay (since no interface was "
1435                          "enabled)", bond->name, bond->active_slave->name,
1436                          bond->active_slave->delay_expires - time_msec());
1437             bond_enable_slave(bond->active_slave, true, tags);
1438         }
1439
1440         if (!old_active_slave) {
1441             tag_set_add(tags, bond->no_slaves_tag);
1442         }
1443
1444         bond->send_learning_packets = true;
1445     } else if (old_active_slave) {
1446         VLOG_WARN_RL(&rl, "bond %s: all interfaces disabled", bond->name);
1447     }
1448 }
1449
1450 /* Returns the tag for 'bond''s active slave, or 'bond''s no_slaves_tag if
1451  * there is no active slave. */
1452 static tag_type
1453 bond_get_active_slave_tag(const struct bond *bond)
1454 {
1455     return (bond->active_slave
1456             ? bond->active_slave->tag
1457             : bond->no_slaves_tag);
1458 }
1459
1460 /* Attempts to make the sum of the bond slaves' statistics appear on the fake
1461  * bond interface. */
1462 static void
1463 bond_update_fake_slave_stats(struct bond *bond)
1464 {
1465     struct netdev_stats bond_stats;
1466     struct bond_slave *slave;
1467     struct netdev *bond_dev;
1468
1469     memset(&bond_stats, 0, sizeof bond_stats);
1470
1471     HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
1472         struct netdev_stats slave_stats;
1473
1474         if (!netdev_get_stats(slave->netdev, &slave_stats)) {
1475             /* XXX: We swap the stats here because they are swapped back when
1476              * reported by the internal device.  The reason for this is
1477              * internal devices normally represent packets going into the
1478              * system but when used as fake bond device they represent packets
1479              * leaving the system.  We really should do this in the internal
1480              * device itself because changing it here reverses the counts from
1481              * the perspective of the switch.  However, the internal device
1482              * doesn't know what type of device it represents so we have to do
1483              * it here for now. */
1484             bond_stats.tx_packets += slave_stats.rx_packets;
1485             bond_stats.tx_bytes += slave_stats.rx_bytes;
1486             bond_stats.rx_packets += slave_stats.tx_packets;
1487             bond_stats.rx_bytes += slave_stats.tx_bytes;
1488         }
1489     }
1490
1491     if (!netdev_open_default(bond->name, &bond_dev)) {
1492         netdev_set_stats(bond_dev, &bond_stats);
1493         netdev_close(bond_dev);
1494     }
1495 }