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

[202205][mellanox][ssm]: Enable Store-And-Forward switching mode #16662

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

nazariig
Copy link
Collaborator

@nazariig nazariig commented Sep 22, 2023

Signed-off-by: Nazarii Hnydyn nazariig@nvidia.com

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

  1. make configure PLATFORM=mellanox
  2. make target/sonic-mellanox.bin

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)

  • 202205

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====
           '//||\\`
             ''``

…00C/SN4700.

Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
@bingwang-ms
Copy link
Contributor

Does this change depend on certain SAI change? Or only changing sai profile is enough?

@bingwang-ms
Copy link
Contributor

MSFT ADO: 24668711

@yxieca yxieca merged commit 214ea08 into sonic-net:202205 Sep 28, 2023
15 checks passed
@nazariig
Copy link
Collaborator Author

Does this change depend on certain SAI change? Or only changing sai profile is enough?

@bingwang-ms yes. The new SAI is a must in addition to these changes. We will upstream it soon

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.

3 participants