Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pimd: Candidate RP/BSR preparation #16259

Merged

Conversation

eqvinox
Copy link
Contributor

@eqvinox eqvinox commented Jun 20, 2024

Preparations for C-RP/C-BSR:

  • make clang-format not wreck the YANG module lists
  • print a debug if we're RP ourselves (helpful when debugging C-RP)
  • allow sending unicast PIM packets that are routed normally by the kernel (C-RP advertisements)
  • pre-plumb in the NHT infrastructure for BSRs to track C-RPs

#15969 rebase & reopen (the old PR has CI stuck in some weird state)

eqvinox added 5 commits June 20, 2024 13:10
The YANG module list is really better as 1-item-per-line.

Signed-off-by: David Lamparter <[email protected]>
This proved helpful in debugging the Candidate-RP code.

Signed-off-by: David Lamparter <[email protected]>
The Candidate-RP code is about to use these.

Signed-off-by: David Lamparter <[email protected]>
The upcoming Candidate-RP code needs to send PIM packets that go through
normal unicast routing, without forcing a specific output interface.
Allow passing in NULL ifp to do that.

Signed-off-by: David Lamparter <[email protected]>
For BSMs, we should track which of the RP candidates in the BSM message
are actually available, before trying to use them (which also puts them
in NHT for that).  This applies for both BSRs as well as BSM receivers.

Signed-off-by: David Lamparter <[email protected]>
pimd uses "v4/v6 dispatch" typedefs.  Don't complain about those in
checkpatch.

Signed-off-by: David Lamparter <[email protected]>
@Jafaral Jafaral merged commit 56afdee into FRRouting:master Jun 24, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants