ospfd: LSDB_LOOP treat it as a loop.

Inform the .clang-format file about LSDB_LOOP and
put the proper indentation for this loop into the
code.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
This commit is contained in:
Donald Sharp 2018-01-17 13:28:17 -05:00
parent 7c0c85a2eb
commit 044506e7f8
9 changed files with 102 additions and 91 deletions

View File

@ -57,3 +57,5 @@ ForEachMacros:
- SUBGRP_FOREACH_ADJ_SAFE - SUBGRP_FOREACH_ADJ_SAFE
- AF_FOREACH - AF_FOREACH
- FOREACH_AFI_SAFI - FOREACH_AFI_SAFI
# ospfd
- LSDB_LOOP

View File

@ -327,8 +327,7 @@ static int ospf_abr_nssa_am_elected(struct ospf_area *area)
struct router_lsa *rlsa; struct router_lsa *rlsa;
struct in_addr *best = NULL; struct in_addr *best = NULL;
LSDB_LOOP(ROUTER_LSDB(area), rn, lsa) LSDB_LOOP (ROUTER_LSDB(area), rn, lsa) {
{
/* sanity checks */ /* sanity checks */
if (!lsa || (lsa->data->type != OSPF_ROUTER_LSA) if (!lsa || (lsa->data->type != OSPF_ROUTER_LSA)
|| IS_LSA_SELF(lsa)) || IS_LSA_SELF(lsa))

View File

@ -1313,22 +1313,28 @@ int ospf_apiserver_handle_sync_lsdb(struct ospf_apiserver *apiserv,
/* Check msg type. */ /* Check msg type. */
if (mask & Power2[OSPF_ROUTER_LSA]) if (mask & Power2[OSPF_ROUTER_LSA])
LSDB_LOOP(ROUTER_LSDB(area), rn, lsa) LSDB_LOOP(ROUTER_LSDB(area), rn, lsa)
apiserver_sync_callback(lsa, (void *)&param, seqnum); apiserver_sync_callback(
lsa, (void *)&param, seqnum);
if (mask & Power2[OSPF_NETWORK_LSA]) if (mask & Power2[OSPF_NETWORK_LSA])
LSDB_LOOP(NETWORK_LSDB(area), rn, lsa) LSDB_LOOP(NETWORK_LSDB(area), rn, lsa)
apiserver_sync_callback(lsa, (void *)&param, seqnum); apiserver_sync_callback(
lsa, (void *)&param, seqnum);
if (mask & Power2[OSPF_SUMMARY_LSA]) if (mask & Power2[OSPF_SUMMARY_LSA])
LSDB_LOOP(SUMMARY_LSDB(area), rn, lsa) LSDB_LOOP(SUMMARY_LSDB(area), rn, lsa)
apiserver_sync_callback(lsa, (void *)&param, seqnum); apiserver_sync_callback(
lsa, (void *)&param, seqnum);
if (mask & Power2[OSPF_ASBR_SUMMARY_LSA]) if (mask & Power2[OSPF_ASBR_SUMMARY_LSA])
LSDB_LOOP(ASBR_SUMMARY_LSDB(area), rn, lsa) LSDB_LOOP(ASBR_SUMMARY_LSDB(area), rn, lsa)
apiserver_sync_callback(lsa, (void *)&param, seqnum); apiserver_sync_callback(
lsa, (void *)&param, seqnum);
if (mask & Power2[OSPF_OPAQUE_LINK_LSA]) if (mask & Power2[OSPF_OPAQUE_LINK_LSA])
LSDB_LOOP(OPAQUE_LINK_LSDB(area), rn, lsa) LSDB_LOOP(OPAQUE_LINK_LSDB(area), rn, lsa)
apiserver_sync_callback(lsa, (void *)&param, seqnum); apiserver_sync_callback(
lsa, (void *)&param, seqnum);
if (mask & Power2[OSPF_OPAQUE_AREA_LSA]) if (mask & Power2[OSPF_OPAQUE_AREA_LSA])
LSDB_LOOP(OPAQUE_AREA_LSDB(area), rn, lsa) LSDB_LOOP(OPAQUE_AREA_LSDB(area), rn, lsa)
apiserver_sync_callback(lsa, (void *)&param, seqnum); apiserver_sync_callback(
lsa, (void *)&param, seqnum);
} }
} }
@ -1336,14 +1342,16 @@ int ospf_apiserver_handle_sync_lsdb(struct ospf_apiserver *apiserv,
if (ospf->lsdb) { if (ospf->lsdb) {
if (mask & Power2[OSPF_AS_EXTERNAL_LSA]) if (mask & Power2[OSPF_AS_EXTERNAL_LSA])
LSDB_LOOP(EXTERNAL_LSDB(ospf), rn, lsa) LSDB_LOOP(EXTERNAL_LSDB(ospf), rn, lsa)
apiserver_sync_callback(lsa, (void *)&param, seqnum); apiserver_sync_callback(lsa, (void *)&param,
seqnum);
} }
/* For AS-external opaque LSAs */ /* For AS-external opaque LSAs */
if (ospf->lsdb) { if (ospf->lsdb) {
if (mask & Power2[OSPF_OPAQUE_AS_LSA]) if (mask & Power2[OSPF_OPAQUE_AS_LSA])
LSDB_LOOP(OPAQUE_AS_LSDB(ospf), rn, lsa) LSDB_LOOP(OPAQUE_AS_LSDB(ospf), rn, lsa)
apiserver_sync_callback(lsa, (void *)&param, seqnum); apiserver_sync_callback(lsa, (void *)&param,
seqnum);
} }
/* Send a reply back to client with return code */ /* Send a reply back to client with return code */
@ -1945,16 +1953,19 @@ void ospf_apiserver_flush_opaque_lsa(struct ospf_apiserver *apiserv,
case OSPF_OPAQUE_LINK_LSA: case OSPF_OPAQUE_LINK_LSA:
for (ALL_LIST_ELEMENTS(ospf->areas, node, nnode, area)) for (ALL_LIST_ELEMENTS(ospf->areas, node, nnode, area))
LSDB_LOOP(OPAQUE_LINK_LSDB(area), rn, lsa) LSDB_LOOP(OPAQUE_LINK_LSDB(area), rn, lsa)
apiserver_flush_opaque_type_callback(lsa, (void *)&param, 0); apiserver_flush_opaque_type_callback(
lsa, (void *)&param, 0);
break; break;
case OSPF_OPAQUE_AREA_LSA: case OSPF_OPAQUE_AREA_LSA:
for (ALL_LIST_ELEMENTS(ospf->areas, node, nnode, area)) for (ALL_LIST_ELEMENTS(ospf->areas, node, nnode, area))
LSDB_LOOP(OPAQUE_AREA_LSDB(area), rn, lsa) LSDB_LOOP(OPAQUE_AREA_LSDB(area), rn, lsa)
apiserver_flush_opaque_type_callback(lsa, (void *)&param, 0); apiserver_flush_opaque_type_callback(
lsa, (void *)&param, 0);
break; break;
case OSPF_OPAQUE_AS_LSA: case OSPF_OPAQUE_AS_LSA:
LSDB_LOOP(OPAQUE_LINK_LSDB(ospf), rn, lsa) LSDB_LOOP(OPAQUE_LINK_LSDB(ospf), rn, lsa)
apiserver_flush_opaque_type_callback(lsa, (void *)&param, 0); apiserver_flush_opaque_type_callback(lsa,
(void *)&param, 0);
break; break;
default: default:
break; break;

View File

@ -661,7 +661,8 @@ static int ospf_ase_calculate_timer(struct thread *t)
if (area->external_routing == OSPF_AREA_NSSA) if (area->external_routing == OSPF_AREA_NSSA)
LSDB_LOOP(NSSA_LSDB(area), rn, lsa) LSDB_LOOP(NSSA_LSDB(area), rn, lsa)
ospf_ase_calculate_route(ospf, lsa); ospf_ase_calculate_route(ospf,
lsa);
} }
/* kevinm: And add the NSSA routes in ospf_top */ /* kevinm: And add the NSSA routes in ospf_top */
LSDB_LOOP(NSSA_LSDB(ospf), rn, lsa) LSDB_LOOP(NSSA_LSDB(ospf), rn, lsa)

