Skip to content

Conversation

nasbdh9
Copy link

@nasbdh9 nasbdh9 commented Jul 30, 2025

Compile the SPD5118 module to allow users to load it.

After loading the module, as long as the system exposes the relevant I2C interface and the memory complies with the SPD5118 standard, it should be possible to read the memory module temperature.

@amotin
Copy link
Collaborator

amotin commented Jul 30, 2025

Debian's kernel config does not have this option. Wonder if it was added later, or they don't have it for a reason. Also on most of server systems things like that should be monitored by BMC, while at least on some older hardware direct access to memory controller's I2C could be dangerous.

@nasbdh9
Copy link
Author

nasbdh9 commented Jul 31, 2025

This is a new driver added in kernel 6.11, which mainly standardizes memory temperature readings. Whether SPD can be written directly in the operating system depends on the design of the memory SPD chip and has little to do with this driver.

Server systems usually have protection mechanisms such as CLTT, and the memory SPD itself is not directly exposed to the operating system.

Compile the SPD5118 module to allow users to load it.

After loading the module, as long as the system exposes the relevant I2C interface
and the memory complies with the SPD5118 standard,
it should be possible to read the memory module temperature.

Signed-off-by: Tan Zien <[email protected]>
@nasbdh9 nasbdh9 force-pushed the truenas/linux-6.12 branch from 54a9b00 to 5d6e0ae Compare July 31, 2025 01:44
@amotin
Copy link
Collaborator

amotin commented Jul 31, 2025

I am not about SPD. IIRC the bus where the sensor reside did not have proper arbitration to prevent simultaneous access by OS and CPU microcode.

@nasbdh9
Copy link
Author

nasbdh9 commented Jul 31, 2025

prevent simultaneous access by OS and CPU microcode.

I'm not sure how this relates to CPU microcode.
OS itself has automatically loaded i2c_piix4, spd5118 driver simply adds a hwmon interface to detected bus.

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