Fix: GetSdr nextRecord can not reach lastRecord

The output of `ipmitool sdr elist` is missing the last
fru message because nextRecord can not reach lastRecord

Tested: `ipmitool sdr elist' works as

Fix: GetSdr nextRecord can not reach lastRecord

The output of `ipmitool sdr elist` is missing the last
fru message because nextRecord can not reach lastRecord

Tested: `ipmitool sdr elist' works as expected

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

show more ...

Add Threshold bit to sensor capabilites

[3:2] Sensor Threshold Access Support
00 - No thresholds
01 - Thresholds are readable
10 - Thresholds are readable and setable

Change-Id: I8ba1ca

Add Threshold bit to sensor capabilites

[3:2] Sensor Threshold Access Support
00 - No thresholds
01 - Thresholds are readable
10 - Thresholds are readable and setable

Change-Id: I8ba1ca4fd108b530c3c1c6637636c5a8ae5682d4
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

sensors: Set event messages enabled bit

Should be set to 1.

Tested-by: ipmi get sensor reading raw cmd

Change-Id: I7597cef9a12ec2839983fda3310b4d4071e690f2
Signed-off-by: James Feist <james.feist@

sensors: Set event messages enabled bit

Should be set to 1.

Tested-by: ipmi get sensor reading raw cmd

Change-Id: I7597cef9a12ec2839983fda3310b4d4071e690f2
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Add fan profile host support

Implement the fan profile portions of get / set fan
profile ipmi commands. These commands have been overused
with too much debug information, currently we only plan
to s

Add fan profile host support

Implement the fan profile portions of get / set fan
profile ipmi commands. These commands have been overused
with too much debug information, currently we only plan
to support the part that the host needs.

Tested-by: Modified the settings on the host and they
were persisted correctly on the next boot

Change-Id: I5736703123154193b8e193ccb14fc2ffb9f5320b
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Fix boost process call for SEL Clear again

The boost::process::seach_path() call to find journalctl caused the
library to fail to register the OEM IPMI commands. This change
uses the hardcoded path

Fix boost process call for SEL Clear again

The boost::process::seach_path() call to find journalctl caused the
library to fail to register the OEM IPMI commands. This change
uses the hardcoded path to journalctl instead.

Tested: Verified that the OEM commands are getting registered from
the library and that SEL Clear executes the journalctl call.

Change-Id: Ibfe3aff170269b4395d9ab5ad5dc81ff68b63886
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

Fix boost process call for SEL Clear

The boost::process::system call to journalctl for SEL clear was failing
from not finding the 'journalctl' command. This change add a
boost::process::search_path

Fix boost process call for SEL Clear

The boost::process::system call to journalctl for SEL clear was failing
from not finding the 'journalctl' command. This change add a
boost::process::search_path to search for the full path to journalctl
which fixes the system call.

Tested: Before the change SEL Clear returned 0xFF. After the change
SEL Clear returns successfully and executes the journalctl call.

Change-Id: I03510608a705c034638a1cabf71f5c2ed24b14ec
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

Remove support for Set SEL Time

Set SEL Time isn't supported, so override it to always return
0xC1.

Tested: Verified that Set SEL Time with zero or more parameters
returns 0xC1.

Change-Id: I0d27a9

Remove support for Set SEL Time

Set SEL Time isn't supported, so override it to always return
0xC1.

Tested: Verified that Set SEL Time with zero or more parameters
returns 0xC1.

Change-Id: I0d27a991442745d953ae4c39d94f7d077995a647
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

Implement get/set platform serial port parameters command

This command is used for BIOS to set/get host serial host configuration.

Tested:
set config:
ipmitool raw 0x32 0x90 1 0
get config:
ipmitoo

Implement get/set platform serial port parameters command

This command is used for BIOS to set/get host serial host configuration.

Tested:
set config:
ipmitool raw 0x32 0x90 1 0
get config:
ipmitool raw 0x32 0x90 0

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

show more ...

IPMI COMMAND: Get LED status

LED status includes system identify/amber/green status.
They are handled by led.controller and defined by Physical.interface.yaml

uint8_t status; //LED Status

IPMI COMMAND: Get LED status

LED status includes system identify/amber/green status.
They are handled by led.controller and defined by Physical.interface.yaml

