libpldmresponder: Implement GetStateSensorReadings

Register the getStateSensorReadings method and get the state sensor PDR
structure according to the SensorId property.

Tested with JSON file:
https

libpldmresponder: Implement GetStateSensorReadings

Register the getStateSensorReadings method and get the state sensor PDR
structure according to the SensorId property.

Tested with JSON file:
https://gist.github.com/lxwinspur/6a40abea7330c25e4d49826e890c4be9

pldmtool raw -d 0x80 0x02 0x21 0x01 0x00 0x01 0x00
Request Message:
08 01 80 02 21 01 00 01 00
Response Message:
08 01 00 02 21 00 01 03 00 00 00

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I4faeb1e85f9fd4a2745f592d37be38c28112f5b9

show more ...

Add parsing JSON for State Sensor PDR

According to spec DSP0248_1.2.0: 28.6 and sensor_pdr.json, parse JSON
and generate PDR structure.

Tested with JSON file:
https://gist.github.com/lxwinspur/6a40

Add parsing JSON for State Sensor PDR

According to spec DSP0248_1.2.0: 28.6 and sensor_pdr.json, parse JSON
and generate PDR structure.

Tested with JSON file:
https://gist.github.com/lxwinspur/6a40abea7330c25e4d49826e890c4be9

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I654913b6fa07f34f405f7dd41a5f1ac0ae2706fb

show more ...

Add state sensor configuration to the sensor_pdr.json

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I239d482f95a66be711952e628ef0b0f27f78bdd7

pdr: Implement terminus locator PDR for BMC

This commit creates the TL PDR for the BMC. The TID and MCTP EID for
BMC is static, 1 and 8 respectively. BMC PDRs are assigned the value
of 1 for PLDMTer

pdr: Implement terminus locator PDR for BMC

This commit creates the TL PDR for the BMC. The TID and MCTP EID for
BMC is static, 1 and 8 respectively. BMC PDRs are assigned the value
of 1 for PLDMTerminusHandle. Added the parser for terminus locator
PDR in pldmtool.

Tested:

/tmp/pldmtool platform GetPdr -d 0

nextRecordHandle: 2
responseCount: 19
recordHandle: 1
PDRHeaderVersion: 1
PDRType: 1
recordChangeNumber: 0
dataLength: 9

PLDMTerminusHandle: 1
validity: valid
TID: 1
containerID: 0
terminusLocatorType: MCTP_EID
terminusLocatorValueSize: 1
EID: 8

Change-Id: I596301d6c676b450ae1f2cef872966b4c40d8bae
Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
Signed-off-by: Sampa Misra <sampmisr@in.ibm.com>

show more ...

libpldmresponder: construct BMC PDRs lazily

BMC's PDRs were constructed when the pldm daemon starts up. However, the
pldm daemon might rely on other services to start in order to create
PDRs. To sol

libpldmresponder: construct BMC PDRs lazily

BMC's PDRs were constructed when the pldm daemon starts up. However, the
pldm daemon might rely on other services to start in order to create
PDRs. To solve this problem, construct the PDRs when the first GetPDR
call comes in. From a practical timing perspective, this ensures the
pldm daemon and the services it needs to build PDRs are all up and
running.

An alternative would be to add 'After' directives in pldm's service
file, however that can get complex when some of the services are not
present on all platforms.

Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Change-Id: Id6ac9fd47b293f7e84e3837432b32b0e3c3f8a5a

show more ...

host-bmc PDR flow: support refreshEntireRepository

Support the refreshEntireRepository format of the
pldmPDRRepositoryChgEvent event, to fetch all PDRs of the host.

Signed-off-by: Deepak Kodihalli

host-bmc PDR flow: support refreshEntireRepository

Support the refreshEntireRepository format of the
pldmPDRRepositoryChgEvent event, to fetch all PDRs of the host.

Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Change-Id: Ic91806e35c34b6fb9421df6be7e801d0c8c7c866

show more ...

