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

ospfd: Unicast direct acknowledgements on P2MP interfaces #17866

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aceelindem
Copy link
Collaborator

The commit c706f0e changed OSPF to send all OSPF Ack packets as multicast on point-to-multipoint (P2MP) interfaces. Experience has proved that sending direct Acks as multicast doesn't offer any convergence benefits and will be more inefficient in situations where the underlying network topology relies on replication for multicast. Additionally, sending all these acknowledgements as multicast complicates the debugging of packet captures.

Sending these direct acks as unicast also elmiinated the case where the same LSA could be sent in a single packet.

@frrbot frrbot bot added the ospf label Jan 16, 2025
@aceelindem
Copy link
Collaborator Author

CI:rerun

The commit c706f0e changed OSPF to
send all OSPF Ack packets as multicast on point-to-multipoint (P2MP)
interfaces. Experience has proved that sending direct Acks as
multicast doesn't offer any convergence benefits and will be more
inefficient in situations where the underlying network topology
relies on replication for multicast. Additionally, sending all these
acknowledgements as multicast complicates the debugging of packet
captures.

Sending these direct acks as unicast also elmiinated the case where
the same LSA could be sent in a single packet.

Signed-off-by: Acee Lindem <[email protected]>
@aceelindem aceelindem force-pushed the aceelindem/ospf-p2mp-unicast-direct-ack branch from 31ff0d9 to e0b6eb9 Compare January 16, 2025 20:10
@aceelindem
Copy link
Collaborator Author

CI:rerun

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant