Skip to content

[doc][vpp] HLD for Router Interface Loopback Packet Action on the VPP dataplane - #2462

Open
aaronber0614 wants to merge 1 commit into
sonic-net:masterfrom
aaronber0614:vpp-iface-loopback-action-hld
Open

[doc][vpp] HLD for Router Interface Loopback Packet Action on the VPP dataplane#2462
aaronber0614 wants to merge 1 commit into
sonic-net:masterfrom
aaronber0614:vpp-iface-loopback-action-hld

Conversation

@aaronber0614

@aaronber0614 aaronber0614 commented Jul 13, 2026

Copy link
Copy Markdown

Why I did it

Document the design for SAI_ROUTER_INTERFACE_ATTR_LOOPBACK_PACKET_ACTION (drop or forward a routed "hairpin" packet whose next hop egresses its ingress RIF) on the VPP dataplane, ahead of the implementation PRs, so reviewers can align on the approach first.

Feature tracking issue: sonic-net/sonic-buildimage#25788

Note: this HLD was moved here from sonic-net/sonic-platform-vpp#258 (now closed) to follow the SONiC convention of keeping design documents in the SONiC repo under doc/vpp/.

How I did it

Added a feature HLD under doc/vpp/ following the existing VPP feature-HLD convention: motivation, the RIF-shape matrix, the plugin approach (with alternatives considered), the SAI and CONFIG_DB surface, the VPP node, binary API and counter design, a test plan with live-DUT validation results, and the pre-existing defects found during enablement.

How to verify it

Documentation only. Renders under doc/vpp/vpp-iface-loopback-action.md.

Which release branch to backport (provide reason below if selected)

None.

Description for the changelog

Add HLD: Router Interface Loopback Packet Action on the VPP dataplane.

Part of sonic-net/sonic-buildimage#25788.


PR series (landing order)

Tracking issue: sonic-net/sonic-buildimage#25788

  1. HLD: [doc][vpp] HLD for Router Interface Loopback Packet Action on the VPP dataplane #2462 (this PR)
  2. sonic_ext plugin, nodes and binary API: [vpp] Add sonic_ext RIF loopback packet action nodes and binary API sonic-platform-vpp#259
  3. vslib/vpp SAI adapter and defect fixes: [vpp] SAI RIF loopback packet action and pre-existing VPP vslib defect fixes sonic-sairedis#1983 (depends on 2)
  4. Testbed virtio ctrl_vlan off: [vpp][testbed] Disable virtio ctrl_vlan on VPP KVM DUTs sonic-mgmt#26035
  5. Enablement, lands last: [vpp][ci] Enable iface_loopback_action test on the t1-lag-vpp testbed sonic-mgmt#26036 (depends on 2, 3, 4)

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@lolyu lolyu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please add signature to this PR

… dataplane

Signed-off-by: aaronber0614 <82425405+aaronber0614@users.noreply.github.com>
@aaronber0614
aaronber0614 force-pushed the vpp-iface-loopback-action-hld branch from 828a22e to 648edaa Compare July 22, 2026 20:23
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

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.

3 participants