* Plans regarding PECI sensors and PCIe inventory with upstream Linux @ 2023-03-22 13:14 Paul Fertser 2023-03-24 21:38 ` Winiarska, Iwona 0 siblings, 1 reply; 3+ messages in thread From: Paul Fertser @ 2023-03-22 13:14 UTC (permalink / raw) To: openbmc Hello, Recently OpenBMC switched to newer Linux version which got upstream support for the PECI bus, with CPU and DIMM modules temperature sensors reported via it. I assume this is essential for all Intel-based server motherboards as monitoring CPU temperature and spinning fans accordingly is one of the core BMC functionalities. However, I wasn't able to find[0] any sign that this code can actually be used in OpenBMC. intelcpusensor daemon exits early as the upstream code doesn't implement the IOCTLs for direct userspace bus operations. Is there any work planned to make it compatible, or was some other way chosen? Another related question is that peci-pcie daemon used for enumerating host PCIe devices can no longer be used due to the same reason, is there some replacement in the works? Where can I track its development? [0] https://gerrit.openbmc.org/q/project:openbmc/dbus-sensors+message:intelcpusensor -- Be free, use free (http://www.gnu.org/philosophy/free-sw.html) software! mailto:fercerpav@gmail.com ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Plans regarding PECI sensors and PCIe inventory with upstream Linux 2023-03-22 13:14 Plans regarding PECI sensors and PCIe inventory with upstream Linux Paul Fertser @ 2023-03-24 21:38 ` Winiarska, Iwona 2023-03-29 13:55 ` Paul Fertser 0 siblings, 1 reply; 3+ messages in thread From: Winiarska, Iwona @ 2023-03-24 21:38 UTC (permalink / raw) To: openbmc, fercerpav; +Cc: Weiss, Zev On Wed, 2023-03-22 at 16:14 +0300, Paul Fertser wrote: > Hello, > > Recently OpenBMC switched to newer Linux version which got upstream > support for the PECI bus, with CPU and DIMM modules temperature > sensors reported via it. I assume this is essential for all > Intel-based server motherboards as monitoring CPU temperature and > spinning fans accordingly is one of the core BMC functionalities. > > However, I wasn't able to find[0] any sign that this code can actually > be used in OpenBMC. intelcpusensor daemon exits early as the upstream > code doesn't implement the IOCTLs for direct userspace bus > operations. Is there any work planned to make it compatible, or was > some other way chosen? Yes, it's currently not compatible with upstream PECI. In January there was a thread related to dbus-sensors present and future: https://lore.kernel.org/openbmc/Y79U52toP0+Y4edh@hatter.bewilderbeest.net/ where Zev mentioned that he started to work on dbus-sensors replacement, which does support (among other things) upstream PECI: https://github.com/zevweiss/omnisensor/blob/master/src/peci.rs > > Another related question is that peci-pcie daemon used for enumerating > host PCIe devices can no longer be used due to the same reason, is > there some replacement in the works? Where can I track its > development? When it comes to pecidev, I didn't plan to have it available upstream. From my perspective - the uAPI exposed by pecidev is too low-level to be used for anything other than development purpose. It would also be very tricky to support certain PECI features reliably (such as wake on peci) in the future if pecidev was available. Regarding peci-pcie, the plan would be to, instead of using pecidev, implement a PECI driver on the kernel side, which exposes the necessary information over sysfs (abstracting away the PECI commands used internally) and use that in peci- pcie. Thanks -Iwona > > [0] > https://gerrit.openbmc.org/q/project:openbmc/dbus-sensors+message:intelcpusensor > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Plans regarding PECI sensors and PCIe inventory with upstream Linux 2023-03-24 21:38 ` Winiarska, Iwona @ 2023-03-29 13:55 ` Paul Fertser 0 siblings, 0 replies; 3+ messages in thread From: Paul Fertser @ 2023-03-29 13:55 UTC (permalink / raw) To: Winiarska, Iwona; +Cc: openbmc, Weiss, Zev Hello Iwona, On Fri, Mar 24, 2023 at 09:38:04PM +0000, Winiarska, Iwona wrote: > On Wed, 2023-03-22 at 16:14 +0300, Paul Fertser wrote: > > Recently OpenBMC switched to newer Linux version which got upstream > > support for the PECI bus, with CPU and DIMM modules temperature > > sensors reported via it. I assume this is essential for all > > Intel-based server motherboards as monitoring CPU temperature and > > spinning fans accordingly is one of the core BMC functionalities. > > Yes, it's currently not compatible with upstream PECI. > > In January there was a thread related to dbus-sensors present and future: > https://lore.kernel.org/openbmc/Y79U52toP0+Y4edh@hatter.bewilderbeest.net/ > where Zev mentioned that he started to work on dbus-sensors replacement, which > does support (among other things) upstream PECI: > > https://github.com/zevweiss/omnisensor/blob/master/src/peci.rs It's a rather cool project, but I do not see it having become part of OpenBMC mainstream just yet. Probably not many have tried even just testing it, as I only see commits from Zev himself. I hope this will change in the future. For now I figured a simple change seems to be enough to allow IntelCPUSensor from dbus-sensors to continue working on current OpenBMC: https://gerrit.openbmc.org/c/openbmc/dbus-sensors/+/62025 > Regarding peci-pcie, the plan would be to, instead of using pecidev, implement a > PECI driver on the kernel side, which exposes the necessary information over > sysfs (abstracting away the PECI commands used internally) and use that in peci- > pcie. When I was asking about plans I meant something with specific dates, not just the general outline :) You can see as it currently stands OpenBMC upstream lost the very essential feature (fan control depending on CPU temperature) for most (all?) Intel-based servers, so I'm surprised there's no particular migration path ready in advance. That said, thank you for the upstreaming work, I see it was attempted before for years and you're the one who finally managed to do the right (from Linux maintainers PoV) thing. Congratulations! -- Be free, use free (http://www.gnu.org/philosophy/free-sw.html) software! mailto:fercerpav@gmail.com ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-03-29 13:56 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-03-22 13:14 Plans regarding PECI sensors and PCIe inventory with upstream Linux Paul Fertser 2023-03-24 21:38 ` Winiarska, Iwona 2023-03-29 13:55 ` Paul Fertser
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.