random-seed: reduce scope of variable

units: automatically initialize the system token if that makes sense

bootctl: show random seed state

bootctl: add new verb for initializing a random seed in the ESP

random-seed: move pool size determination to random-util.[ch]

That way we can reuse it elsewhere.

core: take random seed from boot loader and credit it to kernel entropy pool

sd-boot: read random seed from ESP and pass it to OS

efi: steal glibc sha256 implementation

bootctl: add new feature flag for indicating random seed management support

bootctl: shortcut configuration file parsing

bootctl: drop const from non-pointer function argument

bootctl: use the fact that startswith() returns the suffix

efivars: modernize efi_get_variable() a bit

Primarily, make sure the return parameters are all individually
optional.

efi: add log_oom() helper

efi: modernize file_read() a bit

xattr-util: document that we NUL suffix

fs-util: add fsync_full() helper

We usually combine an fsync() with fsync_directory_of_file() hence let's
add a helper that does both in one.

Merge pull request #13144 from poettering/nspawn-modprobe

nspawn modprobe fixes

network: drop redundant Link::kind check

All CAN devices' iftype are ARPHRD_CAN. So, checking Link::kind is
redundant.

Follow-up for c6ac3729c9ece63185a3c4e7bc1970b98d2feb01.

units: add initrd-network-generator.service

Merge pull request #13179 from yuwata/network-drop-new-settings-from-dhcp-section

network: drop recently added settings from deprecated [DHCP] section

network: drop recently added settings from deprecated [DHCP] section

test-network: use [DHCPv4] and [DHCPv6] sections instead of deprecated [DHCP] section

Merge pull request #13169 from ssahani/bridge-igmp-group-version

Bridge igmp group version

test-network: add test for Bridge.IGMPVersion=

networkd: bridge add support to set IGMP version

Add IFLA_BR_MCAST_IGMP_VERSION

Merge pull request #13166 from yuwata/network-slcan-support

network: slcan support

Merge pull request #13173 from kinvolk/iaguis/fix-norbind

Revert "nspawn: remove unnecessary mount option parsing logic"

tests: pass --werror to meson on Semaphore

core: try to reopen /dev/kmsg again right after mounting /dev

I was debugging stuff during early boot, and was confused that I never
found the logs for it in kmsg. The reason for that was that /proc is
generally not mounted the first time we do log_open() and hence
log_set_target(LOG_TARGET_KMSG) we do when running as PID 1 had not
effect. A lot later during start-up we call log_open() again where this
is fixed (after the point where we close all remaining fds still open),
but in the meantime no logs every got written to kmsg. This patch fixes
that.

