clang-format: update latest spec and reformat

Copy the latest format file from the docs repository and apply.

Change-Id: Ic23653be3ed404980fd0e723d9ed8354e2b32579
Signed-off-by: Patrick Williams <p

clang-format: update latest spec and reformat

Copy the latest format file from the docs repository and apply.

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

show more ...

Logging.Entry: use AdditionalData property again

Change I7cc9591895f219471ac098387f72a35c18e6c750 finished the
migration in phosphor-logging so that AdditionalData is now
a dictionary (as is Additio

Logging.Entry: use AdditionalData property again

Change I7cc9591895f219471ac098387f72a35c18e6c750 finished the
migration in phosphor-logging so that AdditionalData is now
a dictionary (as is AdditionalData2). Use AdditionalData so
that phosphor-logging is free to delete AdditionalData2.

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

show more ...

Logging.Entry: use AdditionalData2 metadata dict

phosphor-logging's dbus entries now expose the metadata as a
`dict<string,string>` rather than requiring additional processing
by callers to split th

Logging.Entry: use AdditionalData2 metadata dict

phosphor-logging's dbus entries now expose the metadata as a
`dict<string,string>` rather than requiring additional processing
by callers to split the metadata. Support this property in selutility
and eliminate the string processing code.

Tested: Compile test.

(There is no Tested statements in the git-log that explain how to
use this code. I do not have a system that uses these OEM SELs.
The code as-is is broken when the phosphor-logging commit is merged
because there is a new property type which will fail to parse in the
dbus message unpacking call.)

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

show more ...

Remove unused variables

Using clang-tidy, it was detected that some variables are no longer
in use. This commit remove unused variables.

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

Remove unused variables

Using clang-tidy, it was detected that some variables are no longer
in use. This commit remove unused variables.

Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: I382e6ce07e66dc5ad082be9b36d217a037e609b5

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: I44441096113929ce96eb1439e2932e6ff3c87f27
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

Modify clearSEL

After clearing the log, adding the log does not start from 1,
use deleteAll method to clearSEL.

Tested :Run `ipmitool sel clear&&ipmitool sel list`:
1 | 05/23/21 | 09:01:11 UTC | E

Modify clearSEL

After clearing the log, adding the log does not start from 1,
use deleteAll method to clearSEL.

Tested :Run `ipmitool sel clear&&ipmitool sel list`:
1 | 05/23/21 | 09:01:11 UTC | Event Logging Disabled #0x8b | Log area reset/cleared | Asserted

Signed-off-by: Lotus Xu <xuxiaohan@bytedance.com>
Change-Id: Ic1e63c3334adf77f54f36bebfed459138a215f06

show more ...

Add sel entry

Add sel entry for ipmi raw command

Tested:`ipmitool raw 0x0a 0x44 0x00 0x01 0x02 0x00 0x00 0x00
0x00 0x20 0x00 0x04 0x14 0x8a 0x6f 0x00 0x00 0x00` for power_button
pressed,`ipmitool s

Add sel entry

Add sel entry for ipmi raw command

Tested:`ipmitool raw 0x0a 0x44 0x00 0x01 0x02 0x00 0x00 0x00
0x00 0x20 0x00 0x04 0x14 0x8a 0x6f 0x00 0x00 0x00` for power_button
pressed,`ipmitool sel list` can see
1 | 01/24/21 | 02:02:06 UTC | Button #0x8a | Power Button pressed | Asserted

Signed-off-by: Lotus Xu <xuxiaohan@bytedance.com>
Change-Id: I5a6bff109e5d12707a3a74570e6b7a584c7117bf

show more ...

sel: Add OEM SEL Record data structure

The code only supports SEL Event Records. Re-define the data structure
to use union to support different kinds of SEL records:
* SEL Event Records
* OEM SEL Re

sel: Add OEM SEL Record data structure

The code only supports SEL Event Records. Re-define the data structure
to use union to support different kinds of SEL records:
* SEL Event Records
* OEM SEL Record, type C0-DF
* OEM SEL Record, type E0-FF

Signed-off-by: Lei YU <yulei.sh@bytedance.com>
Change-Id: I7cf69c3e1d47664815c029308a52c62049c8e691

show more ...

storagehandler: move get & reserve SEL to new API

Rewrite "Get sel info & Reserve SEL" commands to use the new API.

Tested:
1. verified ipmitool sel info is same both before and
after the changes
C

storagehandler: move get & reserve SEL to new API

Rewrite "Get sel info & Reserve SEL" commands to use the new API.

Tested:
1. verified ipmitool sel info is same both before and
after the changes
Command: ipmitool sel info
Output:
SEL Information
Version : 1.5 (v1.5, v2 compliant)
Entries : 0
Free Space : 65535 bytes or more
Percent Used : unknown
Last Add Time : Not Available
Last Del Time : Not Available
Overflow : false
Supported Cmds : 'Delete' 'Reserve'

Command: ipmitool raw 0x0a 0x40
Output: 51 00 00 ff ff ff ff ff ff ff ff ff ff 0a

2. verfied the Reserve SEL command
ipmitool raw 0x0a 0x42
01 00
ipmitool raw 0x0a 0x42
02 00

Change-Id: I37d602293066274a3fc417ad7e59cea3c73315f7
Signed-off-by: jayaprakash Mutyala <mutyalax.jayaprakash@intel.com>
Signed-off-by: Richard Marian Thomaiyar <richard.marian.thomaiyar@linux.intel.com>
Signed-off-by: jayaprakash Mutyala <mutyalax.jayaprakash@intel.com>

show more ...

remove usage of sdbusplus::message::variant

sdbusplus has had its alias of std::variant in place for long
enough. This changes all ipmid references to use std::variant
directly instead of the sdbusp

remove usage of sdbusplus::message::variant

sdbusplus has had its alias of std::variant in place for long
enough. This changes all ipmid references to use std::variant
directly instead of the sdbusplus alias.

Tested-by: building and running ipmid

Change-Id: Id5b4136d4589aa598815edd3ef4202e64a7698e2
Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>

show more ...

storagehandler: move Clear SEL to new IPMI API

Rewritten "Clear SEL" command to use the newly
introduced IPMI provider API.

Tested:
verified operation of clear SEL, and queried
the sel list before

storagehandler: move Clear SEL to new IPMI API

Rewritten "Clear SEL" command to use the newly
introduced IPMI provider API.

Tested:
verified operation of clear SEL, and queried
the sel list before & after clear SEL
execution.

ipmitool raw 0x0a 0x47 0x08 0x00 0x43 0x4c 0x52 0x0
01 // response

Change-Id: Id8b8fa351c9d42632f71ce3445a63491b8ef4d16
Signed-off-by: Pradeep Kumar <pradeep1x.kumar@intel.com>
Signed-off-by: Richard Marian Thomaiyar <richard.marian.thomaiyar@linux.intel.com>

show more ...

storagehandler: move delete SEL to new API

Rewritten "Delete SEL" command to use the newly
introduced IPMI provider API.

Tested:
Verified the behavior to be same as previous change.
ipmitool sel de

storagehandler: move delete SEL to new API

Rewritten "Delete SEL" command to use the newly
introduced IPMI provider API.

Tested:
Verified the behavior to be same as previous change.
ipmitool sel delete 5
Unable to delete entry 5: Requested sensor, data,
or record not found

Change-Id: I3b40be056ed1c9b3bb5ac75387c694abe16403b5
Signed-off-by: Pradeep Kumar <pradeep1x.kumar@intel.com>
Signed-off-by: Richard Marian Thomaiyar <richard.marian.thomaiyar@linux.intel.com>

show more ...

move types.hpp to ipmid/types.hpp for export

types.hpp is required by utility.hpp, which is exported, so it needs to
be exported as well. This moves it to the include/libipmid directory,
changes the

move types.hpp to ipmid/types.hpp for export

types.hpp is required by utility.hpp, which is exported, so it needs to
be exported as well. This moves it to the include/libipmid directory,
changes the Makefile to export it, and changes all the files that
include it so it can be found in the right place.

Change-Id: I30ec365446e4de466c266ec4faa327478460ec05
Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>

show more ...

Fix compile error with g++ 7.3

Get the below error message when compiling with g++ 7.3:
selutility.hpp:131:6: error: 'chrono' in namespace 'std' does not name a type

Change the include order to fix

Fix compile error with g++ 7.3

Get the below error message when compiling with g++ 7.3:
selutility.hpp:131:6: error: 'chrono' in namespace 'std' does not name a type

Change the include order to fix this issue

Change-Id: Ieaac302dd2e411a71736cbbe3f92431e30f3f5ea
Signed-off-by: Yong Li <yong.b.li@linux.intel.com>

show more ...

add .clang-format

Change-Id: I7c2a527b4751a560703a61fcbe9638b150546af5
Signed-off-by: Patrick Venture <venture@google.com>

Add typedef for Property name

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

Add API to Read the DBUS logging entry object paths

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

Add Clear SEL command

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

Add Delete SEL entry command

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

Add Get SEL entry command

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

Add Get SEL Info command

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

Get the timestamp from the log entry

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

Convert a log entry into a SEL record

Check the DBUS logging entry for callout associations, if there
is a mapping between the inventory path and sensor number then
populate the SEL record. Otherwis

Convert a log entry into a SEL record

Check the DBUS logging entry for callout associations, if there
is a mapping between the inventory path and sensor number then
populate the SEL record. Otherwise log a SEL against the sytem
event sensor.

Resolves openbmc/openbmc#1516

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

show more ...

Create IPMI SEL record from Logging DBUS Entry

The Log Id, resolved field and the timestamp is read from the DBUS
entry and mapped to the SEL record. The remaining fields are
populated from the gene

Create IPMI SEL record from Logging DBUS Entry

The Log Id, resolved field and the timestamp is read from the DBUS
entry and mapped to the SEL record. The remaining fields are
populated from the generated code which maps an association to the
sensor number.

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

show more ...