Skip to content

[202012] Support zero buffer profiles#83

Closed
stephenxs wants to merge 1 commit into202012from
zero-profile-202012
Closed

[202012] Support zero buffer profiles#83
stephenxs wants to merge 1 commit into202012from
zero-profile-202012

Conversation

@stephenxs
Copy link
Copy Markdown
Owner

@stephenxs stephenxs commented Sep 27, 2021

This is to backport community PR 8768 to 202012 branch

  1. Add buffer profiles and pool definition for zero buffer profiles
    • If buffer model is static:
      • Apply normal buffer profiles to admin-up ports
      • Apply zero buffer profiles to admin-down ports
    • If buffer model is dynamic:
      • Apply normal buffer profiles to all ports
      • buffer manager will take care when a port is shutdown
  2. To make code clean and reduce redundant code, extract common macros from buffer_defaults_t{0,1}.j2 of all SKUs to two common files:
    • one in Mellanox-SN2700 for single ingress pool mode
    • the other in ACS-MSN2700 for double ingress pool mode
      Those files of all other SKUs will be symbol link to the above files
  3. Adjust example output file of json template for unit test

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

Why I did it

How I did it

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

1. Add buffer profiles and pool definition for zero buffer profiles
   - If buffer model is static:
      - Apply normal buffer profiles to admin-up ports
      - Apply zero buffer profiles to admin-down ports
   - If buffer model is dynamic:
      - Apply normal buffer profiles to all ports
      - buffer manager will take care when a port is shutdown
2. To make code clean and reduce redundant code, extract common macros from buffer_defaults_t{0,1}.j2 of all SKUs to two common files:
    - one in Mellanox-SN2700 for single ingress pool mode
    - the other in ACS-MSN2700 for double ingress pool mode
   Those files of all other SKUs will be symbol link to the above files
3. Adjust example output file of json template for unit test

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@stephenxs stephenxs closed this Oct 26, 2021
stephenxs pushed a commit that referenced this pull request Jun 18, 2022
[sonic-linkmgrd][202012] submodule update

0839af2 Longxiang Lyu Wed Jun 15 08:46:21 2022 +0800 [202012] Fix IP header checksum in handleSendSwitchCommand (#89)
afc4972 Jing Zhang Wed Jun 1 10:33:12 2022 -0700 Revert "Update log level for mux probing and mux state chance (#23)" (#85)
ed52d0a Longxiang Lyu Tue May 31 10:28:30 2022 +0800 Add a command line option to store logs into a separate file (#83)

sign-off: Jing Zhang zhangjing@microsoft.com
stephenxs pushed a commit that referenced this pull request Nov 24, 2023
…utomatically (sonic-net#17174)

#### Why I did it
src/sonic-host-services
```
* 586b1e9 - (HEAD -> master, origin/master, origin/HEAD) Disable systemd auto-restart of dependent services for spineRouters (#83) (5 hours ago) [Deepak Singhal]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stephenxs pushed a commit that referenced this pull request Jul 23, 2025
…ically (sonic-net#23362)

#### Why I did it
src/sonic-dash-ha
```
* c3e8828 - (HEAD -> master, origin/master, origin/HEAD) Dpu db access requires container_name (#83) (2 days ago) [yue-fred-gao]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stephenxs pushed a commit that referenced this pull request Feb 2, 2026
…ly (sonic-net#25252)

#### Why I did it
src/sonic-stp
```
* f8f0c34 - (HEAD -> master, origin/master, origin/HEAD) Use protocol-aware 100ms timer callback (#83) (9 days ago) [Vyacheslav Yamont]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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