Age | Commit message (Collapse) | Author | Files | Lines |
|
Each bit in CapabilitiesFlags dbus property is a chassis
capability. Setting a capability overwrites other capabilities.
Read-and-update of the property results in race condition/confusion.
To tackle this, added a boolean property for each of chassis
capability. The new properties will be read and updated in ipmi
command handling code.
The capabilities listed below are supported by Intel and so enabled
the corresponding bits/properties by default.
1. Intrusion sensor (bit 0)
2. Front panel lockout (bit 1)
3. NMI (bit 2)
The original CapabilitiesFlags property will be deprecated once the
changes are made to use new booleans in all the repos.
Tested:
------
root@intel-obmc:~# ipmitool raw 0x00 0x00
07 22 44 66 88 aa
Change-Id: I7f9ec00c1bf94d550dd000634011ef8e7a460967
Signed-off-by: Karthick Sundarrajan <karthick.sundarrajan@intel.com>
|
|
During stress test, netipmid service will crash with "bad executor" error,
the current Character Accumulate Interval is too small,
after increase it to 100, no more crash issue.
Tested:
Start "ipmitool sel list", "ipmitool sensor list"
and "ipmitool sol looptest 200 500" at the same time,
no "sol looptest fail" error
Change-Id: Idf8ba1fdc3478bda408cf668f671411a02574b68
Signed-off-by: Yong Li <yong.b.li@linux.intel.com>
|
|
bmcweb uses the software interface for bios version query in redfish,
to align with it, the settings service implements the software interface,
this reference needs to be changed too
Tested:
AC cycle the host, BiosVersion is the correct bios version:
https://$BMCIP/redfish/v1/Systems/system
Change-Id: I619de9ff1a2356b7da1c59cfa139344cc2114eeb
Signed-off-by: Yong Li <yong.b.li@linux.intel.com>
|
|
bmcweb uses the Software.Activation interface to query the bios version,
implement the Software.Activation
and Association.Definitions to support the query.
Tested:
https://$BMCIP/redfish/v1/Systems/system contains the BiosVersion:
"BiosVersion": "NA",
Change-Id: I77fdefc5962bfc7715b125ccfec0ec57e9dd0f55
Signed-off-by: Yong Li <yong.b.li@linux.intel.com>
|
|
Added "RequestedApplyTime" to settings service which
will be used in firmware update. This will give option
for user to set the firmware image activation like
activate image 'Immediate' or Activate image on 'OnReset'
etc.
Tested:
Changed the setting using Redfish, rebooted BMC and
validated the preserved 'RequestedApplyTime' setting.
Change-Id: Ib08bb82fcc651f5489d69928ddaa7e1f05ebfdcf
Signed-off-by: Vikram Bodireddy <vikram.bodireddy@linux.intel.com>
|
|
Match it so the waits on things are the same.
Tested: name changed
Change-Id: I88171677a39d8a36fd2976122fe4f725b029735e
Signed-off-by: James Feist <james.feist@linux.intel.com>
|
|
This gets initialized in IPMI to 0 every boot anyways,
might as well just make it not persist.
Tested: Doesn't persist
Change-Id: I068cc0fc48533f334752da68d8f51b208cfdbe48
Signed-off-by: James Feist <james.feist@linux.intel.com>
|
|
{} only initalizes the std::optional, not the thing
in the optional. Intialize it correctly.
Tested: Warnings went away, on reboot changes persisted
Change-Id: I3c40317f077d241f0dc938910a28876f911d8021
Signed-off-by: James Feist <james.feist@linux.intel.com>
|
|
System needs to keep the NMI signal event source by this Property, this
interface is used to support the access the Property with default value.
it's convenient for other repos to use it by moving from "com.intel" to
"xyz.openbmc_project".
Tested
it's tested in system for IPMI OEM command - get/set NMI source
Change-Id: I2df4e577eff30b4a82b2face0c6dead2a20a6e2d
Signed-off-by: Chen Yugang <yugang.chen@linux.intel.com>
|
|
Change-Id: Ib76f920c16519d25bbfa93281445104cbe8a290e
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
|
|
RestartCause is handled by power-control, so remove it from
settings.
Tested:
Combined with the power-control changes, triggered software, command,
button, watchdog, and power restore restart types and confirmed that
the restart cause was correctly reported.
Change-Id: I748c70749bdb7169e483758e6af7a58fef2319a5
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
|
|
Tested:
Combined with the IPMI command, confirmed that when I enable
and disable reset on SMI, the value is correctly stored in
the setting.
Change-Id: I83be31b9edb4758639ab1e242752ae38ef180085
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
|
|
This enables rest api logs
Tested:
https://gerrit.openbmc-project.xyz/c/openbmc/bmcweb/+/22699
Change-Id: Idbdf460ab1655f0d9c3d7a1eded99b3f30f133b4
Signed-off-by: James Feist <james.feist@linux.intel.com>
|
|
NMISourse should be disabled as default, and enabled by NMISource signal
tested
test by NMI Button and NMISources by OEM command.
Change-Id: I920f4610a02f29818e75d43cd042c3ce1aa0b0a5
Signed-off-by: Chen Yugang <yugang.chen@linux.intel.com>
|
|
Add Cold Redundancy configs to settings, so that these configs will not reset
to default after AC cycle.
Tested:
Change properties ColdRedundancyEnabled, ColdRedundancyStatus, PeriodOfRotation
RotationAlgorithm, RotationEnabled, RotationRankOrder by ipmi or redfish
commands, do an AC cycle, use ipmi or redfish commands to get these properties
again, the properties value is not reset to default.
Change-Id: Iba8a82d4bf31ceeabf53c0c5f87705133dee18b0
Signed-off-by: Cheng C Yang <cheng.c.yang@intel.com>
|
|
Add setting to be used by dimm offset
Tested: Used dimm offset ipmi command to set setting
Change-Id: I82e2df30c2093b84756b1cb50360b77940fc408c
Signed-off-by: James Feist <james.feist@linux.intel.com>
|
|
Add the 32-bit POHCounter into settings,
it is used to store the host power on hours.
Tested:
Run “Ipmitool chassis poh” command without errors
Change-Id: I97fa8abdc10d72afb57f870a398f5f9abd5830ef
Signed-off-by: Yong Li <yong.b.li@linux.intel.com>
|
|
issue: set/get system boot options command return 0xff.
This IPMI command set/get system boot options commands
worked fine before. After default.yaml has been
removed and used default.hpp instead, it runs into issue
because some Properties related with boot options are
missed in default.hpp while transferring from default.YAML
to default.hpp.
Now add these Properties to default.hpp, it works fine.
tested
test by IPMI set/get system boot options commands
set boot option:
ipmitool raw 0x00 0x08 0x05 0x80 0x18 0x00 0x00 0x00
get boot option:
ipmitool raw 0x00 0x09 0x05 0x00 0x00
Change-Id: I465ae877fb6c906144127fb0932088cca4aca70d
Signed-off-by: Chen Yugang <yugang.chen@linux.intel.com>
|
|
|
|
Folder names cannot be the same as file names or we
will get conflicts.
Tested: settingsd no longer had issues on efe3
Change-Id: I057358b8a6f8a0ec98c177be8d7396d726438cc4
Signed-off-by: James Feist <james.feist@linux.intel.com>
|
|
This adds the settings needed for the Processor Error Config.
It stores the CATERR and ERR2 reset configuration and the current
count of CATERRs for each CPU and total crashdumps.
Tested:
Verified that the settings are available on DBus.
Combined with the IPMI change, verified that the Get and Set
Processor Error Config commands can get the data and update the
reset configuration both from IPMI directly and from BIOS setup.
Change-Id: I659ec31f59251c03cfd34a19fd0b1c5e20487d48
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
|
|
This adds a setting that defaults to "Unknown" and is updated to
"True" or "False" based on the boot status provided by u-boot on
the command line.
Tested:
Verified that the AC Boot setting is created with the default value
of "Unkown" on each boot.
Combined with the AC Boot check change, this makes the power
restore policy behave as expected.
Change-Id: I6358956464a71078fd8e3ff989a6fcf0b17b1028
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
|
|
|
|
|
|
RestrictionMode property will be managed by prov-mode-mgr
and hence need to remove it from settings daemon.
Change-Id: Ifaf693c1c21226cad9ec84dd992d678ccd83f2f6
Signed-off-by: Richard Marian Thomaiyar <richard.marian.thomaiyar@linux.intel.com>
|
|
Add SOL channel support sol eth0, sol eth1, sol eth2 to phosphor-settings, so
that phosphor-settings can store different channel config for SOL.
Tested:
SOL configuration are stored in xyz.openbmc_project.Settings
path /xyz/openbmc_project/ipmi/sol/eth0 /xyz/openbmc_project/ipmi/sol/eth1
/xyz/openbmc_project/ipmi/sol/eth2
And with ipmi-host and ipmi-net patch, SOL Set/Get config parameter command
can work correctly.
Change-Id: I5ab31c9ab30d8c212b061c9aa3205c8a6fd9fe23
Signed-off-by: Cheng C Yang <cheng.c.yang@intel.com>
|
|
This change adds a flag that allows a setting to not persist.
Tested:
Verified that a setting that is set to not persist gets created
with the default value on each boot.
Combined with the AC Boot parameter change, this makes the power
restore policy behave as expected.
Change-Id: I0a8e8237864ab7ab315c0be6df23b424f0a1ccb3
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
|
|
System need to keep the NMI signal event source by this Property, this
interface to support the access the Property with default value.
test
it's tested in system for IPMI OEM command - get/set NMI source
Change-Id: I8e0c199efe1a12844abf57220ded5e2ac2a2c9a4
Signed-off-by: Chen,Yugang <yugang.chen@linux.intel.com>
|
|
This adds restart cause to settingsd
Tested: was able to set it using watchdog timer
Change-Id: Icc4b9b8734e2b7e87de6cd7068e7f99b87de3f98
Signed-off-by: James Feist <james.feist@linux.intel.com>
|
|
This implements an asio based, non-yaml settingsd. This
should make it so we don't have to go through the process
of creating yaml anymore.
Tested: Modifed thermal_mode setting and on reboot it
persisted
Change-Id: Ie385064aa07838acb16ecd6cc86778d570d586cc
Signed-off-by: Feist, James <james.feist@linux.intel.com>
|