sensor_event: Correct the D-Bus object path

The hardcoded D-Bus object path for hypervisor network config is
corrected to match the settings object path.

Signed-off-by: Tom Joseph <tomjoseph@in.ibm

sensor_event: Correct the D-Bus object path

The hardcoded D-Bus object path for hypervisor network config is
corrected to match the settings object path.

Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
Change-Id: Iba7ca207376a3d42a88a7715e64dcae28735b592

show more ...

libpldmresponder: create PDR when the D-Bus object path exists

On a system where this D-Bus object path does not exist, it should
not create a PDR when the D-Bus object specified here is not found.

libpldmresponder: create PDR when the D-Bus object path exists

On a system where this D-Bus object path does not exist, it should
not create a PDR when the D-Bus object specified here is not found.

Tested: In the PDR config files, the PDR is successfully created when
the objectPath of the D-Bus exists, otherwise, skipped the current
config and the PDR is not created.

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: Ib59fa79cc28d5712161bfb971c37c86c363bafb0

show more ...

libpldmresponder: PDR: check config dir

Gracefully exist if PDR config directory doesn't exist.

Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Change-Id: Iee4210469f275f574fc9c9b8da183003497

libpldmresponder: PDR: check config dir

Gracefully exist if PDR config directory doesn't exist.

Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Change-Id: Iee4210469f275f574fc9c9b8da183003497bfb2d

show more ...

pldmtool: Implement GetFRURecordByOption

Implement GetFRURecordByOption

Tested:

$pldmtool fru GetFRURecordByOption -i 0 -r 0 -f 0
...
FRU Record Set Identifier: 1
FRU Record Type: 1(General)
Numbe

pldmtool: Implement GetFRURecordByOption

Implement GetFRURecordByOption

Tested:

$pldmtool fru GetFRURecordByOption -i 0 -r 0 -f 0
...
FRU Record Set Identifier: 1
FRU Record Type: 1(General)
Number of FRU fields: 5
Encoding Type for FRU fields: 1(ASCII)
FRU Field Type: 3(Part Number)
FRU Field Length: 7
FRU Field Value: 02CY415
FRU Field Type: 4(Serial Number)
FRU Field Length: 12
FRU Field Value: YA1934319126
FRU Field Type: 5(Manufacturer)
FRU Field Length: 3
FRU Field Value: IBM
FRU Field Type: 8(Name)
FRU Field Length: 16
FRU Field Value: PROCESSOR MODULE
FRU Field Type: 10(Version)
FRU Field Length: 2
FRU Field Value: 22
FRU Record Set Identifier: 2
FRU Record Type: 1(General)
Number of FRU fields: 4
Encoding Type for FRU fields: 1(ASCII)
FRU Field Type: 2(Model)
FRU Field Length: 20
FRU Field Value: M393A2K43BB1-CTD
FRU Field Type: 4(Serial Number)
FRU Field Length: 10
FRU Field Value: 0x367f4d92
FRU Field Type: 5(Manufacturer)
FRU Field Length: 19
FRU Field Value: Samsung Electronics
FRU Field Type: 8(Name)
FRU Field Length: 32
FRU Field Value: DDR4-2666 16GiB 64-bit ECC RDIMM
FRU Record Set Identifier: 3
FRU Record Type: 1(General)
Number of FRU fields: 1
Encoding Type for FRU fields: 1(ASCII)
FRU Field Type: 8(Name)
FRU Field Length: 12
FRU Field Value: powersupply0
FRU Record Set Identifier: 4
FRU Record Type: 1(General)
Number of FRU fields: 1
Encoding Type for FRU fields: 1(ASCII)
FRU Field Type: 8(Name)
FRU Field Length: 12
FRU Field Value: powersupply1

