Skip to content

Add dynamic minimum fan speed support#4356

Closed
Junchao-Mellanox wants to merge 20 commits intosonic-net:masterfrom
Junchao-Mellanox:dy-min-speed
Closed

Add dynamic minimum fan speed support#4356
Junchao-Mellanox wants to merge 20 commits intosonic-net:masterfrom
Junchao-Mellanox:dy-min-speed

Conversation

@Junchao-Mellanox
Copy link
Collaborator

- What I did

  1. Add a fan fault policy to check fan fault. If any fan fault, set fan speed to 100% and disable kernel algorithm
  2. Add dynamic minimum fan speed support, minimum fan speed will be determined by air direction, highest temperature and sensor trust state.

- How I did it

  1. Add a condition to check fan fault and change the default policy file
  2. Add a condition to check if dynamic minimum fan speed need change and an action to change minimum fan speed
  3. Add a policy to update dynamic minimum fan speed every 60 seconds, this policy is private that user cannot configure it

- How to verify it

- Description for the changelog

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

Junchao-Mellanox and others added 20 commits March 11, 2020 20:39
Conflicts:
	device/mellanox/x86_64-mlnx_msn2700-r0/plugins/sfputil.py
	platform/mellanox/mlnx-platform-api/sonic_platform/thermal.py
Conflicts:
	platform/mellanox/mlnx-platform-api/sonic_platform/psu.py
Conflicts:
	platform/mellanox/mlnx-platform-api/sonic_platform/psu.py
…et speed; 3. check thermal zone temperature before set speed; 4. enable thermal algorithm by change thermal_zone_mode
Conflicts:
	platform/mellanox/mlnx-platform-api/sonic_platform/thermal.py
	platform/mellanox/mlnx-platform-api/tests/test_thermal_policy.py
mssonicbld added a commit that referenced this pull request Mar 16, 2026
…atically (#26184)

#### Why I did it
src/sonic-utilities
```
* caf471a5 - (HEAD -> 202511, origin/202511) Wait for monit monitor <service> operation to complete during config (#4325) (52 minutes ago) [Hemanth Kumar Tirupati]
* 6a9dcc78 - Add alternate TH5 hwsku names to gcu (#4357) (2 days ago) [mssonicbld]
* 0017a8d4 - [fibshow]: Fix exception when blackhole routes are present (#4356) (2 days ago) [mssonicbld]
```
#### 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