Skip to content

Differentiate between copper cables and optical modules when updating pre-emphasis #599

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

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Differentiate between copper cables and optical modules when updating pre-emphasis #599

Show file tree
Hide file tree
Changes from 12 commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
d1cac5e
Differentiate between copper cables and optical modules when updating…
micas-net Mar 20, 2025
272332f
fix test_xcvrd.py
micas-net Mar 21, 2025
6f95cdc
Update test_xcvrd.py
micas-net Mar 21, 2025
a6bb6c3
Update test_xcvrd.py
micas-net Mar 21, 2025
5a98f47
Update test_xcvrd.py
micas-net Mar 21, 2025
2193180
Update test_xcvrd.py
micas-net Mar 21, 2025
b097d55
Update test_xcvrd.py
micas-net Mar 21, 2025
39a7873
update tests/media_settings.json
micas-net Mar 21, 2025
e109ca0
Update media_settings.json
micas-net Mar 21, 2025
57bf849
Update media_settings.json
micas-net Mar 21, 2025
b4c98a3
Update test_xcvrd.py
micas-net Mar 21, 2025
70dfa5c
Update test_xcvrd.py
micas-net Mar 21, 2025
9e7065d
Update media_settings_parser.py
micas-net May 15, 2025
8127a15
Merge branch 'master' into Differentiate-between-copper-cables-and-op…
micas-net May 15, 2025
1d1aaf9
Update test_xcvrd.py
micas-net May 15, 2025
f2aa7c8
Update test_xcvrd.py
micas-net May 15, 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
22 changes: 22 additions & 0 deletions sonic-xcvrd/tests/media_settings.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -436,6 +436,28 @@
"lane3":"0x144808"
}
}
},
"33": {
"fiber": {
"speed:50G": {
"preemphasis": {
"lane0":"0x144808",
"lane1":"0x144808",
"lane2":"0x144808",
"lane3":"0x144808"
}
}
},
"copper": {
"speed:50G": {
"preemphasis": {
"lane0":"0x144808",
"lane1":"0x144808",
"lane2":"0x144808",
"lane3":"0x144808"
}
}
}
}
}
}
Expand Down
22 changes: 16 additions & 6 deletions sonic-xcvrd/tests/test_xcvrd.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,10 @@
global_media_settings = media_settings_with_comma_dict['GLOBAL_MEDIA_SETTINGS'].pop('1-32')
media_settings_with_comma_dict['GLOBAL_MEDIA_SETTINGS']['1-5,6,7-20,21-32'] = global_media_settings

port_settings_with_comma_dict = copy.deepcopy(media_settings_dict)
port_media_settings = port_settings_with_comma_dict['PORT_MEDIA_SETTINGS'].pop('33')
port_settings_with_comma_dict['PORT_MEDIA_SETTINGS']['33'] = port_media_settings

media_settings_with_regular_expression_dict = copy.deepcopy(media_settings_dict)
media_settings_with_regular_expression_dict['GLOBAL_MEDIA_SETTINGS']['1-32'] = {}
# Generate regular expression patterns for QSFP28-40GBASE-CR4-xxM and QSFP+-40GBASE-CR4-xxM that have the same pre-emphasis value
Expand Down Expand Up @@ -982,18 +986,20 @@ def test_get_media_settings_key(self, mock_is_cmis_api, mock_chassis):
'cable_type': 'Length Cable Assembly(m)',
'cable_length': '255',
'specification_compliance': "{'10/40G Ethernet Compliance Code': '10GBase-SR'}",
'type_abbrv_name': 'QSFP+'
'type_abbrv_name': 'QSFP+',
'media_type_key': 'fiber'
}
}

# Test a good 'specification_compliance' value
result = media_settings_parser.get_media_settings_key(0, xcvr_info_dict, 100000, 2)
assert result == { 'vendor_key': 'MOLEX-1064141421', 'media_key': 'QSFP+-10GBase-SR-255M', 'lane_speed_key': 'speed:50G'}
assert result == { 'vendor_key': 'MOLEX-1064141421', 'media_key': 'QSFP+-10GBase-SR-255M', 'lane_speed_key': 'speed:50G', 'media_type_key': 'fiber'}

