Skip to content

[vpp] Allow routes to override Class E drop#233

Open
rustiqly wants to merge 1 commit into
sonic-net:masterfrom
rustiqly:fix/vpp-class-e-route-override
Open

[vpp] Allow routes to override Class E drop#233
rustiqly wants to merge 1 commit into
sonic-net:masterfrom
rustiqly:fix/vpp-class-e-route-override

Conversation

@rustiqly
Copy link
Copy Markdown
Contributor

Why I did it

SONiC VPP currently inherits upstream VPP's hard drop for 240.0.0.0/4 via FIB_SOURCE_SPECIAL. That source cannot be overridden by normal route programming, so VPP drops Class E destinations even when SONiC has an explicit/default route covering them. This breaks parity with other ASIC platforms and reproduces in test_decap traffic that uses 240.0.0.0/4 destinations.

How I did it

  • Add a VPP patch that changes only the 240.0.0.0/4 default drop source from FIB_SOURCE_SPECIAL to FIB_SOURCE_DEFAULT_ROUTE.
  • Leave multicast 224.0.0.0/4 as FIB_SOURCE_SPECIAL so multicast drop behavior is unchanged.
  • Register the patch in vppbld/patches/series.

How I verified it

  • git diff --check
  • Verified the new patch applies cleanly to the pinned FD.io VPP source file from vppbld/vpp_version (src/vnet/fib/ip4_fib.c).

Fixes sonic-net/sonic-buildimage#27345

Signed-off-by: Rustiqly <rustiqly@users.noreply.github.com>
@mssonicbld
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@prgeor prgeor requested a review from lolyu May 20, 2026 15:59
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.

Bug: [vpp] Hard-coded 240.0.0.0/4 FIB drop in upstream VPP breaks SONiC routing parity (e.g. test_decap)

2 participants