Skip to content

[202412] [FRR] Add support for 514 BGP sessions #1034

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

Merged
merged 3 commits into from
Apr 23, 2025

Conversation

vivekrnv
Copy link
Contributor

Add support for 512 BGP sessions

Backport sonic-net/sonic-buildimage#22390 to 202412

Patch Upstream Commit
0044-zebra-Prevent-starvation-in-dplane_thread_loop.patch 6faad863
0083-bgpd-fix-vty-output-of-evpn-route-target-AS4.patch 20b3ab48
0084-zebra-Ensure-dplane-does-not-send-work-back-to-maste.patch c4115522
0085-zebra-Limit-mutex-for-obuf-to-when-we-access-obuf.patch c58da10d
0086-bgpd-backpressure-Fix-to-pop-items-off-zebra_announc.patch 898852f
0087-zebra-fnc-obuf-could-be-accessed-without-a-lock.patch e7a1fbbcf
0088-zebra-Add-show-fpm-status-json-command.patch 0a9e8ef49
0089-doc-Add-show-fpm-status-json-command-to-documentatio.patch a0c4fe2ca
0090-zebra-avoid-a-race-during-FPM-dplane-plugin-shutdown.patch 277784f
0091-zebra-add-nexthop-counter-to-show-zebra-dplane-comma.patch e36e570c
0092-zebra-Installation-success-should-not-set-NHG-as-val.patch 910b2c5a
0093-zebra-When-reinstalling-a-NHG-set-REINSTALL-flag.patch b2ade8e
0094-zebra-Conslidate-zebra_nhg_set_valid-invalid-functio.patch 6ee9cc68
0095-zebra-Properly-note-that-a-nhg-s-nexthop-has-gone-do.patch 3b9428a7
0096-zebra-be-consistent-about-v6-nexthops-for-v4-routes.patch c93bc371
0097-lib-zebra-Modify-nexthop_cmp-to-allow-you-to-use-wei.patch 75268f01
0098-zebra-Create-Singleton-nhg-s-without-weights.patch ae4a1315
0099-zebra-Allow-blackhole-singleton-nexthops-to-be-v6.patch ae397ad9
0100-zebra-Allow-for-initial-deny-of-installation-of-nhe-.patch 4bf2c11f
0101-zebra-Properly-note-that-a-nhg-s-nexthop-has-gone-do.patch 892e8179
0102-zebra-Reinstall-nexthop-when-interface-comes-back-up.patch 279f427c
0103-zebra-Attempt-to-reuse-NHG-after-interface-up-and-ro.patch 98d56711
0104-zebra-Expose-_route_entry_dump_nh-so-it-can-be-used.patch 4fb44993
0105-zebra-Fix-resetting-valid-flags-for-NHG-dependents.patch 6e95686b
0106-zebra-Fix-leaked-nhe.patch a84d2bc0
0107-zebra-Uninstall-NHG-in-some-situations.patch d1cba73a
0108-zebra-Optimize-invoking-nhg-compare-func.patch 0faa70a5
0109-zebra-Nexthops-need-to-be-ACTIVE-in-some-cases.patch df56b92b
0110-zebra-On-Nexthop-install-failure-don-t-set-Installat.patch 4b2b1a9a
0111-zebra-Bring-up-514-BGP-neighbor-sessions.patch ea399e15
0112-lib-Add-support-for-stream-buffer-to-expand.patch 65b3ee4e
0113-zebra-zebra-crash-for-zapi-stream.patch c122afdb
0114-bgpd-Replace-per-peer-connection-error-with-per-bgp.patch 10c127bc
0115-bgpd-remove-apis-from-bgp_route.h.patch 1d5a8a20
0116-bgpd-batch-peer-connection-error-clearing.patch 4baa9f2d
0117-zebra-move-peer-conn-error-list-to-connection-struct.patch 411abd6b
0118-bgpd-Allow-batch-clear-to-do-partial-work-and-contin.patch b68be906
0119-zebra-send-v6-fast-RA-at-faster-interval.patch c8f12a4f
0120-lib-add-option-to-start-stop-wheel-timer.patch ca0adcdd
0121-bgpd-Paths-received-from-shutdown-peer-not-deleted.patch 2cbfc7ec

Verification:
Verified the changes on topology with scaled BGP tests

@vivekrnv vivekrnv requested a review from lguohan as a code owner April 21, 2025 19:35
@@ -24,6 +24,7 @@
0041-bgpd-backpressure-Fix-to-avoid-CPU-hog.patch
0042-zebra-Use-built-in-data-structure-counter.patch
0043-zebra-Use-the-ctx-queue-counters.patch
0044-zebra-Prevent-starvation-in-dplane_thread_loop.patch
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not too sure if this id getting duplicated will cause any potential issue, but since it is validated, I will still getting it merged.

@vivekrnv as FYI, just in case.

@r12f r12f merged commit 98f1d11 into Azure:202412 Apr 23, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants