Revision Date Author Comments
# 1901ab74 09-May-2024 Delphine CC Chiu <Delphine_CC_Chiu@wiwynn.com>

phosphor-discover-system-state: set after multi-user.target

Set the phosphor-discover-system-state service to run after
multi-user.target to ensure the operation is successful.
Avoid setting the Pow

phosphor-discover-system-state: set after multi-user.target

Set the phosphor-discover-system-state service to run after
multi-user.target to ensure the operation is successful.
Avoid setting the PowerRestoreDelay time still meet BMC not
being ready.
This is particularly important when facing network instability,
which can delay the activation of multi-user.target.

Test Log:
```
....
May 09 18:18:49 bmc phosphor-bmc-state-manager[903]: BMC_READY
May 09 18:18:49 bmc phosphor-bmc-state-manager[903]: Setting the BMCState field to xyz.openbmc_project.State.BMC.BMCState.Ready
May 09 18:18:49 bmc systemd[1]: Started Reboot Host 1 If Enabled.
May 09 18:18:49 bmc systemd[1]: Started Reboot Host 2 If Enabled.
May 09 18:18:49 bmc systemd[1]: Started Reboot Host 3 If Enabled.
May 09 18:18:49 bmc systemd[1]: Started Reboot Host 4 If Enabled.
May 09 18:18:49 bmc systemd[1]: Started Reboot Host 5 If Enabled.
May 09 18:18:49 bmc phosphor-discover-system-state[1206]: The LastbootCause is a watchdog. Power policy execution is currently allowed
May 09 18:18:49 bmc systemd[1]: Started Reboot Host 6 If Enabled.
May 09 18:18:49 bmc phosphor-discover-system-state[1207]: The LastbootCause is a watchdog. Power policy execution is currently allowed
May 09 18:18:49 bmc phosphor-discover-system-state[1206]: One time not set, check user setting of power policy
May 09 18:18:49 bmc phosphor-discover-system-state[1206]: Host power is off, processing power policy xyz.openbmc_project.Control.Power.RestorePolicy.Policy.AlwaysOn
May 09 18:18:49 bmc phosphor-discover-system-state[1206]: power_policy=ALWAYS_POWER_ON, powering host on (0s delay)
May 09 18:18:49 bmc phosphor-discover-system-state[1207]: One time not set, check user setting of power policy
May 09 18:18:50 bmc phosphor-discover-system-state[1208]: The LastbootCause is a watchdog. Power policy execution is currently allowed
May 09 18:18:50 bmc systemd[1]: Started Reboot Host 7 If Enabled.
May 09 18:18:50 bmc phosphor-discover-system-state[1207]: Host power is off, processing power policy xyz.openbmc_project.Control.Power.RestorePolicy.Policy.AlwaysOn
May 09 18:18:50 bmc phosphor-discover-system-state[1207]: power_policy=ALWAYS_POWER_ON, powering host on (0s delay)
May 09 18:18:50 bmc phosphor-discover-system-state[1209]: The LastbootCause is a watchdog. Power policy execution is currently allowed
May 09 18:18:50 bmc phosphor-discover-system-state[1208]: One time not set, check user setting of power policy
May 09 18:18:50 bmc phosphor-host-state-manager[994]: Host state transition request of xyz.openbmc_project.State.Host.Transition.On
....

```

Change-Id: Ic97c331d41af6b3ca5673e711107d475362c72c3
Signed-off-by: Delphine CC Chiu <Delphine_CC_Chiu@wiwynn.com>

show more ...


# 8e9f24d3 23-Feb-2024 Patrick Williams <patrick@stwcx.xyz>

phosphor-discover-system-state: improve service description

A multi-host system currently starts 8 instances of this service
with less than useful service descriptions. Improve the description
to i

phosphor-discover-system-state: improve service description

A multi-host system currently starts 8 instances of this service
with less than useful service descriptions. Improve the description
to include the host in question.

```
[ OK ] Started Reboot If Enabled.
[ OK ] Started Reboot If Enabled.
[ OK ] Started Reboot If Enabled.
[ OK ] Started Reboot If Enabled.
[ OK ] Started Reboot If Enabled.
[ OK ] Started Reboot If Enabled.
[ OK ] Started Reboot If Enabled.
[ OK ] Started Reboot If Enabled.
```

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I3867c30dedf12937db8f96df2d4b34d715828f05

show more ...


# b0986c07 31-May-2023 Andrew Geissler <geissonator@yahoo.com>

bmc-ready: auto power restart function update

This service is responsible for executing the auto power restart logic
for the system. Prior to this change, it would run as a oneshot service
during th

bmc-ready: auto power restart function update

