summaryrefslogtreecommitdiff
path: root/meta-ibs/meta-cp2-5422
diff options
context:
space:
mode:
authorNikita Kosenkov <NKosenkov@IBS.RU>2022-06-14 15:15:24 +0300
committerNikita Kosenkov <NKosenkov@IBS.RU>2022-06-14 15:15:24 +0300
commitce5655be5f5f2d49a90ffc7ec4fcc82b697ec2d9 (patch)
tree61436c9b9675036dd1dacfaaef48c43ea6c9a510 /meta-ibs/meta-cp2-5422
parentc29cccddde306c9d7d06eec7bf2f14058820e061 (diff)
downloadopenbmc-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')
-rw-r--r--meta-ibs/meta-cp2-5422/recipes-x86/chassis/x86-power-control/0001-x86-power-control-added-check-BMC-restart-cause-for-.patch118
-rw-r--r--meta-ibs/meta-cp2-5422/recipes-x86/chassis/x86-power-control_%.bbappend1
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"