Age | Commit message (Collapse) | Author | Files | Lines |
|
After over a year of discussion, Redfish released new power and thermal
schemas as part of the Redfish 2020.4 release.
These new powersubsystem, thermalsubsystem, fan, powersupply, etc
schemas can co-exist with the old power and thermal schemas.
The old schemas and the new schemas are controlled by different options.
The current plan for bmcweb is:
1. Until the new schemas are all in and tested, as a default, enable the
old schemas and disable the new.
2. After the new have been in and tested, enable them. This means the
old and new will co-exist if running the defaults. The sensor
collection behavior will reflect the new schemas, that is all sensors
can be found under the sensor collection.
3. After an OpenBMC release, at a later time, disable the old and add
appropriate deprecation.
This change here, enabling the new schemas, jumps meta-ibm to #2.
This reflects our desire for our Redfish clients to start using the new
schemas.
Some reasons why the ThermalSubsystem/PowerSubsystem are an improvement
on the existing Thermal/Power schemas:
1. They include the latest properties like LocationIndicatorActive.
2. Fans, PowerSupplies, Temperatures were arrays in the old schemas.
This was cumbersome and could hit limits of JSON arrays.
3. Large amount of static data mixed with sensor readings, which hurt
performance in certain cases.
4. Inconsistent definitions of properties vs newer schemas like Processor
and Memory schemas.
Reference:
https://www.dmtf.org/sites/default/files/standards/documents/DSP0268_2020.4.pdf
Tested: Built bmcweb. See the new sensor behavior.
Change-Id: I9a698cbc162a331c21c7dc5138000faac6247f9b
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ia05891430c6c97a89b7bc8ba4558ae496866bf7d
|
|
bitbake offers a choice of DEB, RPM or IPK packaging. To a degree the
choice is functionally arbitrary for image generation but control over
the package format becomes important if we want to:
1. Include runtime package management functionality in the firmware image
2. Mess about with the packages on the build system
With respect to 1 the IPK format and opkg (an ipk package manager) are
designed for embedded systems[1] - by contrast to RPMs have heavier
dependencies and a greater impact on the size and complexity of the
firmware image.
Regarding 2, the embedded nature and the need for opkg to work without
much fuss leads to a lower configuration barrier by comparison to RPMs.
With ipk it becomes possible to reuse the packages built during image
preparation for core analysis without needing to generate an SDK:
```
$ export LD_LIBRARY_PATH=./tmp/work/x86_64-linux/opkg-native/*/recipe-sysroot-native/usr/lib
$ MY_DEBUG_ROOT=tmp/rootfs-debug
$ ./tmp/sysroots-components/x86_64/opkg-native/usr/bin/opkg \
-f ./tmp/work/p10bmc-openbmc-linux-gnueabi/obmc-phosphor-image/*/opkg.conf \
-o $MY_DEBUG_ROOT \
update
$ fakeroot ./tmp/sysroots-components/x86_64/opkg-native/usr/bin/opkg \
-f ./tmp/work/p10bmc-openbmc-linux-gnueabi/obmc-phosphor-image/1.0-r0/opkg.conf \
-o $MY_DEBUG_ROOT \
install dbus-sensors dbus-sensors-dbg
$ gdb-multiarch
(gdb) set solib-absolute-prefix .../tmp/rootfs-debug
(gdb) add-auto-load-safe-path .../tmp/rootfs-debug
(gdb) file tmp/rootfs-debug/usr/bin/nvmesensor
(gdb) core-file obmcdump_17_9597/core.nvmesensor.0.aae91b519d0e4e0e8bbe746e3f6cd25f.2779.9594000000
Core was generated by `/usr/bin/nvmesensor'.
Program terminated with signal SIGABRT, Aborted.
pthread_kill.c:45
45 pthread_kill.c: No such file or directory.
(gdb) bt
pthread_kill.c:45
../sysdeps/posix/raise.c:26
/home/andrew/src/openbmc/openbmc/build/p10bmc/tmp/rootfs-debug/usr/lib/libstdc++.so.6
/home/andrew/src/openbmc/openbmc/build/p10bmc/tmp/rootfs-debug/usr/lib/libstdc++.so.6
/home/andrew/src/openbmc/openbmc/build/p10bmc/tmp/rootfs-debug/usr/lib/libstdc++.so.6
/home/andrew/src/openbmc/openbmc/build/p10bmc/tmp/rootfs-debug/usr/lib/libstdc++.so.6
"xyz.openbmc_project.NVMeSensor", this=0x488f04) at
/usr/include/sdbusplus/bus.hpp:234
../../../../../../workspace/sources/dbus-sensors/src/NVMeSensorMain.cpp:159
(gdb)
```
This approach documented in the Poky Reference Manual:
https://www.yoctoproject.org/docs/1.0/poky-ref-manual/poky-ref-manual.html#platdev-gdb-remotedebug-launch-gdb-inferiorbins
Switch all machines to IPK to align the debugging experience with
upstream's documentation and to facilitate efficient use of packaged
software at runtime.
[1] https://openwrt.org/docs/guide-user/additional-software/opkg
Change-Id: I8ef526add2d7a6790de1b3eb3fb85cd39b864f23
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
|
|
This feature is not needed on p10bmc systems and causes unwanted probing
of i2c hardware and journal entries.
Tested:
- Booted p10bmc machine and made sure it still worked as expected
and no new errors arose
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Idff2477060f5719ad85529daff28ef945e77700e
|
|
This reverts commit bcc5f6b0f24e8ad0b03b8217e88a19ff3002c084.
bcc5f6b0f24e ("Override pldm response time out value") talks about
timeouts due to the endpoint taking some time to respond. However, the
net effect of the change is the response to a retried request races
against the instance ID expiration interval because the retry interval
is effectively equal to the instance ID expiration interval once we
account for some timer slack.
This is demonstrated by the following strace on pldmd, where we can see
a retried request go out, followed by the report that the request
failed, further followed by the response to the request coming in. Note
the values are string-literal-escaped-octal, so the [ 0x80 0x00 0x03 ...
] byte encoding of the GetPLDMVersions request appears as "\200\0\3...":
```
...
11:56:25.046173 socket(AF_UNIX, SOCK_SEQPACKET, 0) = 3
...
11:56:25.183936 connect(3, {sa_family=AF_UNIX, sun_path=@"mctp-mux"}, 11) = 0
11:56:25.190994 write(3, "\1", 1) = 1
...
11:56:25.195272 sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\t\1", iov_len=2}, {iov_base="\200\0\3\0\0\0\0\1\0", iov_len=9}], msg_iovlen=2, msg_controllen=0, msg_flags=0}, 0) = 11
...
11:56:30.202298 sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\t\1", iov_len=2}, {iov_base="\200\0\3\0\0\0\0\1\0", iov_len=9}], msg_iovlen=2, msg_controllen=0, msg_flags=0}, 0) = 11
11:56:30.202820 gettid() = 1918
11:56:30.203029 timerfd_settime64(6, TFD_TIMER_ABSTIME, {it_interval={tv_sec=0, tv_nsec=0}, it_value={tv_sec=3848, tv_nsec=61124978641398328}}, NULL) = 0
11:56:30.203286 epoll_wait(4, [{EPOLLIN, {u32=14373240, u64=14373240}}], 14, 0) = 1
11:56:30.203509 clock_gettime64(CLOCK_REALTIME, {tv_sec=1629806190, tv_nsec=203587376}) = 0
11:56:30.203687 clock_gettime64(CLOCK_MONOTONIC, {tv_sec=3843, tv_nsec=523046301}) = 0
11:56:30.203844 clock_gettime64(CLOCK_BOOTTIME, {tv_sec=3843, tv_nsec=523206110}) = 0
11:56:30.204049 write(2, "Response not received for the re"..., 59) = 59
11:56:30.204427 write(2, " EID = ", 7) = 7
11:56:30.204745 write(2, "9", 1) = 1
11:56:30.205047 write(2, " INSTANCE_ID = ", 15) = 15
11:56:30.205389 write(2, "0", 1) = 1
11:56:30.205719 write(2, " TYPE = ", 8) = 8
11:56:30.205997 write(2, "0", 1) = 1
11:56:30.206266 write(2, " COMMAND = ", 11) = 11
11:56:30.206576 write(2, "3", 1) = 1
11:56:30.206893 write(2, "\n", 1) = 1
11:56:30.209402 write(2, "Failed to receive response for ", 31) = 31
11:56:30.209814 write(2, "getPLDMVersion command, Host see"..., 46) = 46
11:56:30.210969 gettid() = 1918
11:56:30.211171 timerfd_settime64(6, TFD_TIMER_ABSTIME, {it_interval={tv_sec=0, tv_nsec=8549172174085160960}, it_value={tv_sec=0, tv_nsec=8566510441663037440}}, NULL) = 0
11:56:30.211406 epoll_wait(4, [{EPOLLIN, {u32=14373240, u64=14373240}}], 14, 0) = 1
11:56:30.211640 clock_gettime64(CLOCK_REALTIME, {tv_sec=1629806190, tv_nsec=211720512}) = 0
11:56:30.211825 clock_gettime64(CLOCK_MONOTONIC, {tv_sec=3843, tv_nsec=531188829}) = 0
11:56:30.211983 clock_gettime64(CLOCK_BOOTTIME, {tv_sec=3843, tv_nsec=531335706}) = 0
11:56:30.212143 recv(3, NULL, 0, MSG_PEEK|MSG_TRUNC) = 15
11:56:30.212366 recv(3, "\t\1\0\0\3\0\0\0\0\0\5\361\361\360\0", 15, 0) = 15
```
That is, at 11:56:30.202298 we send out the retry for the request
initiated at 11:56:25.195272 and the reply arrives back at
11:56:30.212366, but in between we've already cancelled the request
handler due to the instance ID interval timer expiring.
Resolve this by removing the explicit configuration of the
response-time-out build parameter setting the per-request response time
to 4.8 seconds, setting its value back to the default of two seconds.
Anecdotal testing of with the following shell script produced no
failures (by inspection of the journal as the iterations executed):
```
for i in `seq 1 30`; do echo $i; ( systemctl stop pldmd mctp-demux && echo 1e78902c.kcs > /sys/bus/platform/drivers/ast-kcs-bmc/unbind && sleep 1 && echo 1e78902c.kcs > /sys/bus/platform/drivers/ast-kcs-bmc/bind && systemctl start pldmd && sleep 15 ) || break; done
```
Change-Id: Ide125d686e79376b412fca0105449c8bef722cfe
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
The `p10bmc` machine's JSON config files should now use the JSON config
files installed from the repository.
Change-Id: Ibd9bbc055e4d00a5f799a291e9fd3a9955d774c5
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
The `witherspoon` and `p10bmc` machines' JSON config files should now
use the JSON config files installed from the repository.
Change-Id: I47669745aa7047c4de64ee8aedc98437593a3b62
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
The `witherspoon` and `p10bmc` machines' JSON config files should now
use the JSON config files installed from the repository.
Change-Id: Ie40264435ca5278a50f2aa3c688d882d72cb376b
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
All IBM systems support the SNMP trap notification for error logs. This
means phosphor-dbus-monitor needs to be careful to not start until the
SNMP service is up and running.
Without this dependency, situations can be hit, when lots of errors are
present, where the phosphor-dbus-monitor calls into the SNMP shared
library and hits an unhandled exception when trying to talk to the SNMP
service.
Tested:
- Verified the unhandled exception is no longer seen and
phosphor-dbus-monitor starts after the SNMP service.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I8b4ca94c0cdd7721aa8847dc478f3827f040a654
|
|
To build webui-vue with IBM theme, need to set IBM environment variable
during the webui-vue build.
Yocto has many layers of abstraction. To specify IBM environment
variables, we use --mode npm build Commandline argument, which helps to
set IBM environment variables.
We need to set this environment variable during IBM build only. So
created webui-vue_%.bbappend file, which sets that env variable only
for IBM builds.
More information could be found at
https://github.com/openbmc/webui-vue/blob/master/docs/customization/build.md
Tested with IBM build.
Signed-off-by: Abhishek Patel <Abhishek.Patel@ibm.com>
Change-Id: I6ff997a94eb59b695741ccca95acb7693a4714ff
|
|
Adriana Kobylak (1):
oem: ibm: Add services to reset PHYP NVRAM
Tom Joseph (1):
tools: Extend pldm_fwup_pkg_creator
Change-Id: I9dce25f3137a938c84dd080eaa8f37aa5a5147eb
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
- In this patch, added a setting to enable or disable the
HardwareIsolation (aka Guard) in the OpenPOWER based system.
- By default, the setting is enabled, that's means the
HardwareIsolation feature is enabled for the IBM system so,
the respective BMC components need to take care of the
functionality based on the roles of each application usage
by using this setting if needs.
Signed-off-by: Ramesh Iyyar <rameshi1@in.ibm.com>
Change-Id: If03c98b74f5bc26941ddc825cde81ea307518742
|
|
We're about to introduce a package for mctp utilities, called 'mctp'.
However, the libmctp package already uses that name.
This change renames 'mctp' to the upstream name of 'libmctp'.
Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>
Change-Id: Ia49a40c822e920d3bc36584cbc2c97b83828cfdd
|
|
In the spirit of consistency with the new upstream override
syntax, change phosphor-discovery-service.bbclass to use it
and make tree-wide changes as appropriate.
After the change checked on Witherspoon on QEMU:
```
root@witherspoon:/etc/avahi/services# ls
obmc_console.service obmc_redfish.service obmc_rest.service
```
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ib1fe3cce57a0130378af789abd83b457e0c3a318
|
|
Matt Spinler (1):
Switch SdBusError to generic sdbusplus exception
Change-Id: I9fad0117559672b860270d86fb2538b649b3ee28
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Witherspoon systems are currently used for CI. Currently, CI does not
test the webui. This change will affect Swift systems, as swift.conf
requires witherspoon.conf.
Additionally, removing phosphor-webui saves us 397 kB in filesystem sz.
Built up-to-date images to see the size of the rootfs.squashfs-xz
The size of obmc-phosphor-image-witherspoon-<date>.rootfs.squashfs-xz:
1) With webui-vue: 19.836928 MB
2) With phosphor-webui: 19.78368 MB
3) Without either (i.e. this change): 19.386368 MB.
Testing using commit 7d637776938c77d69752a72a09edc90ebde93164:
1) Code update successful on hardware system.
HW system updated image and rebooted successfully
Signed-off-by: Ali Ahmed <ama213000@gmail.com>
Change-Id: Ib2488dd462851b98f557dd9c76086d48dcbe905c
|
|
Upstream bbclasses changed to typically use the `:${PN}` override
syntax, including the SYSTEMD_ variables. Change our systemd.bbclass
to do the same for consistency and perform a tree-wide variable
replacement.
Spot checked by building bletchley and witherspoon and checking
some of the SYSTEMD_LINK directives on installed packages under qemu.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I20a9dd809bff8af8759488734f80486c7228c6eb
|
|
With the reshuffle that occurred in 38cfcc393d9e ("meta-aspeed:
u-boot-aspeed-sdk: Use provided signing key by default") we no-longer
need to specify these options in p10bmc.conf
Mistakenly, `SOCSEC_SIGN_ENABLE = "1"` was duplicated later in the file
by 699146128388 ("meta-ibm: p10bmc: Assert that we want the SPL signed
by socsec"). Removing the instance as done in this patch still leaves us
with the positive assertion that we want SPL signing enabled.
Change-Id: Iaa9cbe5b0ba4aaf69bb0fba1eb8f6bc62d085a33
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Upstream supports zstd compression now as an alternative to zlib or xz.
zstd is only slightly larger than xz but is significantly faster. This
makes it a better candidate for the journal. Enable it in systemd
instead of xz.
Add an exemption for witherspoon to continue using xz instead. Having
both zstd and xz support built into the image puts us over the flash
limit and we currently need xz for phosphor-debug-collector.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I6fb55b96776aeeea1c544743b08bdbbc2853e7b7
|
|
Testing:
Built image for p10bmc platform, installed on a test system,
and verified netipmid was started on both eth0 and eth1
interfaces. Also, verified that ipmitool commands work as
expected for IP addresses configured on eth0 and eth1 interfaces
Signed-off-by: Shantappa Teekappanavar <sbteeks@yahoo.com>
Change-Id: Ie8399ba1219dac2f9c4ac15b7dfcf7e746750f0d
|
|
We've typically kept these LAYERSERIES_COMPAT to 2 releases: the current
and the upcoming. Remove 'gatesgarth' is it is now 2 releases back.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I5e812a94fed1738898af75c0fdee81996a5bbf20
|
|
Use nonarch_base_libdir instead of base_libdir, because the files are
always in /lib/udev/rules.d/. On ppc64le base_libdir is lib64, and the
files end up in the wrong spot.
Signed-off-by: Anton Blanchard <anton@ozlabs.org>
Change-Id: I3c24dae293f4166f1ed1826d09d4bde7cd7d0357
|
|
Configure the SOCSEC_SIGN_* variables to sign the SPL and exploit the
AST2600 hardware root-of-trust.
Note that this doesn't require that secure-boot is enabled on the
system, the SoC will bootstrap just fine with the signature in place
while secure-boot is disabled. Signing the SPL allows us to switch the
systems over to secure-boot at our leisure.
Change-Id: I07b5c4afb7bacc040cbdce6c82a0fb3a57d0f7f8
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
The hostboot dump collection initiated by watchdog_timeout
is disabled by default. This commit enables hostboot dump
collection on p10bmc platforms for watchdog timeout.
Build watchdog_timeout binary:
$ devtool modify -n openpower-debug-collector <local-source-path>
$ MACHINE=p10bmc bitbake openpower-debug-collector
Test:
Ran watchdog_timeout binary built by bitbake, and verified that
the binary takes the timeout value and times out after the specified
time interval
Signed-off-by: Shantappa Teekappanavar <sbteeks@yahoo.com>
Change-Id: I343aac95fd97feb226d8676c3c34c2902c1c6690
|
|
Running out of flash space on our witherspoon system so need to cut
back some function. Telemetry is fairly new and takes a sizeable chunk
of flash space (200KB)
Tested:
- Verified squashfs went from 19.20MB to 19.00MB with this change
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I1741649f573cd25363167d69b4a802f2f261d93a
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iadecbc8418d901f82fcc741d3e88d2d202fe96fe
|
|
Update with the latest version from upstream.
Change-Id: I1a7da37b0457dab873afaf6445aca360d54b47ca
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
Change-Id: I9116ed7260e369136acb39eec15075db2d4dbeba
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
This commit is add D-bus monitor config.yaml
for errorlog entries for snmp eventing.
Tested by:
Configured SNMP and created errorlog
observed snmp traps recieved on SNMP server
Signed-off-by: Ravi Teja <raviteja28031990@gmail.com>
Change-Id: I59c1f8a3b03eb3aab4a55c3cfb0bfb3e9e600a4d
|
|
This commit would override response time out value for a request
to 4.8 seconds for all the ibm machines.
We have seen in our debugging that in rare cases host takes close
to 5 seconds to respond to the pldm message.This value (4.8 seconds)
is in sync with what phyp maintains for their messages(5 seconds).
As we do lot of fileIO operations, and the number of PDR's tend to grow,
picking 4.8 seconds seems to be less risky path & also larger timeout
would reduce the number of retries from BMC incase if the host does not
respond within 2-3 seconds.
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: Ic0eac42b66dbc2c6d69bff440fe590775f35e390
|
|
Background:
OpenBMC provisions the BMC firmware image with the root account password
in a form which is no longer acceptable to Linux-PAM version 1.5.1.
Specifically, [phosphor-defaults.inc][] sets the password hash into
/etc/shadow as "\$1\$UGMqyqdG\$FZiylVFmRRfl9Z0Ue8G7e/", where $1
indicates the deprecated [MD5 hash algorithm][]. Ref: [wikipedia passwd
entry][]. Beginning around PAM version 1.5.1, when you log in, the
[pam_unix.so module][] authenticates okay but requires the password to
be changed. (For example, you'll get a message like "You are required
to change your password immediately (administrator enforced)." This
behavior is undesirable for OpenBMC project defaults, and is not
tolerated by the project's current continuous integration tools.)
This change is to replace the password hash to keep the same cleartext
password but hashed with an acceptable algorithm.
Specifically, the password hash supplied in phosphor-defaults.inc is
updated to use the same password as before but encoded
with the SHA-512 algorithm. The hash was generated by the
`openssl passwd -6 0penBmc` command. This change ought to be
transparent and forward and backward compatible.
Note various meta-layers use this same hash string in
conf/local.conf.sample files. They are changed to match.
References:
[phosphor-defaults.inc]: https://github.com/openbmc/openbmc/blob/1a977b269ed437bebb9ae7810e3157746ec9174d/meta-phosphor/conf/distro/include/phosphor-defa
ults.inc#L245
[wikipedia passwd entry]: https://en.wikipedia.org/wiki/Passwd
[pam_unix.so module]: https://github.com/linux-pam/linux-pam/tree/master/modules/pam_unix
[MD5 hash algorithm]: https://en.wikipedia.org/wiki/MD5
Tested:
Created image with new password hash and PAM 1.5.1 and checked that
login works okay and does not require the passwod to be changed.
Signed-off-by: Joseph Reynolds <joseph-reynolds@charter.net>
Change-Id: I5b189374f08ba506dbed7f8b9b991f2808cc3bc5
|
|
Background: The OpenBmc project default root account password is set
in meta-phosphor/conf/distro/include/phosphor-defaults.inc and can be
customized in each layer's local.conf file.
Many of these local.conf.sample files had redundant code to set the
password, which probably should not have been there. Removing them
allows the defaults in phosphor-defaults.inc to take effect.
Tested: No. Only meta-ibm was tested.
Signed-off-by: Joseph Reynolds <joseph-reynolds@charter.net>
Change-Id: I76dce00d269d7afa005d7bcfd63f846d3cf45596
|
|
Signed-off-by: Sumit Kumar <sumit_kumar@in.ibm.com>
Change-Id: I62027c739eb4cf95bb1bfc8ccddec30964bb1906
|
|
The phosphor-gpio-presence application is no longer needed for power
supply presence detection after the updates to using libgpiod in the
phosphor-psu-monitor application.
Remove the systemd parts of the bbappend, and remove the obmc conf
files.
Signed-off-by: B. J. Wyman <bjwyman@gmail.com>
Change-Id: I18a7f36e21d18a22f7625aadc3229ce5439c8d6a
|
|
Change Ie1a7c389edb6b7a048836a49283ceb62de51bba5 will be transitioning
the 'Type' property in Inventory.Item.Chassis to an enumeration.
In order to avoid crashing in PIM we need these default values in the
starting YAML to be fully-qualified enumeration names that will match
the values which will start in Ie1a7c.
This code is safe to merge as is without any PDI or PIM changes. Prior
to the PDI change, we must make a change to PIM that allows conversion
automatic from string->enum, leveraging library interfaces available in
sdbusplus. These will be submitted independently.
I checked the codebase for usage of this string. It appears that the
value is currently, effectively, write-only. There is code in bmcweb
that fills in the equivalent Redfish value but currently just hard-codes
the string 'RackMount'.
Tested: Booted Witherspoon in a QEMU model with this change and proposed
changes to sdbusplus + PIM. PIM no longer coredumps with the PDI change
and yields an expected persistence file:
```
$ pwd
/var/lib/phosphor-inventory-manager/xyz/openbmc_project/inventory/system/chassis
$ cat xyz.openbmc_project.Inventory.Item.Chassis
{
"value0": {
"cereal_class_version": 2,
"Type": 3
}
}
```
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Icaf9447f31ccdd945cdf74b3e017682e4aed686f
|
|
Change-Id: Iac6362f744113d9a405e4a4ec30077e513dda2d3
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
As for Rainier and Everest, use KCS2 as the debug-trigger interface.
Change-Id: I2614ffc3d97164658f2ad4fbb916cbf44be152e1
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Currently required for the debug-trigger application.
Change-Id: I585ccc5591b3f66db41bfd9799df05cec902403d
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Replaces the obsolete MCTP LPC driver.
Change-Id: Ic5f7f6ec796bff8f0628d53e09a7437a74399ef4
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
This sensor on the Op_Panel, one of them being a SI7020 type sensor
which needed to be enabled.
Signed-off-by: Bruce Mitchell <bruce.mitchell@linux.vnet.ibm.com>
Change-Id: Ib6a86a6fae52414ad9c6901e5f6f8aa8c8a383c3
|
|
Witherspoon uses a specialized phosphor-fan-control@.service file and
uses the phosphor-fan-control-init@.service file still to just set the
fans ready target. To correctly be included in the FILES directive,
these services need to be explicitly added on wiherspoon machines.
Change-Id: I76129a7833337aaf74fd83b0e28f3083f1a41b0a
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
- ddimm
- capi connector
- nvme
- pcie slot
- pcie cable card top and bottom enclosures
- base op panel
Signed-off-by: Lakshminarayana R. Kammath <lkammath@in.ibm.com>
Change-Id: I7de498f83ab3bace4ce39deb98c31c71c40f2dd4
|
|
MaxBootCycleCount is a dbus property that is used to indicate
the maximum number of boot cycles that the post-code-manager
daemon can store the progress codes for, before it starts
wrapping.
Unlike Intel postcodes, IBM progress codes are huge and it is
possible to get close to 230+ progress codes per boot cycle, so
tweaking this value to 5 boot cycles for IBM systems so that
we can get less number of log entries on a redfish GET request.
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: Ida10e73992b01535d57c844d83366d786cd448e2
|
|
The Altitude measurement is from the computation of
the DPS310 pressure sensor with this
"Expression": "44330.0 * (1.0 - ((P0/101325.0) ^ (1/5.255)))"
Tested: busctl introspect xyz.openbmc_project.VirtualSensor \
/xyz/openbmc_project/sensors/altitude/Altitude returned
results consistent with location of the Rainier system.
Signed-off-by: Bruce Mitchell <bruce.mitchell@linux.vnet.ibm.com>
Change-Id: I07b83dd6b8f4ccf0755ccbdc81c40976a3e571cd
|
|
Phosphor logging PEL infrastructure provide supports to
process SBE chip-op failures and creating PEL.
PEL creation required to process SBE provided FFDC packets.
This FFDC is initially created by SBE using FAPI based
infrastructure for the hardware procedure execution failures.
To extract this data to convert into PEL required format
requires additional processing. This processing required
PHAL(Power Hardware Abstraction layer) based back-end
support.
Added required packages support for PHAL feature enabled
systems.
Test: verified build.
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
Change-Id: I5cf1a30514aa0a5bfdf2fe3a288dac9d5a0b5b47
|
|
OpenBMC requires additional capabilities for the
OpenPOWER systems such as
- A common mechanism to access and control the host
hardware from the BMC.
- A storage mechanism to store the data that can be
used for initialize, control and access the CEC
hardware.
- An infrastructure to initialize the hardware using
the hardware procedures provided by the hardware team.
- Common mechanism to handle the hardware failures for
the host hardware.
These above features are essential for the BMC to provide
better RAS capability and to support lab debug for the
enterprise class systems.
The phal(POWER Hardware Abstraction Layer) aims to provide
a common layout such that the OpenPOWER application can
use this infrastructure to boot, enable RAS features like dump,
diagnostics, and lab debug tooling purpose.
The phal is a combination of open source packages that are
tailored/customised to use in OpenBMC. For instance POWER
hardware access and control is managed through pdbg and
host hardware data modelling is done via devicetree.
phal is group of libraries running in BMC. These libraries
are used by OpenPOWER specific application for host hardware
interactions, Hostboot and SBE initialization, diagnostics
and debugging.
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
Change-Id: I187d9582598743871279a1e407451f4cf4f72668
|
|
Install the udev files to cover the transition between drivers and
devices.
Change-Id: I87ffe3ae82bb7a073a8140adcb308b09fc43c9b9
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
libmctp provides the MCTP implementation for communication between the
host and the BMC.
Change-Id: I6ae5a23fbb34c27db16a152afefb79ca6a649759
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
These were separated in a strange fashion. Perhaps we should reorganise
the whole file in platform-specific sections.
Change-Id: I51b8bb20060447c6e88557824e9465b1713f082f
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Currently /proc/sysrq-trigger is the only supported backend for
debug-trigger. Once we have kdump support, we'll add a kexec backend and
drop the magic sysrq.
Change-Id: I07dfa50474d9f6333428b6fb38020c7b7fd0adec
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|