This service is responsible for executing the auto power restart logic
for the system. Prior to this change, it would run as a oneshot service
during the multi-user.target. This meant the BMC would not achieve
"Ready" state until after this service has completed.

With the recent changes to support not allowing a power on of the system
until the BMC is "Ready", we need to change this service so it is not
holding up that state change.

Systems that enable this new feature to only allow a power on when the
BMC is in a "Ready" state will need to ensure they have a non-zero
PowerRestoreDelay set for their system. This allows the service in this
commit to start, the BMC to get to "Ready" and then the power on request
to be made (if needed).

If in the future there's a need to enable this new feature and also have
a 0s delay, then this service will need to be enhanced to wait for BMC
"Ready".

Tested:
- Set policy to AlwaysOn, confirmed service started, BMC hit "Ready",
service waited 30s, and then issued power on without issue.

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I1ecb8ed9e1cd6babfa2c0fdb51e8a9bd6c83e214

show more ...


# b640ec47 26-Jan-2023 Andrew Geissler <geissonator@yahoo.com>

update service file deps in accordance with recent refactoring

As noted in the following mailing list thread:
https://lists.ozlabs.org/pipermail/openbmc/2022-November/032457.html

and implemented vi

update service file deps in accordance with recent refactoring

As noted in the following mailing list thread:
https://lists.ozlabs.org/pipermail/openbmc/2022-November/032457.html

and implemented via this series of commits:
https://gerrit.openbmc.org/q/topic:op-power-control-ref

Some systemd services had their names changed. Update this service file
to reflect this.

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I8fdefbeb3d5100bc42eb365c59ad80d6b2119c26

show more ...


# d0fa61c5 20-Jan-2023 Andrew Geissler <geissonator@yahoo.com>

service-deps: add missing discover system state dep

The discover system state software makes d-bus calls to the settings
service and does not handle it missing in a graceful manner. Ensure the
setti

service-deps: add missing discover system state dep

The discover system state software makes d-bus calls to the settings
service and does not handle it missing in a graceful manner. Ensure the
setting service is running before starting this service.

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Icb362b2472c3efb135941a7124fc9c3b4b069a00

show more ...


# c328a4cf 18-Mar-2022 Potin Lai <potin.lai@quantatw.com>

discover-system-state: add multi-host support

Add HostObjects class to fetch paths of settings d-bus objects under
/xyz/openbmc_project/control/hostX .

Signed-off-by: Potin Lai <potin.lai@quantatw.

discover-system-state: add multi-host support

Add HostObjects class to fetch paths of settings d-bus objects under
/xyz/openbmc_project/control/hostX .

Signed-off-by: Potin Lai <potin.lai@quantatw.com>
Change-Id: I1dc55ec389e94e6c85aa7031b304f8c4ed15dc71

show more ...


# b2b3d9c2 26-Jan-2022 Andrew Geissler <geissonator@yahoo.com>

pinhole: no power restore policy on pinhole reset

If the user initiated a BMC reboot via the pinhole reset then do not run
the power restore policy. The pinhole reset is mostly used in debug
situati

pinhole: no power restore policy on pinhole reset

If the user initiated a BMC reboot via the pinhole reset then do not run
the power restore policy. The pinhole reset is mostly used in debug
situations where a system is having issues. Disabling the power restore
logic helps keep debug simpler.

Tested:
- The overall pinhole reset logic has not been tested on hardware.
- A variety of tests were done within simulation to validate the
different paths in this series of patches
- Full end to end testing will occur once all function is in place due
to the complexities of physically toggling the pinhole reset

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Ia656b4872620b6a1fc6ba8f82c01f041d43378a2

show more ...


# 38b2e14d 06-May-2020 Andrew Geissler <geissonator@yahoo.com>

oneshot: discover-system-state: remain after exit

This service should only ever run once, after a BMC has first booted.
Ensure this is the case by setting RemainAfterExit to yes.

Signed-off-by: And

oneshot: discover-system-state: remain after exit

This service should only ever run once, after a BMC has first booted.
Ensure this is the case by setting RemainAfterExit to yes.

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I346ab3588ee210709206a41c5e2acc73222561eb

show more ...


# c212353d 14-Jan-2020 Andrew Geissler <geissonator@yahoo.com>

move state manager service files into its repo

OpenBMC is moving towards individual repos hosting and maintaining their
own service files. This allows the corresponding maintainer more control
over

move state manager service files into its repo

OpenBMC is moving towards individual repos hosting and maintaining their
own service files. This allows the corresponding maintainer more control
over their service file and removes the meta-* layer maintainer from
needing to be involved.

Once this is merged, its bump will need to be combined with a change in
the meta-phosphor layer that removes the service files.

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Ib9ebfb16e131bc2743b3fc9fcfdfb7370f455f8f

show more ...