Skip to content
New issue

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

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

Already on GitHub? # to your account

Backport patches to increase critical threshold for ASIC and validate transceiver temperature #185

Merged

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented Jan 12, 2021

Backport new patches to increase the ASIC critical threshold from 110C to 140C, and validate the transceiver critical threshold temperature:

  1. 0022-mlxsw-core-Increase-critical-threshold-for-ASIC-ther.patch torvalds/linux@b06ca3d
  2. 0023-mlxsw-core-Add-validation-of-transceiver-temperature.patch torvalds/linux@57726eb

This change has been verified on all Mellanox devices based on Spectrum-1, Spectrum-2, and Spectrum-3 ASIC

    new file:   patch/0022-mlxsw-core-Increase-critical-threshold-for-ASIC-ther.patch
    modified:   patch/series

Signed-off-by: Kebo Liu <kebol@nvidia.com>
…ture

    new file:   patch/0023-mlxsw-core-Add-validation-of-transceiver-temperature.patch
    modified:   patch/series

Signed-off-by: Kebo Liu <kebol@nvidia.com>
@paulmenzel
Copy link
Contributor

It looks like, this is the same as merge/pull request #184, so this one (#185) can be closed.

@keboliu
Copy link
Collaborator Author

keboliu commented Jan 12, 2021

It looks like, this is the same as merge/pull request #184, so this one (#185) can be closed.

@paulmenzel although the content of these two sets of patches is the same, but #184 is for the 201911 branch and this one is for the master branch. Since the patch numbering is different between these two branches, so I would not expect to cherry-pick. Two different PRs are opened by intention.

@lguohan lguohan merged commit 548e8e0 into sonic-net:master Jan 12, 2021
@keboliu keboliu deleted the master-critical-thermal-threshold-fix branch January 13, 2021 01:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants