loopback: use ERRNO_IS_PRIVILEGE() where appropriate

loop: make 'Failed to configure loopback device' log message clearer

We print the very same log message for loopback block devices and for
loopback network devices. Let's better be clear what kind it is.

docs: use Title Case for section titles

as per:

https://titlecase.com/

scsi_id: retry inquiry ioctl if host_byte is DID_TRANSPORT_DISRUPTED

The inquiry is issued to kernel via ioctl, kernelspace would set
this inquiry command's retry count to 0 which means the command
would not be retried in kernel space even if the LLDs returned
a status which need to be retried. So we should take the retry
in user space.

condition: fix device-tree firmware path

The path /sys/firmware/device-tree doesn't exist. This should be either
/proc/device-tree or /sys/firmware/devicetree.

The first path is only a link. So lets use the second path.

See https://github.com/torvalds/linux/blob/v4.14/drivers/of/base.c#L218.

man: Minor punctuation and word tweak

udev-util: minor cleanups for on_ac_power()

Follow-ups for #24420.

Merge pull request #24475 from yuwata/devpath

sd-device: introduce two helper functions for devnum

network: drop unused timestamp

test-64: run one more subtest on non-KVM environment with relaxed condition

docs: fix incorrect env var name for credentials directory

CREDENTIAL_PATH appears nowhere in the systemd source code.
$CREDENTIALS_DIRECTORY is what is used instead.

shell-completion: drop unused $mode

Fixes #24473.

udevadm: replace find_device_from_path() with sd_device_new_from_path()

test-network: add missing online check

udev: do not kill workers when requested to set the same log level currently assigned

Also refuse invalid log level.

udev/net: drop unused timestamp

test-64: relax number of partitions used in testcase_simultaneous_events() to speed up non-KVM environment

tree-wide: use devpath_from_devnum() and device_open_from_devnum()

Fixes #24465.

sd-device: skip to check diskseq if device is not initialized

sd-device: introduce device_open_from_devnum()

sd-device: introduce devpath_from_devnum()

devnum-util: split-out device_path_make_inaccessible()

sd-device: rename device-util.c -> device-filter.c

The functions provided by the file are only used in sd-device.

growfs,repart: fix misuse of sd_device_get_devpath()

Follow-up for 0f79b3469f167583033d17f9a6fa0e4341a68003.

To obtain device node, sd_device_get_devname() must be used.

test-64: extend timeout for slower env e.g. non-kvm

tree-wide: fix typo

hwdb: Add Dell Professional Sound Bar AE515

loop-util: use filter provided by sd_device_enumerator

Merge pull request #24468 from yuwata/test-udev-debug

test: enable udev debug log

test-50-dissect: generate debugging logs of udevd

test-58-repart: use udevadm control instead of creating service drop-in

Merge pull request #23764 from enr0n/oomd-allow-managed-oom-preference

oomd: allow ManagedOOMPreference on all cgroups

Merge pull request #24463 from mrc0mmand/ubuntu-ci-tweaks

(Not only) Ubuntu CI tweaks

systemd-growfs: use sd_device_new_from_devnum()

Use sd_device_new_from_devnum() instead of
device_path_make_major_minor_sysfs().

test: make the virt detection quiet

Follow-up to cde09b07dfdc132a31672693c037bfc0b5879331.

test: set the default QEMU and nspawn timeouts to 30 minutes

Set both timeouts to some reasonable values instead of just `infinity`
to provide some form of a safe-net in case the test goes haywire and the
environment didn't set the timeouts itself (like our CIs do).

test: respect the global $QEMU_TIMEOUT

CIs set QEMU and nspawn timeouts by themselves which reflect their needs
and possibilities, so let's respect that value, instead of using one
pre-set value which might or might not work for all of them.

Both Ubuntu CI and CentOS CI set these values themselves.

test: use a unique machine name for each nspawn test

to avoid scope clashing in case some previous test crashed and/or didn't
clean up properly. Currently all test machines are called `root`, since
the name is automagically derived from the container path (in this case
`/var/tmp/systemd-test.XXXXXX/root`).

