<div dir="ltr"><div>Good morning,</div><div><br></div><div>I just wanted to make sure you received the v2 of the patch correctly.</div><div>I just realized that maybe something went wrong when I sent the email since it included changes on two different repositories and I sent it as a response to the same thread as the original PR.</div><div>In that case, I will promptly resend the updated version with any changes needed.</div><div>I also understand that you are very busy with the new releases, so do not worry if you need more time for the review.</div><div><br></div><div>Best Regards,</div><div>Matteo</div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, 11 Dec 2025 at 10:16, Matteo Perin <<a href="mailto:matteo.perin@canonical.com">matteo.perin@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The v2 of the patch changes the following:<br>
<br>
- The peer route type now contains information on the interface on which the peer was discovered.<br>
<br>
- The change above allowed to remove the need to configure the "from" option on the BGP peers channel, so it was removed.<br>
<br>
- pxlen of net_add_peer has been set to 0, since the field is unused.<br>
<br>
- RAdv neighbor discovery option has been renamed to router discovery.<br>
<br>
- The peers channel in BGP has been made a regular channel (instead of a bgp_channel).<br>
<br>
- The new BGP channel persist option has been made a protocol-wide option and it now also dictates the behavior of spawed dynamic BGP sessions.<br>
<br>
- Documentation on changes on both RAdv and BGP has been added.<br>
<br>
- Added testcase for the bird-tools testing suite.<br>
<br>
Thank you Ondrej for the review, your insights (especially the advice on changing the channel type) allowed me to refactor how the implementation works a bit and make it tidier (I think).<br>
My response was delayed by a few day so I could also provide some added documentation work and finish implementing automated testing for the bird-tools project.<br>
<br>
Let me know if you have any other suggestion on changes needed.<br>
<br>
Best Regards,<br>
Matteo<br>
<br>
Matteo Perin (4):<br>
  Nest: Add net_peer route type to track discovered peers<br>
  RAdv: Add neighbor discovery based on incoming RAs to RAdv proto<br>
  BGP: Move postponed socket reloop logic from start to start_locked to<br>
    achieve locking requirements<br>
  BGP: Add peers channel and sessions spawn on export to achieve<br>
    automatic peering<br>
<br>
 doc/bird.sgml        |  65 +++++++++++++<br>
 lib/net.c            |  16 ++++<br>
 lib/net.h            |  32 ++++++-<br>
 nest/config.Y        |   3 +-<br>
 nest/proto.c         |   2 +-<br>
 nest/protocol.h      |   3 +<br>
 proto/bgp/attrs.c    |  51 ++++++++++<br>
 proto/bgp/bgp.c      | 216 +++++++++++++++++++++++++++++++++++++++----<br>
 proto/bgp/bgp.h      |  27 +++++-<br>
 proto/bgp/config.Y   |  12 ++-<br>
 proto/bgp/packets.c  |  11 +++<br>
 proto/radv/config.Y  |   3 +-<br>
 proto/radv/packets.c |  41 +++++++-<br>
 proto/radv/radv.c    | 165 ++++++++++++++++++++++++++++++++-<br>
 proto/radv/radv.h    |   4 +<br>
 15 files changed, 622 insertions(+), 29 deletions(-)<br>
<br>
-- <br>
2.43.0<br>
<br>
</blockquote></div>