Age | Commit message (Collapse) | Author | Files | Lines |
|
```
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
|
|
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
|
|
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>
|
|
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
|
|
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
|
|
Matt Spinler (1):
Fix typo
Change-Id: I1b1b75dd92fa3bcba7126f4ba9f11c6787b201e4
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
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
|
|
IBM manufacturing has requested this new feature be enabled on p10bmc
machines.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I47fcc24667dfcccc4314548b29772227280eef4c
|
|
Patrick Williams (1):
build: enable C++20
Change-Id: Ib2bc73d1f7227f9a8494598be123eac66c2f8ace
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
phosphor-dbus-monitor still uses autoconf and has a hard dependency on
phosphor-snmp, causing phosphor-snmp to be brought in on systems that
have no interest in using SNMP. The vast majority of systems do not
supply any config for phosphor-dbus-monitor, so the usage of it does not
seem wide enough to enable the IMAGE_FEATURE by default.
- Remove 'obmc-dbus-monitor' from IMAGE_FEATURE by default.
- Clean up all systems that manually remove the dbus-monitor feature or
package.
- Explicitly add the 'obmc-dbus-monitor' to the following systems:
- meta-ibm: all
- meta-inspur: fp5280g2
- meta-quanta: s6q
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iad6c4104466e21d1b9843dac964a634019eb6935
|
|
This commit enables the hypervisor n/w configuration service
for the IBM systems and when it is enabled, the service file
gets created that starts after the bios manager service.
Tested By:
Service file is created for IBM machines and the service
is up and running when the image is flashed onto the system
Signed-off-by: Asmitha Karunanithi <asmitk01@in.ibm.com>
Change-Id: Iac6d4a52df2a85fa3c8b60cc42fa6905337769de
|
|
1) Move ibm specific plugins from recipe to tools/dreport.d/ibm.d/plugins.d
in the repository
2) Move openpower specific plugins from recipe to
tools/dreport/opepower.d/plugins.d in the repository.
4) Modified to install the plugins from the machine specific
plugin folder from the repository
Reference links
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52852/
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52479
Includes:
phosphor-debug-collector: srcrev bump 02634e5531..124d31f09c
Marri Devender Rao (3):
openpower:dreport: add guarded resolved and ephemeral records to BMC dump
ibm:dreport add vpd persistent data to BMC dump
dreport: Fix network and timedate plugin failures during BMC dump generation
Signed-off-by: Marri Devender Rao <devenrao@in.ibm.com>
Change-Id: I0bd350b9b564c4e98a6765902de76274f90500e4
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I2887152b9451e8f0d812ba6cc5cf20f0ab61f38a
|
|
Patrick Williams (1):
sdbusplus: object: don't use 'bool' argument constructor
Change-Id: I59cee1db5bd10776d6f49ad58056e008d7409fa7
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
phosphor-state-manager: srcrev bump 1ab2b6cea6..211d972d75
Adriana Kobylak (1):
chassis: Monitor the PowerSystemInputs for power status
Allen.Wang (2):
host-state-manager: Add multi-host support
chassis-state-manager: Correct poweroff state Unit
Andrew Geissler (2):
ensure bmc-state-manager stays in quiesce state
generate bmc dump on monitored systemd target fail
Brandon Wyman (1):
Correct pinhole reset logic for chassis state
Mike Capps (1):
Set Critical severity for Blackout errors
Patrick Williams (7):
filesystem: use non-experimental version
meson: remove stdc++fs dependency
meson: add subproject wraps for current dependencies
sdbusplus: object: don't use 'bool' argument constructor
meson: add missing dependencies and wraps
crit-service: update defaults for templated Host/Chassis
sched-host-tran: support multi-host
Potin Lai (3):
chassis-state-manager: Add multi-chassis support
discover-system-state: add multi-host support
host-state-manager: fix host0 failure with reading wrong object path
shamim ali (1):
power-on-hours: ensure not incremented when system off
Some commits went into phosphor-state-manager support that starts the
framework to support multiple chassis and hosts within a system. This
commit templates both the host and chassis services.
Change-Id: I19764aa0c625d4d28d8d0e672658f1a3910335af
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
|
|
Since P9 systems need only to stop the link failover, and P10 systems
need only to set the gigabit link speed, split the nsci-netlink
service into two and make it per-interface. Use the systemd network
check to make sure the link is up first. Use a helper script to fetch
the ifindex for the network interface
Signed-off-by: Eddie James <eajames@us.ibm.com>
Change-Id: I6558787a2234575b528864bb71705964a63c0498
|
|
The powercap range is currently hardcoded by mrw-override-settings.yaml
This commit will remove the hardcoded range and allow the full range of
values to be set.
The actual limits are available via dbus which can be used to validate
the user power cap.
Change-Id: I71e6009d5bc288c0d8480db26ce9e6782bdbd167
Signed-off-by: Chris Cain <cjcain@us.ibm.com>
|
|
New systems should be using the 'use-json' model by default, but it
currently requires a bbappend in order to enable it. Automatically
detect based on the PREFERRED_PROVIDER of virtual/...-config-native if
the intended use is JSON or YAML.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ie674d1b901395b4fd41838b6e978ca1ff00b0277
|
|
Add the power supply temperature readings to the hwmon configuration and
the Rainier and Everest assocation files.
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: I00e7fde4a58da553b9374733b22b8608d0c9da41
|
|
This takes the default critical services list from meta-phosphor and
appends on extra services IBM is interested in.
Ideally, there would be an easy way to just append an extra file to the
command line but this is going to need some more thought. For now, just
override the entire file.
Tested:
- Booted witherspoon up in QEMU and verified new json file was loaded
and that stopping one of the new critical services resulted in a
dump, error log, and BMC Quiesced state.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Icf5db5331072cbcbd0359e0b789b664e31423124
|
|
This commit adds p10bmc specific configuration files to display
CPU and DIMM FRU information. Without these configuration files
the default phosphor recipe configuration files are used, and we
do not get FRU info for all configured CPUs and DIMMs.
The changes are meta-ibm specific, the image was built and tested
against our simulation model.
Signed-off-by: Shantappa Teekappanavar <sbteeks@yahoo.com>
Change-Id: I60dd8ce05941105dfaf6b5e67fa8c7b0bb06534a
|
|
The power_mode and power_ips dbus objects are being consolidated and
moved under the org.open_power.OCC.Control service.
Their default values will now be set via entity-manager.
Verified on Everest hardware.
Change-Id: Ibfa1e55126862db3521855ad0743a1423e555dac
Signed-off-by: Chris Cain <cjcain@us.ibm.com>
|
|
Providing `opkg` in the image allows us to easily install packages
after-the-fact, usually to debug systems in the lab.
This is not a mechanism for providing firmware updates.
No specific configuration is supplied to enable its use, and hosting of
package archives is left as an exercise to the user. That said, bitbake
generates its own package archive in the course of creating the output
image. It also provides a `package-index` target, which after execution,
allows you to expose the archive with no more work than:
```
$ python3 -m http.server --directory tmp/deploy/ipk 8080`
```
The addition of opkg with the chosen options increases the uncompressed
filesystem size by 3855360 bytes (~3.68MiB).
Change-Id: I7731a4ebe08249e2216e8ca1984f8e8d87558687
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|