Age | Commit message (Collapse) | Author | Files | Lines |
|
There were a number of places that inherited this class but did not
use it. A number of classes that did use it did not inherit it.
This was possible because the class was pulled in by the distro
configuration but that has been removed here as well.
Tested: Built and verified a witherspoon image
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Change-Id: I73511cdcc609875b4fe016967c1b84c5200fb703
|
|
Change-Id: I6b55d9993e132d38fb78f4cd2ac21bd8431d37a6
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
When the BMC sends a system power cap to the host, the host
validates it is in a certain range or it will reject it. So,
the BMC also checks the range before sending it at all.
This range comes out of the MRW (machine readable workbook)
XML, and is stored as output power. The user, however,
inputs the power cap value as input power, and so the range
the BMC code checks it against must also be in input power.
To support this, the script that pulls the power cap min
and max values out of the MRW for use in settings validation
will now convert those output power values into input power
by applying the power supply derating factor.
Resolves openbmc/openbmc#2944
Tested: Check that the power cap range that the settings manager
uses is now the input power versions.
Change-Id: Id5c5851d081ad706fda73054b49e67a93d57f76e
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
In the previous review it was noted that it didn't make much sense
for host-check to be it's own provided function in OpenBMC. It's more
of a recommended piece of the base host state function.
Change-Id: I024472e96ea00f411e8348e0221c210f84700019
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Move OpenBMC to Yocto 2.4(rocko)
Tested: Built and verified Witherspoon and Palmetto images
Change-Id: I12057b18610d6fb0e6903c60213690301e9b0c67
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Change-Id: I58c2b601a49d1fc4a179a326a27c7591a0592cc0
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Id72206de9c229cf30d2c51d25bc8c856ff318028
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I5311255a9d45846fdf774177d848635286d24226
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Update to list groups per event along with not requiring the zone
conditions be defined for an event that should be generated for all.
Tested:
Generated code contains events in correct conditions
Single group event generation remains unchanged
Two groups within event included in same generated event
Resolves openbmc/openbmc#2630
Change-Id: I35502f7fa361c93f367f940c260a160742f160b1
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
Alphabetized the whitelist file.
Change-Id: I5d6de780c17524409585680a7c9dea297e7c317f
Signed-off-by: Patrick Venture <venture@google.com>
|
|
Reset boot progress and OS status sensors if host is not
running
Resolves openbmc/openbmc#2804
Change-Id: I1fee2bf7776ae957e87452d9cfa7a4fca647b46f
Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
|
|
This download service is used on systems that do not implement
the UBI code update feature. Therefore it should run at the same
priority as all other services. On the UBI enabled system that
doesn't use it (witherspoon), its been pulled from the image.
Resolves openbmc/openbmc-tools#13
Change-Id: I14693dadda375f9d0108e4fccb1a614dae7388e1
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I06fe51e53e614f24857cedb5c58da68a4d60016b
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I2c8c832a43d3c072f0497d8f483a77ea8f84e76e
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Id49de6ce721cf1b8d7b5194960af69f0e7779598
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Move OpenBMC to Yocto 2.3(pyro).
Tested: Built and verified Witherspoon and Palmetto images
Change-Id: I50744030e771f4850afc2a93a10d3507e76d36bc
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Resolves: openbmc/openbmc#2461
|
|
As with MRW the use of the UBI image scheme is distro policy.
Convert the existing machine feature to a distro feature.
Enable the new distro feature on the systems that use
it(Witherspoon).
Add a distro override and fix-up MF checks to use this override
instead for improved readability.
Change-Id: I57f3bc244523eff5bc043caff8365e334d98cb30
Tested: Built a witherspoon image and validate manifest
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
As with MRW the use of the UBI image scheme is distro policy.
Convert the existing machine feature to a distro feature.
Enable the new distro feature on the systems that use
it(Witherspoon).
Add a distro override and fix-up MF checks to use this override
instead for improved readability.
Tested: Built a Witherspoon image and validated image
Change-Id: I8ab03115bbfc2ecc77cff5c9eb8628903ae88051
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Using MRW or not is distro policy. As such remove the obmc-mrw
machine feature.
Do not enable the remaining obmc-mrw distro feature in the base
phosphor distro configuration.
Create new distro configs for the systems that use MRW today
(i.e. Witherspoon) that enables the obmc-mrw distro feature and point
the local.conf.sample at the new distro configuration.
Change-Id: If4cedc745b3ee942518fbfd4d5cf2cd5b60f346d
Tested: Built a Witherspoon image and validate the manifest
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Add a missing pyyaml dependency to the
phosphor-ipmi-sensor-inventory-mrw-native recipe.
Tested: Built a Witherspoon image
Change-Id: Ia85eb4da107f855d8f2dd9483b4dd34b71674f24
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Change-Id: I53d382a8c253338cf41ba4d6440b4a83e495e803
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I0eed1092768c02f68efe678a38ae012da14feae1
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
With this entry, code will be able to map a gpu_func sensor
that comes down in an eSEL to an inventory item so a callout
can be done.
Resolves openbmc/openbmc#2955
Tested: Checked that the GPU inventory items now show up
in inventory-sensor-gen.cpp
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: Ibb3a100c9bb129fe0a06fc82af40413101be80c5
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
Ignore the return code of fw_printenv in case rwfs_size doesn't
exist, otherwise the function fails and doesn't get set.
Also check that the imgsize variable is set before calling
fw_setenv to prevent accidentally clearing the rwfs_size.
Tested: Verified that rwfs_size gets set when it didn't exist.
Also that it gets updated when the value is different than
the one passed. And that it doesn't get cleared if the function
is called without passing the imgsize argument.
Change-Id: I02e3414e3b22da55890b7343b8805402139b6bad
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
Change-Id: I1b299ee8167a7c9a16b06fb58c3d13ed9c0d97a0
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I693eeb27becfd6290a3bd420e3b403d27a570e4e
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I17a657c313010c0cb25a333381c9a291442200d4
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Id0a813aee8d4a7dee62bb48cf5e7934fd01a3614
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I76d3a7a54142dd36292e7d66b383ffda8fd31ffc
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I3b07d2ae145412a57a74ce1dfc153e81b5d331d3
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
The MAX31785revA specification does not match how it functions.
The MAX31785revA's functionality reports the rotor feedback speeds in a
slow/fast relationship instead of rotor positions, therefore the fan*_0
object will always represent the slowest rotor. In combination with how
the MAX31785revA latches its TACHSEL = 0 during a speed transition, it
made it difficult to fault isolate the rear rotor of a fan.
Using a combination of the fan monitor trust groups on all fan rotors
and enabling a delay timer to determine when a fan rotor becomes
functional, fan monitor is able to correctly reflect each fan rotor and
fan enclosure's functional state in inventory.
Fan faults are unable to be monitored on all fans when transitioning to
a target speed.
Tested:
Each fan rotor's functional state is updated when blocked
Correct amount of time to mark a fan nonfunctional is still honored
A fan is functional after remaining in spec for 5 seconds
Resolves openbmc/openbmc#2798
Change-Id: I834d63f6082eae4cb03ec501f0fb8272a9f5b673
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
Change-Id: I0cea4645f828be91eb7fcf0820d84f40c2b69e95
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I198e97fec23f00aa8b0cf3c5857b50ebca01e424
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Ic4ccebc7c809f051f702821b9c69c3c2c20c7117
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
If an environment variable doesn't exist, fw_printenv returns 1
(error). With the addition of "set -eo pipefail" to the
obmc-flash-bmc script, this error causes the service that sets
the priorities to fail:
witherspoon systemd[1]: Starting Set U-Boot environment variable...
witherspoon obmc-flash-bmc[3883]: ## Error: "66089bab" not defined
witherspoon systemd[1]: obmc-flash-bmc-setenv@66089bab\x3d0.service: Main process exited, code=exited, status=1/FAILURE
witherspoon systemd[1]: Failed to start Set U-Boot environment variable.
Ignore the return code instead, the variable will be empty
if fw_printenv doesn't return a value.
Change-Id: I78b559e0ccb74ae52c73f1cc37b06a139a0d0e14
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
Change-Id: I233142268f40237b0bb96bbd0e508ea3f0f91922
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I4ecedf890b2d19d1345fa80f9278ed198429a588
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Ia242be45464d724f600b3d0d2127a9b44858fc48
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Resolves openbmc/openbmc#2500
Change-Id: I7caaf07f1d9f6d54d2e92a382d675ef450db2c8e
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
|
|
In older versions of coreutils install with -D and -t together has
different behavior such that the destination directory is not created.
Later versions of coreutils this does work but manually install the
target directory so builds on RHEL7 can work.
Change-Id: I42c46198321e4bf8085898b6d7485a3425634aca
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
The default size of the UBI read-write volume is currently set to 4MB.
There are no plans to store more than one BMC rootfs image on the chip
so in a 32MB flash chip, accounting for the current rootfs image size
of ~15MB with plans to grow to ~20MB with the addition of redfish and
other packages, it is safe to increase the read-write volume size to
6MB since the current size is already almost completely taken up if
there are multiple error logs / dumps on the system.
In addition, make the size configurable from a recipe so that the
size can be changed in a per-system basis. And during code update,
check the current size and update it if it's different, this allows
systems to be able to be resized to a new size by performing factory
reset after a code update that sets the new size.
Tested:
- Booted on QEMU and verified non-ubi system (romulus) retains the
current 4MB read-write volume, and ubi system (witherspoon) has a
6MB volume.
- Code updated to an image that has these changes and verified the
rwfs_size env variable changes to 6MB, and that a subsequent factory
reset rebuilds the volume with size 6MB.
Change-Id: I995eb560c1bd87ee95712c731e3d6e55bc0b2735
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
The phosphor-software-manager-updater depends on bash. Add
the runtime dependency.
Change-Id: I619cf8d965be520785752ad6073419a5eb645618
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Provide a Turbo mode setting and the ability to enable/disable
the processor Turbo mode. Default value is enabled.
Change-Id: I17537f845dc17fb850e349a63cd2f93ad25cf225
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
|
|
With openbmc commit c03b9c4a5d5f8e089297d8d4c13540d6b2c9421f
a new U-Boot environment variable was added to perform a
factory reset, and logic was added to obmc-flash-bmc to
always check if this new variable existed so that systems
running older versions of BMC firmware would have this new
variable added since the environment variables are not updated
during a firmware update.
A few openbmc tags have been created since then, and only
witherspoon currently uses this code update script, so it's a
good time to remove this workaround.
Change-Id: I1cb7fdf0e657d24a7641cc4d18776ea20d206c7c
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
Change-Id: Ib78f9f24dc4a33cda8c4268178425df42c394364
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I4e34219fc78cadc11c6122b960b73b976f3c43b9
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
All the fan services associated with controlling, monitoring and
presence detection of fans need to remain active until a poweroff
completes. This helps ensure thermal safety while a system is in
transition to a powered off state.
With the introduction of this new target (which runs last in
the power off path), need to move the host reset service to
running after it to ensure the power on in the reboot path
does not start until the last power off target runs.
Resolves openbmc/openbmc#2762
Change-Id: I4ca671d2c7de66e8bfa4bba607b06c88c0a6e7f9
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
Change-Id: Iaa4c75aa8515d4c30e88cf5f18ef2330d564bcc6
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I1cdb11a84cd74f2bb807c0367d7bcd98107f554d
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Remove a reference to python-rocket from the phosphor-gevent
recipe.
Change-Id: I3a46d1b5d174cdd8733b975ef916897d47ba964a
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Tested: Documentation only - no functional changes
|