<div dir="ltr"><div>This email is about a feature request in phosphor-pid-control, that to support</div><div>optional sensors.</div><div>It is mentioned in comment of <a href="https://gerrit.openbmc-project.xyz/#/c/11003/">https://gerrit.openbmc-project.xyz/#/c/11003/</a></div><div><br></div><div>Background:</div><div>Current phosphor-pid-control checks the temperature sensors and calculate the</div><div>expected fan speed.</div><div>It assumes the temperature sensors are always available. And if the sensor is</div><div>not there, the service throws exceptions.</div><div><br></div><div>The feature request:</div><div>I would like phosphor-pid-control to support "optional" sensors, that it may</div><div>be unavailable when system is running.</div><div>Such sensor is counted when it is available, and is ignored when it is</div><div>missing.</div><div><br></div><div>An use case:</div><div>I want to adjust fan speed based on cpu core or dimm temperature.</div><div>For a typical OpenPOWER P9 system, there could be at most 48 cores, but they</div><div>can be missing depending on the CPU, e.g. if a system is plugged CPUs with 22</div><div>cores, 4 cores temperature will be missing. And in run time, a core may be</div><div>garded if it has problem it will be missing as well.</div><div>The same for dimms, for a system with at most 16 dimms, it depends on how many</div><div>dimms are attached.</div><div>In such case, the config will specify all the temperatures for cpu cores and</div><div>dimms, and phosphor-pid-control shall ignore the missing temperatures.</div><div><br></div><div>Patrick, what do you think about such case?</div><div><br></div><div>Thanks!</div></div>