E.g. (from Ubuntu CI):
```
[23:10:12] --x-- Running TEST-71-HOSTNAME --x--
make: Entering directory '/tmp/autopkgtest.5LjnBV/build.0mE/systemd/test/TEST-71-HOSTNAME'
+ make -C TEST-71-HOSTNAME setup run
TEST-71-HOSTNAME SETUP: test hostnamed
Reusing existing cached image /tmp/autopkgtest.5LjnBV/build.0mE/systemd/test/TEST-71-HOSTNAME/../default.img → /tmp/autopkgtest.5LjnBV/build.0mE/systemd/test/default.img
'/var/tmp/systemd-test.1yy2SS/default.img' -> '/tmp/autopkgtest.5LjnBV/build.0mE/systemd/test/default.img'
I: Masking supporting services
'/var/tmp/systemd-test.1yy2SS/root/etc/systemd/system/systemd-hwdb-update.service' -> '/dev/null'
'/var/tmp/systemd-test.1yy2SS/root/etc/systemd/system/systemd-journal-catalog-update.service' -> '/dev/null'
'/var/tmp/systemd-test.1yy2SS/root/etc/systemd/system/systemd-networkd.service' -> '/dev/null'
'/var/tmp/systemd-test.1yy2SS/root/etc/systemd/system/systemd-networkd.socket' -> '/dev/null'
'/var/tmp/systemd-test.1yy2SS/root/etc/systemd/system/systemd-resolved.service' -> '/dev/null'
TEST-71-HOSTNAME RUN: test hostnamed
+ env --unset=UNIFIED_CGROUP_HIERARCHY --unset=SYSTEMD_NSPAWN_UNIFIED_HIERARCHY timeout --foreground 1200 /bin/systemd-nspawn --register=no --kill-signal=SIGKILL --directory=/var/tmp/systemd-test.1yy2SS/root --setenv=SYSTEMD_UNIT_PATH=/usr/lib/systemd/tests/testdata/testsuite-71.units:/usr/lib/systemd/tests/testdata/units: /lib/systemd/systemd systemd.unit=testsuite.target systemd.wants=testsuite-71.service systemd.wants=end.service
Spawning container root on /var/tmp/systemd-test.1yy2SS/root.
Press ^] three times within 1s to kill container.
Failed to allocate scope: Unit root.scope already exists.
E: nspawn failed with exit code 1
```

test: bump the reboot timeout to 60 seconds

As the reboot might take a bit on oversaturated hypervisors (spotted in
Ubuntu CI).

Merge pull request #24440 from mrc0mmand/TEST-64-tweaks

A couple of TEST-64 tweaks to make it usable without KVM

Merge pull request #24456 from yuwata/network-tcp-congctl

network: introduce TCPCongestionControlAlgorithm=

oomd: fix off-by-one when dumping kill candidates

When we kill a cgroup that is towards the end of the sorted candidate
list (i.e. when we have to resort to killing a candidate with
ManagedOOMPreference=avoid), this cgroup is not logged in the candidate
list. This is due to an off-by-one error when assigning dump_until.

man: update ManagedOOMPreference entry to reflect new behavior

oomd: add tests for oomd_fetch_cgroup_oom_preference

Add a new test function, test_oomd_fetch_cgroup_oom_preference, to test
the ManagedOOMPreference logic. For starters, cut the relevant tests out
of test_oomd_cgroup_context_acquire_and_insert, and add them to the new
function. Then, expand these tests to cover the new behavior.

oomd: loosen the restriction on ManagedOOMPreference

The ManagedOOMPreference property is only honored on cgroups which are
owned by root. This precludes anyone from setting ManagedOOMPreference
on cgroups managed by user managers.

Loosen this restriction in the following way: when processing a
monitored cgroup for kill candidates, honor the ManagedOOMPreference
setting if the monitored cgroup and cgroup candidate are owned by the
same user. This allows unprivileged users to configure
ManagedOOMPreference on their cgroups without affecting the kill
priority of ancestor cgroups.

N.B. that since swap kill operates globally to kill the largest
candidate, it is not appropriate to apply this logic to the swap kill
scenario. Therefore, the existing restriction on ManagedOOMPreference
will remain when calculating candidates for swap kill.

Add a new function, oomd_fetch_cgroup_oom_preference, to assist with
this new logic. To simplify things, move the `user.oomd_{avoid,omit}`
xattr reads to this function so that the xattr reads and uid checks are
performed all at once.

core/cgroup: allow setting oomd xattrs from user manager

The extended attributes 'user.oomd_avoid' and 'user.oomd_omit' are used
to store the setting of the ManagedOOMPrefernce property. Since these
are user extended attributes, the access permissions are defined by the
file permissions, and there is no need to guard this from user managers.

sleep: doc update for multiple battery, acpi_btp support and freeze/thaw user.slice

test: temporarily (?) disable TEST-64 in Ubuntu CI

