From e35ef181a41384446aca614522a7cbb10606dd5b Mon Sep 17 00:00:00 2001 From: Martin Mares Date: Tue, 2 Mar 1999 18:36:09 +0000 Subject: o The if_change_too_big_p change was too high-spirited. Fixed. o Introduced if_find_by_index() o Recognizing two types of interface updates: full update (starting with if_start_update(), ending with if_end_update(), guaranteed to see all existing interfaces) and a partial update (only if_update(), usually due to asynchronous interface notifications). --- nest/iface.c | 37 ++++++++++++++++++++++++++++--------- 1 file changed, 28 insertions(+), 9 deletions(-) (limited to 'nest/iface.c') diff --git a/nest/iface.c b/nest/iface.c index 4d1b6856..9a02a3f6 100644 --- a/nest/iface.c +++ b/nest/iface.c @@ -231,8 +231,6 @@ if_dump_all(void) static inline int if_change_too_big_p(struct iface *i, struct iface *j) { - if ((i->flags ^ j->flags) & IF_UP) /* Going up/down is always OK */ - return 0; if (!ipa_equal(i->ip, j->ip) || /* Address change isn't */ !ipa_equal(i->prefix, j->prefix) || i->pxlen != j->pxlen || @@ -310,16 +308,19 @@ if_update(struct iface *new) if (if_change_too_big_p(i, new)) /* Changed a lot, convert it to down/up */ { DBG("Interface %s changed too much -- forcing down/up transition\n", i->name); - i->flags &= ~IF_UP; - if_notify_change(IF_CHANGE_DOWN | IF_CHANGE_FLAGS, i, NULL); + if (i->flags & IF_UP) + { + i->flags &= ~IF_UP; + if_notify_change(IF_CHANGE_DOWN | IF_CHANGE_FLAGS, i, NULL); + } rem_node(&i->n); goto newif; } c = if_changed(i, new); - if_copy(i, new); /* Even if c==0 as we might need to update i->index et al. */ - i->flags |= IF_UPDATED; if (c) if_notify_change(c, i, new); + if_copy(i, new); /* Even if c==0 as we might need to update i->index et al. */ + i->flags |= IF_UPDATED; return; } @@ -332,6 +333,15 @@ newif: | IF_CHANGE_FLAGS | IF_CHANGE_MTU, NULL, i); } +void +if_start_update(void) +{ + struct iface *i; + + WALK_LIST(i, iface_list) + i->flags &= ~IF_UPDATED; +} + void if_end_update(void) { @@ -341,9 +351,7 @@ if_end_update(void) auto_router_id(); WALK_LIST(i, iface_list) - if (i->flags & IF_UPDATED) - i->flags &= ~IF_UPDATED; - else + if (!(i->flags & IF_UPDATED)) { memcpy(&j, i, sizeof(struct iface)); i->flags = (i->flags & ~(IF_LINK_UP | IF_UP)) | IF_ADMIN_DOWN; @@ -364,6 +372,17 @@ if_feed_baby(struct proto *p) p->if_notify(p, IF_CHANGE_CREATE | ((i->flags & IF_UP) ? IF_CHANGE_UP : 0), i, NULL); } +struct iface * +if_find_by_index(unsigned idx) +{ + struct iface *i; + + WALK_LIST(i, iface_list) + if (i->index == idx) + return i; + return NULL; +} + static void auto_router_id(void) /* FIXME: What if we run IPv6??? */ { -- cgit v1.2.3