commit | b4cbfac10b490cd2a8ca83bfd139fe07bf1fe5a7 | [log] [tgz] |
---|---|---|
author | NodeMan97 <corey.hardesty@icloud.com> | Mon May 09 23:56:39 2022 -0500 |
committer | Andrew Geissler <andrew@geissonator.com> | Wed Jul 06 19:48:46 2022 +0000 |
tree | a3fe80e0e9c79e10fe913b4ab1048d9542d57997 | |
parent | 9e0416f41810f573982961d5f33658727d2f1678 [diff] |
Host reboot configuration support Redfish API now supports a mechanism for the users of the system to configure the number of times a BMC should restart the host firmware when in an error path before giving up. Originally the function "attemptsLeft(uint32_t)" ignored the passed in value and and always set this number to a hardcoded 3(without the user knowing). Changes made here allow for this property to be set by an external user over D-bus for values other than 3. To compliment this functionality and the new DBus property "RetryAttempts" an override was created to ensure that when "RetryAttempts" is set by an external user that the "AttemptsLeft" property is updated accordingly. Versioning was added to avoid a serialization error that would occur if a system was loaded using the previous version that did not include this new property. Testing: - Using the newly defined cereal version of '2' performed correctly. - Removing the new property and reverting cereal class version back to '1' behaved properly. - built and ran with Docker along with the web changes. - verified update of "AttemptsLeft" through a p10bmc running on simics through busctl. - verified that when the "RetryAttempts" policy value is changed by either bmcweb or dbus set-property() that "AttemptsLeft" updates accordingly. busctl set-property xyz.openbmc_project.State.Host ... RetryAttempts u 1 busctl get-property xyz.openbmc_project.State.Host ... RetryAttempts service@p10bmc: u 1 busctl get-property xyz.openbmc_project.State.Host ... AttemptsLeft service@p10bmc: u 1 busctl set-property xyz.openbmc_project.State.Host ... RetryAttempts u 3 busctl set-property xyz.openbmc_project.State.Host ... AttemptLeft u 8 busctl get-property xyz.openbmc_project.State.Host ... AttemptLeft u 3 service@p10bmc: u 3 Signed-off-by: Corey Hardesty <corey.hardesty@icloud.com> Change-Id: I70a8457c88fb76e118937d79e8dbe15518df7264
This repository contains the software responsible for tracking and controlling the state of different objects within OpenBMC. This currently includes the BMC, Chassis, and Host. The most critical feature of phosphor-state-manager software is its support for requests to power on and off the system by the user.
This software also enforces any restore policy (i.e. auto power on system after a system power event or bmc reset) and ensures its states are updated correctly in situations where the BMC is rebooted and the chassis or host are in on/running states.
This repository also provides a command line tool, obmcutil, which provides basic command line support to query and control phosphor-state-manager applications running within an OpenBMC system. This tool itself runs within an OpenBMC system and utilizes D-Bus APIs. These D-Bus APIs are used for development and debug and are not intended for end users.
As with all OpenBMC applications, interfaces and properties within phosphor-state-manager are D-Bus interfaces. These interfaces are then used by external interface protocols, such as Redfish and IPMI, to report and control state to/by the end user.
phosphor-state-manager makes extensive use of systemd. There is a writeup here with an overview of systemd and its use by OpenBMC.
phosphor-state-manager follows some basics design guidelines in its implementation and use of systemd:
phosphor-state-manager implements states and state requests as defined in phosphor-dbus-interfaces for each object it supports.
Ready
once all services within the default.target have executed. The only state change request you can make of the BMC is for it to reboot itself.On
or Off
.Off
, Running
, Quiesced
(error condition), or in DiagnosticMode
(collecting diagnostic data for a failure)As noted above, phosphor-state-manager provides a command line tool, obmcutil, which takes a state
parameter. This will use D-Bus commands to retrieve the above states and present them to the user. It also provides other commands which will send the appropriate D-Bus commands to the above properties to power on/off the chassis and host (see obmcutil --help
within an OpenBMC system).
The above objects also implement other D-Bus objects like power on hours, boot progress, reboot attempts, and operating system status. These D-Bus objects are also defined out in the phosphor-dbus-interfaces repository.
The RestorePolicy defines the behavior the user wants when the BMC is reset. If the chassis or host is on/running then this service will not run. If they are off then the RestorePolicy
will be read and executed by phosphor-state-manager code.
In situations where the BMC is reset and the chassis and host are on and running, its critical that the BMC software do two things:
Note that some of this logic is provided via service files in system-specific meta layers. That is because the logic to determine if the chassis is on or if the host is running can vary from system to system. The requirement to create the files defined below and ensure the common targets go active is a must for anyone wishing to enable this feature.
phosphor-state-manager discovers state vs. trying to cache and save states. This ensure it's always getting the most accurate state information. It discovers the chassis state by checking the pgood
value from the power application. If it determines that power is on then it will do the following:
On
The chassis@0-on file is removed once the obmc-chassis-poweron@0.target becomes active (i.e. all service have been successfully started which are wanted or required by this target).
The logic to check if the host is on sends a command to the host, and if a response is received then similar logic to chassis is done:
The host@0-on file is removed once the obmc-host-start@0.target and obmc-host-startmin@0.target become active (i.e. all service have been successfully started which are wanted or required by these targets).
To build this package, do the following steps: 1. meson build 2. ninja -C build To clean the repository again run `rm -rf build`.