The machines are still way too slow for the test to work properly
without accel.

test: bump the base VM memory to 768M

as with 512M some tests occasionally trip off OOM-killer (e.g.
TEST-64 + multipath).

test: lower the # of iterations with plain QEMU

test: require KVM only for specific sub-tests

since other sub-tests run relatively fine with TCG as well.

test: check for other hypervisors as well

EC2 machines run on Xen, so account for that as well when checking KVM
availability.

test: lower the # of mpath devices to 16

to make the test suitable for slower machines.

Merge pull request #24459 from poettering/tmpfiles-graceful-copy

tmpfiles: make absent source of "C" copy lines graceful

Merge pull request #24460 from poettering/stub-measure-fix

sd-stub: some minor refactoring of the cpio/TPM code

cpio: don't claim we measured anything if we weren't even asked

This is a bugfix.

Follow-up for 599fe002a135570406504998dbb7a42dabc963da

cpio: split out cpio TPM measurement logic from pack_cpio()

No code change, just some refactoring, so that we can reuse the
measurement logic later elsewhere.

test-network: add test for TCPCongestionControlAlgorithm=

network: introduce TCPCongestionControlAlgorithm=

Closes #24432.

Merge pull request #24457 from poettering/smbios-condition-fix

SMBIOS condition fix

tmpfiles: move validation/normalization of path before we use it

We need to normalize/validate the path (i.e. first column) of
tmpfiles.d/ lines before we start using the path, otherwise we'll use it
before it's known to be good. This matters since for some line types the
path is mangled into the argument column (i.e. sevents column), and we
should only do that once we know it's in a good state.

tmpfiles: in C lines, make missing source graceful error

I don't see where it would ever be a good thing that file copies done
via tmpfiles.d/ C lines cause the tmpfiles operation to fail if their
source happens to be missing. It's a problem if we can't set up the
destination properly (which is the job of systemd-tmpfiles after all),
but if the source is simply missing (NB: setting up the source is the job of
of the rules writer) this shouldn't be a problem.

This is useful for copying stuff into place if it happens to exist. For
example, if systemd-stub passes additional data into the initrd's
/.extra/ directory, we can copy it into a better place (e.g. /run/) with
this, where it will survive the initrd→host transition.

This mirrors behaviour of the recently added "^" line modifier which may
be used source "w" lines from credentials – there two the behaviour is
to simply skip the line if the source is missing.

test: don't accept failure as succes

condition: check for last not first ')' in firmware test expressions

I possess a machine with ')' in its BIOS version string, which will
cause the current parser to mistake it as the closing ')' of the
smbios-fields() expression.

Let's make sure we always fo for the last, not the first ')', hence.

test: wait for loop device to be removed

Follow-up for bca762ce1abafd24016eba7310fdd38c758419f8.

Fixes #24450.

test-sd-device: run subsystem filter test several times

The test is quite racy. Let's wait for all queued udev events to be
processed, and run the test several times.

Fixes #24400.

sd-netlink: fix attribute type for RTAX_CC_ALGO

resolve: control system hostname synthesis via environment variable

sleep: freeze and thaw user.slice to save resource while suspend

tree-wide: change --kill-who to --kill-whom

getopt allows non-ambiguous abbreviations, so backwards-compat is maintained, and
people can use --kill-who (or even shorter abbreviations). English is flexible,
so in common speach people would use both forms, even if "whom" is technically
more correct. The advantage of using the longer form in the code is that we
effectively allow both forms, so we stop punishing people who DTGCT¹, but still
allow people to use the spoken form if they prefer.

1. Do the gramatically correct thing

Merge pull request #24454 from bluca/ci

Fix Ubuntu/SUSE mkosi

mkosi: add back packages removed from OpenSUSE build

Some packages were removed from the OpenSUSE build, which broke the
unit tests. Add them back.

Fixes: 37d35150cbb5 ("mkosi: Ensure we build all features/components in mkosi")

mkosi: disable isc-dhcp-server again

It's still failing in Ubuntu:

● isc-dhcp-server.service  loaded failed failed ISC DHCP IPv4 server
● isc-dhcp-server6.service loaded failed failed ISC DHCP IPv6 server

Just disable them via the kernel command line masking.

journal: rename special journal field _SYSTEM_CONTEXT= → _RUNTIME_SCOPE=

Previously the field "_SYSTEM_CONTEXT" knew he values "initrd" + "main". Let's change
this to "_RUNTIME_SCOPE" and "initrd" + "system".

