summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndrew Geissler <andrewg@us.ibm.com>2017-06-30 16:43:05 -0500
committerAndrew Geissler <andrewg@us.ibm.com>2017-06-30 16:45:51 -0500
commit181f8364f81fc8864e9ae86f28de13f80cd9bbf9 (patch)
tree91836315c276fba446be75cbe503cb0913c35275
parent4f309e8c68907218bc808af94475c1b056d0ae6e (diff)
downloadphosphor-state-manager-181f8364f81fc8864e9ae86f28de13f80cd9bbf9.tar.gz
phosphor-state-manager-181f8364f81fc8864e9ae86f28de13f80cd9bbf9.zip
Make the BMC reboot irreversible
Using irreversible ensures no other conflicting units are started (or even queued) Resolves openbmc/openbmc#1161 Change-Id: Ia937fc0271be896e5bc469129335ae29c7a79d69 Signed-off-by: Andrew Geissler <andrewg@us.ibm.com>
-rw-r--r--bmc_state_manager.cpp6
1 files changed, 3 insertions, 3 deletions
diff --git a/bmc_state_manager.cpp b/bmc_state_manager.cpp
index 9929059..3539a30 100644
--- a/bmc_state_manager.cpp
+++ b/bmc_state_manager.cpp
@@ -122,11 +122,11 @@ void BMC::executeTransition(const Transition tranReq)
SYSTEMD_OBJ_PATH,
SYSTEMD_INTERFACE,
"StartUnit");
-
- method.append(sysdUnit, "replace");
+ // The only valid transition is reboot and that
+ // needs to be irreversible once started
+ method.append(sysdUnit, "replace-irreversibly");
this->bus.call(method);
-
return;
}
OpenPOWER on IntegriCloud