Age | Commit message (Collapse) | Author | Files | Lines |
|
This saved 49152 bytes of compressed image space in our platform.
Signed-off-by: Josh Lehan <krellan@google.com>
Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: I4b5b4c7cd2bfffb9720bae02624519ee10ca73af
|
|
With the latest Yocto we see failures:
```
17:14:36 | CMake Error at /home/jenkins-op/workspace/ci-openbmc/distro/ubuntu/label/docker-builder/target/tiogapass/build/work/arm1176jzs-openbmc-linux-gnueabi/phosphor-ipmi-ipmb/0.1+gitAUTOINC+8227626764-r0/recipe-sysroot-native/usr/share/cmake-3.21/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
17:14:36 | Could NOT find PkgConfig (missing: PKG_CONFIG_EXECUTABLE)
```
Add the correct inherit for ipmb-ipmb. Pre-emptively add for any
recipe which does 'inherit cmake'.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ib3327af0f19268d1f5b70259c3445b0985da9fa0
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I588025b614416c43aa2d053765ab53bacf890cb5
|
|
6be1787fdc13405fa9daa6755c28534bbcaa7458 reverted this change
temporarily to get around a yocto recipe refactor that was preventing us
from upgrading cleanly. Unfortunately, there's nothing more permanent
than a temporary hack, so this patch never got reapplied.
Recently, we've had issues with Palmetto failing to build boost::regex
with an error:
./boost/regex/v5/mem_block_cache.hpp:91:11: error: 'static_mutex' in
namespace 'boost' does not name a type
Considering OpenBMC doesn't use boost regex (or a bunch of the other
boost libraries) we shouldn't be building them.
For reference, 57acb4680f999e0b2ab96e87fa5313defddeb577 was the original
patch, and contains more details on the advantages of this, but the keys
are we reduce the build times significantly by only building the two
libraries OpenBMC uses, and we avoid bugs like this in boost modules we
don't care about, as we're only building the modules we use.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I4006618c6553d72bf7f47baadb0c3e35e141bdc9
|
|
This variable was added to bitbake.conf so lets use it. Using it is
best pratice and enables things like alternative base_libdir paths.
Change-Id: I181d12e02ca7918614dacec68bd8e033932eb0ab
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Upstream now has an identical recipe at:
meta-openembedded/meta-oe/recipes-devtools/boost-url/boost-url_git.bb
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I99636ef4eb317e39af5fc053b5c8e61d4af6ce3d
|
|
This reverts commit 6be1787fdc13405fa9daa6755c28534bbcaa7458.
openbmc/openbmc#3720 tracks the issue we've been having with the latest
poky update. For now we're going to have to revert it which means we
need to go back to handling the older boost.
Lets revert this so we go back to not minimizing our boost library which
will cause a bit larger library in flash but allow us to support both
the old and new boost packaging.
(From meta-phosphor rev: 3ee1f9df23ab04871297676c265dfcf0a8db38f4)
Change-Id: Ib45de998931142ea74dffddafd960b7513717a18
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Yocto upstream moved context and coroutine directly into BOOST_LIBS,
which means that they're not added back for our arm variant.
https://github.com/lgirdk/poky/commit/624262541605c476f5a0bd415b08d00379c15b00
This causes our build to fail with "cannot find -lboost_coroutine"
errors. This should resolve it.
Tested:
Fixing CI break. No functional testing done.
(From meta-phosphor rev: 6be1787fdc13405fa9daa6755c28534bbcaa7458)
Signed-off-by: Ed Tanous <ed@tanous.net>
Change-Id: I60d7bb8ea25a08bfdb399f8addfaf560fd7a7b7c
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Yocto upstream moved context and coroutine directly into BOOST_LIBS,
which means that they're not added back for our arm variant.
https://github.com/lgirdk/poky/commit/624262541605c476f5a0bd415b08d00379c15b00
Temporarily remove our BOOST_LIBS override so we can pull the upstream
yocto in, then come back and add specifically what we need again.
Tested:
None, letting CI verify it
(From meta-phosphor rev: 8c30e9b47641854502a31c0a8402d3142586fd55)
Change-Id: I4f174a2473525a02e5c5c080ffc90929f5e4770e
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
The warrior version of rng-tools introduced an optional PACKAGECONFIG,
which was enabled by default, called `libjitterentropy`. For BMC chips
which have a hardware RNG, this is not a necessary feature, and causes
a noticeable delay in the start of the rngd. On systems with very low
IO activity we are seeing cases of what appears to be hangs.
Define a machine feature which can be used to disable the PACKAGECONFIG
for machines which contain a BMC chip with a hardware RNG. Aspeed has
one which is supported; Nuvoton currently does not.
(From meta-phosphor rev: e60e3098c981ece700a855ddc0daf2bdddf0a7da)
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ie4a26e6ca22fe0d7ee9470062f5b26ac733454f2
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Currently when there is no ldap configuration,
nslcd daemon starts with default config which has
URI ldap://127.0.0.1, then we notice unnecessary nslcd
LDAP lookups for all local groups.
this commit updates config with "nss_initgroups_ignoreusers ALLLOCAL"
this option filters out all LDAP lookups for all local groups.
Tested by:
1.reboot system and check traces
2.restarting nslcd service and check traces
(From meta-phosphor rev: cde6c778ea919861d585d46d9599e42d3af04e66)
Signed-off-by: Ravi Teja <raviteja28031990@gmail.com>
Change-Id: I8005582be960a35aedff10353f408d297a3713fb
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
3 years ago, we pushed a patch to allow the usage of boost libraries,
and deleted the phrase "If you give a mouse a cookie". Seemingly, we've
done a decent job of not asking for glass of milk, despite the cookie
being given.
https://github.com/openbmc/docs/commit/56d79455f15ef07be326e59fde4b47ca5866a30c
This commit adds some enforcement of this by removing the libraries that
are disallowed by the coding standard, and ensuring that only the boost
headers are passed through the build. This means that it reduces a
large bottleneck in the build, and saves a significant amount of build
time (about 5 minutes the last time I measured it).
After a discussion a long time ago, it was deemed that coroutine and
context would be allowed in, given that they solved a significant
problem, and didn't add a large binary size to the image. This patchset
preserves those two libraries.
I've had this patch locally for a long time to improve my build times;
I only remembered it because boost-url broke it given that by default
it builds in c++11 mode, which requires boost. As such, I have modified
our build of it to build in standalone mode, which relies on c++17
semantics, and does not require boost libraries. This is added as a
bbappend, given that it will likely be deleted from meta-phosphor
shortly, once we merge in the latest yocto tree, which has accepted it.
Tested:
Built with the patchset below. obmc-phosphor-image builds in
significantly less time, which no sstate cache.
(From meta-phosphor rev: 69e46c2f6f94563762c57382f7dfcc6ac84229b5)
Signed-off-by: Ed Tanous <ed@tanous.net>
Change-Id: I44b849ad395092254a0df84e219e6e78b47625f1
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
This adds boost-url, a header only url parsing library
based on and created by one of the authors of boost
beast. This will be used to replace QueryString in
bmcweb, which is a part that was originally from crow
that has been flagged for issues many times.
Usage:
https://gerrit.openbmc-project.xyz/c/openbmc/bmcweb/+/34973
Tested:
Using bmcweb, url params worked
(From meta-phosphor rev: c1546bf1305e91f1ce27fa0b34f873cd6c82a6de)
Change-Id: I04638ad66525ba0fa9248c95bbf04b1456e6fd4a
Signed-off-by: James Feist <james.feist@linux.intel.com>
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Add CXX to packageconfig and change the devtools to
RDEPEND on libgpiod-tools instead as they are the actual
tools. Without changing it to the tools when we enable
cxx the tools do not by default get installed by Yocto.
Tested: gpioinfo still functioned properly on BMC, and was
able to bitbake package using CXX headers for dbus-sensors.
(From meta-phosphor rev: 76d4c46294c83ea0240a319b4e74c06b57dc51c1)
Change-Id: I5143130bbfbe97f8af4818231d502f36f88fe4ae
Signed-off-by: James Feist <james.feist@linux.intel.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
This gives more CPU time to the daemons and requests are are more
important to run in a timely matter. We can see that we reach ipmid
coming up and multi-user ~20s faster on an ast2500.
Before priority reduction:
root@zaius:~# journalctl -u phosphor-ipmi-host -o short-monotonic
-- Logs begin at Thu 2019-05-16 07:27:57 UTC, end at Thu 2019-05-16 07:31:25 UTC. --
[ 86.855857] zaius systemd[1]: Starting Phosphor Inband IPMI...
[ 91.624926] zaius ipmid[1423]: JSON file not found
[ 93.436815] zaius systemd[1]: Started Phosphor Inband IPMI.
[ 94.615676] zaius ipmid[1423]: Loading whitelist filter
[ 94.686476] zaius ipmid[1423]: Set restrictedMode = false
[ 95.791197] zaius ipmid[1423]: New interface mapping
root@zaius:~# journalctl -u multi-user.target -o short-monotonic
-- Logs begin at Thu 2019-05-16 07:27:57 UTC, end at Thu 2019-05-16 07:31:25 UTC. --
After priority reduction:
[ 106.580622] zaius systemd[1]: Reached target Multi-User System.
root@zaius:~# journalctl -u phosphor-ipmi-host -o short-monotonic
-- Logs begin at Thu 2019-05-16 07:38:53 UTC, end at Thu 2019-05-16 07:41:20 UTC. --
[ 70.097045] zaius systemd[1]: Starting Phosphor Inband IPMI...
[ 73.578294] zaius ipmid[1423]: JSON file not found
[ 75.026965] zaius systemd[1]: Started Phosphor Inband IPMI.
[ 76.143958] zaius ipmid[1423]: Loading whitelist filter
[ 76.301370] zaius ipmid[1423]: Set restrictedMode = false
[ 76.945443] zaius ipmid[1423]: Command in process, no attention
[ 77.062437] zaius ipmid[1423]: New interface mapping
root@zaius:~# journalctl -u multi-user.target -o short-monotonic
-- Logs begin at Thu 2019-05-16 07:38:53 UTC, end at Thu 2019-05-16 07:41:20 UTC. --
[ 83.550371] zaius systemd[1]: Reached target Multi-User System.
(From meta-phosphor rev: fb85bf908cf4106e131dafdb653064cb34106992)
Change-Id: Ie22e5044bb0ef2d2f893a98f657e2083a1789e26
Signed-off-by: William A. Kennington III <wak@google.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
The upstream rng-tools ships a systemd unit as a standard part of the
package now so we don't need to define our own custom one.
(From meta-phosphor rev: 2ced95119fe18b7ac75ea27df869043445734787)
Change-Id: I0b9aaeb17ff85f0ccabb268e10ca48501763da66
Signed-off-by: William A. Kennington III <wak@google.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Reset the following subtrees on thud HEAD:
poky: 87e3a9739d
meta-openembedded: 6094ae18c8
meta-security: 31dc4e7532
meta-raspberrypi: a48743dc36
meta-xilinx: c42016e2e6
Also re-apply backports that didn't make it into thud:
poky:
17726d0 systemd-systemctl-native: handle Install wildcards
meta-openembedded:
4321a5d libtinyxml2: update to 7.0.1
042f0a3 libcereal: Add native and nativesdk classes
e23284f libcereal: Allow empty package
030e8d4 rsyslog: curl-less build with fmhttp PACKAGECONFIG
179a1b9 gtest: update to 1.8.1
Squashed OpenBMC subtree compatibility updates:
meta-aspeed:
Brad Bishop (1):
aspeed: add yocto 2.6 compatibility
meta-ibm:
Brad Bishop (1):
ibm: prepare for yocto 2.6
meta-ingrasys:
Brad Bishop (1):
ingrasys: set layer compatibility to yocto 2.6
meta-openpower:
Brad Bishop (1):
openpower: set layer compatibility to yocto 2.6
meta-phosphor:
Brad Bishop (3):
phosphor: set layer compatibility to thud
phosphor: libgpg-error: drop patches
phosphor: react to fitimage artifact rename
Ed Tanous (4):
Dropbear: upgrade options for latest upgrade
yocto2.6: update openssl options
busybox: remove upstream watchdog patch
systemd: Rebase CONFIG_CGROUP_BPF patch
Change-Id: I7b1fe71cca880d0372a82d94b5fd785323e3a9e7
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Package app phosphor-ldap-conf. This app implements an API to update
the LDAP config file i.e /etc/nslcd.conf.
This commit also creates a file:/etc/nslcd.conf.default, which is to
be restored as /etc/nslcd.conf when we disable LDAP configuration.
(From meta-phosphor rev: 6d7f640d702b5853268bec58e8529516e7884493)
Change-Id: I7183652732c2200b5cc0a644ddb179c8784eb0fb
Signed-off-by: Nagaraju Goruganti <ngorugan@in.ibm.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
A cereal recipe was added recently to meta-openembedded. Use it so we
can deprecate our version of it.
(From meta-phosphor rev: cc51cd9910f3480cf4d2e0bff8aa6b95162e5bc1)
Change-Id: I7746dbd1eda3011b870b85770092b2d020800af7
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
This has been merged into meta-oe, use the meta-oe version
as it is more up to date.
(From meta-phosphor rev: d32b6818513b069ad59fa60a5301d5a52ebf4d2f)
Change-Id: I1487849fba34e9363d96ca4bef9b5b692fb13b46
Signed-off-by: James Feist <james.feist@linux.intel.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Update to latest release and fixup recipe to use cmake.
Copy styling from rapidjson in meta-oe as these have basically
the same purpose.
Tested-by: Verified image still built
(From meta-phosphor rev: 9a6e2eebe418cfb5608f477caa327864e4037fdc)
Change-Id: Iceb244a43e7a24dbdd5e2f9d4c5470796cb85d0b
Signed-off-by: James Feist <james.feist@linux.intel.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
nss-pam-ldapd uses the openldap API to interact with the
ldap server.
openldap recipe includes the openldap daemon as a LDAP server
and the libraries implementing the LDAP protocol.
BMC would be acting as LDAP client so we don't need those
pacakages which is related to LDAP server.
Packages which is being removed through this commit are
mdb,passwd,dnssrv,monitor,meta etc,these are slapd backends.
Tested: This commit reduces the size of the image by 1.5M.
(From meta-phosphor rev: dd63751bbbaad25b745aa210b587d6818c083ece)
Change-Id: Id8a64c3775b3c771258072e866cfd0c070af57db
Signed-off-by: Ratan Gupta <ratagupt@in.ibm.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Bringing the LDAP authentication module support in openbmc stack
requires to pull the nss-pam-ldapd which allows the LDAP
server to provide the user, passwd, group info that we normally get
from the /etc flat files.
nss-pam-ldapd provides libnss-ldap and pam_ldap module which
delegate the work to the nslcd(daemon) that queries the LDAP server.
pam_ldap uses the openldap client API to interact with the LDAP server.
nss-pam-ldapd files are pulled from
http://git.yoctoproject.org/cgit/cgit.cgi/meta-cloud-services/
tree/recipes-support/nss-pam-ldapd
meta-cloud-services sha: 38cc19fb3a813673051de314aafabda0545d8466
Tested: Adding the "ldap" distro feature brings the nss-pam-ldapd
and its dependencies into the image and removing the "ldap" from the
distro feature doesn't bring the nss-pam-ldapd and its dependencies.
(From meta-phosphor rev: 4835bb3901a4bff777a97d4f363e3a731b87f21c)
Change-Id: Ifa5da20e7ac47b0c9d9af305ae621252e6d765f3
Signed-off-by: Ratan Gupta <ratagupt@in.ibm.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Adopt a more conventional directory hierarchy. meta-phosphor is still
a _long_ way from suitable for hosting on yoctoproject.org but things
like this don't help.
(From meta-phosphor rev: 471cfcefa74b8c7ceb704cb670e6d915cf27c63b)
Change-Id: I3f106b2f6cdc6cec734be28a6090800546f362eb
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|