Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update qos config to clear queues for bounced back traffic #10176

Merged
merged 12 commits into from
Apr 5, 2022
Merged

Update qos config to clear queues for bounced back traffic #10176

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
0d37da4
Update qos config to clear queues for bounced back traffic
bingwang-ms Mar 8, 2022
6565c2e
Address comments
bingwang-ms Mar 9, 2022
0e0fffe
Fix UT
bingwang-ms Mar 9, 2022
9a491e1
Fix UT
bingwang-ms Mar 9, 2022
a44c156
Update qos table
bingwang-ms Mar 30, 2022
598cf86
Check NEIGH type
bingwang-ms Mar 30, 2022
f1d9c0f
Fix UT
bingwang-ms Mar 30, 2022
eb399f8
Fix py2 UT
bingwang-ms Mar 30, 2022
5da008b
Merge branch 'master' into clear_q_for_tunnel_remapping
bingwang-ms Mar 30, 2022
ba5b35d
Fix py2 UT
bingwang-ms Mar 30, 2022
da726a6
Remove TC_TO_QUEUE_MAP for leaf
bingwang-ms Mar 31, 2022
dab2fc0
Merge branch 'master' into clear_q_for_tunnel_remapping
bingwang-ms Apr 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion device/arista/x86_64-arista_7050cx3_32s/Arista-7050CX3-32S-C32/qos.json.j2
View file
Open in desktop

This file was deleted.

1 change: 1 addition & 0 deletions device/arista/x86_64-arista_7050cx3_32s/Arista-7050CX3-32S-C32/qos.json.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
../Arista-7050CX3-32S-D48C8/qos.json.j2
85 changes: 85 additions & 0 deletions device/arista/x86_64-arista_7050cx3_32s/Arista-7050CX3-32S-D48C8/qos.json.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1,86 @@
{% if 'subtype' in DEVICE_METADATA['localhost'] and DEVICE_METADATA['localhost']['subtype'] == 'DualToR' %}
{%- macro generate_dscp_to_tc_map() %}
"DSCP_TO_TC_MAP": {
"AZURE": {
"0" : "1",
"1" : "1",
"2" : "2",
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
"3" : "3",
"4" : "4",
"5" : "1",
"6" : "6",
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
"7" : "1",
"8" : "0",
"9" : "1",
"10": "1",
"11": "1",
"12": "1",
"13": "1",
"14": "1",
"15": "1",
"16": "1",
"17": "1",
"18": "1",
"19": "1",
"20": "1",
"21": "1",
"22": "1",
"23": "1",
"24": "1",
"25": "1",
"26": "1",
"27": "1",
"28": "1",
"29": "1",
"30": "1",
"31": "1",
"32": "1",
"33": "1",
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
"34": "1",
"35": "1",
"36": "1",
"37": "1",
"38": "1",
"39": "1",
"40": "1",
"41": "1",
"42": "1",
"43": "1",
"44": "1",
"45": "1",
"46": "5",
"47": "1",
"48": "7",
"49": "1",
"50": "1",
"51": "1",
"52": "1",
"53": "1",
"54": "1",
"55": "1",
"56": "1",
"57": "1",
"58": "1",
"59": "1",
"60": "1",
"61": "1",
"62": "1",
"63": "1"
}
},
"TC_TO_PRIORITY_GROUP_MAP": {
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
"AZURE": {
"0": "0",
"1": "0",
"2": "2",
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
"3": "3",
"4": "4",
"5": "0",
"6": "6",
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
"7": "7"
}
},
{%- endmacro %}
{% endif %}

{%- include 'qos_config.j2' %}
85 changes: 85 additions & 0 deletions device/arista/x86_64-arista_7260cx3_64/Arista-7260CX3-C64/qos.json.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1,86 @@
{% if ('type' in DEVICE_METADATA['localhost'] and DEVICE_METADATA['localhost']['type'] == 'LeafRouter') or ('subtype' in DEVICE_METADATA['localhost'] and DEVICE_METADATA['localhost']['subtype'] == 'DualToR') %}
{%- macro generate_dscp_to_tc_map() %}
"DSCP_TO_TC_MAP": {
neethajohn marked this conversation as resolved.
Show resolved Hide resolved
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
"AZURE": {
"0" : "1",
"1" : "1",
"2" : "2",
"3" : "3",
"4" : "4",
"5" : "1",
"6" : "6",
"7" : "1",
"8" : "0",
"9" : "1",
"10": "1",
"11": "1",
"12": "1",
"13": "1",
"14": "1",
"15": "1",
"16": "1",
"17": "1",
"18": "1",
"19": "1",
"20": "1",
"21": "1",
"22": "1",
"23": "1",
"24": "1",
"25": "1",
"26": "1",
"27": "1",
"28": "1",
"29": "1",
"30": "1",
"31": "1",
"32": "1",
"33": "1",
"34": "1",
"35": "1",
"36": "1",
"37": "1",
"38": "1",
"39": "1",
"40": "1",
"41": "1",
"42": "1",
"43": "1",
"44": "1",
"45": "1",
"46": "5",
"47": "1",
"48": "7",
"49": "1",
"50": "1",
"51": "1",
"52": "1",
"53": "1",
"54": "1",
"55": "1",
"56": "1",
"57": "1",
"58": "1",
"59": "1",
"60": "1",
"61": "1",
"62": "1",
"63": "1"
}
},
"TC_TO_PRIORITY_GROUP_MAP": {
"AZURE": {
"0": "0",
"1": "0",
"2": "2",
"3": "3",
"4": "4",
"5": "0",
"6": "6",
"7": "7"
}
},
{%- endmacro %}
{% endif %}

