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

Disable SA_EQUALS_DA trap on DNX LC SKUs #17206

Merged

Conversation

arista-nwolfe
Copy link
Contributor

This change was submitted directly to 202205 but it's also needed in master and 202305 with SAI9.x
#13346

There has been a couple CSPs for this as well:
CS00012273013 - [7.1][J2, J2c+] Disable SA Equals DA trap on DNX
CS00012320965 - SAI9.2: iBGP doesn't work due to SA_EQUALS_DA trap

If SA_EQUALS_DA trap is enabled iBGP won't work as the Ethernet-IB0 ports are expected to get packets with SA==DA.
E.G. iBGP not working:

nfc405-3# show ip bgp summary -d all

IPv4 Unicast Summary:
BGP router identifier 8.0.0.1, local AS number 65100 vrf-id 0
BGP table version 136206
RIB entries 101651, using 19516992 bytes of memory
Peers 5, using 3709880 KiB of memory
Peer groups 4, using 256 bytes of memory


Neighbhor      V     AS    MsgRcvd    MsgSent    TblVer    InQ    OutQ  Up/Down    State/PfxRcd    NeighborName
-----------  ---  -----  ---------  ---------  --------  -----  ------  ---------  --------------  --------------
10.0.0.1       4  65200       1915       2054         0      0       0  01:22:11   34050           ARISTA01T3
10.0.0.5       4  65200       1924       2060         0      0       0  01:22:39   34050           ARISTA03T3
10.0.0.7       4  65200       1914       2051         0      0       0  01:22:08   34050           ARISTA04T3
10.0.0.11      4  65200       1914       2051         0      0       0  01:22:07   34050           ARISTA06T3
10.0.254.3     4  65100          0          0         0      0       0  never      Connect         nfc405-7

Backport:

  • 202305

@wenyiz2021
Copy link
Contributor

[line 864] Error: sai_disable_srcmacqedstmac_ctrl is not permitted
Result: ../../../device/arista/x86_64-arista_7800r3_48cq2_lc/Arista-7800R3-48CQ2-C48/jr2-a7280cr3-32d4-40x100G.config.bcm FAILED the config check!

@wenyiz2021
Copy link
Contributor

@arlakshm @rlhui for viz

@wenyiz2021
Copy link
Contributor

@arlakshm @lguohan kindly ask for merging of this PR, thanks!

@lguohan lguohan merged commit 865f33c into sonic-net:master Nov 29, 2023
@StormLiangMS
Copy link
Contributor

@wenyiz2021 do we have ADO to track the cherry pick?

@wenyiz2021
Copy link
Contributor

wenyiz2021 commented Nov 30, 2023

hi @StormLiangMS I just created ADO: 26018742 to track cherry pick for 202305

yxieca pushed a commit that referenced this pull request Dec 4, 2023
This change was submitted directly to 202205 but it's also needed in master and 202305 with SAI9.x
#13346

There has been a couple CSPs for this as well:
CS00012273013 - [7.1][J2, J2c+] Disable SA Equals DA trap on DNX
CS00012320965 - SAI9.2: iBGP doesn't work due to SA_EQUALS_DA trap

If SA_EQUALS_DA trap is enabled iBGP won't work as the Ethernet-IB0 ports are expected to get packets with SA==DA.

In the VOQ chassis design, for outgoing control plane packets, the packets goes the recycle port for routing, therefore the dmac of the packet should be the asic router mac. The source mac is assigned by the kernel, so it is also the asic router mac.
@wenyiz2021
Copy link
Contributor

@StormLiangMS could you help to backport tp 202305 branch? ADO: 26018742

@mssonicbld
Copy link
Collaborator

@arista-nwolfe PR conflicts with 202305 branch

@wenyiz2021
Copy link
Contributor

@arista-nwolfe
hi Nathan, please raise a separate PR in 202305

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

Successfully merging this pull request may close these issues.

6 participants