Gitiles
Code Review Sign In
gerrit.openbmc.org / openbmc / phosphor-host-ipmid / 8ba1d87405365a202dea8e3d73e40475d0b450dc
commit8ba1d87405365a202dea8e3d73e40475d0b450dc[log] [tgz]
authorAndrew Geissler <geissonator@yahoo.com>Mon May 03 13:06:35 2021 -0500
committerAndrew Geissler <geissonator@yahoo.com>Fri May 07 21:12:30 2021 +0000
tree46f7c93e51562bdf4a8626ce0c53bf70cfb9f56d
parenta55c953dd2bba16f2d43b186f920b18da4a3493f [diff]
host-condition: reduce timeout from 30 to 3 sec

Some new software is on the way to refactor the way OpenBMC firmware
detects if the host is running. Future commits in this series will
elaborate on this.

A long running observation of this existing design is that it never
takes the host 30 seconds to respond if it's actually running. In fact
it's always less then one second if it's actually running because ths
host is much faster then the BMC hardware. The 30s timeout has always
caused confusion because of how long it takes when the host is clearly
not running.

A new phosphor-dbus-interface has moved the interface to determine if
the host is up and running into a simple property read. To implement
that property read, we need to ensure this command returns as quickly as
possible.

This commit reduces the time we give the host to respond to a heartbeat
command from 30 to 3 seconds.

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Id288f929a9948da84687dd4455b697263dcf6ac2
1 file changed
tree: 46f7c93e51562bdf4a8626ce0c53bf70cfb9f56d
  1. app/
  2. dbus-sdr/
  3. docs/
  4. include/
  5. libipmid/
  6. libipmid-host/
  7. scripts/
  8. softoff/
  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. bootstrap.sh
  20. chassishandler.cpp
  21. chassishandler.hpp
  22. configure.ac
  23. dcmihandler.cpp
  24. dcmihandler.hpp
  25. elog-errors.hpp
  26. entity_map_json.cpp
  27. entity_map_json.hpp
  28. error-HostEvent.hpp
  29. fruread.hpp
  30. generate_whitelist.sh
  31. globalhandler.cpp
  32. globalhandler.hpp
  33. groupext.cpp
  34. host-cmd-manager.cpp
  35. host-cmd-manager.hpp
  36. host-interface.cpp
  37. host-interface.hpp
  38. host-ipmid-whitelist.conf
  39. ipmi_fru_info_area.cpp
  40. ipmi_fru_info_area.hpp
  41. ipmid-new.cpp
  42. ipmisensor.cpp
  43. ipmiwhitelist.hpp
  44. LICENSE
  45. MAINTAINERS
  46. Makefile.am
  47. read_fru_data.cpp
  48. read_fru_data.hpp
  49. README.md
  50. sample.cpp
  51. sample.h
  52. selutility.cpp
  53. selutility.hpp
  54. sensordatahandler.cpp
  55. sensordatahandler.hpp
  56. sensorhandler.cpp
  57. sensorhandler.hpp
  58. settings.cpp
  59. settings.hpp
  60. storageaddsel.cpp
  61. storageaddsel.hpp
  62. storagehandler.cpp
  63. storagehandler.hpp
  64. sys_info_param.cpp
  65. sys_info_param.hpp
  66. systemintfcmds.cpp
  67. systemintfcmds.hpp
  68. testaddsel.cpp
  69. testit.cpp
  70. transporthandler.cpp
  71. transporthandler.hpp
  72. whitelist-filter.cpp
README.md

To build this package, do the following steps:

1. ./bootstrap.sh
2. ./configure ${CONFIGURE_FLAGS}
3. make

To clean the repository run ./bootstrap.sh clean.