Gitiles
Code Review Sign In
gerrit.openbmc.org / openbmc / phosphor-host-ipmid / bae471cb4e9faeaf183664252b51e8c39d71fc95
commitbae471cb4e9faeaf183664252b51e8c39d71fc95[log] [tgz]
authorWilliam A. Kennington III <wak@google.com>Fri Jun 15 10:38:01 2018 -0700
committerEmily Shaffer <emilyshaffer@google.com>Fri Sep 21 21:18:06 2018 +0000
tree232536e41160a071d53760936968ba2e0af8ab5f
parent1f4bc1f079243df22aa1d47e82625620ff2de634 [diff]
app/watchdog: Only log internal failures once

Logging internal failures to the phosphor-logger is a slow process that
can take up to 5 seconds. If we do this for each watchdog reset for a
relatively fast watchdog we will clog up the channel and fill the ESEL
log with errors. We probably don't want that kind of information to
build up. Instead, only report errors to the ESEL when the state of the
internal watchdog transitions from HEALTHY -> UNHEALTHY.

Tested:
    Works on zaius when booting and terminating the watchdog daemon
    while it is ticking. Produces the expected single error in the log
    and only delays the IPMI channel once.

Change-Id: I8110ba66c4a85e188666a34cb6d055bdbec30622
Signed-off-by: William A. Kennington III <wak@google.com>
1 file changed
tree: 232536e41160a071d53760936968ba2e0af8ab5f
  1. app/
  2. docs/
  3. host-ipmid/
  4. scripts/
  5. softoff/
  6. test/
  7. xyz/
  8. .build.sh
  9. .clang-format
  10. .gitignore
  11. .travis.yml
  12. apphandler.cpp
  13. apphandler.h
  14. bootstrap.sh
  15. chassishandler.cpp
  16. chassishandler.h
  17. configure.ac
  18. dcmihandler.cpp
  19. dcmihandler.hpp
  20. elog-errors.hpp
  21. error-HostEvent.hpp
  22. fruread.hpp
  23. generate_whitelist.sh
  24. globalhandler.cpp
  25. globalhandler.h
  26. groupext.cpp
  27. host-cmd-manager.cpp
  28. host-cmd-manager.hpp
  29. host-interface.cpp
  30. host-interface.hpp
  31. host-ipmid-whitelist.conf
  32. ipmi_fru_info_area.cpp
  33. ipmi_fru_info_area.hpp
  34. ipmid.cpp
  35. ipmid.hpp
  36. ipmisensor.cpp
  37. ipmiwhitelist.hpp
  38. LICENSE
  39. MAINTAINERS
  40. Makefile.am
  41. net.cpp
  42. net.hpp
  43. oemrouter.cpp
  44. read_fru_data.cpp
  45. read_fru_data.hpp
  46. README.md
  47. sample.cpp
  48. sample.h
  49. selutility.cpp
  50. selutility.hpp
  51. sensordatahandler.cpp
  52. sensordatahandler.hpp
  53. sensorhandler.cpp
  54. sensorhandler.h
  55. settings.cpp
  56. settings.hpp
  57. storageaddsel.cpp
  58. storageaddsel.h
  59. storagehandler.cpp
  60. storagehandler.h
  61. systemintfcmds.cpp
  62. systemintfcmds.h
  63. testaddsel.cpp
  64. testit.cpp
  65. timer.cpp
  66. timer.hpp
  67. transporthandler.cpp
  68. transporthandler.hpp
  69. types.hpp
  70. utils.cpp
  71. utils.hpp
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.