#
9f38152a |
| 17-Oct-2024 |
Thang Tran <thuutran@amperecomputing.com> |
reject Transition request while firmware being flashed
When users request to flash BMC/Host... firmware, the BMC should reject turning on the host or rebooting BMC requests to protect the system. As
reject Transition request while firmware being flashed
When users request to flash BMC/Host... firmware, the BMC should reject turning on the host or rebooting BMC requests to protect the system. As defined in the PDIs [1], the ActivationBlocksTransition interface is used to prevent power-on during flashing. The phosphor-state-manager shall look for this interface to make decision should transition request is executed or not. When BMC rejects the transition request, an error is thrown as defined in PDIs [2].
[1] https://github.com/openbmc/phosphor-dbus-interfaces/blob/master/yaml/xyz/openbmc_project/Software/ActivationBlocksTransition.interface.yaml [2] https://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/75231
Tested: Enable check-fwupdate-before-do-transition option 1. Request to flash Host's firmware via Redfish or WebUi 2. Request to turn off the Host. $ipmitool power off 3. BMC turns off the power. 4. Request to turn on the Host. $ipmitool power on 5. BMC rejects above request. No power action is executed. 6. Request to reboot BMC. $ipmitool mc reset cold 7. BMC did not reboot. 8. Request turn on the Host after flashing done. 9. BMC turns on the Power. 10. Request to reboot BMC. 11. BMC reboots.
Disable check-fwupdate-before-do-transition option 1. Request to flash Host's firmware via Redfish or WebUi 2. Request to turn off the Host. $ipmitool power off 3. BMC turns off the power. 4. Request to turn on the Host. $ipmitool power on 5. BMC does not reject this request, but no power action is handled 6. When flashing done, BMC turns on the power.
Change-Id: If3998e58d24cfb72d13b01eb5b8c8cc5a39b3c95 Signed-off-by: Thang Tran <thuutran@amperecomputing.com>
show more ...
|
#
d01e1eaa |
| 03-Dec-2024 |
Eddie James <eajames@linux.ibm.com> |
secure-boot: Update TPM device path
Updates to the device tree necessitate updating this path to check whether the TPM is supposed to be present.
Change-Id: I20e890213c4a990aab8c74dd17b5f7eb4968d11
secure-boot: Update TPM device path
Updates to the device tree necessitate updating this path to check whether the TPM is supposed to be present.
Change-Id: I20e890213c4a990aab8c74dd17b5f7eb4968d112 Signed-off-by: Eddie James <eajames@linux.ibm.com>
show more ...
|
#
f566c964 |
| 22-Jul-2024 |
Amithash Prasasd <amithash@meta.com> |
Switch to Use phosphor-dbus-interface defined constants
We are redefining a lot of the constants which are already defined in the phosphor-dbus-interface auto-generated headers. This change switches
Switch to Use phosphor-dbus-interface defined constants
We are redefining a lot of the constants which are already defined in the phosphor-dbus-interface auto-generated headers. This change switches the code to use those instead of allowing compile time definition of those.
Allowing for compile time definition would probably break clients who are not aware of the new paths anyway.
Tested by build a harma image and loading in qemu to test the added interfaces are unchanged. ``` busctl introspect xyz.openbmc_project.State.BMC \ /xyz/openbmc_project/state/bmc0 busctl introspect xyz.openbmc_project.State.Host \ /xyz/openbmc_project/state/host0 busctl introspect xyz.openbmc_project.State.Chassis \ /xyz/openbmc_project/state/chassis0 ```
Change-Id: Ib4c77d2789c13f509b75a2b1837ea454e53e8ae9 Signed-off-by: Amithash Prasasd <amithash@meta.com>
show more ...
|
#
966f2362 |
| 18-Apr-2024 |
Patrick Williams <patrick@stwcx.xyz> |
host-gpios: enable by default
We should tend to have a default meson.option policy that enables as much code as possible so that it is covered by CI. The `host-gpios` option was disabled and we did
host-gpios: enable by default
We should tend to have a default meson.option policy that enables as much code as possible so that it is covered by CI. The `host-gpios` option was disabled and we did not have a PACKAGECONFIG for it in openbmc/openbmc. With Ia737a5645dcf472efdc75a9af7260eff60ef90ed, I have added a PACKAGECONFIG for this setting and it will be disabled by default in our Yocto builds. Enable it here to regain the CI coverage.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I952095a294059a27ae1066d6abdc9abcc5ba709a
show more ...
|
#
31cddb75 |
| 27-Feb-2024 |
Andrew Geissler <geissonator@yahoo.com> |
Add option to enable/disable forced warm reboots
Some OpenBMC systems don't support a forced warm reboot. Provide a compile option to enable/disable.
Tested: - Confirmed that if feature is disabled
Add option to enable/disable forced warm reboots
Some OpenBMC systems don't support a forced warm reboot. Provide a compile option to enable/disable.
Tested: - Confirmed that if feature is disabled, then AllowedHostTransitions did not have ForceWarmReboot option.
Change-Id: I207b8fa7e3f60efbf406397ff4df980a5d2677ed Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
show more ...
|
#
a15b81b4 |
| 16-Aug-2023 |
George Liu <liuxiwei@inspur.com> |
meson.options: Fix some warnings
Warning messages: ``` meson.options:129: WARNING: Project targets '>=1.1.1' but uses feature deprecated since '1.1.0': "boolean option" keyword argument "value" of t
meson.options: Fix some warnings
Warning messages: ``` meson.options:129: WARNING: Project targets '>=1.1.1' but uses feature deprecated since '1.1.0': "boolean option" keyword argument "value" of type str. use a boolean, not a string
meson.options:144: WARNING: Project targets '>=1.1.1' but uses feature deprecated since '1.1.0': "boolean option" keyword argument "value" of type str. use a boolean, not a string ```
Signed-off-by: George Liu <liuxiwei@inspur.com> Change-Id: I69f41cca5cc792e89306531d380d52b19ec41775
show more ...
|
#
65438b6d |
| 15-Aug-2023 |
George Liu <liuxiwei@inspur.com> |
meson_options.txt: Support for reading options from meson.options
Support has been added for reading options from meson.options instead of meson_options.txt[1]. These are equivalent, but not using t
meson_options.txt: Support for reading options from meson.options
Support has been added for reading options from meson.options instead of meson_options.txt[1]. These are equivalent, but not using the .txt extension for a build file has a few advantages, chief among them many tools and text editors expect a file with the .txt extension to be plain text files, not build scripts.
[1] https://mesonbuild.com/Release-notes-for-1-1-0.html#support-for-reading-options-from-mesonoptions
Signed-off-by: George Liu <liuxiwei@inspur.com> Change-Id: I705afd61af8cfcad6e66008ff45f285fa99ec78b
show more ...
|