Fix trace of pldm return codes for yocto update

Fix Yocto update compile failure

Change-Id: I392cefaefb5aa3d44e83b1edb7df0dbaaf11d86b
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

Fix PLDM dependency string

After systemd daemon-reload an error was found:
open_power.OCC.Control.service:7: Failed to add dependency on xyz.openbmc_project.PLDM, ignoring: Invalid argument

Chang

Fix PLDM dependency string

After systemd daemon-reload an error was found:
open_power.OCC.Control.service:7: Failed to add dependency on xyz.openbmc_project.PLDM, ignoring: Invalid argument

Change-Id: Id1c6ff9661347e3b7dc3e6d764dbe5ef4dc451bd
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

Implement processor throttle dbus properties

- create processor throttle dbus objects for each OCC (processor)
- update throttle properties based on OCC poll response data
or safe mode status.

Th

Implement processor throttle dbus properties

- create processor throttle dbus objects for each OCC (processor)
- update throttle properties based on OCC poll response data
or safe mode status.

Throttle data will be made available via Redfish

NAME TYPE SIGNATURE RESULT/VALUE FLAGS
org.freedesktop.DBus.Introspectable interface - - -
.Introspect method - s -
org.freedesktop.DBus.Peer interface - - -
.GetMachineId method - s -
.Ping method - - -
org.freedesktop.DBus.Properties interface - - -
.Get method ss v -
.GetAll method s a{sv} -
.Set method ssv - -
.PropertiesChanged signal sa{sv}as - -
xyz.openbmc_project.Control.Power.Throttle interface - - -
.ThrottleCauses property as 0 emits-change
.Throttled property b false emits-change

Example of throttled processor (due to a power limit):
as 1 "xyz.openbmc_project.Control.Power.Throttle.ThrottleReasons.PowerLimit"

Change-Id: I0af9d82fab9d694427d0adaa45f4a372d25fbc12
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest .clang-format from the docs repository and reformat the
repository.

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

show more ...

Add Chris as maintainer

I would like to help out on the reviews. I have been reviewing
occ-control code for over a year and I am ready to share some of the
maintanence burden going forward.

Here ar

Add Chris as maintainer

I would like to help out on the reviews. I have been reviewing
occ-control code for over a year and I am ready to share some of the
maintanence burden going forward.

Here are my contributions to occ-control repository:
https://github.com/openbmc/openpower-occ-control/commits?author=cjcain

Change-Id: I4cb1d30ae6b9e7ce0eb2bebbbf156558245f93d4
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

regenerate-meson: re-run with latest from sdbusplus

The sdbus++-gen-meson has a new version, which requires regenerating
all the meson in this repository. Re-run the `regenerate-meson`
script.

Cha

regenerate-meson: re-run with latest from sdbusplus

The sdbus++-gen-meson has a new version, which requires regenerating
all the meson in this repository. Re-run the `regenerate-meson`
script.

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

show more ...

meson: remove deprecated get_pkgconfig_variable

