Skip to content

Conversation

@roger530-ho
Copy link
Collaborator

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

  1. If temperature reading from the MAC or CPU, the APIs will return a fallback value that causes the fan to operate at 100% speed. Default to reading MAC temperature from SDK; if it fails, fall back to FPGA reading and apply a +5°C offset for compensation.

  2. If the BMC supports thermal policy, the system reports temperature readings to the BMC at 10-second intervals.

  3. If the BMC becomes unresponsive or fails, thermal control falls back to CPU-based monitoring

  4. Update the transceiver temperature threshold from 75°C to 70°C.

  5. Refactored to modular class-based architecture with dual BMC/CPU thermal control paths.

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)

1. If temperature reading from the MAC or CPU, the APIs will return a
   fallback value that causes the fan to operate at 100% speed.
   Default to reading MAC temperature from SDK; if it fails, fall back
   to FPGA reading and apply a +5°C offset for compensation.

2. If the BMC supports thermal policy, the system reports temperature
   readings to the BMC at 10-second intervals.

3. If the BMC becomes unresponsive or fails, thermal control falls
   back to CPU-based monitoring

4. Update the transceiver temperature threshold from 75°C to 70°C.

5. Refactored to modular class-based architecture with
   dual BMC/CPU thermal control paths.
@roger530-ho roger530-ho requested a review from PJHsieh January 5, 2026 03:32
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.

1 participant