Age | Commit message (Collapse) | Author | Files | Lines |
|
openpower-debug-collector may end up requesting a SBE or hostboot dump.
Ensure that if stop-instruction is also scheduled in the same systemd
target, that we first stop instruction to get the hardware in a known
state and then start the dump collection.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I1156a20b16545ff42e20d4a8e3fc1089786ba605
|
|
Manojkiran Eda (1):
Add OWNERS file
Shantappa Teekappanavar (1):
Watchdog timeout support in SBE boot window
Change-Id: I26e2d03199ca2ad15d9f7dde930854b1de4442bd
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Upstream yocto made a change recently that brought to light a bug in
some of our recipes. If your meson makefiles utilize the dependency()
function then the recipe must also include pkgconfig.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I48c062226275c6e110bde27e815d2643c369c14c
|
|
OBMC_MACHINE_FEATURES is a variable that probably shouldn't even exist
as all it does is provide a level of indirection to the real
MACHINE_FEATURES (in phosphor-base.inc, OBMC_ is added to the base).
There are a few recipes that have developed checks against
OBMC_MACHINE_FEATURES for enabling recipe-level features when they
should be checking the standard MACHINE_FEATURES variable. Clean up
this recipe-level indirection.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Id437b93aa6c1de9a27726eea9b7e2c30e71ab3bf
|
|
Shantappa Teekappanavar (1):
Remove checks for AutoReboot in watchdog timeout
Change-Id: I49aa3ae4fa6b007965a6ddb1832189ac52da1f6f
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
|
|
Patrick Williams (1):
exception: switch to public sdbus exception
Change-Id: I65e07d8dfb54ccc5d6f028683f61ebd4b92b1122
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Shantappa Teekappanavar (1):
watchdog: Create PEL when AutoReboot is false and watchdog times out
Change-Id: If2349e70cbe42036fb653df0153ac5d889f43252
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Dhruvaraj Subhashchandran (1):
Add fmtlib support
Shantappa Teekappanavar (1):
watchdog: Collect hostboot dump when watchdog times out
Change-Id: I7f569d253215b5c0d78a1826fe115021d2f76216
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Change-Id: I5ca16c6fa7bc616bfa9bc41c4f86d32fd92b0287
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
See openbmc/openpower-debug-collector#2
The move to meson broke something in this path that is not obvious to
me. It's also not clear if this function is really even needed.
For now, disable so we can get back to the SDK building and give us some
time to look further into this.
Tested:
- Verified SDK now builds
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I459ecd7279004c47f89fadc39e224020725c3ae8
|
|
Shantappa Teekappanavar (1):
meson: remove old automake files
Change-Id: I1164d7964e27e4355ecd37fdb915889e95c2afd3
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
The openpower-debug-collector now supports meson
build infrastructure to build the binaries. Switching
to meson build system instead of using autotools.
The enable/disable install_error_yaml is an optional
feature used to check the availability of the required
packages to build binaries using autotools. In meson
build system, this feature is removed since the availability
of the packages are always checked, and therefore removing
it from the recipe file.
Added version bump 222131c to enable meson build support
Tested:
- Image build: MACHINE=witherspoon-tacoma bitbake obmc-phosphor-image
- Updated witherspoon-tacoma BMC with newly created image
- Verified watchdog_timeout and checkstop_app are installed
at /usr/bin directory
Signed-off-by: Shantappa Teekappanavar <sbteeks@yahoo.com>
Change-Id: Idc0fc31dbe1e604b4d010590ef18cc5f3638d65b
|
|
Patrick Williams (1):
bootstrap: fix shellcheck warnings
Change-Id: I29de0d05cb3b87ad2cffd630f906c05b8ef46cfd
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Jayanth Othayoth (1):
Update clang-format
Change-Id: Id987582fae4eb1f888c0434f2ae14621fbea88be
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
When mpreboot is enabled, there is a more advanced form of diagnostic
and recovery put into the obmc-host-crash@.target. In this case we do
not want the basic error reported by the service within this repo.
Tested:
- Verified correct services installed for mpreboot on Rainier and
Tacoma machines
- Verified non-mpreboot services installed on Witherspoon machine
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: If29ad7a83c1e1551a7eb319179eca504e9f93678
|
|
The OpenBMC community has been moving away from the use of the
SYSTEMD_LINK macro and going towards more explicit commands to create
the needed wants/requires relationships for systemd.
This commit does that in preparation for some other work in this area
utilizing the new mpreboot distro feature. No functional changes were
done with this commit.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I01086e51400cf94498c62d9bc6cbcc75813b20f6
|
|
(From meta-openpower rev: 0b6a5a4200476300470668c9b9f5526bf153e315)
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Id64a0ffda3d81ea267115a62422166534ca2eab9
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
(From meta-openpower rev: 45c1fcd1ea9a3e0cf476853e719adcffd28221ad)
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
Change-Id: I96ba2d6854b992e619f5c5fc60f90de6ff8d560f
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Some recipes have abstracted the git revision into a separate file
and have a line 'require ${PN}.inc'. The issue with PN is that it
changes to have a -native or -nativesdk for those contexts. The
variable ${BPN} (base package name) should be used instead.
(From meta-openpower rev: df35a2536cf347e9a4e5b15ae48b59630722d08d)
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I12040750b349bc36601c00a1b355d81ded2a0a01
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
sdbusplus has changed the package layout so that sdbusplus-native and
sdbus++-native are now ${PYTHON_PN}-sdbus++-native. Update recipes
that need to use sdbus++ (or its associated python modules) or remove
the DEPENDS if it is no longer necessary.
(From meta-openpower rev: d555f82d1a10a4c65377ae9b090996220a635c0d)
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ie79b13b1cce3ee14e89d895d325c021c68cf89bd
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Out of the box bitbake examines built applications, automatically adds
runtime dependencies and thus ensures any library packages dependencies
are automatically added to images, sdks, etc. There is no need to list
them explicitly in a recipe.
Dependencies change over time, and listing them explicitly is likely
prone to errors - the net effect being unnecessary shared library
packages being installed into images.
Consult
https://www.yoctoproject.org/docs/latest/mega-manual/mega-manual.html#var-RDEPENDS
for information on when to use explicit runtime dependencies.
(From meta-openpower rev: 42fbcf6040860dc6b0b553febeabedd241f22a27)
Change-Id: If56c264d3807f604ae8ba9c948095ba61093285d
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Fixup PV in openpower-debug-collector_git.bb
(From meta-openpower rev: fe60aee974fbe0f059705b6cf5dc9cf22f505c3b)
Change-Id: Ie9d0ea2b158f17c12e00ac9bf3f60ed513d3550b
Signed-off-by: Patrick Venture <venture@google.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Jayanth Othayoth (2):
MAINTAINERS: Added Reviewer information
Enable clang code format
(From meta-openpower rev: b05da16b2b2fb42862ee81cbd9bd5e030310fcf9)
Change-Id: If170aae3a14a10436235ba7da2d4ecdce0755072
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Vernon Mauery (1):
openpower-debug-collector: use c++17
(From meta-openpower rev: 962bda8e055b63d52b74c3690d57010e5bbcdb42)
Change-Id: Ia836f14527e191f408a0e338847361b3b6d7853d
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Rename openpower-debug-collector.bb to openpower-debug-collector_git.bb
per OE norms.
(From meta-openpower rev: c69639aff81c9aba43d0f221101fe64653493311)
Change-Id: I0725aa7d9d7b74320fa767fe2dd3e8051270f702
Signed-off-by: Patrick Venture <venture@google.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Adopt a more conventional directory hierarchy. meta-openpower is still
a _long_ way from suitable for hosting on yoctoproject.org but things
like this don't help.
(From meta-openpower rev: 2c8db5661cecd7e4cc20bcb96df76490e62cfb67)
Change-Id: I415caaa28ec728f24e755ddb549012f24874fefe
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|