Skip to content

[QoS] Added Spectrum-5 as supported ASIC #17908

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

Merged
liat-grozovik merged 2 commits into from
Jun 8, 2025
+45 −4
Merged

[QoS] Added Spectrum-5 as supported ASIC #17908

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4c7949d
Fixed QoS tests on 5640 platform - Added SPC5 as supported ASIC
AharonMalkin Apr 9, 2025
a65e877
Update qos_sai_base.py
AharonMalkin Apr 23, 2025
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
9 changes: 7 additions & 2 deletions tests/qos/files/mellanox/qos_param_generator.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,11 @@ def __init__(self, qos_params, asic_type, speed_cable_len, dutConfig, ingressLos
'cell_size': 192,
'headroom_overhead': 47,
'private_headroom': 30
},
'spc5': {
'cell_size': 192,
'headroom_overhead': 47,
'private_headroom': 30
}
}
self.asic_type = asic_type
Expand Down Expand Up @@ -86,7 +91,7 @@ def collect_qos_configurations(self):
headroom = xon + xoff
ingress_lossless_size = int(
math.ceil(float(self.ingressLosslessProfile['static_th']) / self.cell_size)) - xon
if self.asic_type == "spc4":
if self.asic_type == "spc4" or self.asic_type == 'spc5':
pg_q_alpha = self.ingressLosslessProfile['pg_q_alpha']
port_alpha = self.ingressLosslessProfile['port_alpha']
pool_size = int(math.ceil(float(self.ingressLosslessProfile['pool_size']) / self.cell_size))
Expand Down Expand Up @@ -122,7 +127,7 @@ def collect_qos_configurations(self):
self.qos_parameters['dst_port_id'] = dst_testPortIds[0]
pgs_per_port = 2 if not self.dualTor else 4
occupied_buffer = 0
if self.asic_type == "spc4":
if self.asic_type == "spc4" or self.asic_type == 'spc5':
for i in range(1, ingress_ports_num_shp):
for j in range(pgs_per_port):
pg_occupancy = int(math.ceil(
Expand Down
36 changes: 36 additions & 0 deletions tests/qos/files/mellanox/special_qos_config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,3 +36,39 @@ qos_params:
pkts_num_margin: 8
wm_q_shared_lossy:
packet_size: 1200
spc5:
profile:
pkts_num_leak_out: 1
xoff_1:
packet_size: 800
xoff_2:
packet_size: 800
xoff_3:
packet_size: 800
xoff_4:
packet_size: 800
wm_pg_headroom:
packet_size: 800
wm_q_shared_lossless:
packet_size: 700
hdrm_pool_size:
packet_size: 800
xon_1:
packet_size: 800
xon_2:
packet_size: 800
xon_3:
packet_size: 800
xon_4:
packet_size: 800
lossy_queue_1:
packet_size: 1200
pkts_num_margin: 5
wm_pg_shared_lossless:
packet_size: 800
pkts_num_margin: 7
wm_pg_shared_lossy:
packet_size: 1200
pkts_num_margin: 8
wm_q_shared_lossy:
packet_size: 1200
4 changes: 2 additions & 2 deletions tests/qos/qos_sai_base.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,8 @@ class QosBase:
SUPPORTED_T1_TOPOS = ["t1-lag", "t1-64-lag", "t1-56-lag", "t1-backend", "t1-28-lag", "t1-32-lag",
"t1-isolated-d28u1"]
SUPPORTED_PTF_TOPOS = ['ptf32', 'ptf64']
SUPPORTED_ASIC_LIST = ["pac", "gr", "gr2", "gb", "td2", "th", "th2", "spc1", "spc2", "spc3", "spc4", "td3", "th3",
"j2c+", "jr2", "th5"]
SUPPORTED_ASIC_LIST = ["pac", "gr", "gr2", "gb", "td2", "th", "th2", "spc1", "spc2", "spc3", "spc4", "spc5",
"td3", "th3", "j2c+", "jr2", "th5"]

BREAKOUT_SKUS = ['Arista-7050-QX-32S']

Expand Down
Loading