]> git.ipfire.org Git - thirdparty/systemd.git/commit
core: downgrade errors about BPF loading when called from socket_bind_supported() 19835/head
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Mon, 7 Jun 2021 08:20:30 +0000 (10:20 +0200)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Mon, 7 Jun 2021 08:28:46 +0000 (10:28 +0200)
commit1a9e33aee36b4c353628068507ba1dd7d76b6e14
treee98e99f263f83ca0349fc489a20ed51669387d08
parent5b35b56eae21ccaef9911a5f01b8bc4ce762d5b7
core: downgrade errors about BPF loading when called from socket_bind_supported()

prepare_socket_bind_bpf() is called from two sites: socket_bind_supported() and
socket_bind_install_impl(). For the latter, when errors occur we certainly want
to log, since they'll be fatal for the unit.  But for the former, we should be
quiet, at least on the "expected" errors like lack of permissions. I kept error
on map resizing and such, which should not fail, at log_warning(). They are not
fatal when called from socket_bind_suppported(), but still a sign that
something is off.

Currently BPF filters can only be used by privileged users. Thus each systemd
--user will fail in socket_bind_supported(). With the patch, we only log this
at debug level.

https://lwn.net/ml/bpf/cover.1620499942.git.yifeifz2@illinois.edu/ gives some
hope that unprivileged access will be possible, so let's keep the code trying.
We might get lucky and get support for filters in user mode without any changes
on our side.
src/core/socket-bind.c