Age | Commit message (Collapse) | Author | Files | Lines |
|
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>
|
|
Jayanth Othayoth (1):
phal: libipl enum name change IPL_NO_ERR to IPL_ERR_OK
Change-Id: I85095bdc2ef4c9be28280e3c9b61de650ee41870
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Willy Tu (1):
updater: Cleanup Active blob to support multiple stages
Change-Id: I996c83a647ed5e0bb376f93a55321c66cc3308fa
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Manojkiran Eda (2):
Remove executable permissions for config file
Remove unused function in fru parser code
Change-Id: I7baf892e270e760e9934ee7c1029f461ef6fc4cf
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Shawn McCarney (1):
regulators: Add detectPhaseFaults() to Device
Change-Id: Ibb7bb699b8d5eaab71821f28f6462e17bbc74a93
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Charles Boyer (1):
service_files: remove StopWhenUnneeded flag in FruDevice service
Konstantin Aladyshev (1):
entity-manager: Fix typos in log messages
Oskar Senft (1):
schema: Add EntityId and EntityInstance to JSON schema.
Patrick Williams (1):
exception: add errno
Zev Weiss (1):
configurations: add ASRock E3C246D4I board
Change-Id: I7c2a5d804f558ba0f94441c52f4d047ce46adb26
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
watchdog-timeout service is needed only if the PHAL machine features
is not present/enabled. In p10bmc is PHAL machine features based, so
removing the watchdog-timeout service.
This commit:
- Does not install openpower-debug-collector-watchdog-timeout@.service
file in the /lib/systemd/system directory for PHAL machine features
based systems
- Removes openpower-debug-collector-watchdog-timeout@0.service link to
the service file, openpower-debug-collector-watchdog-timeout@.service,
on PHAL machine features based systems
Testing:
Installed the image on p10bmc system and verified that the systemd
service file openpower-debug-collector-watchdog-timeout@.service was
not created, and the link file,
openpower-debug-collector-watchdog-timeout@0.service, also was not
created.
Signed-off-by: Shantappa Teekappanavar <sbteeks@yahoo.com>
Change-Id: I7bafd9b7e232043cddbcce76af51fd457169f759
|
|
This commit:
- fixes missing "@" in the
openpower-debug-collector-watchdog-timeout@.service file path.
- Adds openpower-debug-collector-watchdog-timeout@.service file to the
/lib/systemd/system directory.
Testing:
Installed the image on simics and verified that the watchdog-timeout
systemd service file was added to /lib/systemd/system directory.
Signed-off-by: Shantappa Teekappanavar <sbteeks@yahoo.com>
Change-Id: Id6d9309d35bf218a2b458548f8a9dc6b7cc0a58c
|
|
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
|
|
Amitay Isaacs (2):
libipl/p10: libpdbg api always returns -1 on error
libipl: Improve naming IPL_ERR_NILL to IPL_ERR_OK
Chirag Sharma (1):
libipl: Check to not apply resolved guard records
Jayanth Othayoth (6):
libipl: Add support for hostboot attribute overrides
libipl: Add support for disabling denial list based SCOM
libipl: pdbg v3.3 related changes
libipl: Add sbe ffdc data logging support
libipl: Log FFDC data if mpipl_continue chip-op fails
libipl: ipl_error_callback_func prototype change
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
Change-Id: I817d48e88a484b70119314f3e54e5029c233c632
|
|
Make the following changes to fix compilation errors:
- add honister to the LAYERSERIES_COMPAT variable,
- increase CONF_VERSION to 2,
- use new bitbake override syntax
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Change-Id: I971f0f0f7110e0350994ced151ca429444885056
|
|
AMD EthanolX CRB uses 24LC128 EEPROM as a device that is supposed
to store FRU information.
Add necessary Linux driver that adds support for this chip.
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Change-Id: Iaf54bfd5bfbc3266d2a18861ee47339df78c777b
|
|
Since the upstream has renamed the mctp package name (debf2b5),
it will fail when building fp5280g2.
Synchronously update the mctp package name of fp5280g2.
Tested: built fp5280g2 image successfully.
Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: Ifdf4bc7955d182617532537f1f5fc59931d27eca
|
|
Patrick Williams (1):
exception: add errno
Change-Id: I124f8fa8535a5db6c5469b6e760fd1253e424b84
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (2):
exception: create subclass for generated errors
sdbus++: add errno property to errors
Change-Id: I1ce4f1006af90300322b71a3cd52f5716589df40
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Ed Tanous (2):
Make dbus-sensors compile with clang-13
clang-tidy disable bugprone-narrowing-conversions
Change-Id: I078acb48b9c61cdf7ba870fef9bb2de8804bdaa7
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Krzysztof Grobelny (1):
added support for Collection Functions
Lukasz Kazmierczak (1):
Updated subprojects revisions to fix build issues
Change-Id: I63885a6ae212ac7ab6b58381e9d0a2defe2f3fd6
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Jayanth Othayoth (1):
phal: ipl_error_callback_func prototype related changes
Change-Id: I6c3e9b4184b2293c15679b94ff45d37d016189ef
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Alexander Filippov (1):
format: fix logging of a boolean property
Change-Id: I27c4012725c090eea4e17c93e7072cfdf019aec8
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Dixsie Wolmers (1):
Fix broken link on firmware page
Konstantin Aladyshev (8):
Russian translations for the IA log section changes
Russian translations for the IA health section changes
Russian translations for the IA control section changes
Russian translations for the IA configuration section changes
Russian translations for the IA access section changes
Russian translations for the IA resource section changes
Russian translation for the system attention indicators
Russian translations for the export all and delete all buttons
Change-Id: I3ce91e1bf2920843fcfcabd6dd969b0629bf1640
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Patrick Williams (1):
exception: add errno
Shawn McCarney (2):
regulators: Add PhaseFaultDetection to Device
regulators: Simplify Device test cases
Change-Id: I21a846a695166ac704b849d764576a1c9ffb173d
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|