[quagga-dev 10143] [RFC] bgpd: mark route nodes scheduled into work queue

Stephen Hemminger shemminger at vyatta.com
Fri Jan 4 02:43:21 GMT 2013


The flag bit BGP_NODE_PROCESS_SCHEDULED is checked but never set.
This causes route node to be scheduled multiple times under load.
Shouldn't it be set here?

---
 bgpd/bgp_route.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/bgpd/bgp_route.c b/bgpd/bgp_route.c
index b7efe74..484cbb5 100644
--- a/bgpd/bgp_route.c
+++ b/bgpd/bgp_route.c
@@ -1656,6 +1656,7 @@ bgp_process (struct bgp *bgp, struct bgp_node *rn, afi_t afi, safi_t safi)
         break;
     }
   
+  SET_FLAG (rn->flags, BGP_NODE_PROCESS_SCHEDULED);
   return;
 }
 
-- 
1.7.10.4





More information about the Quagga-dev mailing list