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

[202211][Mellanox] Update HW-MGMT package to new version V.7.0030.1011(#16239) #16295

Merged
merged 2 commits into from
Oct 10, 2023
Merged

[202211][Mellanox] Update HW-MGMT package to new version V.7.0030.1011(#16239) #16295

merged 2 commits into from
Oct 10, 2023

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented Aug 26, 2023 •
edited
Loading

Why I did it

backport #16239

  1. Update Mellanox HW-MGMT package to newer version V.7.0030.1011
  2. Replace the SONiC PMON Thermal control algorithm with the one inside the HW-MGMT package on all Nvidia platforms
  3. Support Spectrum-4 systems
Work item tracking
  • Microsoft ADO (number only):

How I did it

  1. Update the HW-MGMT package version number and submodule pointer
  2. Remove the thermal control algorithm implementation from Mellanox platform API
  3. Revise the patch to HW-MGMT package which will disable HW-MGMT from running on SIMX
  4. Update the downstream kernel patch list

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

  • []
  • []

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@keboliu
[Mellanox] Update HW-MGMT package to new version V.7.0030.1010
e08709b 
Signed-off-by: Kebo Liu <kebol@mellanox.com>
@keboliu keboliu requested a review from lguohan as a code owner August 26, 2023 07:48
@keboliu keboliu requested a review from StormLiangMS August 26, 2023 07:51
liat-grozovik
liat-grozovik previously approved these changes Aug 28, 2023
View reviewed changes
@keboliu
Copy link
Collaborator Author

keboliu commented Aug 29, 2023

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@keboliu
Copy link
Collaborator Author

keboliu commented Aug 30, 2023

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@keboliu
Update hw-mgmt version to 7.0030.1011
e0856cf 
Signed-off-by: Kebo Liu <kebol@nvidia.com>
@keboliu keboliu changed the title [202211][Mellanox] Update HW-MGMT package to new version V.7.0030.1010(#16239) [202211][Mellanox] Update HW-MGMT package to new version V.7.0030.1011(#16239) Aug 31, 2023
@liat-grozovik
Copy link
Collaborator

@StormLiangMS can you please help to merge? pls confirm the dependency is included as well

@keboliu
Copy link
Collaborator Author

keboliu commented Sep 24, 2023

@StormLiangMS would you please help to merge this PR?

@StormLiangMS
Copy link
Contributor

enhancement, but only take effect on Mellanox platform.

@keboliu
Copy link
Collaborator Author

keboliu commented Sep 26, 2023

enhancement, but only take effect on Mellanox platform.

Hi Storm, this new HW-MGMT is to support Mellanox SPC4 platforms, w/o it SPC4 won't work in this branch.

StormLiangMS
StormLiangMS approved these changes Oct 10, 2023
View reviewed changes
Copy link
Contributor

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS merged commit b56b34a into sonic-net:202211 Oct 10, 2023
@keboliu keboliu deleted the 202211_hw_mgmt_7.0030.1010 branch October 10, 2023 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@StormLiangMS StormLiangMS StormLiangMS approved these changes

@liat-grozovik liat-grozovik liat-grozovik left review comments

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

Assignees
No one assigned
Labels
Platform: Mellanox
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@keboliu @mssonicbld @liat-grozovik @StormLiangMS