diff options
author | Nikita Kosenkov <NKosenkov@IBS.RU> | 2022-06-14 15:15:24 +0300 |
---|---|---|
committer | Nikita Kosenkov <NKosenkov@IBS.RU> | 2022-06-14 15:15:24 +0300 |
commit | ce5655be5f5f2d49a90ffc7ec4fcc82b697ec2d9 (patch) | |
tree | 61436c9b9675036dd1dacfaaef48c43ea6c9a510 /meta-ibs/meta-cp2-5422 | |
parent | c29cccddde306c9d7d06eec7bf2f14058820e061 (diff) | |
download | openbmc-ce5655be5f5f2d49a90ffc7ec4fcc82b697ec2d9.tar.xz |
Revert "x86-power-control: added check BMC restart cause for AlwaysOn policy"
This reverts commit c29cccddde306c9d7d06eec7bf2f14058820e061, reversing
changes made to 77997ff4ed369387c85847dc2420af321af2ac24.
Diffstat (limited to 'meta-ibs/meta-cp2-5422')
2 files changed, 0 insertions, 119 deletions
diff --git a/meta-ibs/meta-cp2-5422/recipes-x86/chassis/x86-power-control/0001-x86-power-control-added-check-BMC-restart-cause-for-.patch b/meta-ibs/meta-cp2-5422/recipes-x86/chassis/x86-power-control/0001-x86-power-control-added-check-BMC-restart-cause-for-.patch deleted file mode 100644 index 92a0da1675..0000000000 --- a/meta-ibs/meta-cp2-5422/recipes-x86/chassis/x86-power-control/0001-x86-power-control-added-check-BMC-restart-cause-for-.patch +++ /dev/null @@ -1,118 +0,0 @@ -From f49794106c6993b0a53b3b6634b44ae3d2f5cd82 Mon Sep 17 00:00:00 2001 -From: Nikita Kosenkov <NKosenkov@IBS.RU> -Date: Thu, 9 Jun 2022 14:26:30 +0300 -Subject: [PATCH] x86-power-control: added check BMC restart cause for AlwaysOn - policy - ---- - src/power_control.cpp | 80 +++++++++++++++++++++++++++++++++++++++++-- - 1 file changed, 78 insertions(+), 2 deletions(-) - -diff --git a/src/power_control.cpp b/src/power_control.cpp -index 05aa45c..380ba7e 100644 ---- a/src/power_control.cpp -+++ b/src/power_control.cpp -@@ -31,6 +31,7 @@ - #include <filesystem> - #include <fstream> - #include <string_view> -+#include <regex> - - namespace power_control - { -@@ -857,6 +858,74 @@ void PersistentState::saveState() - appStateStream << stateData.dump(indentationSize); - } - -+enum class BMCRestartCause -+{ -+ power = 1, // After cold BMC boot -+ watchdog, // After warm BMC reboot -+ unknown // Unknown reboot cause -+}; -+ -+BMCRestartCause getBmcRestartCause() -+{ -+ // U-boot write restart cause to bootargs -+ std::string cmdlineFilePath{"/proc/cmdline"}; -+ std::ifstream cmdlineFile{cmdlineFilePath}; -+ if(!cmdlineFile.is_open()) -+ { -+ lg2::error("BMCRestartCause: cannot open {FILE}", "FILE", -+ cmdlineFilePath); -+ -+ return BMCRestartCause::unknown; -+ } -+ -+ // Read cmd line from file -+ std::stringstream cmdlineBuffer; -+ cmdlineBuffer << cmdlineFile.rdbuf(); -+ cmdlineFile.close(); -+ -+ std::string cmdline{cmdlineBuffer.str()}; -+ if(cmdline.size() == 0) -+ { -+ lg2::error("BMCRestartCause: file {FILE} is empty", "FILE", -+ cmdlineFilePath); -+ -+ return BMCRestartCause::unknown; -+ } -+ -+ // Parse cmdline with regex -+ std::smatch regexResult; -+ std::regex regexExpr("resetreason=((?:power|watchdog))"); -+ if (!std::regex_search(cmdline, regexResult, regexExpr)) -+ { -+ lg2::error("BMCRestartCause: error parse file {FILE}", "FILE", -+ cmdlineFilePath); -+ -+ return BMCRestartCause::unknown; -+ } -+ -+ // Initialize restart cause -+ BMCRestartCause bmcRestartCause = BMCRestartCause::unknown; -+ -+ if(regexResult.ready()) -+ { -+ // The second element is needed -+ // as it contains the result of the second capturing group -+ if(regexResult.str(1) == "power") -+ { -+ bmcRestartCause = BMCRestartCause::power; -+ } -+ else if (regexResult.str(1) == "watchdog") -+ { -+ bmcRestartCause = BMCRestartCause::watchdog; -+ } -+ -+ lg2::info("BmcRestartCause: current BMC restart cause={CAUSE}", "CAUSE", -+ regexResult.str(1)); -+ } -+ -+ return bmcRestartCause; -+} -+ - static constexpr char const* setingsService = "xyz.openbmc_project.Settings"; - static constexpr char const* powerRestorePolicyObject = - "/xyz/openbmc_project/control/host0/power_restore_policy"; -@@ -1083,8 +1152,15 @@ void PowerRestoreController::invoke() - if (powerRestorePolicy == - "xyz.openbmc_project.Control.Power.RestorePolicy.Policy.AlwaysOn") - { -- sendPowerControlEvent(Event::powerOnRequest); -- setRestartCauseProperty(getRestartCause(RestartCause::powerPolicyOn)); -+ // We check the reason for the BMC restart and -+ // if the BMC is after a cold boot, -+ // then start the "AlwaysOn" policy -+ if (getBmcRestartCause() == BMCRestartCause::power) -+ { -+ sendPowerControlEvent(Event::powerOnRequest); -+ setRestartCauseProperty( -+ getRestartCause(RestartCause::powerPolicyOn)); -+ } - } - else if (powerRestorePolicy == - "xyz.openbmc_project.Control.Power.RestorePolicy.Policy.Restore") --- -2.35.1 - diff --git a/meta-ibs/meta-cp2-5422/recipes-x86/chassis/x86-power-control_%.bbappend b/meta-ibs/meta-cp2-5422/recipes-x86/chassis/x86-power-control_%.bbappend index cbda0f130a..d8a4104b65 100644 --- a/meta-ibs/meta-cp2-5422/recipes-x86/chassis/x86-power-control_%.bbappend +++ b/meta-ibs/meta-cp2-5422/recipes-x86/chassis/x86-power-control_%.bbappend @@ -1,7 +1,6 @@ FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:" SRC_URI:append = " \ file://power-config-host0.json \ - file://0001-x86-power-control-added-check-BMC-restart-cause-for-.patch \ " RDEPENDS:${PN}:append = " bash" |