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

[Mellanox] Update buffer calculations for Mellanox-SN5600-C224O8 SKU #20992

Merged

Conversation

noaOrMlnx
Copy link
Collaborator

@noaOrMlnx noaOrMlnx commented Dec 2, 2024

Why I did it

Update buffer calculations for Mellanox-SN5600-C224O8 HwSKU.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Update the lossless pull size

How to verify it

Run sonic-mgmt QoS test

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

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

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)

@noaOrMlnx noaOrMlnx marked this pull request as ready for review December 5, 2024 09:36
@liat-grozovik liat-grozovik merged commit 5614f51 into sonic-net:master Dec 12, 2024
22 checks passed
JibinBao added a commit to JibinBao/sonic-mgmt that referenced this pull request Jan 10, 2025
1. For spc4, there is only lossy buffer, so the buffer for lossless buffer will be taken by lossy buffer, if the packet size is too small, the packet number sent to occupy the shared buffer will increase a lot, so it will lead that descriptor will be exhausted, so update testQosSaiPgSharedWatermark, testQosSaiQSharedWatermark and testQosSaiLossyQueue accordingly.
2. Remove the test config of scheduler.block_data_plane, otherwise it might rise yang validation error when do config reload
3. list the relevant Prs:
   sonic-net/sonic-buildimage#20992
   sonic-net/sonic-buildimage#21056
   sonic-net/sonic-buildimage#20991
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202411: #21409

dgsudharsan pushed a commit to dgsudharsan/sonic-buildimage that referenced this pull request Jan 15, 2025
VladimirKuk pushed a commit to Marvell-switching/sonic-buildimage that referenced this pull request Jan 21, 2025
…onic-net#20992)

- Why I did it
Update buffer calculations for Mellanox-SN5600-C224O8 HwSKU.

- How I did it
Update the lossless pull size

- How to verify it
Run sonic-mgmt QoS test
ayurkiv-nvda pushed a commit to ayurkiv-nvda/sonic-buildimage that referenced this pull request Jan 22, 2025
tshalvi pushed a commit to tshalvi/sonic-buildimage that referenced this pull request Jan 23, 2025
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.

5 participants