$pldmtool fru GetFRURecordByOption -i 0 -r 0 -f 8
FRU Record Set Identifier: 1
FRU Record Type: 1(General)
Number of FRU fields: 1
Encoding Type for FRU fields: 1(ASCII)
FRU Field Type: 8(Name)
FRU Field Length: 16
FRU Field Value: PROCESSOR MODULE
FRU Record Set Identifier: 2
FRU Record Type: 1(General)
Number of FRU fields: 1
Encoding Type for FRU fields: 1(ASCII)
FRU Field Type: 8(Name)
FRU Field Length: 32
FRU Field Value: DDR4-2666 16GiB 64-bit ECC RDIMM
FRU Record Set Identifier: 3
FRU Record Type: 1(General)
Number of FRU fields: 1
Encoding Type for FRU fields: 1(ASCII)
FRU Field Type: 8(Name)
FRU Field Length: 12
FRU Field Value: powersupply0
FRU Record Set Identifier: 4
FRU Record Type: 1(General)
Number of FRU fields: 1
Encoding Type for FRU fields: 1(ASCII)
FRU Field Type: 8(Name)
FRU Field Length: 12
FRU Field Value: powersupply1

$pldmtool fru GetFRURecordByOption -i 0 -r 0 -f 4
FRU Record Set Identifier: 1
FRU Record Type: 1(General)
Number of FRU fields: 1
Encoding Type for FRU fields: 1(ASCII)
FRU Field Type: 4(Serial Number)
FRU Field Length: 12
FRU Field Value: YA1934319126
FRU Record Set Identifier: 2
FRU Record Type: 1(General)
Number of FRU fields: 1
Encoding Type for FRU fields: 1(ASCII)
FRU Field Type: 4(Serial Number)
FRU Field Length: 10
FRU Field Value: 0x367f4d92
FRU Record Set Identifier: 3
FRU Record Type: 1(General)
Number of FRU fields: 0
Encoding Type for FRU fields: 1(ASCII)
FRU Record Set Identifier: 4
FRU Record Type: 1(General)
Number of FRU fields: 0
Encoding Type for FRU fields: 1(ASCII)

Signed-off-by: John Wang <wangzqbj@inspur.com>
Change-Id: Ia4bf6f938061dad69ab243704cfa9ac8da5fc80e

show more ...

libpldmresponder: Implement GetFRURecordByOption

Implement handler for GetFRURecordByOption command

Signed-off-by: John Wang <wangzqbj@inspur.com>
Change-Id: If2bc90872b2521f1771aa800de6fbce569a5b3

libpldmresponder: Implement GetFRURecordByOption

Implement handler for GetFRURecordByOption command

Signed-off-by: John Wang <wangzqbj@inspur.com>
Change-Id: If2bc90872b2521f1771aa800de6fbce569a5b339

show more ...

Emit the Event.StateSensorEvent signal