Add Logitech touchpads, sort vendors alphabetically (#13167)

hwdb: Add Logitech touchpads, such as Logitech Wireless Touchpad, Logitech T650 and others

Sort vendors alphabetically.

update NEWS with more recently commited stuff

Revert "timesyncd: add Conflicts for ntpd and chronyd"

This reverts commit 971a7a1526a6c1d5cb439a8a41dc65ccd4e3a66f.

These unit names are typically different on distributions, let's not
hardcode those. Stuff like this should probably live in the distro
RPM/.deb, but not upstream, where we should be distro agnostic and
agnostic to other higher level packages like this.

test: add some test for norbind

This regressed so let's add a test to make sure it doesn't happen again.

Revert "nspawn: remove unnecessary mount option parsing logic"

This reverts commit 72d967df3e27186dd014bed2c6e7400cc32d84c5.

Revert this because it broke the `norbind` option of the bind flags
because it does bind-mounts unconditionally recursive.

Let's bring the old logic back.

Fixes: #13170

Merge pull request #13168 from ssahani/fix-dhcp-route-no-parsing

network: fix DHCP route rumber parsing.

sysctl: Enable ping(8) inside rootless Podman containers

This makes ping(8) work without CAP_NET_ADMIN and CAP_NET_RAW because
those aren't effective inside rootless Podman containers.

It's quite useful when using OSTree based operating systems like Fedora
Silverblue, where development environments are often set up using
rootless Podman containers with helpers like Toolbox [1]. Not having
a basic network utility like ping(8) work inside the development
environment can be inconvenient.

See:
https://lwn.net/Articles/422330/
http://man7.org/linux/man-pages/man7/icmp.7.html
https://github.com/containers/libpod/issues/1550

The upper limit of the range of group identifiers is set to 2147483647,
which is 2^31-1. Values greater than that get rejected by the kernel
because of this definition in linux/include/net/ping.h:
  #define GID_T_MAX (((gid_t)~0U) >> 1)

That's not so bad because values between 2^31 and 2^32-1 are reserved
on systemd-based systems anyway [2].

[1] https://github.com/debarshiray/toolbox
[2] https://systemd.io/UIDS-GIDS.html#summary

network: support slcan

The device driver does not have IFLA_INFO_KIND attribute. So, we need to
check iftype.

Closes #13150.

network: fix DHCP route rumber parsing.

Merge pull request #13145 from poettering/nss-gateway-fix

filter _gateway NSS resolving to only use main routing table

man: highlight the different concepts behind h/H and t/T

Fixes: #13151

tree-wide: drop netinet/ether.h from socket-util.h and sd-netlink.h

Merge pull request #13118 from bluca/shutdown_watchdog_kexec

core: add KExecWatchdogSec and rename ShutdownWatchdogSec to RebootWatchdogSec

Merge pull request #13143 from poettering/logind-inhibit-restart

logind restart inhibition fixes

meson: reorder alphabetically

local-addresses: filter out any routing tables but the main one

Fixes: #13132

netlink: move local-addresses.[ch] to src/shared

This code is not part of the public API of sd-netlink, nor used by it
internally and hence should not be in the sd-netlink directory.

Also, move the test case for it to src/test/.

[systemctl] Don't print ExecXYZEx= when doing 'systemctl status'

The info printed in this function is the same as the non-Ex version of the
property so there's no point double printing.

Other places that print ExecXYZEx= properties are left alone since the
displayed information is different.

factory: add default /etc/issue file

Booting up an image with --volatile=yes otherwise looks so naked, so
let's include this file in the default factory too. It's common and
simple and should be safe to ship.

update TODO

bootctl: show correct error code

efi: drop unused define

efi: remove trailing whitespace in string

fuzzit: ignore library version

hwdb: run 'meson hwdb-update'

libudev: Update list of possible actions

Add "move" action as per src/libsystemd/sd-device/device-private.c

core: add spdx header to all-units.h

The specific header file is probably not copyrightable anyway, since
it's so trivial, but let's still add the SPDX header line so that a
systematic check for the line does't spit out this header needlessly.

core: rename ShutdownWatchdogSec to RebootWatchdogSec

This option is only used on reboot, not on other types of shutdown
modes, so it is misleading.
Keep the old name working for backward compatibility, but remove it
from the documentation.

core: add KExecWatchdogSec option

Rather than always enabling the shutdown WD on kexec, which might be
dangerous in case the kernel driver and/or the hardware implementation
does not reset the wd on kexec, add a new timer, disabled by default,
to let users optionally enable the shutdown WD on kexec separately
from the runtime and reboot ones. Advise in the documentation to
also use the runtime WD in conjunction with it.

Fixes: a637d0f9ecbe ("core: set shutdown watchdog on kexec too")

Merge pull request #12977 from yuwata/network-route-type-local-12975

network: do not touch kernel-created multicast route

hwdb: add HHKB Pro JP keyboard lack of LEDs

The HHKB Pro JP has no leds whatsoever,  record that. The sysfs
file "capabilities/led" says "1f".

Merge pull request #13148 from poettering/v243-news-more

more v243 news preparation

logind: don't unlink session fifo when exiting logind

Let's only close our fds, but not unlink it. That's done when the
session is stopped.

This should make sure the fd will survive daemon restarts.

logind: make id const, since it points into another buffer

logind: at start-up automatically clean up orphaned inhibitors

logind: un-export and voidify a few functions

Let's minimize scope of functions and make sure that functions that
semantically should never fail can't return errors.

logind: rework allocation/freeing of inhibitors

Let's follow our modern style (i.e. return proper errors, use structure
initialization and _cleanup_).

Most importantly: remove state file and FIFO removal from
inhibitor_free() and let's move it to inhibitor_stop().

