On Friday 07 December 2018 20:24:49 Guenter Roeck wrote: > Anyway, I would like to get some feedback if this can cause regressions > on systems which don't support that many sensors and maybe report something > completely different if one tries to read the high-numbered sensors. > I seem to recall that there was a reason for checking the type and not just > trying to read sensor values. There can be also different problem for sensors which are turned off. E.g. on notebooks with switchable graphic cards which have included temperature sensors. When graphic card is turned off, then SMM returns error when asking for temperature value (for obvious reason). But temperature type still returns correct value "this is GPU sensors". So we cannot replace temp_type check by temp_value check. It introduce race condition between "starting GPU" and initializing dell-smm hwmon. Now linux kernel has support for dynamic turning ON/OFF switchable GPU, so we need to care about these race conditions too. -- Pali Rohár pali.rohar@xxxxxxxxx
