Age | Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: XP Chen <xiao-peng.chen@fii-na.com>
Change-Id: I540022c8547367d46f1f2fa52d6a144d41856ec4
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ia05891430c6c97a89b7bc8ba4558ae496866bf7d
|
|
The `pfx` variable was not correctly being initialized
Change-Id: Iafc0a2ca8dbfe943a9bc8c1649ae68fb00d267a1
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
The return statements should have been exits.
Change-Id: I39a190250656ba676ea2ccbe570b88e3f18e5121
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
In the current state soft-off service is a hard dependency
(requires) for the host shut down target, so in any case where
the soft-off service fails with an error condition, the shutdown
target fails too.
This commit intends to make soft-off service as a soft dependency(wants)
so that the shutdown target can still proceed even though the
pldmsoftoff service fails for whatever reason.
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: I5650702c6eb71adab10fdb6493c9d582a9cb0129
|
|
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>
|
|
Switching to IPK usage breaks the building of this package.
During the installation there are no required groups.
This commit fixes the issue.
Change-Id: Ibe80f3bf80e851005f637e47ca669e47035dba6d
Signed-off-by: Alexander Filippov <a.filippov@yadro.com>
|
|
Possibly a bug in the CMSG logic in glibc, workaround being submitted to
systemd as https://github.com/systemd/systemd/pull/20752.
Change-Id: Ie8aeecd6bbfd329c2b9c18980d823d4722c01428
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
The clk patch is a prerequisite for the backport of the ADC patches.
The ADC patches are not yet merged, but are under active review and are
expected to be merged in v5.16. Version 6 of the patchset was merged.
Billy Tsai (14):
iio: adc: aspeed: set driver data when adc probe.
dt-bindings: iio: adc: Add ast2600-adc bindings
iio: adc: aspeed: completes the bitfield declare.
iio: adc: aspeed: Keep model data to driver data.
iio: adc: aspeed: Restructure the model data
iio: adc: aspeed: Add vref config function
iio: adc: aspeed: Use model_data to set clk scaler.
iio: adc: aspeed: Use devm_add_action_or_reset.
iio: adc: aspeed: Support ast2600 adc.
iio: adc: aspeed: Fix the calculate error of clock.
iio: adc: aspeed: Add func to set sampling rate.
iio: adc: aspeed: Add compensation phase.
iio: adc: aspeed: Support battery sensing.
iio: adc: aspeed: Get and set trimming data.
Dmitry Baryshkov (1):
clk: divider: add devm_clk_hw_register_divider
Eddie James (4):
ARM: dts: aspeed: everest: Add I2C bus 15 muxes
ARM: dts: aspeed: rainier: Add system LEDs
ARM: dts: aspeed: rainier: Add eeprom on bus 12
ARM: dts: aspeed: rainier: Remove gpio hog for GPIOP7
Joel Stanley (1):
Revert "block: nbd: add sanity check for first_minor"
Michael Walle (1):
clk: divider: add devm_clk_hw_register_divider_table()
Change-Id: I120d78fa77df01e239eeb67b3209cebfe2973491
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
This includes security and bug fixes from the 5.10.61 through to
5.10.65 stable releases.
Change-Id: I754ab3d2c0ede419f082d42e245c9e8dd27166b5
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
Change-Id: Id980b8175dc5621f394771941de065689239fe3f
Signed-off-by: Willy Tu <wltu@google.com>
|
|
Allow to read the host console output via SOL.
Change-Id: I249dcf485bcc34bb7c461feece8999240a33faa1
Signed-off-by: Arthur Heymans <arthur.heymans@9elements.com>
Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
|
|
Override variable syntax change, honister compat, and a CONF_VERSION
bump.
Fixes openbmc/openbmc#3816.
Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: Ia41d159d8118e5dba4b213bb0f7c2d61753c8ad3
|
|
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
|
|
Lei YU (1):
meson_options: Enable all features by default
Change-Id: Ia287c5818adaab33c4f09487b8cdd3cb8f190742
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Tejas Patil (1):
Add support for MTU property
Change-Id: I76c79f663a143c1e6cc72cf0d66eaf0bea7d50aa
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Brandon Kim (1):
Add Model property to Cpu Interface
Change-Id: I30076561b0553b27f8d60afdc85e904a6cce1cfe
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
George Liu (2):
Add GeorgeLiu as the Reviewer
Update clang-format
Vishwanatha Subbanna (1):
Add Patrick as the Maintainer
Change-Id: I075aa936f782e993a6427cf266054421f8bfd1f1
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Jean-Marie Verdun (1):
Add initial support for polarity on a couple of GPIOs
Zev Weiss (2):
Append node number to dbus names more directly
Fix powerControlDir creation failure error message
Change-Id: I7299ba7115a2ea17ef7daefc9da1fc3796da29e4
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Alpana Kumari (2):
Dimm memory size support
Enable dynamic presence detect of FRUs
Matt Spinler (1):
Support size_t properties on extraInterfaces
Santosh Puranik (1):
Set one-time properties and PrettyName
alpana07 (1):
DD2: Enable Dimms
Change-Id: I3446ce4e07f69c4cddabc5a7867e5eec6e599a36
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Sneha Patel (1):
Remove unnecessary span from toggle component
Change-Id: If216d4dd69019e0fe2ba483f4c650917b2681550
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Shawn McCarney (7):
regulators: Clear phase fault history in Device
regulators: Simplify Chassis test cases
regulators: Add detectPhaseFaults() to Chassis
regulators: Simplify System test cases
regulators: Add detectPhaseFaults() to System
regulators: Enable phase fault detection
regulators: Fix race condition in monitor disable
Change-Id: I3ca7bf31b10fb03548c99c04057da770695a1df6
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Tim Lee (1):
bmc-state-manager: Add support BMC Reboot Cause feature
Change-Id: I1758df4efde9dec8981c400d95bee740cf1737a7
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Jayaprakash Mutyala (1):
Master Write-Read: Validate reserved field content
Change-Id: I2390ef052ea1c593681ebf3a547ca92d03f2b6bc
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
George Liu (1):
Update clang-format
Change-Id: I0ad894ebd8caf2bc62b500f4e7e167bdb9e8dc81
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
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>
|
|
Currently, all received RAs are dropped by systemd-networkd with an
internal error code.
Change-Id: I8df0b587e4d6504c624940ee04f923ba6f166c3f
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
DRM_FBDEV_EMULATION previously selected FB and was default y as long as DRM
was enabled. In commit f611b1e7624c ("drm: Avoid circular dependencies for
CONFIG_FB") the select was replaced with a depends on FB, disabling the
drivers that depended on it.
As the AST2500 uses the GFX device, and the uart-renderer process uses
fbterm to display to the FB device, we want CONFIG_FB=y for that system.
This is harmless for kernels that do not have the offending commit.
Signed-off-by: Joel Stanley <joel@jms.id.au>
Change-Id: I57213b60f7fc06c89112a9825c7ed8ed7ff5e298
|
|
Andrei Kartashev (1):
Refactor handling CPUInfo array
Change-Id: I35c6200e0a67b2b962cd629ea4c9efb3a8cfa698
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Mansi Joshi (1):
Fix SMMDR send directory info call
Change-Id: I7205ebf2167ac89fd0c518d1ed4c1efa709fe7d9
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Jayanth Othayoth (4):
PHAL: Helper function to set PDATA_INFODB
PHAL: set PDATA_INFODB value for phal-export-devtree app
PHAL: set PDATA_INFODB value during import devtree
PHAL: disable attributes tool debug traces
Change-Id: Ia969a9811580b6d79c59c5f941acaab5b8f67be3
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (1):
clang-format: apply clang-12 changes
Change-Id: Ifae2213b7017b98ce3530c44fe8c9cc256187703
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Tom Joseph (1):
Refactor the verbose tracing in PLDM
Change-Id: I5bed3b2087068a6359db4b146056388cf20ecbef
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Konstantin Aladyshev (2):
Russian translations for the POST code page
Sync ru-RU translation file with the en-US file
Change-Id: If1f76c6350bc9b393569abae2e2e23424c2b8ebf
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Matt Spinler (1):
Find P10 OCCs by looking in /dev
Change-Id: Ie474fb85f66704346e43435cb7d172daf27bba4d
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (2):
build: add phosphor-logging dep and wrap
logging: replace direct journal calls with lg2
Change-Id: Ib0eedad0b41c9848a09da3313717ca594d5ad59a
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Ali Ahmed (1):
Add SystemType to Systems/hypervisor
Ed Tanous (4):
Remove unused variables in connection class
Rearrange/clean code in connection
fix clang formatting
Clear UserSession in between requests
John Edward Broadbent (2):
Change ownership of boost::req to crow::req
Corrects issues in session
Change-Id: Idaacee582adc86dcbe70d8dfa59fa6984fb255fa
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Adriana Kobylak (1):
clang updates
Patrick Williams (1):
clang-format: apply clang-12 changes
Change-Id: I458f6e24203a239399cc54b009df790ef7c491b7
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I494bb2352ba068f02dab761fabc7cfcae6a70eb9
|
|
In prep for Ib408e97ff99863cd91bcbcb43a4738d773f4e21c, add the
necessary dependency for phosphor-logging.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I2a67d940da9598250e9884c35006f670f68efc72
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I75f71a404016949179cca78605ae5be4613249e3
|
|
The `run_git` in phosphor's os-release.bbappend assumes that the
git repository is in `${COREBASE}`, which is the location of the
`meta/` subdirectory. This is true when building exclusively the
`openbmc` tree but may not be the case when external users attempt
to use the `openbmc` tree as a git-submodule. Create an indirection
(`OS_RELEASE_ROOTPATH`) which defaults to `COREBASE`, but allows
an override for those other use-cases.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I4e0171e065379c5030a36ea5be774b5db78bfe34
|
|
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>
|
|
`install -d ${D}${sysconfdir}/default` has been configured
in the libmctp_git.bb file, remove it.
Tested:
1. Successfully build libmctp and image-bmc
2. Saw the `/etc/default/mctp` file in image-bmc
Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I162d10b4a09ebe87388f4e3e4d9f79aa32a94230
|
|
We want to use sets instead of separate rules for each address. This
also ensures that packets coming from internal sources are matched as
internal packets.
Change-Id: Iff87b81c48c7491a74af1a2cead4cabcb56d81a0
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
We want to reduce the size of our target binaries, and LTO saves us
about 300K.
Change-Id: Id1fe52b48a16999385251ae927012ff3e494e2f3
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
Patrick Williams (3):
elog-gen: minor changes to reduce clang-format diffs
elog-gen: sort exceptions for file stability
clang-format: apply clang-12 changes
Change-Id: If208b6a1bbec647c2b1c7208cbf893a3b9d68ddb
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Matt Spinler (1):
Put PowerState On/Off enums in quotes
Change-Id: Iffc7ef537cf0dc5a8869d678363bdf737a463ab7
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|