Revision Date Author Comments
# 933bee76 09-May-2023 Andrew Geissler <geissonator@yahoo.com>

systemd: no installation in templated targets

Upstream yocto introduced a change via e510222 (systemd-systemctl:
fix instance template WantedBy symlink construction).

This fixes a bug that we in Op

systemd: no installation in templated targets

Upstream yocto introduced a change via e510222 (systemd-systemctl:
fix instance template WantedBy symlink construction).

This fixes a bug that we in OpenBMC had been taking advantage of in that
we were able to document our templated target dependencies without it
actually doing anything. The real installation of services within
targets occurs in our bitbake recipes due to the complexity of chassis
and host instances on a per machine basis.

Leave the dependency information in the service files but comment them
out. It's useful to be able to look at a service and understand which
targets it's going to be installed into by the bitbake recipes.

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

show more ...


# f2e53726 03-May-2022 Andrew Geissler <geissonator@yahoo.com>

pvm_sys_dump_active: ensure no failure

Setting this BIOS attribute is a "best try" type function. If it fails,
do not cause this service to fail. The BIOS is used to provide a warning
to the user bu

pvm_sys_dump_active: ensure no failure

Setting this BIOS attribute is a "best try" type function. If it fails,
do not cause this service to fail. The BIOS is used to provide a warning
to the user but is not a hard requirement.

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

show more ...


# 67513c14 26-Apr-2022 Andrew Geissler <geissonator@yahoo.com>

mpreboot: collect bmc dump

When a mpreboot is started, it indicates the host firmware has
encountered a critical failure. At times, the BMC firmware could be
involved in this error. Collect a BMC du

mpreboot: collect bmc dump

When a mpreboot is started, it indicates the host firmware has
encountered a critical failure. At times, the BMC firmware could be
involved in this error. Collect a BMC dump in parallel with the mpreboot
so that the BMC team will also have some debug data in these scenarios.

This is a "best attempt" type situation so do not let the failure of
generating a BMC dump, impact the overall mpreboot flow.

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

show more ...


# 03eaa3e8 02-May-2022 Andrew Geissler <geissonator@yahoo.com>

pvm_sys_dump_active: utilize proper systemd syntax

Was noticing this warning in the journal:
/lib/systemd/system/op-clear-sys-dump-active@.service:17: Ignoring unknown escape sequences: "busctl set-

pvm_sys_dump_active: utilize proper systemd syntax

Was noticing this warning in the journal:
/lib/systemd/system/op-clear-sys-dump-active@.service:17: Ignoring unknown escape sequences: "busctl set-property xyz.openbmc_project.BIOSConfigManager /xyz/openbmc_project/bios_config/manager xyz.openbmc_project.BIOSConfig.Manager PendingAttributes a{s\(sv\)} 1 pvm_sys_dump_active xyz.openbmc_project.BIOSConfig.Manager.AttributeType.Enumeration s Disabled"

A bit of internet searching pointed out that systemd requires two
backslashes

Tested:
- Verified services still work as expected and we no longer get the
above warning.

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

show more ...


# 9ca8a117 07-Apr-2022 Andrew Geissler <geissonator@yahoo.com>

update host bios to indicate mpreboot

A new IBM-specific BIOS was introduced via the following commit:
https://gerrit.openbmc-project.xyz/c/openbmc/pldm/+/52268

This new BIOS value tracks when a me

update host bios to indicate mpreboot

A new IBM-specific BIOS was introduced via the following commit:
https://gerrit.openbmc-project.xyz/c/openbmc/pldm/+/52268

This new BIOS value tracks when a memory preserving reboot starts until
the time that the generated dump is offloaded from the system.

Utilizing the openpower-proc-control repository for this function,
since this is where mpreboot is controlled and is specific to systems
with this feature.

PHYP is responsible for setting the BIOS to disabled once the offload
has completed. The requirement on the BMC is to just ensure it gets
reset on any power off or system AC loss.

Tested:
Updated op-proc-control recipes to put this new service in the
multi-user.target and obmc-host-stop@0.target.
- Caused mpreboot, verified it was set to enabled
- Rebooted BMC in above condition, verified it was still enabled
- Powered off system, verified it was reset to disabled
- Set to enabled at BMC Ready, rebooted BMC and verified it was reset to
disabled

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

show more ...


# 750f9227 24-Feb-2022 Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>

Execute mp reboot prior to host dump delete service

The entries for the dumps stored in the host memory needs to
be deleted during a power off but that should be skipped
in the case of a memory pres

Execute mp reboot prior to host dump delete service

The entries for the dumps stored in the host memory needs to
be deleted during a power off but that should be skipped
in the case of a memory preserving reboot.

Add changes to make sure the host dump clear service
executes after enter mp reboot service, so dump clear
service can be skipped if the indicater file exists.

Tests:
Feb 24 11:38:53 systemd[1]: Finished Start memory preserving reboot host0.
Feb 24 11:38:53 systemd[1]: Condition check resulted in Remove host dump entries during poweroff being skipped.

Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
Change-Id: I736a2c6245b494db054b086a1b63348838d926a7

show more ...


# bbb53393 14-Feb-2022 Adriana Kobylak <anoo@us.ibm.com>

PHAL: Add service dependency to DEVTREE creation

Start the services that require DEVTREE after the
openpower-update-bios-attr-table.service since that's the one that
sets up the DEVTREE symlink. Als

PHAL: Add service dependency to DEVTREE creation

Start the services that require DEVTREE after the
openpower-update-bios-attr-table.service since that's the one that
sets up the DEVTREE symlink. Also update the path to DEVTREE to the one
defined by the meson options.

Tested: Verified phal started after the bios attr service file, powered
on, and issued reboot at power on without any errors. Verified service
files had devtree path of:
Environment="PDBG_DTB=/media/hostfw/running/DEVTREE"

Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
Change-Id: If41f86b287b70c48d17c6f4b2af9aa0adfb81488

show more ...


# b5fa942a 03-Dec-2021 Andrew Geissler <geissonator@yahoo.com>

host-restart: set restart reason on mpreboot

When a memory preserving reboot (caused by a host firmware crash) is the
reason for a host restart, ensure the RestartCause is set appropriately.

This p

host-restart: set restart reason on mpreboot

When a memory preserving reboot (caused by a host firmware crash) is the
reason for a host restart, ensure the RestartCause is set appropriately.

This property is used by other software on some systems to guide
partition behaviors in the host code.

The phosphor-state-manager software is responsible for resetting the
RestartCause when the host is powered off.

Tested:
- Verified RestartCause was correctly set to Host.RestartCause.HostCrash
when executing the service.

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

show more ...


# 1dc0829e 22-Jun-2021 Andrew Geissler <geissonator@yahoo.com>

service-files: move to own sub-directory

Some more service files are on their way for new features so put all of
them in a sub-directory to keep the main one a bit cleaner

Signed-off-by: Andrew Gei

service-files: move to own sub-directory

Some more service files are on their way for new features so put all of
them in a sub-directory to keep the main one a bit cleaner

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

show more ...