{%- include 'qos_config.j2' %}
1 change: 0 additions & 1 deletion device/arista/x86_64-arista_7260cx3_64/Arista-7260CX3-D108C8/qos.json.j2
View file
Open in desktop

This file was deleted.

1 change: 1 addition & 0 deletions device/arista/x86_64-arista_7260cx3_64/Arista-7260CX3-D108C8/qos.json.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
../Arista-7260CX3-C64/qos.json.j2
1 change: 0 additions & 1 deletion device/arista/x86_64-arista_7260cx3_64/Arista-7260CX3-Q64/qos.json.j2
View file
Open in desktop

This file was deleted.

1 change: 1 addition & 0 deletions device/arista/x86_64-arista_7260cx3_64/Arista-7260CX3-Q64/qos.json.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
../Arista-7260CX3-C64/qos.json.j2
29 changes: 27 additions & 2 deletions files/build_templates/qos_config.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,13 @@
{%- endfor %}
{%- set port_names_active = port_names_list_active | join(',') -%}

{%- set port_names_list_extra_queues = [] %}
{%- for port in PORT_ACTIVE %}
{% if (generate_dscp_to_tc_map is defined) and ('type' in DEVICE_METADATA['localhost'] and DEVICE_METADATA['localhost']['type'] == 'LeafRouter' and DEVICE_NEIGHBOR is defined and 'name' in DEVICE_NEIGHBOR[port] and 'T0' in DEVICE_NEIGHBOR[port]['name']) or
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
('subtype' in DEVICE_METADATA['localhost'] and DEVICE_METADATA['localhost']['subtype'] == 'DualToR' and DEVICE_NEIGHBOR is defined and 'name' in DEVICE_NEIGHBOR[port] and 'T1' in DEVICE_NEIGHBOR[port]['name']) %}
{%- if port_names_list_extra_queues.append(port) %}{%- endif %}
{% endif %}
{%- endfor %}

{%- set pfc_to_pg_map_supported_asics = ['mellanox', 'barefoot', 'marvell'] -%}
{%- set backend_device_types = ['BackEndToRRouter', 'BackEndLeafRouter'] -%}
Expand All @@ -36,7 +43,7 @@
{
{% if generate_tc_to_pg_map is defined %}
{{- generate_tc_to_pg_map() }}
{% else %}
{% elif generate_dscp_to_tc_map is not defined %}
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
"TC_TO_PRIORITY_GROUP_MAP": {
"AZURE": {
"0": "0",
Expand Down Expand Up @@ -87,6 +94,8 @@
"7": "7"
}
},
{% elif generate_dscp_to_tc_map is defined %}
{{- generate_dscp_to_tc_map() }}
{% else %}
"DSCP_TO_TC_MAP": {
"AZURE": {
Expand Down Expand Up @@ -206,7 +215,13 @@
{% if asic_type in pfc_to_pg_map_supported_asics %}
"pfc_to_pg_map" : "AZURE",
{% endif %}
"pfc_enable" : "3,4"
{% if port in port_names_list_extra_queues %}
"pfc_enable" : "2,3,4,6",
"pfcwd_sw_enable" : "3,4"
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
{% else %}
"pfc_enable" : "3,4",
"pfcwd_sw_enable" : "3,4"
{% endif %}
}{% if not loop.last %},{% endif %}

{% endfor %}
Expand Down Expand Up @@ -266,7 +281,12 @@
{% endfor %}
{% for port in PORT_ACTIVE %}
"{{ port }}|2": {
{% if port in port_names_list_extra_queues %}
"scheduler" : "scheduler.1",
"wred_profile": "AZURE_LOSSLESS"
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
{% else %}
"scheduler": "scheduler.0"
{% endif %}
},
{% endfor %}
{% for port in PORT_ACTIVE %}
Expand All @@ -276,7 +296,12 @@
{% endfor %}
{% for port in PORT_ACTIVE %}
"{{ port }}|6": {
{% if port in port_names_list_extra_queues %}
"scheduler" : "scheduler.1",
"wred_profile": "AZURE_LOSSLESS"
bingwang-ms marked this conversation as resolved.
Show resolved Hide resolved
{% else %}
"scheduler": "scheduler.0"
{% endif %}
}{% if not loop.last %},{% endif %}

{% endfor %}
Expand Down
Loading