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.

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

[202012][Mellanox]Fix lpmode set when logical port is larger than 64 (#14138) #14202

Merged
merged 1 commit into from
Mar 14, 2023

Commits on Mar 10, 2023

  1. [Mellanox]Fix lpmode set when logical port is larger than 64 (sonic-n…

    …et#14138)
    
    - Why I did it
    In sfplpm API, the number of logical ports is hardcoded as 64. When a system contains more port than this, the SDK APIs would fail with a syslog as below
    
    Mar 7 03:53:58.105980 r-leopard-58 ERR syncd#SDK: [MGMT_LIB.ERR] Slot [0] Module [0] has logport [0x00010069] in enabled state
    Mar 7 03:53:58.105980 r-leopard-58 ERR syncd#SDK: [SDK_MGMT_LIB.ERR] Failed in __sdk_mgmt_phy_module_pwr_attr_set, error: Internal Error
    Mar 7 03:53:58.106118 r-leopard-58 ERR pmon#-c: Error occurred when setting power mode for SFP module 0, slot 0, error code 1
    
    - How I did it
    Remove the hardcoded value of 64. Obtained the number of logical ports from SDK
    
    - How to verify it
    Manual testing
    dgsudharsan committed Mar 10, 2023
    Configuration menu
    Copy the full SHA
    207782f View commit details
    Browse the repository at this point in the history