Skip to content

[Accton][minipack3bta] Disable src-mac==dst-mac drop for L3 loopback fwd test#1392

Open
jeankuo wants to merge 1 commit into
facebook:mainfrom
jeankuo:minipack3bta_disable_srcmac_eq_dstmac_drop
Open

[Accton][minipack3bta] Disable src-mac==dst-mac drop for L3 loopback fwd test#1392
jeankuo wants to merge 1 commit into
facebook:mainfrom
jeankuo:minipack3bta_disable_srcmac_eq_dstmac_drop

Conversation

@jeankuo

@jeankuo jeankuo commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Pre-submission checklist

  • I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running pip install -r requirements-dev.txt && pre-commit install
  • pre-commit run

Summary

AgentL3ForwardingTest.ttl255 injects a packet whose src MAC == dst MAC (the interface/router MAC) out of a port, relies on loopback to re-ingress on a front panel port, and expects it to be L3-routed to the next hop. On BCM78920 (minipack3bta) the front-port ingress drops a frame whose src MAC == dst MAC before routing, so the egress port never counts the packet and the test fails.

This sets sai_disable_srcmacqedstmac_ctrl: 0x1 in the minipack3bta agent config bcm_device:0:global section to disable that drop.

Test Plan

On-box (BCM78920 / SAI_15_4_EA_ODP): the ttl255 packet (src == dst MAC) injected via diag tx with port loopback is dropped at ingress (RDBGC_0) and not routed; the same packet with a different src MAC routes and egresses correctly. With this config the drop is disabled so the loopback-based forwarding test can route.

AgentL3ForwardingTest.ttl255_20260717_184821.zip

…fwd test

AgentL3ForwardingTest.ttl255 injects a packet whose src MAC == dst MAC (both the interface/router MAC) out of a port, relies on loopback to re-ingress on a front panel port, and expects it to be L3-routed. On BCM78920 the front-port ingress drops a frame whose src MAC == dst MAC before routing, so the egress port never sees the packet and the test fails.

Set sai_disable_srcmacqedstmac_ctrl: 0x1 in the minipack3bta agent config global section to disable this drop, allowing the loopback forwarding test to pass. Verified on-box: the same packet with a different src MAC routes and egresses correctly.
@jeankuo
jeankuo requested a review from a team as a code owner July 17, 2026 10:39
@meta-cla meta-cla Bot added the CLA Signed label Jul 17, 2026
@jeankuo jeankuo changed the title [minipack3bta] Disable src-mac==dst-mac drop for L3 loopback fwd test [Accton][minipack3bta] Disable src-mac==dst-mac drop for L3 loopback fwd test Jul 17, 2026
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.

1 participant