This makes sure that state files/FIFOs are not removed when the we
terminate logind, i.e. that they can survive logind restarts.

Fixes: #11825

logind: use free_and_replace() where that makes sense

logind: add logging to inhibitor_load()

Also make some parsing errors, fatals and others (that just care fore
'decoration') non-fatal.

The single caller of inhibitor_load() didn't log about any errors, hence
let's do this in our function, similar to how this is done in
session_load() already.

logind: unify inhibitor signal generation in a single function

logind: cast to (void) when we ignore a syscall return value that is potentially dangerous

logind: drop redundant session_id_valid() check

session_new() checks that as first step anyway, no need to do this
beforehand.

logind: use log_warning_errno() return value where we can

Also, change a couple of log error levels to LOG_WARNING, where we
encounter an error but then end up ignoring it.

logind: don't claim we'd preallocate VTs when we shortcut it

logind: use gcc empty structure initialization

logind: check return value of inhibitor_start()

logind: add missing OOM check in client tool

meson: bump version for package and .so

sd-daemon: don't mention strerror_safe() in examples in public headers

It's an internal function we define, noone else should bother.

NEWS: more additions in preparation von v243

Merge pull request #13136 from keszybz/readd-ntp-units.d

ntp-units.d support

test-network: add tests for issue #6088

network: do not touch kernel-created multicast route

Fixes #6088.

Merge pull request #13109 from poettering/revert-kbd-mode

Revert of #12378 ("VT kbd reset check")

man: document the modprobe hack for DeviceAllow=

units: make sure logind can properly reference drm char class

Similar to the previous fix.

nspawn: explicitly load units beforehand so that DeviceAllow= syntax works

Yuck, but I don't see any prettier solution.

Fixes: #13130

hwdb: add entry for Zowie FK2 mouse (#13139)

Merge pull request #13133 from keszybz/pstore-return-value

pstore: refuse to run if arguments are specified

timesyncd: add Conflicts for ntpd and chronyd

Users might end up with more than one of those service enabled, through
admin mistake, or broken installation scriptlets, or whatever. On my machine,
I had both chronyd and timesyncd happilly running at the same time. If
more than one is enabled, it's better to have just one running. Adding
Conflicts will make the issue more visible in logs too.

timesyncd: add ntp-units.d/ entry for timesync

Priority is 80. At least in Fedora, chrony uses 50, and ntpd 60.
timesyncd has lower priority, because if people install those other packages,
it's most likely on purpose. timesyncd is always installed and provides
less functionality.

timedated: add back support for ntp-units.d/

We removed support for foreign services (and ntp-units.d/) in b72ddf0f4.
Support for foreign services was added back in 5d280742, but through an
environment variable.

The problem with the env var approach is that it only works as a mechanism
to select one item, and doesn't work nicely as a mechinism to create a list
of items through drop-ins (because the env var can be easily overridden, but not
extended). Having a list of "ntp providers" is important to be able to reliably disable
all of them when that is requested.

Another problem is that nobody ever bothered to care about our new "standard".
ntp-units.d/ is a nice simple format that works and is already supported by
chrony and ntpd and timedatex. If we were to introduce and ask people to follow
a new standard, there should be some good reason for this. The idea with env
vars has lower functionality, requires systemd-specific syntax. We should just
re-adopt the format that we originally introduced and that seems to work for
everyone, and more on to more interesting problems.

meson: catch programs which ignore arguments

Don't try to use <<<. It seems to not work properly with old bash
versions or something.

Merge pull request #13131 from yuwata/pstore-follow-ups

pstore: several minor follow-ups

pstore: refuse to run if arguments are specified

(This is why the --help chech passed.)

pid1: make sure we look at /proc/1/cmdline when parsing config

In recent systemd-nspawn we wouldn't parse init args like systemd.log-level=debug.
This is because we wouldn't even look at /proc/1/cmdline.

$ systemd-nspawn -n cat /proc/1/stat
1 (cat) R 0 1 1 34816 ....
                ^^^^^
34816 is 136:0 a.k.a. /dev/pts/0.

pstore: run only when /sys/fs/pstore is not empty

pstore: use log_setup_service()

pstore: do not add FILE= journal entry if content_size == 0