Gitiles
Code Review Sign In
gerrit.openbmc.org / openbmc / phosphor-host-ipmid / ee3983b0c235af45f98e5ac6f638c3d35091cd41
commitee3983b0c235af45f98e5ac6f638c3d35091cd41[log] [tgz]
authorChau Ly <chaul@amperecomputing.com>Fri Jan 05 07:33:45 2024 +0000
committerChauLy-Vietnam <chaul@amperecomputing.com>Mon Mar 03 02:35:26 2025 +0000
tree8a499626535dd9844939e49cc0b03300300101e0
parentc6837ed72897291ef6ee50bfe81962478090450c [diff]
Correct chassis's PowerState reading condition

phosphor-dbus-interfaces defines 4 values for PowerState property under
xyz.openbmc_project.State.Chassis interface [1]. The current logic
causes all the other values except for
`xyz.openbmc_project.State.Chassis.PowerState.On` to be interpreted as
`Off` in `ipmitool power status`. This patch supports interpreting
CurrentPowerState value to adapt all states following the logic in the
Tested section below.

Tested:
`D-Bus value ends with` <=> `$ipmitool power status`
*.Off <=> Chassis Power is off
*.TransitioningToOn <=> Chassis Power is off
*.On <=> Chassis Power is on
*.TransitioningToOff <=> Chassis Power is on

[1]: https://github.com/openbmc/phosphor-dbus-interfaces/blob/master/yaml/xyz/openbmc_project/State/Chassis.interface.yaml

Signed-off-by: Chau Ly <chaul@amperecomputing.com>
Change-Id: Ic8e9f0a10b061137126ed3e24ad71e3bf80a5a34
1 file changed
tree: 8a499626535dd9844939e49cc0b03300300101e0
  1. app/
  2. dbus-sdr/
  3. docs/
  4. include/
  5. libipmid/
  6. scripts/
  7. softoff/
  8. subprojects/
  9. test/
  10. user_channel/
  11. xyz/
  12. .build.sh
  13. .clang-format
  14. .gitignore
  15. .shellcheck
  16. .travis.yml
  17. apphandler.cpp
  18. apphandler.hpp
  19. chassishandler.cpp
  20. chassishandler.hpp
  21. dcmihandler.cpp
  22. dcmihandler.hpp
  23. error-HostEvent.hpp
  24. fruread.hpp
  25. generate_whitelist.sh
  26. generate_whitelist_create.sh
  27. globalhandler.cpp
  28. groupext.cpp
  29. host-cmd-manager.cpp
  30. host-cmd-manager.hpp
  31. host-interface.cpp
  32. host-interface.hpp
  33. host-ipmid-whitelist.conf
  34. ipmi_fru_info_area.cpp
  35. ipmi_fru_info_area.hpp
  36. ipmiallowlist.hpp
  37. ipmid-new.cpp
  38. ipmisensor.cpp
  39. LICENSE
  40. meson.build
  41. meson.options
  42. OWNERS
  43. read_fru_data.cpp
  44. read_fru_data.hpp
  45. README.md
  46. selutility.cpp
  47. selutility.hpp
  48. sensordatahandler.cpp
  49. sensordatahandler.hpp
  50. sensorhandler.cpp
  51. sensorhandler.hpp
  52. settings.cpp
  53. settings.hpp
  54. storageaddsel.cpp
  55. storageaddsel.hpp
  56. storagehandler.cpp
  57. sys_info_param.cpp
  58. sys_info_param.hpp
  59. systemintfcmds.cpp
  60. systemintfcmds.hpp
  61. transportconstants.hpp
  62. transporthandler.cpp
  63. transporthandler.hpp
  64. whitelist-filter.cpp
README.md

phosphor-host-ipmid

Compile ipmid with default options

meson builddir
ninja -C builddir

Compile ipmid with yocto defaults

meson builddir -Dbuildtype=minsize -Db_lto=true -Dtests=disabled
ninja -C builddir

If any of the dependencies are not found on the host system during configuration, meson automatically gets them via its wrap dependencies mentioned in ipmid/subprojects.

Enable/Disable meson wrap feature

meson builddir -Dwrap_mode=nofallback
ninja -C builddir

Enable debug traces

meson builddir -Dbuildtype=debug
ninja -C builddir

Generate test coverage report

meson builddir -Db_coverage=true -Dtests=enabled
ninja -C builddir test
ninja -C builddir coverage