Why? The sysext logic has a very similar concept of "scopes", declaring
whether a sysext image is intended for the initrd or the main system.
Let's thus use the same naming for both.

sysext's extension-release files hence know SYSEXT_SCOPE=initrd|system,
and the journal messages know _RUNTIME_SCOPE=initrd|system, which makes
this reasonably systematic.

Follow-up for: cae8edd93ca2ef90c41cb9b6322b6908d12947b5

(This is not an API break, since no version with this commit has ever
been released.)

pid1: extend "ConditionFirmware=" for checking SMBIOS system identification information

mkosi: Print logs of failing tests

core/mount: voidify unit_acquire_invocation_id()

Follow-up for 01400460ae16c6522d11d08dd9a4b0928e7980d9.

Fixes CID#1491960.

Merge pull request #24442 from yuwata/udev-retry-event

udev: make requeued event actually restarted

virt: fix alignment

Follow-up for f5558306746c0791dab666e76d51e7ef1ce38362.

test: make TEST-63 more reliable on slower machines

Otherwise we might never hit the trigger limit and wait indefinitely.

Found when trying to run the test on an EC2 xen machine without a nested
virt in CentOS CI (in preparations for some ... unforseseen consequences).

udev: drop unnecessary calls of event_queue_start()

As the subsequent call of on_post() will call it if necessary.

This also drop unnecessary call of event_source_disable() for killing
idle workers, as the event source is disabled in event_queue_start().

udev: certainly restart event for previously locked device

If udevd receives a uevent for a locked block device, then the event
is requeued. However, the queued event will be processed only when at
least one sd_event_source is processed. Hence, if udevd has no event
under processing, or receives no new uevent, etc., then the requeued
event will be never processed.

Follow-up for 400e3d21f8cae53a8ba9f9567f244fbf6f3e076c.

Fixes #24439.

mkosi: use the just built systemd-sysusers instead of groupadd

It disappeared again from SUSE Tumbleweed. Just use sysusers since we
just built it.

Merge pull request #24408 from keszybz/execstart-escape

Properly escape ExecStart= commandlines in transient units

Merge pull request #24242 from msekletar/terminate-idle-sessions

Add option to stop idle sessions after specified timeout

shutdown: Fix last try detection

This was missed in 5125b67742567de79c12e31f434a11d787b4f6b1.

Merge pull request #24429 from poettering/selinux-code-improv

minor selinux-util.c modernization

systemd-growfs: remove dependency on udev symlinks

systemd-growfs currently requires that udev ran first to work (because
/dev/block/ symlinks must exist) but that is not what happens when
we're not using initrd and systemd is PID1.

time-util: fix overflow condition in usec_sub_signed()

If the delta specified is INT64_MIN, and we negate that we'd end up at
INT64_MAX+1 which is outside of the int64_t type. Hence let's treat this
case specifically to avoid unintended overflows.

core: allow disabling system time correction if rtc returns time far in the future

There might be (embedded) systems that get never updated (things like
e.g. entertainment systems of trains, for example) and where the adjustment of
the system clock (introduced by b10abe4bba61aebe4c667c412741193f11886298) would
do the wrong thing even if the difference between the systemd build time and
the rtc is 15 years or more.

This patch allows disabling the adjustment by setting
'clock-valid-range-usec-max' meson option to 0 or to a negative value.

journald: use rate-limited logging to log 'Failed to write entry' message

Merge pull request #24370 from keszybz/sysusers-equivs

Use /bin/bash for root shell and suppress some warnings from sysusers

selinux-util: add safety size check before doing strdupa()

selinux-util: modernizations

Let's no bother with reading return value of libselinux API calls into
r, if the actual error is in errno anyway.

Let's remove one unnecessary strdup()

Let's rename a return parameter ret_xyz

Merge pull request #24072 from poettering/remove-cgroupsv1-docs

decgroupsv1ification: first steps – remove from docs, and generate warnings

tests: add test for StopIdleSessionSec= option

logind: add option to stop idle sessions after specified timeout

Thanks to Jan Pazdziora <jpazdziora@redhat.com> for providing a patch
which implemeted a PoC of this feature.

Merge pull request #24420 from keszybz/mouse-power

Rework on_ac_power()

basic/user-util: rename output param to ret, shorten code

basic/user-util: avoid filesystem access check

The check of u==UID_NOBODY is just a register comparison, but
synthesize_nobody() requires a system call, so let's invert the order in the
condition. Since most calls into this module are not for nobody, we should
save one syscall in the common case.