Skip to content

[Mellanox][Switching Mode] Enable Store-And-Forward switching mode on specific platforms #16781

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 9, 2023
Merged

[Mellanox][Switching Mode] Enable Store-And-Forward switching mode on specific platforms #16781

merged 1 commit into from
Oct 9, 2023

Conversation

nazariig
Copy link
Collaborator

@nazariig nazariig commented Oct 5, 2023 •
edited by liat-grozovik
Loading

Signed-off-by: Nazarii Hnydyn [email protected]

Why I did it

  • To enable Store-And-Forward switching mode for SN2700/SN3800/SN4600C/SN4700 on specific and requested SKUs. Default SKU remain untouched.
Work item tracking
  • N/A

How I did it

  • Added vendor SAI config options

How to verify it

  1. make configure PLATFORM=mellanox
  2. make target/sonic-mellanox.bin
  3. run sonic-mgmt test suits while this option is enabled.

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)

  • master

Description for the changelog

  • N/A

Link to config_db schema for YANG module changes

  • N/A

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

      .---.        .-----------
     /     \  __  /    ------
    / /     \(  )/    -----
   //////   ' \/ `   ---
  //// / // :    : ---
 // /   /  /`    '--
//          //..\\
       ====UU====UU====
           '//||\\`
             ''``

This was referenced Oct 5, 2023
Merged
Merged
StormLiangMS pushed a commit that referenced this pull request Oct 8, 2023
@nazariig
[ssm]: Enable Store-And-Forward switching mode for SN2700/SN3800/SN46…
ea40778 
…00C/SN4700. (#16761)

BACKPORT: #16781

Why I did it
To enable Store-And-Forward switching mode for SN2700/SN3800/SN4600C/SN4700
Work item tracking
N/A
How I did it
Added vendor SAI config options
How to verify it
make configure PLATFORM=mellanox
make target/sonic-mellanox.bin
@liat-grozovik liat-grozovik changed the title [mellanox][ssm]: Enable Store-And-Forward switching mode [Mellanox][ssm]: Enable Store-And-Forward switching mode Oct 8, 2023
@liat-grozovik liat-grozovik changed the title [Mellanox][ssm]: Enable Store-And-Forward switching mode [Mellanox][ssm] Enable Store-And-Forward switching mode Oct 8, 2023
@liat-grozovik liat-grozovik changed the title [Mellanox][ssm] Enable Store-And-Forward switching mode [Mellanox][Switching Mode] Enable Store-And-Forward switching mode Oct 8, 2023
@nazariig
Copy link
Collaborator Author

nazariig commented Oct 9, 2023

@StormLiangMS / @bingwang-ms can you please help to merge?

@liat-grozovik liat-grozovik mentioned this pull request Oct 9, 2023
Merged
11 tasks
@liat-grozovik liat-grozovik merged commit 875a6d9 into sonic-net:master Oct 9, 2023
@liat-grozovik liat-grozovik changed the title [Mellanox][Switching Mode] Enable Store-And-Forward switching mode [Mellanox][Switching Mode] Enable Store-And-Forward switching mode on specific platforms Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liat-grozovik liat-grozovik liat-grozovik approved these changes

@volodymyrsamotiy volodymyrsamotiy Awaiting requested review from volodymyrsamotiy

@lguohan lguohan Awaiting requested review from lguohan

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft

@bingwang-ms bingwang-ms Awaiting requested review from bingwang-ms

Assignees
No one assigned
Labels
Cherry Pick Conflict_202205 Enhancement ➕ Included in 202305 Branch Platform: Mellanox Request for 202205 Branch Request for 202305 Branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nazariig @liat-grozovik @StormLiangMS