uint8_t status; //LED Status
//[1:0] = Reserved
//[3:2] = Status(Amber)
//[5:4] = Status(Green)
//[7:6] = System Identify
//Status definitions:
//00b = Off
//01b = Blink
//10b = On
//11b = invalid
TestBy: ipmitool raw 0x30 0xb0

Change-Id: Ifce7c037f332cbe3fafd19e4e0acd9da6e06d12b
Signed-off-by: Kuiying Wang <kuiying.wang@intel.com>

show more ...

Make reservationID of 0 invalid

You must first get a reservationID before doing a non
zero offset sdr read, to do this, let it be initialized
to zero, but make zero not allowed later on.

Tested-by:

Make reservationID of 0 invalid

You must first get a reservationID before doing a non
zero offset sdr read, to do this, let it be initialized
to zero, but make zero not allowed later on.

Tested-by:
Commands acted correctly with ID 0

~# ipmitool raw 0xa 0x23 0x0 0x0 0xa 0x0 0x1 0x8
Unable to send RAW command (channel=0x0 netfn=0xa lun=0x0 cmd=0x23 rsp=0xc5): Reservation cancelled or invalid
~# ipmitool raw 0xa 0x23 0x0 0x0 0xa 0x0 0x0 0x8
0b 00 0a 00 51 01 3b 20 00 0a

Change-Id: I6efed09eb71bd69433f53c8ec530061775d1b087
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Add SEL commands for Jounal-based SEL entries

This change adds support for SEL IPMI commands using
journal-based SEL entries.

It adds Get SEL Info, Get SEL Entry, Add SEL Entry, and
Clear SEL, to g

Add SEL commands for Jounal-based SEL entries

This change adds support for SEL IPMI commands using
journal-based SEL entries.

It adds Get SEL Info, Get SEL Entry, Add SEL Entry, and
Clear SEL, to get and add entries using the journal.

This is going into intel-ipmi-oem for now because of conflicts
with existing SEL and sensor numbering implemenations. Once
the implemenations can be aligned, these commands can move to
the common IPMI implementation.

Tested: Verified that each of the above commands works as
expected.

Change-Id: Ib7c3832dfaf2bd60e828e368e029a2754c774b07
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

intel-ipmi-oem: Update with new ipmid/api.h path

The non-Yocto build is failing after the changes to the
ipmid-api.h path, so this change updates to the new
ipmid/api.h path.

Tested: Built with and

intel-ipmi-oem: Update with new ipmid/api.h path

The non-Yocto build is failing after the changes to the
ipmid-api.h path, so this change updates to the new
ipmid/api.h path.

Tested: Built with and without Yocto and confirmed both
complete successfully.

Change-Id: If00684c5ca65d99879f6d606709c82d80044af95
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

Move std::experimental::nullopt_t to std

Now that we're on std::optional, use std::nullopt_t instead.

Change-Id: I81c959f3696746089b3177ed1af5f5de89e3f0f0
Signed-off-by: Ed Tanous <ed.tanous@intel.

Move std::experimental::nullopt_t to std

Now that we're on std::optional, use std::nullopt_t instead.

Change-Id: I81c959f3696746089b3177ed1af5f5de89e3f0f0
Signed-off-by: Ed Tanous <ed.tanous@intel.com>

show more ...

Change sensorutils to be header-only

In the future sensorutils may need to move to a common repo, so
making it header-only so it can be shared and moved more easily.

Change-Id: I901be5e7721a5fa0f5e

Change sensorutils to be header-only

In the future sensorutils may need to move to a common repo, so
making it header-only so it can be shared and moved more easily.

Change-Id: I901be5e7721a5fa0f5e308145d55ca3998b11811
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

storage-device: fix fru-device well known name

This has been updated, fix it here too.

Change-Id: I667582fc2c276bd768ef793f90e3af1be7389503
Signed-off-by: James Feist <james.feist@linux.intel.com>

Fix read std::vector message issue.

Using sdbusplus::message::variant<std::vector<>> to read
vector property message through dbus, and using
sdbusplus::message::variant_ns::get<std::vector<>> to
get

Fix read std::vector message issue.

Using sdbusplus::message::variant<std::vector<>> to read
vector property message through dbus, and using
sdbusplus::message::variant_ns::get<std::vector<>> to
get vector property.

TestBy: ipmitool raw 0x30 0x9a

Change-Id: Ide0efdd76c5280c4cfbf942c28b2c87219ba7155
Signed-off-by: Kuiying Wang <kuiying.wang@intel.com>