When the PLDM daemon receives a state sensor EventMessages, it emits the
StateSensorEvent signal (with TID, sensorID, sensorOffset, eventState,
previousEventSt

Emit the Event.StateSensorEvent signal

When the PLDM daemon receives a state sensor EventMessages, it emits the
StateSensorEvent signal (with TID, sensorID, sensorOffset, eventState,
previousEventState as the signal data).

This commit implements a DBus interface defined at
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-dbus-interfaces/+/33552
to provide StateSensorEvent signal.

Tested: Tested ok in fp5280g2 system.
1、Using pldmtool to send a sensor event message:
pldmtool raw -d 0x80 0x02 0x0A 0x01 0x01 0x00 0x2 0x00 0x01 0x03 0x04 0x05
2、Using dbus-monitor tool we can see the signal being emitted:
signal time=6467.863313 sender=:1.107 -> destination=(null destination)
serial=37 path=/xyz/openbmc_project/pldm;
interface=xyz.openbmc_project.PLDM.Event; member=StateSensorEvent
byte 1
uint16 2
byte 3
byte 4
byte 5

Signed-off-by: Chicago Duan <duanzhijia01@inspur.com>
Change-Id: Ica22e2b16e334a748b60145c527cd70564751d60

show more ...

Build FRU table lazily

The FRU table is created when the PLDM daemon starts and depends on
BMC inventory collection to populate it completely. There is no D-Bus
signal or target that PLDM daemon can

Build FRU table lazily

The FRU table is created when the PLDM daemon starts and depends on
BMC inventory collection to populate it completely. There is no D-Bus
signal or target that PLDM daemon can rely on to figure completion of
inventory. This can cause some of the inventory to not show up in the
FRU table if the inventory collection is not complete when the PLDM
daemon starts.

An easy solution to this problem is to do the FRU table creation
lazily. The FRU table will be created when the FRU commands or Get PDR
command is handled the first time. The entity association PDR's are
created when the FRU table is built. So Get PDR commands expects the
building of the FRU table as a pre condition.

Tested:

Ensured FRU table is created when the GetFRURecordTableMetadata,
GetFRURecordTable and GetPDR command is handled by the PLDM daemon the
first time. Next time already built table is returned for the FRU
commands.

Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
Change-Id: I0deb723f30a30a667d0e80c9f9f6aced5ab23a67

show more ...

common: code re-org

Move common files under a common/ directory.

Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Change-Id: I0eaf91d4ee94c1e78228da0c50892f82e91912fa

pldmd: code re-org

Move code specific to the pldm main app to its own directory.

Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Change-Id: I9341ebb02b7fcf8f62fbbe922e1fa6fe3fbcb9ec

host-bmc: code re-org

Move code specific to Host BMC PLDM communication into its own
directory.

Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Change-Id: I0a88874bdccfe6cd71099ff41b8b5c65491

host-bmc: code re-org

Move code specific to Host BMC PLDM communication into its own
directory.

Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
Change-Id: I0a88874bdccfe6cd71099ff41b8b5c654915316d

show more ...

pldmd: D-Bus to host effecter translation

This commit implements a mechanism to move the Host's boot state
from 'not started' to 'boot complete' by setting the relevant
Host effecter when the associ

pldmd: D-Bus to host effecter translation

This commit implements a mechanism to move the Host's boot state
from 'not started' to 'boot complete' by setting the relevant
Host effecter when the associated D-Bus property is set in the BMC.

Also added an example JSON to match D-Bus to host effecters

Change-Id: I41025d99d2b4b3452d4c51b03efe3750e159328b
Signed-off-by: Sampa Misra <sampmisr@in.ibm.com>

show more ...

clang-format: update to latest from docs repo

Since `Cpp11` is an alias for `Latest` and we should tend towards using the
latest C++ standard, update the C++ standard to Latest.

https://github.com/

clang-format: update to latest from docs repo

Since `Cpp11` is an alias for `Latest` and we should tend towards using the
latest C++ standard, update the C++ standard to Latest.

https://github.com/llvm/llvm-project/commit/e5032567903de19962333c4bf7d2edceaf4f9824#diff-b49a097415dff2837d9626d422c58ba8R82
https://github.com/openbmc/docs/blob/master/style/cpp/.clang-format

Also, other OpenBMC repos are doing the same.

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I449e88bb4e1e262060110e1a8f3e8db3ddfc74cf

show more ...

fru: Fix typo in a json example

The root path of PIM is /xyz/openbmc_project/inventory

Signed-off-by: John Wang <wangzqbj@inspur.com>
Change-Id: I15fe343c2e9e49b756a8f7ff77fd34eb7ae62e95

fru: Add default configuration for fru record

Add mapping between the D-Bus properties to the PLDM FRU
properties can be in the code. Use this mapping when fru
jsons are not available

Tested:

Pick

fru: Add default configuration for fru record

Add mapping between the D-Bus properties to the PLDM FRU
properties can be in the code. Use this mapping when fru
jsons are not available

Tested:

Picked this change[30624], and tested on fp5280g2, saw the D-Bus
properties is mapped to the PLDM FRU properties as expected

[30624] https://gerrit.openbmc-project.xyz/c/openbmc/pldm/+/30624

1. Remove the fru configurations:
root@fp5280g2:rm -rf /usr/share/pldm/fru

2. Start pldmd
root@fp5280g2: pldmd&

Test
root@fp5280g2: pldmtool fru getfruRecordTable
FRUTableLength : 95
Total number of Record Set Identifiers in table : 2
Total number of records in table : 2
FRU Record Set Identifier: 1
FRU Record Type: General
Number of FRU fields: 4
Encoding Type for FRU fields: ASCII

FRU Table Data:
FRU Field Type: Part Number
FRU Field Length: 7
FRU Field Value: 02CY296
FRU Field Type: Serial Number
FRU Field Length: 12
FRU Field Value: YA1934415306
FRU Field Type: Manufacturer
FRU Field Length: 3
FRU Field Value: IBM
FRU Field Type: Version
FRU Field Length: 2
FRU Field Value: 22
FRU Record Set Identifier: 2
FRU Record Type: General
Number of FRU fields: 3
Encoding Type for FRU fields: ASCII

FRU Table Data:
FRU Field Type: Model
FRU Field Length: 20
FRU Field Value: 9ASF1G72PZ-2G6D1
FRU Field Type: Serial Number
FRU Field Length: 10
FRU Field Value: 0x1514b700
FRU Field Type: Manufacturer
FRU Field Length: 17
FRU Field Value: Micron Technology

Signed-off-by: John Wang <wangzqbj@inspur.com>
Change-Id: I0e5ec3e4e1ede17c06ef716c6d6231156eddfd44

show more ...

libpldmresponder: bios: fixed resp in setAttribute

Add response encode in setBIOSAttributeCurrentValue.

Signed-off-by: Xiaochao Ma <maxiaochao@inspur.com>
Change-Id: I12ce91a4dd00c6ddce1f41ed5206b7

libpldmresponder: bios: fixed resp in setAttribute

Add response encode in setBIOSAttributeCurrentValue.

Signed-off-by: Xiaochao Ma <maxiaochao@inspur.com>
Change-Id: I12ce91a4dd00c6ddce1f41ed5206b793f4926629

show more ...

bios: check the attribute value to update

Check the attribute value to update according
to the configuration from attribute table

Signed-off-by: John Wang <wangzqbj@inspur.com>
Change-Id: I7a1e8a6e

bios: check the attribute value to update

Check the attribute value to update according
to the configuration from attribute table

Signed-off-by: John Wang <wangzqbj@inspur.com>
Change-Id: I7a1e8a6ede33bc536dcb76081a88d47185919eea

show more ...

Changed get/setDateTime to bmc instead of host

We need to remove the TimeOwner feature and set the date time
to in manual mode to bmc.

Ref: https://lists.ozlabs.org/pipermail/openbmc/2020-April/021

Changed get/setDateTime to bmc instead of host

We need to remove the TimeOwner feature and set the date time
to in manual mode to bmc.

Ref: https://lists.ozlabs.org/pipermail/openbmc/2020-April/021409.html

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I59e7b85c2ded885410a1dd9d67fdb3424dab41a8

show more ...

Fix failure to get file path of bios

Failed to get the biosTable dir, and caused the process to exit when the
removeTables method was called.

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-

Fix failure to get file path of bios

Failed to get the biosTable dir, and caused the process to exit when the
removeTables method was called.

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I24ead5703b6026d1fa7da8060b447457ef9eef91

show more ...

pldm_events: Update PlatformEventMessage implementation

The PlatformEventMessage command implementation is extended to handle
PLDM events for state sensors for which PDR is available. The sensor
inf

pldm_events: Update PlatformEventMessage implementation

The PlatformEventMessage command implementation is extended to handle
PLDM events for state sensors for which PDR is available. The sensor
info is looked up with the HostPDRHandler and the BMC action is looked
up in the StateSensorHandler

Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
Change-Id: I6ff5c31bf05dc4b31f39ac25b6e1b399766013b6

show more ...