Skip to content

Commit 2a4bc3e

Browse files
authored
Fix buffer configs for TH5 C224 SKU (#21624)
Why I did it The values in this config were found to be incorrect when testing. With fewer interfaces than the C256, the reserved buffer space is smaller and therefore the shared limit in this config must be smaller. The BUFFER_QUEUE macro was also not present, causing the defaults to be used which referenced lossless profiles. Work item tracking Microsoft ADO (number only):
1 parent a257ac9 commit 2a4bc3e

File tree

2 files changed

+24
-4
lines changed

2 files changed

+24
-4
lines changed

device/arista/x86_64-arista_7060x6_64pe/Arista-7060X6-64PE-C224O8/buffers_defaults_t0.j2

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,12 @@
55
{%- macro generate_buffer_pool_and_profiles() %}
66
"BUFFER_POOL": {
77
"ingress_lossy_pool": {
8-
"size": "166607744",
8+
"size": "166266368",
99
"type": "ingress",
1010
"mode": "dynamic"
1111
},
1212
"egress_lossy_pool": {
13-
"size": "166607744",
13+
"size": "166266368",
1414
"type": "egress",
1515
"mode": "dynamic"
1616
}
@@ -28,3 +28,13 @@
2828
}
2929
},
3030
{%- endmacro %}
31+
{%- macro generate_queue_buffers(ports) %}
32+
"BUFFER_QUEUE": {
33+
{% for port in ports.split(',') %}
34+
"{{ port }}|0-9": {
35+
"profile" : "egress_lossy_profile"
36+
}
37+
{%- if not loop.last -%},{% endif %}
38+
{% endfor %}
39+
}
40+
{%- endmacro %}

device/arista/x86_64-arista_7060x6_64pe/Arista-7060X6-64PE-C224O8/buffers_defaults_t1.j2

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,12 @@
55
{%- macro generate_buffer_pool_and_profiles() %}
66
"BUFFER_POOL": {
77
"ingress_lossy_pool": {
8-
"size": "166607744",
8+
"size": "166266368",
99
"type": "ingress",
1010
"mode": "dynamic"
1111
},
1212
"egress_lossy_pool": {
13-
"size": "166607744",
13+
"size": "166266368",
1414
"type": "egress",
1515
"mode": "dynamic"
1616
}
@@ -28,3 +28,13 @@
2828
}
2929
},
3030
{%- endmacro %}
31+
{%- macro generate_queue_buffers(ports) %}
32+
"BUFFER_QUEUE": {
33+
{% for port in ports.split(',') %}
34+
"{{ port }}|0-9": {
35+
"profile" : "egress_lossy_profile"
36+
}
37+
{%- if not loop.last -%},{% endif %}
38+
{% endfor %}
39+
}
40+
{%- endmacro %}

0 commit comments

Comments
 (0)