# Test a bad 'specification_compliance' value
xcvr_info_dict[0]['specification_compliance'] = 'N/A'
xcvr_info_dict[0]['media_type_key'] = 'copper'
result = media_settings_parser.get_media_settings_key(0, xcvr_info_dict, 100000, 2)
assert result == { 'vendor_key': 'MOLEX-1064141421', 'media_key': 'QSFP+-*', 'lane_speed_key': 'speed:50G'}
assert result == { 'vendor_key': 'MOLEX-1064141421', 'media_key': 'QSFP+-*', 'lane_speed_key': 'speed:50G', 'media_type_key': 'copper'}
# TODO: Ensure that error message was logged

xcvr_info_dict_for_qsfp28 = {
Expand All @@ -1015,6 +1021,7 @@ def test_get_media_settings_key(self, mock_is_cmis_api, mock_chassis):
"vendor_date": "2020-11-11",
"vendor_oui": "00-77-7a",
"application_advertisement": "N/A",
"media_type_key": 'copper',
}
}
result = media_settings_parser.get_media_settings_key(
Expand All @@ -1024,6 +1031,7 @@ def test_get_media_settings_key(self, mock_is_cmis_api, mock_chassis):
"vendor_key": "AVAGO-XXX-YYY-ZZZ",
"media_key": "QSFP28-100GBASE-SR4 or 25GBASE-SR-50.0M",
"lane_speed_key": "speed:25G",
'media_type_key': 'copper',
}

mock_is_cmis_api.return_value = True
Expand All @@ -1034,7 +1042,8 @@ def test_get_media_settings_key(self, mock_is_cmis_api, mock_chassis):
'cable_type': 'Length Cable Assembly(m)',
'cable_length': '255',
'specification_compliance': "sm_media_interface",
'type_abbrv_name': 'QSFP-DD'
'type_abbrv_name': 'QSFP-DD',
"media_type_key": 'copper'
}
}

Expand All @@ -1051,7 +1060,7 @@ def test_get_media_settings_key(self, mock_is_cmis_api, mock_chassis):

mock_api.get_application_advertisement = MagicMock(return_value=mock_app_adv_value)
result = media_settings_parser.get_media_settings_key(0, xcvr_info_dict, 100000, 2)
assert result == { 'vendor_key': 'MOLEX-1064141421', 'media_key': 'QSFP-DD-sm_media_interface', 'lane_speed_key': 'speed:100GBASE-CR2' }
assert result == { 'vendor_key': 'MOLEX-1064141421', 'media_key': 'QSFP-DD-sm_media_interface', 'lane_speed_key': 'speed:100GBASE-CR2' ,'media_type_key': 'copper'}

