Fix pldmd verbosity command line argument

This commit would attempt to fix the command line argument --verbose.

The fact that verbose option requires an argument 0 or 1 is not
particularly intuitiv

Fix pldmd verbosity command line argument

This commit would attempt to fix the command line argument --verbose.

The fact that verbose option requires an argument 0 or 1 is not
particularly intuitive and a documented OpenBMC anti-pattern.

https://github.com/openbmc/docs/blob/master/anti-patterns.md#non-standard-debug-application-options-and-logging

Resolves : https://github.com/openbmc/pldm/issues/7
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: I50d82ce492bd815b589627d3713b1a7fc1db997b

show more ...

make pldmd a dbus systemd service

With the introduction of PLDM supporting the new
xyz.openbmc_project.Condition.HostFirmware interface and other services
depending on that interface being on D-Bus,

make pldmd a dbus systemd service

With the introduction of PLDM supporting the new
xyz.openbmc_project.Condition.HostFirmware interface and other services
depending on that interface being on D-Bus, it's time to convert the
pldmd service to a D-Bus service.

This allows other services to put a "After=pldmd" in their service and
be assured that pldmd has put its service on the bus before the service
is considered done.

Tested:
- Verified with the other "bmc-reset-host-up" commits that this fixed a
synchronization issue between phosphor-state-manager and pldmd.

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

show more ...

systemd: Remove mctp-demux dependency

mctp-demux-daemon is undergoing a transition from service to socket
activation. Move the ownership of the service relationship into the
service units provided b

systemd: Remove mctp-demux dependency

mctp-demux-daemon is undergoing a transition from service to socket
activation. Move the ownership of the service relationship into the
service units provided by libmctp. This will end in a reversal of the
dependencies with pldmd connecting to the @mctp-mux socket to launch
mctp-demux-daemon on-demand.

Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I7cd3dbaf40a0bc9535b647efb9a0a0d4e2702192

show more ...

Remove Inventory Manager dependency for pldm

- Inventory Manager was added as dependency when pldm
daemon used to build the BMC FRU Record table at start.

- As the design was later changed , we n

Remove Inventory Manager dependency for pldm

- Inventory Manager was added as dependency when pldm
daemon used to build the BMC FRU Record table at start.

- As the design was later changed , we now build the BMC FRU
Record table only after we get the first GETPDR command from
host (happens during Host Poweron operation).

- We can assume that the Inventory manager is all up and
running before the host is poweredon, so we can remove the
inventory dependency from the pldm dependency list.

Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: Iee667d488e03f4055a02fb845a2432c22abe4e95

show more ...

Add a new pldm verbosity option

- The intent behind this commit is to add a new
compile time meson option to control the pldm verbosity.

- The value that is given for meson option goes into the

Add a new pldm verbosity option

- The intent behind this commit is to add a new
compile time meson option to control the pldm verbosity.

- The value that is given for meson option goes into the
default systemd environment file which is picked by the
pldmd service unit before it starts.

- As the pldmd consumes the default pldmd file at runtime,
we should be able to alter the verbosity by changing the
content in /etc/default/pldm_verbosity file.

Tested By:
1. meson buildir
check the pldmd file (make sure verbosity is 0 by default)
2. meson builddir -Dverbosity=1
check the generated pldm_verbosity file (make sure verbosity is 1)
3. Flash the image & check if the pldm is loaded with the right verbosity
that is set during compilation.
4. Change the verbosity in the /etc/default/pldm_verbosity file & restart
pldmd.service, and check if the daemon picks up the change.

Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: Ica5607b801143b2806d343e3b82b043f078fcd22

show more ...

Upstream pldm-hypervisor services under OEM

- The intent behind this commit is to create a folder
for all the pldm services(including the oem services)
& installs the respective services base

Upstream pldm-hypervisor services under OEM

- The intent behind this commit is to create a folder
for all the pldm services(including the oem services)
& installs the respective services based on the meson
configuration.
- This commit would also remove a redundant systemd dependency
which was missed in earlier cleanup.

Tested By:
1. meson buiddir -Doem-ibm=enabled/disabled
2. ninja -C builddir
3. ninja install

Post the install step, was able to see the respective systemd
unit files in the systemd_unit_dir (lib/systemd/system)

Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: I4d9e093be25ac7a8a94bcccc9f8c1e7d3de1c68c

show more ...