| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a condition check for fan1 that will only create fan1 associated
functional properties and actively monitor fan1 if the condition passes.
The condition checks the CoolingType is not WaterCooled.
Tested:
Fan1 functional properties are created when condition passes
Fan1 is monitored when condition passes
Change-Id: I0d9a91ceea500f82e4ccad2dbdc887a14286cf95
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The phoshor-fan-monitor-init and phosphor-fan-control-init
package names that the SYSTEMD_OVERRIDE directives were given
don't actually exist, as those apps are just part of the
phosphor-fan-control and phosphor-fan-monitor packages.
Fixup these directives so the fan watchdog configuration is
used for these services which means if they fail the fan
watchdog will trip as desired.
Tested: Check that phosphor-fan-control-init@0.service.d and
phosphor-fan-monitor-init@0.service.d now show up, and killed
fan-control-init and watched that the watchdog triggered.
Change-Id: I1786891ec5da7045b4f0a9f72d5ae29f454fa9a0
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Testing: Used html from Deepak/Rahul that subscribed to
ambient temp sensor and verified the events showed up in
the web socket via my browser console.
Resolves openbmc/openbmc#2881
Resolves openbmc/openbmc#925
Change-Id: I606a68de7b5c76ae6253dc1722d087e953b7ca8b
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
phosphor-gevent will be moved to internal http,
nginx will be the front end web server, and will
redirect all traffic internally to gevent.
test cases passed with...
tox -e palmetto -- tests/test_rest_interfaces.robot
Testing:
- Before testing suite completed in 1:18
After nginx control 443 suite finished in 0:45 seconds
- Also verified web interface still loads when going
to system name in web browser (http and https)
- Verified code update with full 20MB image still works
- Verified dump can be generated and retrieved
- Verified that when the same IP requests 2 image uploads
at same time, Nginx serializes them
Change-Id: Ifcd61cf22bd8f6ea0536fe706eba8e45ec85e7c3
Signed-Off-by: Chris Austen <austenc@us.ibm.com>
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Ultimate goal is to allow anyone to add one or more http services.
Doing this still allows other groups to not be required to use
them
https://lists.ozlabs.org/pipermail/openbmc/2017-September/009231.html
Also note, this commit launches nginx on 8081. Once advice on
how to get it to launch I'll be moving it to 443 and gevent to
8081.
Initially this will only be used in the meta-ibm layer.
Signed-off-by: Chris Austen <austenc@us.ibm.com>
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I34b076898469f2fa9a0186ca3fb3abc77ae52fe7
|
|
|
|
|
|
|
|
|
| |
One use case on some systems for this layer will be to pull
in Nginx
Signed-Off-By: Chris Austen <austenc@us.ibm.com>
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I1ac55bc182e7d24704d0fe3405438d8e1c97f1a4
|
|
|
|
|
|
|
|
|
|
|
|
| |
This enables the fbterm, uart-render-controller and liberation font
packages and configuration to the Romulus build.
This allows the machine to have the host UART output rendered to a
framebuffer and output on the BMC's graphics device up until the host
loads it's PCIe VGA driver.
Change-Id: I01b755a7ba67a5ba5232b9d786d7ce868bab2d0d
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
|
|
|
|
|
|
|
|
|
| |
These packages are now part of meta-phosphor, so they can be removed
from the meta-witherspoon directory.
The feature is still enabled by their inclusion in
meta-witherspoon/recipes-phosphor/images/obmc-phosphor-image.bbappend.
Change-Id: I653e2b3c96bf9e9ab4d607f60355ff68432e855a
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
|
|
|
| |
Change-Id: I5ec2b7fea98ea30cc4af9e14b00f3c1f0c3967fc
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a temp sensor provided by the OCC is faulted, set the fans to full
speed(10500 RPMs)
Tested:
Set Functional property of an OCC provided sensor to `true` and
fans are set to 10500.
Fan speeds drop when the same Functional property is set to `false`
Resolves: openbmc/openbmc#2222
Change-Id: Iddd9049fc9677ca587422a5bf4104c0d3787af1a
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The values are just outside of the allowed margins of the
fan floor and max speed values.
Tested: Test that the thresholds show up in the web UI
and that the web UI shows when they have been
crossed.
Resolves openbmc/openbmc#3114
Change-Id: I6420252d98b974921d6858dbecb3b52f5a8c646a
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a new udev rule to start the phosphor-hwmon instance for
the MAX31785 fan controller differently than other hwmon devices.
Also introduce different hwmon config files for air and water cooled
systems where the water cooled system doesn't use fan 1.
The new udev rule starts a service that will check the cooling type
and then link in the appropriate config file before starting the
actual phosphor-hwmon service.
Note that this phosphor-hwmon service is still stopped the same as
any other phosphor-hwmon service.
Tested: Test that phosphor-hwmon is started using the correct
config files on air and water cooled systems.
Change-Id: Ia93d97df024e7b6bff5eb379ddea4730fe34b55b
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This service will be started in the udev rule for the MAX31785
hotplug add event. It will wait for the phosphor-cooling-type
service to be done, and then run the script to start the
phosphor-hwmon instance for the MAX31785 fan controller after it
chooses the correct config file to use.
Tested: Test that this service correctly runs the script and that
phosphor-hwmon does get started.
Change-Id: I4495f7073d63004d4dd8333df8964d69beaae4ea
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The phosphor-hwmon instance for the MAX31785 fan controller
chip needs to use different config files based on if the system
is air or water cooled, since different numbers of fans are used.
This script will check the cooling type, link in the correct file
based on that cooling type, and then start the actual hwmon
service.
Tested: Tested as part of end to end testing of this feature.
Change-Id: Ie3efee5d3c6671911b505223152d32f3f5c5c259
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change the type of the phosphor-cooling-type service to
a oneshot, and also tell it to remain after exit.
This is so that any services that depend on this won't start
until the the phosphor-cooling-type application is done so
they can rely on the cooling type property being set in the
inventory.
Tested: Checked that a service that depends on this one
and gets the cooling type will read the correct value.
Change-Id: Id4852983a31787b19bee8a2106a46128c294b2ce
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This allows the SoC to switch to the alternate BMC flash device when we
hit a kernel panic.
Tested: Instrumented which chip the system was currently booted from,
booted a kernel with a broken root= configuration to trigger a
panic, observed the system reboot due to a watchdog event,
confirmed the chip used by the reboot was different.
Change-Id: I763ebadc3e26cc9e420e2469277970fbde974459
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
|
|
|
| |
Change-Id: Idaa61f76d942a547eed7a5a34f191e3cc5d99e15
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
| |
Change-Id: Ic8fb916da6f5d047182f7660787d61892929196e
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The OpenPOWER host firmware requires OpenBMC to clear out certain
"volatile" sections of it's flash chip in certain scenarios:
- Any fresh power on
- Any boot/reboot of the server where a certain "volatile" sensor is
enabled
This commit utilizes the obmc-host-start target, which is only run during
fresh power on operations, to ensure the volatile sensor is set. The new
obmc-host-startmin target is then utilized to check for this sensor and
clear the host volatile flash sections if the sensor is set. In this
way, the volatile flash sections are cleared on all fresh power on
operations and in any reboot loop that the host has set the sensor.
Testing:
- Clean Power On (cleared as expected)
Starting Enable the clearing of the Volatile PNOR partitions in host0...
Started Enable the clearing of the Volatile PNOR partitions in host0.
Starting Clear the Volatile PNOR partitions in host0 if Enabled...
Clear /var/lib/phosphor-software-manager/pnor/rw/HB_VOLATILE
Started Clear the Volatile PNOR partitions in host0 if Enabled.
- Host Reboot with host running (cleared as expected)
Starting Clear the Volatile PNOR partitions in host0 if Enabled...
Clear /var/lib/phosphor-software-manager/pnor/rw/HB_VOLATILE
Started Clear the Volatile PNOR partitions in host0 if Enabled.
- Reboot during hostboot, before istep 16 (nothing cleared as expected)
Starting Clear the Volatile PNOR partitions in host0 if Enabled...
Started Clear the Volatile PNOR partitions in host0 if Enabled.
Resolves openbmc/openbmc#2397
Change-Id: I56a897ce8919a95f9e80ab730ecabf880723f1f6
Signed-off-by: Michael Tritz <mtritz@us.ibm.com>
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For IPMI v2.0/RMCP+ sessions, authentication, integrity and
confidentiality algorithms are involved. The algorithms used in
combination are called Cipher Suites. Each cipher suite has the
id as the key and a collection of algorithms.
Each algorithm has a number assigned in the IPMI specification.
Cipher data information can be read by the Get Channel Cipher
suites command, and the RMCP+ messaging Cipher Suite entries
support and the RMCP+ messaging Cipher Suite Entries parameter
of the Get LAN configuration parameters command.
Change-Id: I3460046a0c72acd5cb2f9066f264142a2378120c
Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Match up the GPU power sensor labels with the temperature ones,
so they are both GPU0 through GPU5 instead of P0_GPU0 through
P1_GPU2.
Resolves openbmc/openbmc#3111
Change-Id: I509349049b8528415084e45c09200c16d2628a31
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a follow up commit of 1065559c, that is to use ${EXTRA_ARGS}
instead of "--continue=", so that this option becomes an optional one
without argument.
For services need extra arguments, e.g. Romulus' id-button service,
specify "--continue" in its config file.
Tested: Verify the generated gpio monitor service uses ${EXTRA_ARGS}
environment. On Romulus, checkstop monitor starts without extra
argument, and id_button monitor starts with --continue argument.
Change-Id: I09c5ad7f97f1a40caabfc9ae394e68798396dffe
Signed-off-by: Lei YU <mine260309@gmail.com>
|
|
|
|
|
|
|
|
| |
Add the fbterm, uart-render-controller and liberation font packages to
the witherspoon build.
Change-Id: Ifa20e9455f06f2ae4d7e9547f62e70ea102b4c89
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
|
|
|
|
|
|
|
|
|
| |
This change adds a small daemon to monitor the state of the host (power
and video device status), and start/stop the fbterm UART renderer as
appropriate.
Change-Id: I6c7cf7a88e9a19cd87d6b781a4a1519597882345
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
|
|
|
|
|
|
|
|
|
| |
For text-to-video rendering, we need a font. This change adds a small
overlay to the existing liberation fonts package, to only use the
monospace font.
Change-Id: I7e0bc307dceb95732f94616d9249df34b484b629
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change adds a definition for fbterm, customised for rendering UART
data to a local framebuffer device.
fbterm upstream isn't very active - last release was in 2008. So, use a
newer git tree, which includes a couple of fixes and customisations for
our usage in uart rendering.
Change-Id: Ib19e879283548dbe403401b6fecfaaeb8cf74c1c
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
|
|
|
|
|
| |
Change-Id: I9fbf94ebb374a764bc4afdff9b4293a89bc75830
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
| |
Change-Id: I5247e8c03ba3aac7af517e228cd5ef6554f7e58f
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
This task will generate a single JSON file containing all of
the errors across all of the *.errors.yaml files.
It also compares this error list to the errors in the policy
table and shows which errors are in one but are missing in
the other.
Change-Id: Ic81b4625bf66bd603f15e5a647857b41fec12a52
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is used by the ibm-logging code, and contains
additional information about the error log entries
that can show up on a Witherspoon. This file is maintained
by the IBM service team, and checked in as-is. A script
runs during the build to pull out the fields used by
the BMC code.
It also enables the use of the com.ibm.Logging.Policy
D-Bus interface.
Resolves openbmc/openbmc#2998
Change-Id: Idf08e397f4e93112ece5cbfe9db67f5682637c32
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Installs the ibm-log-manager application and the
com.ibm.Logging.service. The lifetime of this service
matches that of the phosphor-logging service.
After building the code, the recipe also condenses the
policyTable.json file into a version to go onto the flash.
The policyTable.json in this layer is empty, and the system
specific ones should be installed from their own layers by
bbappending this recipe from there.
Change-Id: I5310da06e9beb69c7d53a16d0beb44232a937716
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This recipe adds the function to assert/deassert identify LED Group
when the ID button is pressed:
1. Add phosphor-gpio-monitor for the ID button GPIO key;
2. Start id-button-pressed.service when the key is pressed;
3. The service starts a script to check the identify LED group state,
and assert or de-assert it.
Tested: Assert/De-assert identify LED group by pressing the ID button
on Romulus.
Change-Id: I782dbc15563a313418aa90bee1f48fc97a6d25fb
Signed-off-by: Lei YU <mine260309@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a sensor provided from an OCC contains a value of
0x00(unavailable), the sensor will be removed from dbus by hwmon. An
InterfacesRemoved signal is triggered when this occurs and this enables
fan control to remove that sensor's interface(and associated properties)
from its internal cache used across set speed events.
Tested:
An InterfacesRemoved signal is received for an unavailable sensor
That sensor's interface and properties are removed from the cache
Change-Id: Ia6343688c2824a984d23a9b16ea9e709d5ed2f9e
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The witherspoon power engineer has requested the lowering of the
over-current warning values
Testing: Going to let CI validate basic boot. The power engineer has
done his own more intense testing of these values.
Resolves openbmc/openbmc#3099
Change-Id: I4a2b2bb132dd43e05828e4c0ed3448dfd4237301
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
| |
Change-Id: I91c566d76c4a5c16ebf36d01a26c31db1212e4b8
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
| |
Added "mutability" flag to indicate if a sensor is READ/WRITE/RW.
Change-Id: I3dcc27526adc2a5d6b1206e8beee66093220763a
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 9aeb8f0961e8d431192321c188af62b7e1831ad5.
Systemd is failing to ping the watchdog while shutting down, resulting
in a BMC reset during the shutdown. This results in an unclean shutdown
and side switching to the alternate BMC chip.
Resolves openbmc/openbmc#3094
Change-Id: I3ddc21a3cd3a1d63403786ca63c8671a72bd4c13
Signed-off-by: Eddie James <eajames@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Enable signature verification in the phosphor-software-manager code
for witherspoon. This causes an error to be logged if updating to
an unsigned image, or image signed with a different key than the one
on the system, and if field mode is set, it'll stop the activation
process.
Tested: Signature verification is enforced on witherspoon,
verified error is logged with and without field mode enabled, and
activation is prevented with field mode enabled.
Change-Id: Ifc8f8054f8d852cc16942af9cbf58d60aff3fc33
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
|
|
|
| |
Change-Id: I44b5726279a4b53a285c0da7003a53d49c281e49
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
| |
Change-Id: I7a678b64290c99a53dbecb326be767b00e3b7aeb
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Add /com/ibm and com.ibm to the lists that the mapper
uses to know what paths and interfaces to handle.
As this is in the meta-ibm layer, it is only applied to
IBM systems.
Change-Id: I8c3624f4bff86af178b4d951632019d81dcaaf51
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
| |
These D-Bus interfaces are only for IBM systems.
Change-Id: I62255887d9cb7059472ad33ccf6fddc84d476cf0
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
| |
This is in preparation for adding recipes that
are only for IBM systems.
Change-Id: Id5be84d78c9a01fd2cf9dc2abb68cabfa6cedd51
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
| |
Added "mutability" flag to indicate if a sensor is READ/WRITE/RW.
Change-Id: Ibe8913a0c72d8c97bdb9288a642901cea34f3a23
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Handle dependent service failures:
1. When ambient temperature is unavailable, set default floor speed;
2. When occ services are unavailable, set to max speed.
Tested: Verify that the default floor speed is set when outlet temp
sensor is missing;
Verify that the max speed is set when occ service is killed;
With https://gerrit.openbmc-project.xyz/#/c/9584/, verify that
the floor speed is set back according to outlet temp when the
sensor and occ service are back.
Change-Id: I395f99ac8a6e76fdeb11a189294b096ee6256b76
Signed-off-by: Lei YU <mine260309@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When an EAGAIN return code(11) is returned for any sensor from an OCC,
remove that sensor from dbus.
Functionally requires: https://gerrit.openbmc-project.xyz/#/c/9893/
Tested:
OCC sensors are removed from dbus when EAGAIN is returned
Change-Id: I2fee78ff7876b8750555454436dd7b53eb3a0e78
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
| |
Should only enable this for specific machines, since the watchdog
configuration is machine-dependent.
Resolves openbmc/openbmc#2427
Change-Id: I03169b5dca6dccabe5eef2b5c759fe9b075450ac
Signed-off-by: Eddie James <eajames@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a plugin for systems in the meta-ibm layer
to collect the com.ibm.Logging objects in a dump.
Resolves openbmc/openbmc#3043
Tested: Run dreport and check that the IBM data is there.
Change-Id: Iaa01a07633265c767649caaf8b9b1580f31ce7cd
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
| |
Change-Id: Id9250fda132995b64f29d8e72fe2597915069506
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
| |
Change-Id: I0418f3f06dc5653d8a538bb0268822907a8195ad
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|