Age | Commit message (Collapse) | Author | Files | Lines |
|
Bonnell is a new IBM system which will be supported within the p10bmc
image. Load its device tree into the p10bmc image.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Idad1aed9aebafcca847a1c2a7749bb29f432601f
|
|
The latest pldm code removed the phyla nvram reset services as they are
No longer controlled via BMC, rather they are controlled by the ibm
hypervisor using a bios attribute.
pldm bump contains the following commits :
Jayashankar Padath (1):
oem-ibm: Remove extra bytes in resource dump request
Manojkiran Eda (3):
oem_ibm: Move the NVRAM files to the hostfw directory
oem_ibm : Add HRESET state setid & various states
oem_ibm: Add new file type for Running lids
Pavithra Barithaya (1):
PLDM:OEM Pass service authorization credentials to PHYP
Change-Id: I91ada72493f6d934cdb52e23e88783bcaae853e5
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Added p10bmc specific dev_id.json file to support IPMI IANA number
Testing:
Used curl command to verify IANA number
$ ipmitool -I lanplus -C 17 -p 623 -U user -P password -H host raw 0x06
0x01
00 80 0f 32 02 8d 41 a7 00 4f 42 00 00 00 00
Signed-off-by: Shantappa Teekappanavar <shantappa.teekappanavar@ibm.com>
Change-Id: I6ca8f343c06aac2303137aa5d78145b59d7099ce
|
|
We also need to move the conf-notes.txt files.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ic44e015c0216b526de4fec277ad42f162bca1f33
|
|
```
WARNING: EXTRA_OEMESON:append:p10bmc += is not a recommended operator combination, please replace it.
WARNING: EXTRA_OEMESON:append:witherspoon-tacoma += is not a recommended operator combination, please replace it.
```
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I5dc503b322d6cb90feb6d6261802fd50ebca31b2
|
|
The latest poky commit is requiring us to have all of
our template configs in a subdirectory instead of directly in
the `conf` directory. Without this we end up with errors during
setup like:
```
Error: TEMPLATECONF value (which is .../openbmc/meta-facebook/meta-bletchley/conf) must point to meta-some-layer/conf/templates/template-name
```
Fix this by moving all of our template files into the 'default'
template subdirectory (following the pattern of poky) and modifying
`setup` as necessary to follow.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iecefde73d55acbb6bc63ae3d68c4311adaf327ae
|
|
IBM systems utilize /tmp and /var for a variety of purposes so setup the
health monitor to look at the remaining space in these file systems. No
action at this time other then a log in the journal to provide a clue
when looking at an issue.
Tested:
- Filled up /tmp and verified this log:
“Aug 19 19:59:43 p10bmc health-monitor[326]: ASSERT: sensor Storage_TMP is above the upper threshold critical high”
- Removed big file from /tmp and verified this log:
"Aug 19 20:01:34 p10bmc health-monitor[326]: DEASSERT: sensor Storage_TMP is under the upper threshold critical high”
Signed-off-by: NodeMan97 <corey.hardesty@icloud.com>
Change-Id: I8a7921dffa6b756a4ab19a0e807e558ccd35f996
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
The p10bmc machines require the phal and op-fsi features to be enabled
to function properly.
Tested:
- Confirmed both features enabled on p10bmc machine
- Confirmed only op-fsi was enabled on witherspoon
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Id01cae6246204a025f4401d0d2c26ccda78d7d77
|
|
The phosphor-fan-presence repository added meson support, so start using
it.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: If850d3d379ad82dc8f9691efd7a0561f350543dc
|
|
Adding telemetry bbappend to customize telemetry meson options
for IBM. IBM has different needs than upstream for telemetry capacity
in terms of numbers of sensors readings and rate of requests.
bbappend file allows IBM to stay up to date with upstream Telemetry
while maintain custom build options more suited to IBM's usecases.
Increase meson option for total maximum Metrics/MetricProperties
limit from 200 to 300. For IBM management agent use cases (HMC), the max
number of Metrics needs to greater than 200.
Also for IBM management agent use cases, telemetry reports need a higher
append limit for metric readings. 32768 was determined as
1) a power of 2,
2) higher than HMC's current usage,
3) Ability to collect one hour of data for 250 sensors reading
once every 30 seconds ==> a general guess as to maximum readings
needed in one report.
Testing for maximum metrics increase:
- Monitored CPU usage of bmcweb for both rainier and
everest systems. bmcweb usage spiked for around 2 seconds with 300
MetricProperty MRD.
- MRD was periodic on 15 interval with 300 MetricProperties averaged
over 30 seconds on everest system with 360 sensors. Max bmcweb usage
was 9% for brief instant when MRD post call is made.
Testing for appendLimit increase:
- Set AppendLimit up to 11000 with 150 sensors on 400+ sensor system.
- Monitored 'top' CPU usage for telemetry binary and bmcweb binary
which never ran above 5%. AppendLimit generated new metric entries
for the report.
- Retrieve report did seem to take a little longer as report grew but
not significantly.
- Tested various other lower limits and sensors
Signed-off-by: Ali Ahmed <ama213000@gmail.com>
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
Change-Id: I418c3e90c565770cd59b02c294754ff98df29a55
|
|
The bmcweb bummp adding this meson option merged so this can go.
IBM does not use phosphor-pid-control instead using phosphor-fan-presence
and such.
This Redfish OEM fan data includes PATCHing. Turn this PATCHing off and
the whole Manager OEM Fan data. This has been done downstream already.
We have also observed 500 errors with getting the thermalMode.
"Jan 24 16:34:57 rain534 bmcweb[435]: (2022-01-24 16:34:57) [ERROR
"managers.hpp":1196] GetPIDValues: Can't get thermalModeIface
/xyz/openbmc_project/control/thermal/0"
Tested: No longer see this InternalErrors. Been running with this
downstream.
Tested on a Tacoma.
Before on a PATCH:
curl -k -X PATCH https://$bmc/redfish/v1/Managers/bmc -d '{"Oem":{"OpenBmc":{"Fan":{"Profile":"Acoustic"}}}}'
{
"error": {
"@Message.ExtendedInfo": [
{
"@odata.type": "#Message.v1_1_1.Message",
"Message": "The request failed due to an internal service...
With this change and the bmcweb change (instead see a PropertyUnknown)
curl -k -X PATCH https://$bmc/redfish/v1/Managers/bmc -d '{"Oem":{"OpenBmc":{"Fan":{"Profile" : "Acoustic"} }}}'
{
"error": {
"@Message.ExtendedInfo": [
{
"@odata.type": "#Message.v1_1_1.Message",
"Message": "The property %1 is not in the list of valid properties for the resource.",
"MessageArgs": [
"Oem"
],
"MessageId": "Base.1.13.0.PropertyUnknown",
...
Change-Id: Ic988e2949ad65b31eb0b9bc6ecbdceb997d8cdda
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
|
|
Note: for witherspoon systems these options are still enabled
Tested:
1) Build works for both p10bmc and tacoma
2) Web sockets did not appear and Redfish output was as desired
Signed-off-by: Ali Ahmed <ama213000@gmail.com>
Change-Id: I0aa710433455b4ca5f17012ecd867846d87a3cfb
|
|
The OpenBMC u-boot tree now has support for the AST2500 based Romulus
and can use a single configuration for NCSI and non-NCSI boards.
Change-Id: I6daa155fcafa0128b119d67bf47cf629ac4b7977
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
The IPS systems will use the same OTP configuration but different RoT
keys.
Change-Id: I73ca9a79092cdd74bef509824a4123445ecbd003
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Better communicate whose P10 systems are the target of the
configuration.
Change-Id: I561750df9b32512753d9ce644088c2130ebb6a55
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Enable generating multiple OTP binaries by specifying multiple OTP
configuration files.
Change-Id: I1ce81cd2c1d0f501578b927eb31cce5a37cdcf1f
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Yocto append syntax should use assignment only and not `+=`.
Adjust this to fix the warning:
```
WARNING: ... :append:p10bmc += is not a recommended operator
combination, please replace it.
```
Fixes 6b7c28775d91c65816b1716edefe4e3299e1aa50.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I90ca66a4651b843e3fe96c13ba0242f7b6fca58d
|
|
These are some defaults IBM has been carrying downstream. Send upstream
since they are in the meta-ibm layer.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I239f0cbe29a118f51143974fc9bfaeaeffe80070
|
|
The power supply INPUT_HISTORY is only supported on certain power
supplies. The power supplies that ship with Rainier 2S2U, Rainier 4U,
and Everest support the INPUT_HISTORY.
The MEX I/O drawer power supplies do not support INPUT_HISTORY.
Use associations from Rainier/Everest power supply inventory entries to
the input history aggregation interfaces to facilitate Redfish filtering
of PowerSupplyMetrics for input history.
Change-Id: I9afb2e57fefbf0cc10378bffe2efe4f4548920ca
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
|
|
Patrick Williams (1):
MAINTAINERS: remove file
Change-Id: I28fe4fffae8f0f8fe459b995cad6e404684095d8
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
phosphor-defaults.inc now automatically inherits extrausers and
configures the default root/0penBmc account, so individual platforms no
longer need to specify either explicitly.
Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: Ic2b725f0eaadfafef67069449497e57438fa0fba
|
|
Change the power-on dependency of the fan watch and fan watchdog reset
services to a wants instead of requires so the system isn't powered off
if they were to fail with power on.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I603747083567c1b3335c640b70da97ed7fac5332
|
|
For the moment, phosphor-webui is deprecated and unmaintained. The
replacement for it is webui-vue, which already has feature parity, and
many more features than its predecessor. Several calls have been made
asking if there is anyone still using phosphor-webui and would like to
maintain it into the future, but no such volunteer has stepped up.
Moving to webui-vue is the recommended option these days, and this
patchset is done under the assumption that these platforms would
actually like to move off the deprecated UI, and just don't have patches
to that effect. For the machine maintainers reading this, if you would
like to stay on phosphor-webui, please start that discussion here about
who would like to maintain it.
This commit ports the last few platforms over to webui-vue.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I9c74442b850aceca130545f37d5bc694a4f4424d
|
|
Default was initially set to represent seconds. The interface
documentation says that the value is representative of microseconds
,as well as, the corresponding code in phosphor-state-manager.
Change-Id: I183c5a248748589e172df58d07f2b2c2ed0f554c
Signed-off-by: Corey Hardesty <corey.hardesty@icloud.com>
|
|
Fan/NIC sensor values crossed thresholds
Added support to trigger hard shutdown alarms and power off the
slots of the platform if the Fan/NIC sensors values getting low
compared to threshold values.
Added platform specific service file and script to take
platorm specific actions.
TESTED : Verified all the slots getting powered Off when fan/nic
sensor
values crossed threshold values.
Signed-off-by: Kumar Thangavel <thangavel.k@hcl.com>
Change-Id: I6deb6707e574a375fac8f2379f47661a5a0ed332
|
|
Patrick Williams (2):
OWNERS: switch 'matches' to 'matchers'
sdbusplus: use shorter type aliases
Change-Id: I9e015a77580779f881057fe6969f6608f2a374c3
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
There are a few 'matchers' statements specified in OWNERS files that
do not do anything. git doesn't track directories, so we cannot do
an 'exact' match on a directory. Switch these 'matchers' to be a
'partial_regex' so that any file included in these paths will match.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I47329241bc9e6eba6f4ecd1b92b7ebe5d7230939
|
|
Jiaqing Zhao (1):
Network: Remove deprecated properties in SystemConfiguration
Remove deprecated properties from meta-ibm.
Change-Id: I99464a4737edaa177f51344701e5c578c106c2bd
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
|
|
IBM systems should default to "Restore" for RestorePolicy. This will
ensure a system is automatically restored to the last state prior to the
power loss.
Tested:
- Built p10bmc image and verified generated source file had correct
default in it.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I2bf890b22545665097c678ca52a8b3cd77e9f5b3
|
|
Commit 7298b24 introduced the concept of a phosphor-settings file that
supported multiple hosts. Adapt the meta-ibm override of host settings
to utilize this new concept.
Tested:
- Verified expected PowerCap settings were in generated
settings_manager.hpp file
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Iacccb71c09970d55cfc44deecb40e15782e98c1a
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Id772552ca99651c47778ee2c04ba7f5ce187ec0b
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I4e9748641b861f194c66a8fd168901e73c09a51f
|
|
New yocto has gotten more strict (and found some bugs in our recipes)
that require correct FILESEXTRAPATHS and correct machine installs.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I5511d543be0d2d17eba88ca63afa5372b8fda3a4
|
|
New yocto has gotten more strict and requires the specific paths be set
for machines.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: If7fb3410c2c513b355260ed28869822741b9d344
|
|
This change will disable the IPMI network interface by default on the
first boot of a p10bmc system after a factory reset. After the first
boot the service is prevented from running again and from there on
adheres to the customers settings.
It is necessary that IPMI network services are started for a brief
moment in time to ensure they show up in the systemd service list.
This is important to ensure that the service-config-manager, which is
responsible for enabling/disabling these services can properly locate
them in the systemd service list.
Tested:
- Issued 5 mini-factory resets to mimic a fresh system boot
Verified service-config-manager identified both IPMI service and
socket files for eth0 and eth1
Verified IPMI was disabled by default
Verified could enable IPMI via the web UI and could issue IPMI
commands using ipmitool with a user that was in the IPMI group
Verified that once IPMI was enabled, it stayed enabled through BMC
reboots
Verified when IPMI was disabled via the GUI, it stayed disabled
through BMC reboots
Signed-off-by: Corey Hardesty <corey.hardesty@icloud.com>
Change-Id: Idad03366c1eb11cbfd7f4ad700976c177eb0121f
|
|
The OpenBMC u-boot tree now has support for the AST2400 based Palmetto
and can use a single configuration for NCSI and non-NCSI boards.
Change-Id: I7190b09bdffbfe045bb7c140bd5626932e86bdb6
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
u-boot has been updated to refer to p10bmc instead of rainier. Given
this file is also shared between everest and rainier change the name.
Change-Id: I6a643e09b782dcdbe86bc5a39c0fd14a82be45d8
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
Harvey.Wu (2):
dbus-sdr: fix SEL record Generator ID to fit SPEC
dbus-sdr: fix the vector size in sdrWriteTable and sdrStatsTable
Jian Zhang (1):
Add data types to Value to extend setDbusProperty
Vernon Mauery (1):
fix header install
Willy Tu (4):
cleanup: Remove all warning errors for the Meson build support
build: Add meson build for phosphor-host-ipmi
build: move all shared_library/module to library with version suffix
build: Remove dynamicsensor library if it is not enabled
Change-Id: I92503c6625d1ec2bf88c0a28afac9b4eeb8fbd75
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: Willy Tu <wltu@google.com>
|
|
This commits provides an option to override
the softoff timeout value from 120 minutes to
45 minutes.
Signed-off-by: Sagar Srinivas <sagar.srinivas@ibm.com>
Change-Id: I2c425a97179d425b3bc9d5e59a431f046456e1d9
|
|
Joel Stanley (2):
ARM: dts: Rename Rainier to P10 BMC
ARM: dts: ast2600: Fix indentation
Change-Id: I732324d89b7c0f731160d69942051e4d4cadc778
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
Matt Spinler (1):
Fix typo
Change-Id: I1b1b75dd92fa3bcba7126f4ba9f11c6787b201e4
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Some cards were built without TPMs populated. However, we signed builds
for these cards. Remove the key corresponding to the signatures on these
builds and sign builds for cards with TPMs populated with a subsequent
key to prevent non-TPM builds from being used on TPM-enabled systems.
Change-Id: I511a310750319b0e3dc2e028285a77d8cf07d7c7
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
After updates Gerrit is now hosted at https://gerrit.openbmc.org.
Urls in the documentation are now updated to reflect this change.
Signed-off-by: Corey Hardesty <corey.hardesty@icloud.com>
Change-Id: I3a7f7d338ca1ce600ccf7b41fa03bd62007925ba
|
|
Add a sysctl configuration file to direct sysctl to enable the
Linux kernel's panic on OOM option.
Tested on Everest hardware:
root@ever10bmc:~# cat /proc/sys/vm/panic_on_oom
2
Signed-off-by: Eddie James <eajames@linux.ibm.com>
Change-Id: Iaf6462c8911ee15deafe5a4fc4bc51f9db864afd
|
|
Don't use /usr/bin/env, it's an antipattern.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: If388f5427bb2ca45718d0ca99452bf3f147861dd
|
|
/usr/bin/env is now an antipattern.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I7e8488b984db0258f68107b23a0e3a7ed149f763
|
|
It's time to require this for p10 based machines now
Tested:
- Confirmed by default it is now "Required"
root@p10bmc:~# pldmtool bios GetBIOSAttributeCurrentValueByHandle -a hb_tpm_required
{
"CurrentValue": "Required"
}
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I0959ffad3d4a38b542087be4a6ca6949ff1b4356
|
|
The dbus sink actions will cause phosphor-dump-manager to take a full
BMC dump if the debug action is received on the trigger source.
Change-Id: If57612a75ca7f68ed2636c9b23dc04972d99626d
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
For the same reason as supply-monitor (a dependency on org.open_power
dbus interfaces).
Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: Ibcf52fc180da744cced04d4720036d8bc84bf89c
|
|
This is the recommended keyword to use and it appears to fix a subtle
bug of some sort where the new secure-check is overriding other
features.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Ic724ec7611741d6286926e96cb0b61530cdcf638
|