View File

@ -1872,8 +1872,7 @@ struct ospf_lsa *ospf_translated_nssa_refresh(struct ospf *ospf,
if (area->external_routing != OSPF_AREA_NSSA && !type7) if (area->external_routing != OSPF_AREA_NSSA && !type7)
continue; continue;
LSDB_LOOP(NSSA_LSDB(area), rn, lsa) LSDB_LOOP (NSSA_LSDB(area), rn, lsa) {
{
if (lsa->data->id.s_addr if (lsa->data->id.s_addr
== type5->data->id.s_addr) { == type5->data->id.s_addr) {
type7 = lsa; type7 = lsa;

View File

@ -183,8 +183,7 @@ void ospf_router_id_update(struct ospf *ospf)
struct route_node *rn; struct route_node *rn;
struct ospf_lsa *lsa; struct ospf_lsa *lsa;
LSDB_LOOP(EXTERNAL_LSDB(ospf), rn, lsa) LSDB_LOOP (EXTERNAL_LSDB(ospf), rn, lsa) {
{
/* AdvRouter and Router ID is the same. */ /* AdvRouter and Router ID is the same. */
if (IPV4_ADDR_SAME(&lsa->data->adv_router, if (IPV4_ADDR_SAME(&lsa->data->adv_router,
&ospf->router_id)) { &ospf->router_id)) {