linux icon indicating copy to clipboard operation
linux copied to clipboard

Enable temperature sensor support for DDR5

Open nasbdh9 opened this issue 4 months ago • 5 comments

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.

nasbdh9 avatar Jul 30 '25 05:07 nasbdh9

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.

amotin avatar Jul 30 '25 17:07 amotin

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.

nasbdh9 avatar Jul 31 '25 01:07 nasbdh9

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.

amotin avatar Jul 31 '25 18:07 amotin

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.

nasbdh9 avatar Jul 31 '25 19:07 nasbdh9

This should be included in debian upstream, doesn't truenas have synced kernel config?

https://lists.debian.org/debian-kernel/2024/12/msg00237.html

nasbdh9 avatar Oct 26 '25 14:10 nasbdh9

This was resolved in the updated debian_amd64.config.

https://github.com/truenas/linux/pull/232/files#diff-3861af575dc632ce9f06be397573fc372945155bbb1b9afbf6cbcdcfa5eb96afR5349

nasbdh9 avatar Dec 09 '25 08:12 nasbdh9