Skip to content

[Mellanox] Use hw-mgmt thermal control service#127

Closed
Junchao-Mellanox wants to merge 1 commit intomasterfrom
hw-mgmt-thermal-control
Closed

[Mellanox] Use hw-mgmt thermal control service#127
Junchao-Mellanox wants to merge 1 commit intomasterfrom
hw-mgmt-thermal-control

Conversation

@Junchao-Mellanox
Copy link
Copy Markdown
Owner

@Junchao-Mellanox Junchao-Mellanox commented Feb 9, 2022

Why I did it

Use hw-mgmt thermal control service on Mellanox platform.

Benefit:

  1. hw-mgmt thermal control works close with kernel
  2. hw-mgmt thermal control has no extra dependency while SONiC thermal control depends on PMON
  3. hw-mgmt thermal control can work at real time while SONiC thermal control works every 1 minute
  4. hw-mgmt thermal control takes effect right after system boot while SONiC thermal control works after PMON is up

Impact:

  1. No code change for thermalctld
  2. No code change for CLI
  3. No code change for platform API
  4. No code change for other vendor
  5. No impact to thermal/fan data collecting function

How I did it

  1. Remove thermal policies from thermal control configuration file
  2. Enable hw-mgmt thermal control service by default

How to verify it

Manual test on 4700

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

@Junchao-Mellanox Junchao-Mellanox deleted the hw-mgmt-thermal-control branch June 12, 2023 04:40
Junchao-Mellanox pushed a commit that referenced this pull request Jan 12, 2024
…automatically (sonic-net#17753)

#### Why I did it
src/sonic-mgmt-framework
```
* ca0656c - (HEAD -> master, origin/master, origin/HEAD) remove tls cipher suite settings (#127) (12 hours ago) [Mai Bui]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Junchao-Mellanox pushed a commit that referenced this pull request Mar 7, 2025
…atically (sonic-net#671)

#### Why I did it
src/sonic-utilities
```
* 57957a8a - (HEAD -> 202412, origin/202412) [code sync] Merge code from sonic-net/sonic-utilities:202411 to 202412 (#127) (55 minutes ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Junchao-Mellanox pushed a commit that referenced this pull request Nov 24, 2025
…ically (sonic-net#24435)

#### Why I did it
src/sonic-dash-ha
```
* 149774b - (HEAD -> master, origin/master, origin/HEAD) Add a PR template (#129) (3 days ago) [yue-fred-gao]
* 711a0be - Sort the output of show actor command (#128) (3 days ago) [yue-fred-gao]
* 15fbd5c - Fix format for protobuf fields in show command. (#127) (3 days ago) [dypet]
* f2d47b9 - Add wait for loopback script. (#126) (3 days ago) [dypet]
* 570db05 - Fix issue #118: use hostname to build service path (#122) (3 days ago) [yue-fred-gao]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants