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

[Nokia][Platform] Update the BCM config file to disable Dos attack when the MAC_SA==MAC_DA #18446

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

saksarav-nokia
Copy link
Contributor

Why I did it

The SAI code was modified to trap and drop the packets if the MAC_SA==MAC_DA and soc variable sai_disable_srcmacqedstmac_ctrl=1 needs to be set in bcm config. There was a PR #13346 which added updated bcm config in 202205, but never merged to master.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Updated BCM config

How to verify it

verified that syncd and swss come up in master with SAI 10.x

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@saksarav-nokia saksarav-nokia force-pushed the saksarav-nokia-bcm-config1 branch from 0511bd4 to 4812b63 Compare March 22, 2024 21:35
@saksarav-nokia
Copy link
Contributor Author

@judyjoseph @mlok-nokia for viz

@judyjoseph
Copy link
Contributor

judyjoseph commented Mar 22, 2024

Thanks @saksarav-nokia, looks like this change was part of a SAI 7.1 update PR 13346

master and 202205 was using different SAI versions, SAI 7.1 was used for 202205 alone.

Hence this PR was not merged into master first and cherry-picked to 202205 branch.

@judyjoseph
Copy link
Contributor

@rhui @lguohan , please help merge this change

@lguohan lguohan merged commit d5e2f64 into sonic-net:master Mar 27, 2024
19 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.

3 participants