Skip to content

[SmartSwitch] [Nvidia] Remove PG 3 and 4 for Internal Ports#122

Closed
vivekrnv wants to merge 4 commits intomasterfrom
remove_lossless_pgs
Closed

[SmartSwitch] [Nvidia] Remove PG 3 and 4 for Internal Ports#122
vivekrnv wants to merge 4 commits intomasterfrom
remove_lossless_pgs

Conversation

@vivekrnv
Copy link
Owner

Why I did it

PG's 3 & 4 for Internal ports today have lossy buffer profiles attached, but this is redundant and can be removed as they occupy extra overhead even though it's not significant

Work item tracking
  • Microsoft ADO (number only):

How I did it

Make the cable length for Dpc ports to 0m and create a new TC_PG_MAP for internal ports

How to verify it

  1. Run config qos reload --no-dynamic-buffer, config save -y and config reload -y
redis-cli -n 4 hget "CABLE_LENGTH|AZURE" Ethernet224
"0m"
redis-cli -n 4 hget "PORT_QOS_MAP|Ethernet232" tc_to_pg_map
"AZURE_DPC"

root@r-bobcat-01:/home/admin# sonic-db-cli CONFIG_DB HGETALL "TC_TO_PRIORITY_GROUP_MAP|AZURE_DPC"
{'0': '0', '1': '0', '2': '0', '3': '0', '4': '0', '5': '0', '6': '0', '7': '7'}

redis-cli -n 0 keys "BUFFER_PG*Ethernet232*"
1) "BUFFER_PG_TABLE:Ethernet232:0"

redis-cli -n 0 keys "BUFFER_PG*Ethernet88*"
1) "BUFFER_PG_TABLE:Ethernet88:3-4"
2) "BUFFER_PG_TABLE:Ethernet88:0"
2025 Feb 14 11:36:15.333024 sonic NOTICE swss#buffermgrd: :- doSpeedUpdateTask: Not creating/updating PG profile for port Ethernet224. Cable length is set to 0m
2025 Feb 14 11:36:15.333024 sonic NOTICE swss#buffermgrd: :- doSpeedUpdateTask: Not creating/updating PG profile for port Ethernet232. Cable length is set to 0m
2025 Feb 14 11:36:15.333142 sonic NOTICE swss#buffermgrd: :- doSpeedUpdateTask: Not creating/updating PG profile for port Ethernet240. Cable length is set to 0m
2025 Feb 14 11:36:15.333142 sonic NOTICE swss#buffermgrd: :- doSpeedUpdateTask: Not creating/updating PG profile for port Ethernet248. Cable length is set to 0m

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

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

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)

vivekrnv and others added 3 commits February 14, 2025 20:30
Signed-off-by: Vivek Reddy <[email protected]>
Signed-off-by: Vivek Reddy <[email protected]>
vivekrnv pushed a commit that referenced this pull request Nov 12, 2025
…ically (sonic-net#24435)

#### Why I did it
src/sonic-dash-ha
```
* 149774b - (HEAD -> master, origin/master, origin/HEAD) Add a PR template (#129) (3 days ago) [yue-fred-gao]
* 711a0be - Sort the output of show actor command (#128) (3 days ago) [yue-fred-gao]
* 15fbd5c - Fix format for protobuf fields in show command. (#127) (3 days ago) [dypet]
* f2d47b9 - Add wait for loopback script. (#126) (3 days ago) [dypet]
* 570db05 - Fix issue #118: use hostname to build service path (#122) (3 days 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.

3 participants