show more ...

Fix variant usage

Replace variant.get with ::get<type>(variant)

Change-Id: I42e690e0da4a17afe7cc9c3d0835bbf530abbe58
Signed-off-by: James Feist <james.feist@linux.intel.com>

Update variant usage to use visit

This is required for std::variant

Change-Id: Id1e59cb316b710762b1ffa90cc7af616eaedff53
Signed-off-by: James Feist <james.feist@linux.intel.com>

Move getSensorConnection to header file

Move getSensorConnection to header file, so that the same
can be used by ipmi-providers Set Sensor override command

Change-Id: Ic6c313eb2bc07710c175b7908647a

Move getSensorConnection to header file

Move getSensorConnection to header file, so that the same
can be used by ipmi-providers Set Sensor override command

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

show more ...

Implement Intel get/set shutdown policy IPMI OEM commands

Tested By:
ipmitool raw 0x30 0x60 1
ipmitool raw 0x30 0x62

Change-Id: I8ca7429b569a465420c9b878fe29f8617c8ef76c
Signed-off-by: Yong Li <yon

Implement Intel get/set shutdown policy IPMI OEM commands

Tested By:
ipmitool raw 0x30 0x60 1
ipmitool raw 0x30 0x62

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

show more ...

Fix display of sensor units

An earlier change modified getSensorTypeStringFromPath() and
removed the comparison function for the sensor units map.
Because they are char* and the type string failed,

Fix display of sensor units

An earlier change modified getSensorTypeStringFromPath() and
removed the comparison function for the sensor units map.
Because they are char* and the type string failed, the default
sort and find map functions don't work and sensor units appear
as unspecified.

This change fixes the type string and adds the char* comparison
function back in so find() can correctly find the sensor unit.

Tested: Ran the 'sensor list' command and verified that the units
are correctly displayed.

Change-Id: Iac29b763dfd899ced9cb8885ddfa0f2edb8c4d64
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

Add Intel-specific IPMI OEM commands

Includes various OEM commands to get and set info on Intel
platforms (e.g. processor error configuration, power restore
delay, PCIe add-in card info, etc.).

Cha

Add Intel-specific IPMI OEM commands

Includes various OEM commands to get and set info on Intel
platforms (e.g. processor error configuration, power restore
delay, PCIe add-in card info, etc.).

Change-Id: I65f1e610a57f127088cc09bf993c82e7247be193
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

Add Intel-specific IPMI FRU commands

Includes commands that are required to support the 'ipmitool
fru print' command.

Change-Id: If7768de1441cc620ad6db4c99c4254305cf5bbbc
Signed-off-by: Jason M. Bi

Add Intel-specific IPMI FRU commands

Includes commands that are required to support the 'ipmitool
fru print' command.

Change-Id: If7768de1441cc620ad6db4c99c4254305cf5bbbc
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

Add Intel-specific IPMI sensor commands

Also includes SDR storage commands that are required to support
the 'ipmitool sensor list' command.

Change-Id: Id1830097d93882114085fce723f0b92367b2db48
Sign

Add Intel-specific IPMI sensor commands

Also includes SDR storage commands that are required to support
the 'ipmitool sensor list' command.

Change-Id: Id1830097d93882114085fce723f0b92367b2db48
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

Intel OEM commands

Intel platform defines a set of OEM commands. They are required by
Intel BIOS.

This change adds 5 OEM commands:
1.get chassis identifier
Return chassis serial number

2.set syste

Intel OEM commands

Intel platform defines a set of OEM commands. They are required by
Intel BIOS.

This change adds 5 OEM commands:
1.get chassis identifier
Return chassis serial number

2.set system GUID
Sets the system GUID obtained according to the format of the IPMI
2.0 Get System GUID command. While the setting is persistent in the
BMC, the BIOS may choose to over-write the value during POST.

3.set BIOS ID
BIOS version string

4.get device info
BMC/HSC/ME version

5.get AIC(add-in-card) FRU
Add-in-card FRU info

Tested: Test with ipmitool raw command on Intel S2600WF
Platform. BIOS calling is also verified by checking ipmi handler
log (journalctl -ef).

Change-Id: I658c5845cd1252f55dc5d24dfc142281b6a38363
Signed-off-by: Jia, Chunhui <chunhui.jia@linux.intel.com>

show more ...