| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
| |
Change-Id: Ie2060c060d24a09bef4344882ffc98fc13e70981
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
| |
Change-Id: I44e2841df9e7a13c350785b7e487bdc95a68cb9e
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
| |
Change-Id: I2a944653ad4a7318c0cdf8884fcea2e460b44dcd
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
| |
Add the sync manager which watches for files to be synced
to the alternate BMC chip.
Change-Id: I543dacf389629c024b35d1b855745bbcc0bbd848
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
|
|
|
| |
Change-Id: I75cd0dcc7f42ae34cab232312f1f453f28cd480d
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
| |
Change-Id: Id843f7bceaaf76797d138f5e40325d288642572a
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
| |
Change-Id: I20e9eff239665a76be1813cd1aa8bb49eee04382
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Enable signature verification in the openpower-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: PNOR 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: I6b8b74f146066da058e137779faf9af157f7131b
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make both romulus-ipmi-inventory-map-native and
romulus-ipmi-fru-read-inventory-native recipes use the same yaml config.
Resolves: openbmc/openbmc#3134
Tested: Verify the generated fru-read-gen.cpp reflect the yaml config
and ipmi fru command prints the correct FRU information.
Change-Id: If52a883e0901d44cea473ac67e89d4361cd5354c
Signed-off-by: Lei YU <mine260309@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The OCC related sensor fault events should be subscribed after the OCCs
active precondition state passes. This keeps from registering for OCC
provided sensor fault signals prior to the OCC driver being bound and
active.
Tested:
OCC fault sensor signals only registered for after being active
Change-Id: I032ebb0145b5d16e4d54e4873429916e05d3cf48
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
|
|
|
| |
Change-Id: I396a84f6a3e01f882dbdcdcf7fef6fa9177f9d82
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Set the default value of the TPMEnable sensor to true on
Witherspoon systems to catch systems that have a missing
TPM card populated.
Tested: Verified the settings value for TPMEnable is set
to true (1) when the BMC boots up.
Change-Id: I445e67dc53d3bac5dc7f6f613ac03202aab567c4
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Add the definition for the verify_pnor_signature package
to the base openpower-software-manager recipe.
This package is disabled by default.
Tested: Built witherspoon image.
Change-Id: I8154e53d1db902fab824cc52b0ddccae21ccfd1a
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The power values being displayed were basically instantaneous
readings as measured by the APSS monitor device, and as such
could show a lot of peaks and valleys that would be undesirable
to surface.
In the future they may get added back in with some averaging done.
In the meantime, the user can use the nvidia-smi command line tool
on the host to get the proper power values.
Tested: No GPU power sensors in D-Bus.
Resolves openbmc/openbmc#3183
Change-Id: Ie2d92560867f29dbf852c53c55b8d371f5fef06f
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
| |
The GPU power hwmon sensors recently changed names to match
the GPU temperature sensor naming conventions. Update this
YAML to reflect that.
Change-Id: I5100ee514bed3ec7901deddcfd3bd9d619d8cd51
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The code will now copy the correct air or water cooled
config file to the correct name so phosphor-hwmon can find it.
It was just a link before, but that would cause problems with
code updates as this directory is preserved, so if older
code was put back on the link would point to nothing and
phosphor-hwmon would never start for the MAX31785.
Now with the copy the old code can still pick up this file.
The only hole this has is if someone goes from the new code
back to the old code, and then swaps air/water cooling, as the
old code will then pick up the config file generated by the
newer code, which now won't match the actual config. This
would be a very unlikely scenario.
Change-Id: I45db3c3ac8265caae30163a4e90035189e861e82
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
| |
Pick up the latest RAS policy table, which is used to
add the EventID and Description fields to error logs on
Witherspoon.
Change-Id: I18c0e855372d438a5c5171fcf6fbf56bd7376aed
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Testing in certain network environments has show the 10s
timeout is too small. It's unclear if the issue is with the
client or server side. When it hits, it occurs during the
upload of BMC and PNOR images.
The error on the client side is a errno 104 (Connection reset
by peer). The error on the server side is a 408 which in nginx
indicates a timeout for the client_body_timeout setting or the
client_header_timeout setting. It's doubtful the client_header
could be the issue here so only the client_body_timeout was
focused on.
The send_timeout value can also result in a dropped client connection.
The default for both the client_body_timeout and the send_timeout is
60s. In openbmc these were changed to 10s to match with OWASP
guidelines. This commit bumps them up to 30s to address this issue.
Change-Id: I7e360ed03fc533e5ca77f829859a82737fe9d5bf
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
| |
This is required for all systems that use hostboot.
Change-Id: I1ff15e4538fa64e369e5ac4b7fb45b68b828f98d
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
|
|
|
|
|
| |
This installs the system service and bash program.
Change-Id: If6d73a9ac37d20782dfc0f86f3170b84371461a7
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
|
|
|
|
|
|
|
| |
This runs the hb_settings script at boot time in order to configure the
SIO registers. It only needs to be done once at boot, as the registers
stick and are not changed in normal operation of the BMC.
Change-Id: I1b24836f2c7e6dfee0a665ea77b6ca7b810553ef
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
|
|
|
|
|
|
|
|
|
|
| |
This shell script is used to configure the SIO registers that are read
by Hostboot at boot time in order to configure settings before IPMI and
other forms of communication are possible.
In the past these were hardcoded in the kernel, which is not an option
now that we support non OpenPower machines in OpenBMC.
Change-Id: If0f4e62cf28849648d1b1611da42e469b786eb86
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
|
|
|
| |
Change-Id: I4b4d98c36e34de0e53e98d0f5eb83558ba049c86
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The OpenPower part number and serial number should be
coming from the VP and VS keywords in the OPFR record.
Resolves openbmc/openbmc#3174
Tested: Check that the serial and part number fields are
filled in with valid values in the inventory.
Change-Id: I06109a24518975c54be6bf3ec6feb66b89be100a
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
| |
Change-Id: Icb65579a2a8ef454223ae5c61c0cbf13b4340fe4
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change to using an indirect lookup with the *_function_id file
to determine what the OCC's hwmon power values represent, similar
to how the OCC temps used the *_label files.
The current code was hardcoding them, assuming hwmon would number
them the same every time, but that is not the case.
The function IDs are documented here:
https://github.com/open-power/docs/blob/master/occ/OCC_P9_FW_Interfaces.pdf
Resolves openbmc/openbmc#3152
Change-Id: Id4c50c7175a8108c317d9c301ad4a682a4f4c89f
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
|
|
|
|
|
|
| |
Initial a new platfom for Power 9 system
Change-Id: Ia20d44037b46baffad78b7b7eb164064b564a982
Tested: Verified lanyang image build
Signed-off-by: Brian Yang <yang.brianc.w@inventec.com>
|
|
|
|
|
| |
Change-Id: I4dd9d87e8f9b70ea6fdcdd017f433ea62b9ee54f
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
|
|
|
|
|
|
|
|
|
| |
-commit #f24faab provides basic infra for FRUs which are accessible
through host but host is not sending the details for those FRUs.
-GPU FRUs are of type mentioned at commit #f24faab.This commit
hardcodes the WITHERSPOON GPU FRU info into YAML.
Change-Id: I783bf77d8e95abf687d2605e2c97a5bb10504c70
Signed-off-by: Nagaraju Goruganti <ngorugan@in.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Witherspoon requires an dev_id.json file whose content is partially
derived from data provided by the os-release package. os-release is
updated for each commit, as some of its content (VERSION and
VERSION_ID) can be derived from `git describe`. As dev_id.json was
provided by the phosphor-ipmi-host package, every commit transitively
triggered a rebuild of phosphor-ipmi-host in order to satisfy
Witherspoon's requirements.
Always rebuilding phosphor-ipmi-host is unhelpful for several reasons:
* It needlessly reduces CI throughput, as it is likely the commits in
question do not modify the phosphor-ipmi-host package.
* GCC suffers from what appears to be an unfixable[1] bug[2] that causes
phoshor-ipmi-host to consume large (>5GiB) amounts of RAM when
compiling some (at least Witherspoon) sensor configurations.
To avoid this, separate the configuration files out into
virtual/phosphor-ipmi-config and phosphor-ipmi-config packages that
phosphor-ipmi-host RDEPENDS on. Witherspoon provides an alternative
implementation in witherspoon-ipmi-config to mangle dev_id.json to its
particular requirements.
A virtual is used rather than a simple bbappends for Witherspoon, as the
bbappend approach breaks builds of machines other than Witherspoon if
Witherspoon is built first: The Witherspoon-specific dev_id.json file is
deployed in its mangled form into e.g. a Zaius image. Specifically, the
following sequence will trigger the issue:
$ TEMPLATECONF=.../witherspoon.conf . openbmc-env
$ bitbake obmc-phosphor-image
$ rm -rf conf
$ TEMPLATECONF=.../zaius.conf . openbmc-env
$ bitbake obmc-phosphor-image
[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80290#c26
[2] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80290
Change-Id: Ib9629fc77b29e2deeab3f1c3a145d9e966c14ec4
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
|
|
|
|
|
|
|
| |
Having bt in the image was causing license errors.
As this dependency will be replaced with KCS, set the
provider to empty for now.
Change-Id: Ifd41faffb5e8a9e0bf4f4dad1a987b71ba18a744
Signed-off-by: James Feist <james.feist@linux.intel.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
| |
Change-Id: I1dd5497890eb613ec0a23d523bd27b5153d5e33b
Signed-off-by: Andrew Geissler <geissonator@yahoo.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>
|