bgpd/bgp_damp.c | 12 ++++++------ bgpd/bgp_fsm.c | 18 +++--------------- bgpd/bgp_packet.c | 2 +- bgpd/bgp_route.c | 26 +++++++++++++------------- bgpd/bgp_snmp.c | 8 ++++---- bgpd/bgpd.c | 22 +++++++++++++++------- bgpd/bgpd.h | 1 + isisd/isis_pdu.c | 2 +- lib/Makefile.am | 3 +++ lib/command.c | 10 +++++----- lib/memory.h | 8 +++++--- lib/prefix.c | 2 +- ospfd/ospf_lsdb.c | 1 - ospfd/ospf_packet.c | 7 ++++--- zebra/router-id.c | 16 ++++++---------- 15 files changed, 68 insertions(+), 70 deletions(-)
New commits: commit 3f3dc2bd5f8cbde24cb581ed0aed0e372bfd556c Author: Mathias Krause <[email protected]> Date: Wed Jan 20 17:39:13 2010 +0300 lib: fix PIE build flags Because the final executables are built as position independent executables (PIE) -- when configure has detected compiler supported for PIE -- the objects in the library archive must be built in that way, too. Otherwise the runtime linker has to do unneccesary relocation for each start of the program. Even worse, the programs won't even be able to start on a (hardened) kernel that doesn't allow those relocations to happen by preventing making the .text segment writable (PaX's MPROTECT feature comes to mind). The attached patch fixes this issue by adding the appropriate flags to the Makefile.am. commit 65957886bfd0dd9d95360d8b015781fc82cc09be Author: Stephen Hemminger <[email protected]> Date: Fri Jan 15 16:22:10 2010 +0300 bgp: use monotonic clock for time of day BGP uses time() to get system time of day; but that value fluctuates with time adjustments from NTP. This can cause premature flapping of peer sessions and other failures. Use the system monotonic clock supported by Quagga thread library to avoid issue. See: http://bugzilla.vyatta.com/show_bug.cgi?id=4467 * bgpd/bgp_fsm.c * bgp_uptime_reset(): dismiss function * bgpd/bgpd.c * bgp_clock(): new function * bgpd/bgp_damp.c * bgp_reuse_timer(): employ bgp_clock() instead of time(NULL) * bgp_damp_withdraw(): idem * bgp_damp_update(): idem * bgp_damp_scan(): idem * bgp_damp_info_vty(): idem * bgp_damp_reuse_time_vty(): idem * bgpd/bgp_fsm.c * bgp_routeadv_timer(): idem * bgp_stop(): idem * bgp_establish(): idem * bgpd/bgp_packet.c * bgp_update_receive(): idem * bgpd/bgp_route.c * bgp_update_rsclient(): idem * bgp_update_main(): idem * bgp_static_update_rsclient(): idem * bgp_static_update_main(): idem * bgp_static_update_vpnv4(): idem * bgp_aggregate_route(): idem * bgp_aggregate_add(): idem * bgp_redistribute_add(): idem * bgpd/bgp_snmp.c * bgpPeerTable(): idem * bgpTrapEstablished(): idem * bgpTrapBackwardTransition(): idem * bgpd/bgpd.c * peer_create(): idem * peer_uptime(): idem * bgp_master_init(): idem commit 2cd754de60210748e61069fed2c9d4086d24c21e Author: Paul Jakma <[email protected]> Date: Thu Jan 14 16:26:12 2010 +0300 ospfd: Fix debug messages that were masked by DISCARD_LSA * ospf_packet.c: (ospf_ls_upd) DISCARD_LSA continues, and so should be after debug messages, not before them. commit efbbcaf1aea686d118019514dcde92b13c92b0fc Author: Paul Jakma <[email protected]> Date: Thu Jan 14 16:19:40 2010 +0300 ospfd: remove unneeded memset from a very hot function * ospf_lsdb.c: (lsdb_prefix_set) memset is unneeded, as all fields are initialised explicitly, and this function can be in the top-3 of a profile when there are a lot of LSAs. commit 0917f7e7c5920b70295c44bd8aad537bc3ee8581 Author: David Ward <[email protected]> Date: Wed Jan 13 20:10:56 2010 +0300 lib: fix memory logging * lib/memory.h * mtype_zcalloc(): correct function prototype * XFREE(): make both definitions consistent in setting the pointer to NULL after freeing the memory These changes will only have an effect if MEMORY_LOG is defined (it is not by default). commit b6516829595ed4fc47af955de8bda9ef5be14ffd Author: Stephen Hemminger <[email protected]> Date: Mon Dec 21 18:50:43 2009 +0300 zebra: change router-id selection algo The router-id table looks like is supposed to be sorted in current quagga code, but the nodes are not added with the sorting function. The sorting function is host byte order dependent. The values need to converted before comparison. Fixing this causes Zebra to choose the largest IP address as router-id, rather than the last address. This probably will surprise some users. The other option would be to just remove the comparison function and keep the existing LIFO behavior. Lastly, simple subtraction works well for comparing. * zebra/router-id.c * router_id_add_address(): employ listnode_add_sort() * router_id_cmp(): employ ntohl(), then compare integers commit 2d362d1099f346202cb628665ff4ae563b2594f5 Author: Stephen Hemminger <[email protected]> Date: Mon Dec 21 12:54:58 2009 +0300 lib: make some structures constant * isisd/isis_pdu.c * maskbit: this pre-initialized array is not modified * lib/prefix.c * maskbit: idem * lib/command.c * facility_map: idem * itoa64: idem * default_motd: make local var static * facility_name(): update local var accordingly * facility_match(): idem http://suva.vyatta.com/git/?p=vyatta-quagga.git;a=commitdiff;h=3f3dc2bd5f8cbde24cb581ed0aed0e372bfd556c http://suva.vyatta.com/git/?p=vyatta-quagga.git;a=commitdiff;h=65957886bfd0dd9d95360d8b015781fc82cc09be http://suva.vyatta.com/git/?p=vyatta-quagga.git;a=commitdiff;h=2cd754de60210748e61069fed2c9d4086d24c21e http://suva.vyatta.com/git/?p=vyatta-quagga.git;a=commitdiff;h=efbbcaf1aea686d118019514dcde92b13c92b0fc http://suva.vyatta.com/git/?p=vyatta-quagga.git;a=commitdiff;h=0917f7e7c5920b70295c44bd8aad537bc3ee8581 http://suva.vyatta.com/git/?p=vyatta-quagga.git;a=commitdiff;h=b6516829595ed4fc47af955de8bda9ef5be14ffd http://suva.vyatta.com/git/?p=vyatta-quagga.git;a=commitdiff;h=2d362d1099f346202cb628665ff4ae563b2594f5 _______________________________________________ svn mailing list [email protected] http://mailman.vyatta.com/mailman/listinfo/svn
