diff options
author | Maria Matejka <mq@ucw.cz> | 2022-07-22 15:37:21 +0200 |
---|---|---|
committer | Maria Matejka <mq@ucw.cz> | 2022-07-22 15:37:21 +0200 |
commit | 4d48ede51dfff9a59572a6b7a21a1bbf159dec60 (patch) | |
tree | 3821019a0178ba7551f1c9d564bbabe8c5c28ed8 /nest/rt-table.c | |
parent | b5c8fce284a2bb529aee6ff201fe044f09a0e418 (diff) |
Revert "Export table: Delay freeing of old stored route."
This reverts commit cee0cd148c9b71bf47d007c850193b5fbf9486c1.
This change is not needed in version 2 and the surrounding code has
disappeared mostly in version 3.
Diffstat (limited to 'nest/rt-table.c')
-rw-r--r-- | nest/rt-table.c | 63 |
1 files changed, 27 insertions, 36 deletions
diff --git a/nest/rt-table.c b/nest/rt-table.c index 4500c888..abb29fe1 100644 --- a/nest/rt-table.c +++ b/nest/rt-table.c @@ -620,29 +620,6 @@ rte_cow_rta(rte *r, linpool *lp) return r; } -/** - * rte_free - delete a &rte - * @e: &rte to be deleted - * - * rte_free() deletes the given &rte from the routing table it's linked to. - */ -void -rte_free(rte *e) -{ - rt_unlock_source(e->src); - if (rta_is_cached(e->attrs)) - rta_free(e->attrs); - sl_free(e); -} - -static inline void -rte_free_quick(rte *e) -{ - rt_unlock_source(e->src); - rta_free(e->attrs); - sl_free(e); -} - static int /* Actually better or at least as good as */ rte_better(rte *new, rte *old) { @@ -799,14 +776,8 @@ do_rt_notify(struct channel *c, net *net, rte *new, rte *old, int refeed) } /* Apply export table */ - struct rte *old_exported = NULL; - if (c->out_table) - { - if (!rte_update_out(c, net->n.addr, new, old, &old_exported, refeed)) - return; - } - else if (c->out_filter == FILTER_ACCEPT) - old_exported = old; + if (c->out_table && !rte_update_out(c, net->n.addr, new, old, refeed)) + return; if (new) stats->exp_updates_accepted++; @@ -836,9 +807,6 @@ do_rt_notify(struct channel *c, net *net, rte *new, rte *old, int refeed) } p->rt_notify(p, c, net, new, old); - - if (c->out_table && old_exported) - rte_free_quick(old_exported); } static void @@ -1198,6 +1166,29 @@ rte_validate(rte *e) return 1; } +/** + * rte_free - delete a &rte + * @e: &rte to be deleted + * + * rte_free() deletes the given &rte from the routing table it's linked to. + */ +void +rte_free(rte *e) +{ + rt_unlock_source(e->src); + if (rta_is_cached(e->attrs)) + rta_free(e->attrs); + sl_free(e); +} + +static inline void +rte_free_quick(rte *e) +{ + rt_unlock_source(e->src); + rta_free(e->attrs); + sl_free(e); +} + static int rte_same(rte *x, rte *y) { @@ -3258,7 +3249,7 @@ again: */ int -rte_update_out(struct channel *c, const net_addr *n, rte *new, rte *old0, rte **old_exported, int refeed) +rte_update_out(struct channel *c, const net_addr *n, rte *new, rte *old0, int refeed) { struct rtable *tab = c->out_table; struct rte_src *src; @@ -3302,7 +3293,7 @@ rte_update_out(struct channel *c, const net_addr *n, rte *new, rte *old0, rte ** /* Remove the old rte */ *pos = old->next; - *old_exported = old; + rte_free_quick(old); tab->rt_count--; break; |