Age | Commit message (Collapse) | Author | Files | Lines |
|
In the current state soft-off service is a hard dependency
(requires) for the host shut down target, so in any case where
the soft-off service fails with an error condition, the shutdown
target fails too.
This commit intends to make soft-off service as a soft dependency(wants)
so that the shutdown target can still proceed even though the
pldmsoftoff service fails for whatever reason.
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: I5650702c6eb71adab10fdb6493c9d582a9cb0129
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I588025b614416c43aa2d053765ab53bacf890cb5
|
|
PLDM is adding infrastructure to register callbacks for the PLDM
requester code.
https://gerrit.openbmc-project.xyz/c/openbmc/pldm/+/43465
function2 is header only and already leveraged in sdeventplus and
phosphor-ipmi-flash. function 2 provides move-only
fu2::unique_function.
Using function2 had an image size increase of 4K for PLDM daemon
with build-type minsize.
Signed-off-by: Tom Joseph <rushtotom@gmail.com>
Change-Id: I427219b51cee7b29b7e62cc77d597143f21fcda0
|
|
This reverts commit 284d56bb877be545f59cf10883d28a37f306fc5b.
Testing has uncovered some corner cases where the host is in a good
enough state to respond to the initial shutdown request, but not in a
good enough state to actually shut itself down. This results in the
quiesce operation hitting the 2 hour timeout that is used in this
scenario. 2 hours to reach a quiesce state is not acceptable. This is
going to take some more design and discussion.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I45a39879da6e7af8c573a1d1d2577f98579df6ee
|
|
When the obmc-host-quiesce@.target was initially created the idea was to
leave the system in the state it was in. In most cases the host had
crashed.
As OpenBMC has grown, the use cases for the quiesce target have grown.
For example the Quiesce On Hardware Error design:
https://github.com/openbmc/docs/blob/master/designs/fail-boot-on-hw-error.md
When the QuiesceOnHwError setting is enabled, the host will be put in a
Quiesce state to indicate that an error has occurred. In a lot of cases
though the host actually can (and does) continue to boot. This causes a
lot of confusion for the user (host state shows Quiesce but host is
still booting).
This commit adds the pldmSoftPowerOff.service to the quiesce target to
ensure the host is given a chance (if it's running) to gracefully shut
itself down. If the host does not respond then the soft off service will
follow it's timeout flow and exit without error.
Tested:
- Verified when quiesce error is injected that the PLDM soft off service
was called
(From meta-phosphor rev: dfa2d6beb496b2265402950caf524c7f79850c46)
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Ie6d07011a5ac5f6964435364ff6e332a77e5a3d4
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
This officially enables the pldm soft off service on systems which
choose to install the pldm package.
Tested:
- Confirmed that with all "softoff" Topic patches that a Rainier system
correctly waited for the host to power down before turning off
chassis power.
(From meta-phosphor rev: 5e124cdc223024621eb79027d89f070e89a67e67)
Change-Id: If5e2d53852d4dbb3a0f8e39a2cdf3ef53f6aa3cf
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Add the pldmSoftPowerOff.service in the pldm_git.bb.
srcrev bump 1b180d8a6d..184f60263a :
Chicago Duan (1):
soft-off: Send soft off request to the host and wait the host soft off responds
(From meta-phosphor rev: 4236597f531f95e736714f35e43b13f8882187cc)
Signed-off-by: Chicago Duan <duanzhijia01@inspur.com>
Change-Id: If5cf32f01b6c554dca68f3a35a4bd71760c15e5f
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
openbmc/pldm now has an in-repo systemd service file. Package the same.
Also need the srcrev bump with this commit because the two are co-reqs.
(From meta-phosphor rev: 127d89742398dec1681fe309002fa4e4fb3c2e24)
Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Change-Id: I2703ca9a416cfc4631d2210fc42723a45851d226
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Remove boost depends in pldm_git.bb .
PLDM was used crc function from boost before.
But now, the pldm has replaced the boost crc method with its own:
https://gerrit.openbmc-project.xyz/c/openbmc/pldm/+/31139
(From meta-phosphor rev: 0e2b30d04ee1814e08dd496d4c3595db3caffda7)
Signed-off-by: Xiaochao Ma <maxiaochao@inspur.com>
Change-Id: I64e558576ad4f065c5db023d5ff26de4c5d26b08
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
PLDM has no such dependency, and it will cause a circular dependency
when the openpower-pels phosphor-logging extension is enabled.
Add a dependency on sdeventplus.
(From meta-phosphor rev: 6fd86c4ebfa1a9b467531b457498f84bc0b5fd3d)
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I8278310fbce7d067a65278c507a7545b2a4c5015
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
The recipe installs the following:
- libpldm
- libpldmresponder
Details about these libs can be found at
https://github.com/openbmc/pldm/blob/master/README.md.
The recipe also installs the pldm daemon (pldmd), however it doesn't
provide a service file to start the daemon. This is because pldmd
depends on the MCTP demux daemon, which in turn would have system
specific invocations via its service file. Hence the service files for
pldm (and libmctp) would likely be installed via system specific
bbappends.
(From meta-phosphor rev: 394e867e7778d76c84aea52f64d1e3a60c968c01)
Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Change-Id: I969840119aa51502dce2b5b1cf4d6085ee4e8fd2
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|