Skip to content

Commit 10871c6

Browse files
noaOrMlnxyanjundeng
authored andcommitted
[Mellanox] Update SN56xx relevant SKUs buffers_defaults_objects.j2 (sonic-net#22067)
- Why I did it To have buffers updated to new requirements to following SKUs: Mellanox-SN5600-C256S1 Mellanox-SN5600-C224O8 Mellanox-SN5610N-C256S2 Mellanox-SN5610N-C224O8 Mellanox-SN5640-C512S2 Mellanox-SN5600-C448O16 sonic-mgmt test update: sonic-net/sonic-mgmt#17788 - How I did it Set port.alpha to 1/64. Set queue.alpha for queues 1, 2, and 3 to 1. Set queue.alpha for queue 4 to infinity or 128 - How to verify it Load an image with the change on Mellanox system, check SDK dumps and run sonic-mgmt test.
1 parent 76957ca commit 10871c6

File tree

2 files changed

+17
-13
lines changed

2 files changed

+17
-13
lines changed

device/mellanox/x86_64-nvidia_sn5600-r0/Mellanox-SN5600-C256S1/buffers_defaults_objects.j2

+8-13
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@
7171
"ingress_lossy_profile": {
7272
"pool":"ingress_lossless_pool",
7373
"size":"0",
74-
"dynamic_th":"3"
74+
"dynamic_th":"7"
7575
},
7676
"egress_lossless_profile": {
7777
"pool":"egress_lossless_pool",
@@ -81,22 +81,22 @@
8181
"egress_lossy_profile": {
8282
"pool":"egress_lossy_pool",
8383
"size":"9216",
84-
"dynamic_th":"7"
84+
"dynamic_th":"-6"
8585
},
8686
"q_lossy_profile": {
8787
"pool":"egress_lossy_pool",
8888
"size":"0",
8989
"dynamic_th":"0"
9090
},
91-
"q_lossy_minus_7_profile": {
91+
"q_lossy_minus_3_profile": {
9292
"pool": "egress_lossy_pool",
9393
"size": "0",
94-
"dynamic_th" : "-7"
94+
"dynamic_th" : "-3"
9595
},
96-
"q_lossy_minus_3_profile": {
96+
"q_lossy_infinity_profile": {
9797
"pool": "egress_lossy_pool",
9898
"size": "0",
99-
"dynamic_th" : "-3"
99+
"dynamic_th" : "7"
100100
}
101101
},
102102
{%- endmacro %}
@@ -149,18 +149,13 @@
149149
{%- macro generate_queue_buffers(port_names_active, port_names_inactive) %}
150150
"BUFFER_QUEUE": {
151151
{% for port in port_names_active.split(',') %}
152-
"{{ port }}|0": {
152+
"{{ port }}|0-3": {
153153
"profile" : "q_lossy_profile"
154154
},
155155
{% endfor %}
156-
{% for port in port_names_active.split(',') %}
157-
"{{ port }}|1-3": {
158-
"profile" : "q_lossy_minus_7_profile"
159-
},
160-
{% endfor %}
161156
{% for port in port_names_active.split(',') %}
162157
"{{ port }}|4": {
163-
"profile" : "q_lossy_profile"
158+
"profile" : "q_lossy_infinity_profile"
164159
},
165160
{% endfor %}
166161
{% for port in port_names_active.split(',') %}

device/mellanox/x86_64-nvidia_sn5600-r0/Mellanox-SN5600-C256S1/qos.json.j2

+9
Original file line numberDiff line numberDiff line change
@@ -167,4 +167,13 @@
167167
},
168168
{%- endmacro %}
169169

170+
{%- macro generate_lossless_tc_list(lossless_tc_list) %}
171+
{#
172+
This macro intentionally does nothing, leaving LOSSLESS_TC as an empty list
173+
Use the following to generate other lists like 3, 4
174+
{%- if lossless_tc_list.append('3') -%}{%- endif -%}
175+
{%- if lossless_tc_list.append('4') -%}{%- endif -%}
176+
#}
177+
{%- endmacro %}
178+
170179
{%- include 'qos_config.j2' %}

0 commit comments

Comments
 (0)