Since meson 0.56, the `get_pkgconfig_variable` has been deprecated. In
meson 0.58 the `get_variable` was enhanced to no longer require the
`pkgconfig

meson: remove deprecated get_pkgconfig_variable

Since meson 0.56, the `get_pkgconfig_variable` has been deprecated. In
meson 0.58 the `get_variable` was enhanced to no longer require the
`pkgconfig` keyword argument. Ensure meson 0.58 is required and update
the usage of all `get_pkgconfig_variable` and `get_variable` to be the
modern variant.

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

show more ...

Handle OCC active sensor updates prior to host runtime

On some systems, occ-control was getting notified that the OCCs were
active before the host reached runtime state. This would prevent
occ-contr

Handle OCC active sensor updates prior to host runtime

On some systems, occ-control was getting notified that the OCCs were
active before the host reached runtime state. This would prevent
occ-control from starting communication with the OCCs.

The fix will ignore the early OCC Active sensor enabled messages and
once the host gets to runtime, it will re-query the sensors to ensure
they are still active.

Verified on fresh boot, warm boot, BMC reset, warm boot after BMC reset
on a system that exhibited the early sensors and one that did not.

Also removes an unnecessary InternalFailure when a sensor was cleared,
but no OCC objets were found.

Change-Id: Idb6c107cf83d12272aef9179045de73298e6d6b6
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

Check host state before attempting OCC communication

HBRT sends a PLDM message to occ-control when the OCCs go active. If the
system is powered down close to that time, there is a window where the
P

Check host state before attempting OCC communication

HBRT sends a PLDM message to occ-control when the OCCs go active. If the
system is powered down close to that time, there is a window where the
PLDM message could get sent even when the host is no longer running.
This commit will confirm the host is actually running when the message
is received; if not, the OCC communication willl not be allowed.

Change-Id: Ia209a5893e8294f1b10bcb143bc59831205223ab
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

Trace PLDM response on unexpected states

Commit will trace the PLDM response packet when querying the OCC active
state sensors if the state is not one of the expected values:
PLDM_STATE_SET_OPERAT

Trace PLDM response on unexpected states

Commit will trace the PLDM response packet when querying the OCC active
state sensors if the state is not one of the expected values:
PLDM_STATE_SET_OPERATIONAL_RUNNING_STATUS_IN_SERVICE
PLDM_STATE_SET_OPERATIONAL_RUNNING_STATUS_STOPPED
PLDM_STATE_SET_OPERATIONAL_RUNNING_STATUS_DORMANT

Change-Id: I87d144b68aed76e473ebf28348ade3df910a5c5b
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

meson: add dependency for nlohmann-json

The code uses nlohmann-json but didn't have a dependency. Add one
so that the project builds in subproject builds.

Signed-off-by: Patrick Williams <patrick@

meson: add dependency for nlohmann-json

The code uses nlohmann-json but didn't have a dependency. Add one
so that the project builds in subproject builds.

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

show more ...

meson: change dependency order for subproject build

Both this repository and phosphor-logging need
phosphor-dbus-interfaces. Meson seems to be unable to find the
wrapfile-based definition of PDI if

meson: change dependency order for subproject build

Both this repository and phosphor-logging need
phosphor-dbus-interfaces. Meson seems to be unable to find the
wrapfile-based definition of PDI if phosphor-logging is specified
first. Swap the ordering so that subproject builds work.

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

show more ...

Remove Hypervisor check for PowerVM target

Remove the dbus query to determine if system is PowerVM.
It is assumed that all systems are PowerVM.

Change-Id: I50d3bb76b469383618aa280c591459674f337130

Remove Hypervisor check for PowerVM target

Remove the dbus query to determine if system is PowerVM.
It is assumed that all systems are PowerVM.

Change-Id: I50d3bb76b469383618aa280c591459674f337130
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

Move the service files to the occ-control repo

To align with the OpenBMC direction, the OCC service files are being
moved into the owning repo, instead of in the meta-openpower directories
in openbm

Move the service files to the occ-control repo

To align with the OpenBMC direction, the OCC service files are being
moved into the owning repo, instead of in the meta-openpower directories
in openbmc.

Change-Id: Icef03a173249dd559f7ae41a8026b11656004d53
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

beautysh: re-format

beautysh is enabled in the openbmc-build-scripts on Bash/Zsh/POSIX-sh
files to have a consistent formatting. Re-run the formatter on the
whole repository.

Change-Id: Ia3f0f61de

beautysh: re-format

beautysh is enabled in the openbmc-build-scripts on Bash/Zsh/POSIX-sh
files to have a consistent formatting. Re-run the formatter on the
whole repository.

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

show more ...

python: fix flake8 warnings and format with black

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

Use format_as rather than operator << overload

format_as seems to be the preferred way to handle enums
according to fmtlib documentation.

Signed-off-by: Eddie James <eajames@linux.ibm.com>
Change-I

Use format_as rather than operator << overload

format_as seems to be the preferred way to handle enums
according to fmtlib documentation.

Signed-off-by: Eddie James <eajames@linux.ibm.com>
Change-Id: I46946b711468d8bd572669a7b182e6df2326f757

show more ...

Update fmtlib fix

The actual operator overload was missing from the previous fix.

Signed-off-by: Eddie James <eajames@linux.ibm.com>
Change-Id: I64fc9c347ce3ed16eb8bf8d944fbf066b4c923b8

Provide a fmtlib ostream formatter for SysPwrMode enum

fmtlib removed support for automatically formatting enums as integers
in version 9. See https://github.com/fmtlib/fmt/issues/1841.

Signed-off-

Provide a fmtlib ostream formatter for SysPwrMode enum

fmtlib removed support for automatically formatting enums as integers
in version 9. See https://github.com/fmtlib/fmt/issues/1841.

Signed-off-by: Eddie James <eajames@linux.ibm.com>
Change-Id: I6a5d04187e55c94d39d8b5ee334f9a37c93081ce

show more ...

Add journal traces to occ-control PELs

When creating a PEL, the last lines of the journal for the app will get
saved into the PEL for additional debug.

Change-Id: Ifa05a00ffdc57833859d719d0e7d8b81c

Add journal traces to occ-control PELs

When creating a PEL, the last lines of the journal for the app will get
saved into the PEL for additional debug.

Change-Id: Ifa05a00ffdc57833859d719d0e7d8b81ccadb5c8
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

Commit OCC related errors as informational

Since the PELs generated by occ-control will end up triggering a PM
Complex reset, they can get logged as informational. Hostboot/HTMGT will
determine when

Commit OCC related errors as informational

Since the PELs generated by occ-control will end up triggering a PM
Complex reset, they can get logged as informational. Hostboot/HTMGT will
determine when recovery attempts has been exhausted and will log an
unrecoverable 2616 PEL indicating the system is in safe mode.

Change-Id: Iab3d3a5d1605a4fea7362b96f5e05570cd0a1746
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

yaml: move to subdirectory

A pending sdbusplus change (I38b69bc67b6a3d80cda1b508a76e106a50d8ab93)
is going to require YAML to be in a specific subdirectory and not in
the root. Move here in prep fo

yaml: move to subdirectory

A pending sdbusplus change (I38b69bc67b6a3d80cda1b508a76e106a50d8ab93)
is going to require YAML to be in a specific subdirectory and not in
the root. Move here in prep for that change.

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

show more ...

MAINTAINERS: remove file

The MAINTAINERS file is deprecated in favor of OWNERS.

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

sdbusplus:p10: object: don't use 'bool' argument constructor

These were missed in the prior commit to handle this due to this file
only being built when p10 is enabled.

Signed-off-by: Andrew Geissl

sdbusplus:p10: object: don't use 'bool' argument constructor

These were missed in the prior commit to handle this due to this file
only being built when p10 is enabled.

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

show more ...

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are for:
* bus_t
* exception_t
* manager_t
* match_t
* message_t
* object_t
* slot_t

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

show more ...