diff options
| author | Andrew Geissler <andrewg@us.ibm.com> | 2017-06-30 16:43:05 -0500 |
|---|---|---|
| committer | Andrew Geissler <andrewg@us.ibm.com> | 2017-06-30 16:45:51 -0500 |
| commit | 181f8364f81fc8864e9ae86f28de13f80cd9bbf9 (patch) | |
| tree | 91836315c276fba446be75cbe503cb0913c35275 | |
| parent | 4f309e8c68907218bc808af94475c1b056d0ae6e (diff) | |
| download | phosphor-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.cpp | 6 |
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; } |

