diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2018-04-09 21:41:37 +0200 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2018-04-18 00:43:47 +0200 |
| commit | d02e9432b5182e573a5889a8c83139f9202766fd (patch) | |
| tree | 371192bea282c668c69b2fa88773ecb7cf5d4fb3 /pbrd/pbr_map.c | |
| parent | pbrd: don't collapse pbr debugs (diff) | |
| download | frr-d02e9432b5182e573a5889a8c83139f9202766fd.tar.xz frr-d02e9432b5182e573a5889a8c83139f9202766fd.zip | |
pbrd: Allow pbr-policy add/removal to install/remove rules
The pbrm->installed variable was being used only in a couple
of places and it has no real bearing on whether or not
we should install a rule or not. Remove this value.
Ticket: CM-20429
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'pbrd/pbr_map.c')
| -rw-r--r-- | pbrd/pbr_map.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/pbrd/pbr_map.c b/pbrd/pbr_map.c index db17cdd8b..21b23ae21 100644 --- a/pbrd/pbr_map.c +++ b/pbrd/pbr_map.c @@ -154,7 +154,7 @@ void pbr_map_add_interface(struct pbr_map *pbrm, struct interface *ifp_add) bf_assign_index(pbrm->ifi_bitfield, pmi->install_bit); pbr_map_check_valid(pbrm->name); - if (pbrm->valid && !pbrm->installed) + if (pbrm->valid) pbr_map_install(pbrm); } @@ -311,8 +311,6 @@ struct pbr_map_sequence *pbrms_get(const char *name, uint32_t seqno) QOBJ_REG(pbrms, pbr_map_sequence); listnode_add_sort(pbrm->seqnumbers, pbrms); - - pbrm->installed = false; } return pbrms; @@ -566,8 +564,6 @@ void pbr_map_install(struct pbr_map *pbrm) for (ALL_LIST_ELEMENTS_RO(pbrm->seqnumbers, node, pbrms)) for (ALL_LIST_ELEMENTS_RO(pbrm->incoming, inode, pmi)) pbr_send_pbr_map(pbrms, pmi, true); - - pbrm->installed = true; } void pbr_map_init(void) |