On Wed, Dec 11, 2024 at 07:17:11AM +0100, Claudio Jeker wrote:
> When a peer is removed in the RDE the Adj-RIB-Out can be cleared in the
> background. So implement a reaper and put the dead peers on a zombielist.

I thought we already had too many reapers in the tree?

> Now in peer_down run the reaper once so that session with only a few
> prefixes are immediatly gone. Not sure if that is premature optimisation.
> 
> While there fixup peer_down prototype and instead call it from
> peer_shutdown(). The shutdown code is a bit funky and only runs when
> started with debug since it is stupid to clean up before exit.

I'm not sure I like the 0/1 return of prefix_adjout_reaper(), but bgpd
has a few different API conventions already, so probably not the moment
to get fussy about thiss and -1/0 wouldn't be much better anyway.

There's an extra empty line after peer_work_pending()

ok tb