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] Backport single pool to 201911 #5194

Merged
merged 1 commit into from
Aug 15, 2020

Conversation

stephenxs
Copy link
Collaborator

- Why I did it
This is to backport PR #4886 to 201911

Calculate pool size in t1 as 24 * downlink port + 8 * uplink port

  • Take both port and peer MTU into account when calculating headroom
  • Worst case factor is decreased to 50%
  • Mellanox-SN2700-C28D8 t0, assume 48 * 50G/5m + 8 * 100G/40m ports
  • Mellanox-SN2700 (C32)
    • t0: 16 * 100G/5m + 16 * 100G/40m
    • t1: 16 * 100G/40m + 16 * 100G/300m

Signed-off-by: Stephen Sun stephens@nvidia.com

- How I did it

- How to verify it

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

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

…lation

This is to backport the sonic-net#4886 to 201911

Calculate pool size in t1 as 24 * downlink port + 8 * uplink port

- Take both port and peer MTU into account when calculating headroom
- Worst case factor is decreased to 50%
- Mellanox-SN2700-C28D8 t0, assume 48 * 50G/5m + 8 * 100G/40m ports
- Mellanox-SN2700 (C32)
  - t0: 16 * 100G/5m + 16 * 100G/40m
  - t1: 16 * 100G/40m + 16 * 100G/300m

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@abdosi
Copy link
Contributor

abdosi commented Aug 15, 2020

is it good to merge ?

@stephenxs
Copy link
Collaborator Author

is it good to merge ?

Yeah, I think so.

@abdosi abdosi merged commit f6a8678 into sonic-net:201911 Aug 15, 2020
@abdosi abdosi self-requested a review August 15, 2020 14:52
@stephenxs stephenxs deleted the backport-single-pool-to-201911 branch August 15, 2020 14:53
StormLiangMS pushed a commit to StormLiangMS/sonic-buildimage that referenced this pull request Jul 18, 2022
…lation (sonic-net#5194)

This is to backport the sonic-net#4886 to 201911

Calculate pool size in t1 as 24 * downlink port + 8 * uplink port

- Take both port and peer MTU into account when calculating headroom
- Worst case factor is decreased to 50%
- Mellanox-SN2700-C28D8 t0, assume 48 * 50G/5m + 8 * 100G/40m ports
- Mellanox-SN2700 (C32)
  - t0: 16 * 100G/5m + 16 * 100G/40m
  - t1: 16 * 100G/40m + 16 * 100G/300m

Signed-off-by: Stephen Sun <stephens@nvidia.com>
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.

2 participants