X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=lib%2Fmac-learning.c;h=c8582f3123c98d577596d9dcc47a4e54635e648b;hb=142181fcd734a2afff9fe13e54fe51c7a2c824d2;hp=a9d414d2fa449081a5691af25f7ca65bebdf7360;hpb=a0bc29a541fc7dc6e20137d5558e2094d614e6ab;p=openvswitch diff --git a/lib/mac-learning.c b/lib/mac-learning.c index a9d414d2..c8582f31 100644 --- a/lib/mac-learning.c +++ b/lib/mac-learning.c @@ -29,10 +29,10 @@ #include "tag.h" #include "timeval.h" #include "util.h" - -#define THIS_MODULE VLM_mac_learning #include "vlog.h" +VLOG_DEFINE_THIS_MODULE(mac_learning) + /* Returns the number of seconds since 'e' was last learned. */ int mac_entry_age(const struct mac_entry *e) @@ -175,11 +175,14 @@ is_learning_vlan(const struct mac_learning *ml, uint16_t vlan) * that now need revalidation. * * The 'vlan' parameter is used to maintain separate per-VLAN learning tables. - * Specify 0 if this behavior is undesirable. */ + * Specify 0 if this behavior is undesirable. + * + * 'lock_type' specifies whether the entry should be locked or existing locks + * are check. */ tag_type mac_learning_learn(struct mac_learning *ml, const uint8_t src_mac[ETH_ADDR_LEN], uint16_t vlan, - uint16_t src_port) + uint16_t src_port, enum grat_arp_lock_type lock_type) { struct mac_entry *e; struct list *bucket; @@ -209,32 +212,42 @@ mac_learning_learn(struct mac_learning *ml, e->port = -1; e->vlan = vlan; e->tag = make_unknown_mac_tag(ml, src_mac, vlan); + e->grat_arp_lock = TIME_MIN; } - /* Make the entry most-recently-used. */ - list_remove(&e->lru_node); - list_push_back(&ml->lrus, &e->lru_node); - e->expires = time_now() + MAC_ENTRY_IDLE_TIME; - - /* Did we learn something? */ - if (e->port != src_port) { - tag_type old_tag = e->tag; - e->port = src_port; - e->tag = tag_create_random(); - COVERAGE_INC(mac_learning_learned); - return old_tag; + if (lock_type != GRAT_ARP_LOCK_CHECK || time_now() >= e->grat_arp_lock) { + /* Make the entry most-recently-used. */ + list_remove(&e->lru_node); + list_push_back(&ml->lrus, &e->lru_node); + e->expires = time_now() + MAC_ENTRY_IDLE_TIME; + if (lock_type == GRAT_ARP_LOCK_SET) { + e->grat_arp_lock = time_now() + MAC_GRAT_ARP_LOCK_TIME; + } + + /* Did we learn something? */ + if (e->port != src_port) { + tag_type old_tag = e->tag; + e->port = src_port; + e->tag = tag_create_random(); + COVERAGE_INC(mac_learning_learned); + return old_tag; + } } + return 0; } /* Looks up MAC 'dst' for VLAN 'vlan' in 'ml'. Returns the port on which a - * frame destined for 'dst' should be sent, -1 if unknown. */ + * frame destined for 'dst' should be sent, -1 if unknown. 'is_grat_arp_locked' + * is an optional parameter that returns whether the entry is currently + * locked. */ int mac_learning_lookup(const struct mac_learning *ml, - const uint8_t dst[ETH_ADDR_LEN], uint16_t vlan) + const uint8_t dst[ETH_ADDR_LEN], uint16_t vlan, + bool *is_grat_arp_locked) { tag_type tag = 0; - return mac_learning_lookup_tag(ml, dst, vlan, &tag); + return mac_learning_lookup_tag(ml, dst, vlan, &tag, is_grat_arp_locked); } /* Looks up MAC 'dst' for VLAN 'vlan' in 'ml'. Returns the port on which a @@ -242,11 +255,14 @@ mac_learning_lookup(const struct mac_learning *ml, * * Adds to '*tag' (which the caller must have initialized) the tag that should * be attached to any flow created based on the return value, if any, to allow - * those flows to be revalidated when the MAC learning entry changes. */ + * those flows to be revalidated when the MAC learning entry changes. + * + * 'is_grat_arp_locked' is an optional parameter that returns whether the entry + * is currently locked.*/ int mac_learning_lookup_tag(const struct mac_learning *ml, const uint8_t dst[ETH_ADDR_LEN], uint16_t vlan, - tag_type *tag) + tag_type *tag, bool *is_grat_arp_locked) { if (eth_addr_is_multicast(dst) || !is_learning_vlan(ml, vlan)) { return -1; @@ -255,6 +271,11 @@ mac_learning_lookup_tag(const struct mac_learning *ml, dst, vlan); if (e) { *tag |= e->tag; + + if (is_grat_arp_locked) { + *is_grat_arp_locked = time_now() < e->grat_arp_lock; + } + return e->port; } else { *tag |= make_unknown_mac_tag(ml, dst, vlan); @@ -293,6 +314,6 @@ mac_learning_wait(struct mac_learning *ml) { if (!list_is_empty(&ml->lrus)) { struct mac_entry *e = mac_entry_from_lru_node(ml->lrus.next); - poll_timer_wait((e->expires - time_now()) * 1000); + poll_timer_wait_until(e->expires * 1000LL); } }