Skip to content

Dual tor t1 c64 202012#107

Closed
stephenxs wants to merge 11 commits into202012from
dual-tor-t1-c64-202012
Closed

Dual tor t1 c64 202012#107
stephenxs wants to merge 11 commits into202012from
dual-tor-t1-c64-202012

Conversation

@stephenxs
Copy link
Copy Markdown
Owner

@stephenxs stephenxs commented May 7, 2022

Why I did it

Support Mellanox-SN4600c-C64 as a T1 switch in dual ToR scenario

This is not a clean PR since some PRs by the community have not been merged.
The following files have been updated based on the community will-be-merged PRs.

  • device/mellanox/x86_64-mlnx_msn4600c-r0/Mellanox-SN4600C-C64/buffers_defaults_t1.j2
  • device/mellanox/x86_64-mlnx_msn4600c-r0/Mellanox-SN4600C-C64/qos.json.j2
  • device/mellanox/x86_64-mlnx_msn2700-r0/Mellanox-SN2700-D48C8/buffers_defaults_objects.j2
  • files/build_templates/bufers_config.j2: generate_pg_profiles_with_extra_lossless_pgs_with_inactive_ports
  • files/build_templates/qos_config.j2: PFC_PRIORITY_TO_PRIORITY_GROUP_MAP

How I did it

Provide buffer pool sizes for Mellanox-SN4600c-C64 T1.
Support generate_pg_profiles_with_extra_lossless_pgs_with_inactive_ports
Adjust PFC_PRIORITY_TO_PRIORITY_GROUP_MAP

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

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

stephenxs pushed a commit that referenced this pull request May 12, 2022
Why I did it
Update submodule sonic-restapi
e83e0e8 Fix Ctype_char larger than address space issue in 32-bit armhf (#107)
@stephenxs
Copy link
Copy Markdown
Owner Author

sonci_build 2052 passed

@stephenxs
Copy link
Copy Markdown
Owner Author

sonic_ci 1482 passed

@stephenxs stephenxs force-pushed the dual-tor-t1-c64-202012 branch from 85865e8 to da87482 Compare May 18, 2022 14:39
stephenxs and others added 4 commits June 5, 2022 02:39
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
…ng macro are defined

Move '_with_extra_queue' and 'with_extra_queue_with_inactive_port' version to the beginning.
For some vendors, all versions are defined.
The generic version will be called if it is defined at the beginning

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@stephenxs stephenxs force-pushed the dual-tor-t1-c64-202012 branch from da87482 to 64a50ce Compare June 5, 2022 02:44
stephenxs and others added 7 commits June 5, 2022 03:51
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
* [Tunnel PFC] Add property for tunnel PFC

Replace the config.bcm file with j2 template file
- Add 'sai_remap_prio_on_tnl_egress=1' property when device metadata local
- Host subtype is 'dualtor'
- Change sai.profile foe the new config.bcm.j2
…onic-net#11018)

Why I did it
As part of PCBB changes, we need to enable 2 extra lossless queues. The changes in this PR are done to adjust only the reserved sizes on Th2 for the additional 2 lossless queues
Calculations are done based on 40 downlinks for T1 and 16 uplinks for dual ToR

How to verify it
Verified that the rendering works fine on Th2 dut
Unit tests have been updated to reflect the modified buffer sizes when pcbb is enabled. There are existing testcases that will test the original buffer sizes when pcbb is disabled. With these changes, was able to build sonic-config-engine wheel successfully

Signed-off-by: Neetha John <nejo@microsoft.com>
It doesn't make sense since both cases result in the same code

Signed-off-by: Stephen Sun <stephens@nvidia.com>
…queues (sonic-net#11018)"

This reverts commit 21f14dc.

unit test needs to be cherry-picked.
@stephenxs stephenxs closed this Jun 7, 2022
stephenxs pushed a commit that referenced this pull request May 16, 2024
…utomatically (sonic-net#18960)

#### Why I did it
src/sonic-host-services
```
* 4caed11 - (HEAD -> 202311, origin/202311) Fix issue: delayed service will not be restarted if swss crashes (#107) (10 hours ago) [Junchao-Mellanox]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stephenxs pushed a commit that referenced this pull request May 24, 2024
…utomatically (sonic-net#18959)

#### Why I did it
src/sonic-host-services
```
* 10bfa90 - (HEAD -> master, origin/master, origin/HEAD) Add new API to support config replace (#125) (25 hours ago) [ganglv]
* 1a8ff81 - Fix issue: delayed service will not be restarted if swss crashes (#107) (3 days ago) [Junchao-Mellanox]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stephenxs pushed a commit that referenced this pull request Sep 16, 2025
…ically (sonic-net#23947)

#### Why I did it
src/sonic-dash-ha
```
* d14d54b - (HEAD -> master, origin/master, origin/HEAD) Implement cleanup logic for all the actors (#102) (10 hours ago) [yue-fred-gao]
* 4e3706a - Fix ha state. (#107) (20 hours ago) [dypet]
* 58dbc27 - fix show hamgrd actor command (#108) (20 hours ago) [yue-fred-gao]
```
#### 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.

4 participants