@pytest.mark.parametrize("data_found, data, expected", [
(True, [('speed', '400000'), ('lanes', '1,2,3,4,5,6,7,8'), ('mtu', '9100')], (400000, 8, 0)),
Expand Down Expand Up @@ -1153,7 +1162,8 @@ def test_is_si_per_speed_supported(self):
(media_settings_empty, 7, {'vendor_key': 'AMPHANOL-5678', 'media_key': 'QSFP-DD-active_cable_media_interface', 'lane_speed_key': 'speed:100GAUI-2'}, {}),
(media_settings_with_regular_expression_dict, 7, {'vendor_key': 'UNKOWN', 'media_key': 'QSFP28-40GBASE-CR4-1M', 'lane_speed_key': 'UNKOWN'}, {'preemphasis': {'lane0': '0x16440A', 'lane1': '0x16440A', 'lane2': '0x16440A', 'lane3': '0x16440A'}}),
(media_settings_with_regular_expression_dict, 7, {'vendor_key': 'UNKOWN', 'media_key': 'QSFP+-40GBASE-CR4-2M', 'lane_speed_key': 'UNKOWN'}, {'preemphasis': {'lane0': '0x18420A', 'lane1': '0x18420A', 'lane2': '0x18420A', 'lane3': '0x18420A'}}),
(media_settings_with_regular_expression_dict, 7, {'vendor_key': 'UNKOWN', 'media_key': 'QSFP+-40GBASE-CR4-10M', 'lane_speed_key': 'UNKOWN'}, {'preemphasis': {'lane0': '0x1A400A', 'lane1': '0x1A400A', 'lane2': '0x1A400A', 'lane3': '0x1A400A'}})
(media_settings_with_regular_expression_dict, 7, {'vendor_key': 'UNKOWN', 'media_key': 'QSFP+-40GBASE-CR4-10M', 'lane_speed_key': 'UNKOWN'}, {'preemphasis': {'lane0': '0x1A400A', 'lane1': '0x1A400A', 'lane2': '0x1A400A', 'lane3': '0x1A400A'}}),
(port_settings_with_comma_dict, 33, {'vendor_key': 'UNKOWN', 'media_key': 'UNKOWN', 'lane_speed_key': 'speed:50G', 'media_type_key': 'fiber'}, {'preemphasis': {'lane0': '0x144808', 'lane1': '0x144808', 'lane2': '0x144808', 'lane3': '0x144808'}})
])
def test_get_media_settings_value(self, media_settings_dict, port, key, expected):
with patch('xcvrd.xcvrd_utilities.media_settings_parser.g_dict', media_settings_dict):
Expand Down
27 changes: 26 additions & 1 deletion sonic-xcvrd/xcvrd/xcvrd_utilities/media_settings_parser.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
VENDOR_KEY = 'vendor_key'
MEDIA_KEY = 'media_key'
LANE_SPEED_KEY = 'lane_speed_key'
MEDIA_TYE_KEY = 'media_type_key'
DEFAULT_KEY = 'Default'
# This is useful if default value is desired when no match is found for lane speed key
LANE_SPEED_DEFAULT_KEY = LANE_SPEED_KEY_PREFIX + DEFAULT_KEY
Expand Down Expand Up @@ -138,12 +139,27 @@ def get_media_settings_key(physical_port, transceiver_dict, port_speed, lane_cou
else:
media_key += '-' + '*'

media_type_ex = "fiber"
Copy link
Collaborator

@prgeor prgeor Mar 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@philo-micas why we need this change? I mean what is the current limitation in existing parser?

Copy link
Author

@micas-net micas-net Mar 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The port may support both copper cables and optical modules, but the pre-emphasis of copper cables and optical modules may be different. the purpose of adding this is to solve the problem that the device port cannot distinguish the pre-emphasis value of copper cables and optical modules.

try:
if (type(media_len) != int) and (type(media_len) != float):
media_len = None
for key, value in transceiver_dict[physical_port].items():
if type(value) != str:
continue
if value.lower().find("copper") >=0:
media_type_ex = "copper"
break
except Exception as e:
helper_logger.log_error("Invalid value for port {}: {}".format(physical_port, str(e)))
helper_logger.log_error("media_type {}".format(media_type_ex))

lane_speed_key = get_lane_speed_key(physical_port, port_speed, lane_count)
# return (vendor_key, media_key, lane_speed_key)
return {
VENDOR_KEY: vendor_key,
MEDIA_KEY: media_key,
LANE_SPEED_KEY: lane_speed_key
LANE_SPEED_KEY: lane_speed_key,
MEDIA_TYE_KEY: media_type_ex
}


Expand Down Expand Up @@ -263,6 +279,7 @@ def get_media_settings(key, media_dict):
for keys in g_dict[PORT_MEDIA_SETTINGS_KEY]:
if int(keys) == physical_port:
media_dict = g_dict[PORT_MEDIA_SETTINGS_KEY][keys]
helper_logger.log_notice("port {} media_dict {}".format(physical_port, media_dict))
break

if len(media_dict) == 0:
Expand All @@ -280,6 +297,14 @@ def get_media_settings(key, media_dict):
return get_media_settings_for_speed(media_dict[DEFAULT_KEY], lane_speed_key)
elif len(default_dict) != 0:
return default_dict
else:
if "copper" in key[MEDIA_TYE_KEY]:
media_dict = media_dict["copper"]
else:
media_dict = media_dict["fiber"]
helper_logger.log_notice("media_settings lane speed {}".format(lane_speed_key))
helper_logger.log_notice("port {} media_dict {}".format(physical_port, media_dict))
return get_media_settings_for_speed(media_dict, lane_speed_key)
else:
if len(default_dict) != 0:
return default_dict
Expand Down
Loading