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.

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

[Dell] S5232f default buffer configuration changes #3278

Merged
merged 9 commits into from
Aug 8, 2019
Merged

[Dell] S5232f default buffer configuration changes #3278

merged 9 commits into from
Aug 8, 2019

Conversation

AshokDaparthi
Copy link
Contributor

  • What I did
  1. Corrected the ingress and egress lossy and lossless buffer pool and profile values.
  2. Single pool for lossy and lossless traffic for PFC priority 3 and 4.
  3. In HWSKU DellEMC-S5232f-P-100GG/25G/10G. Defaults to lossy profile. No Lossless support.
  4. 2 default mmu_init.data files (TD3-DEFAULT-LOSSLESS-P3P4 and "TD3-DEFAULT" created in SAI code. This will have cpu pool configuration and MMU init related configurations.
  • TD3-DEFAULT.data file is only supports lossy.
  • TD3-DEFAULT-LOSSLESS-P3P4.data file is supports lossless on P3 and P4.

@AshokDaparthi
Copy link
Contributor Author

@lguohan - Could you help on this

@lguohan lguohan merged commit fe6664b into sonic-net:master Aug 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lguohan lguohan lguohan approved these changes

Assignees
No one assigned
Labels
Enhancement ➕ Platform: DellEMC
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AshokDaparthi @lguohan @jleveque