diff options
| author | Paul Jakma <paul.jakma@sun.com> | 2006-09-14 03:02:02 +0000 |
|---|---|---|
| committer | Paul Jakma <paul.jakma@sun.com> | 2006-09-14 03:02:02 +0000 |
| commit | 9fde6624fc480995449d8243fe85602d89927eb6 (patch) | |
| tree | 6591e476329ec88bd9ba5fcc008c626d32b3fdb5 /bgpd/ChangeLog | |
| parent | ca058a30b1ea57f83871ab4cf1c9a91ea4064d52 (diff) | |
[bgpd] simplify peer refcounts, squash slow peer leak
2006-09-14 Paul Jakma <paul.jakma@sun.com>
* (general) fix the peer refcount issue exposed by previous, by
just removing refcounting of peer threads, which is mostly
senseless as they're references leading from struct peer,
which peer_free cancels anyway. No need to muck around..
* bgp_fsm.h: Just remove the refcounting from the various
TIMER/READ/WRITE/EVENT ON/OFF/ADD macros.
* bgp_fsm.c: (bgp_stop) use BGP_EVENT_FLUSH, no refcounts attached
to events anymore.
(bgp_event) remove peer_unlock, events not refcounted.
* bgpd.c: (peer_free) flush events before free.
Diffstat (limited to 'bgpd/ChangeLog')
| -rw-r--r-- | bgpd/ChangeLog | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/bgpd/ChangeLog b/bgpd/ChangeLog index 02aaf3ab..393b00d4 100644 --- a/bgpd/ChangeLog +++ b/bgpd/ChangeLog @@ -1,5 +1,18 @@ 2006-09-14 Paul Jakma <paul.jakma@sun.com> + * (general) fix the peer refcount issue exposed by previous, by + just removing refcounting of peer threads, which is mostly + senseless as they're references leading from struct peer, + which peer_free cancels anyway. No need to muck around.. + * bgp_fsm.h: Just remove the refcounting from the various + TIMER/READ/WRITE/EVENT ON/OFF/ADD macros. + * bgp_fsm.c: (bgp_stop) use BGP_EVENT_FLUSH, no refcounts attached + to events anymore. + (bgp_event) remove peer_unlock, events not refcounted. + * bgpd.c: (peer_free) flush events before free. + +2006-09-14 Paul Jakma <paul.jakma@sun.com> + * (general) Fix some niggly issues around 'shutdown' and clearing by adding a Clearing FSM wait-state and a hidden 'Deleted' FSM state, to allow deleted peers to 'cool off' and hit 0 |