Merge pull request #19921 from yuwata/service-verify-bus-type

core/service: fix assertion when Type=dbus but BusName= is not specified

unit: use alias name of man page

man: fix sd_bus_add_node_enumerator() ret_nodes

ret_nodes is NULL terminated, the return value isn't a count.

man: add an example to configure default route on device with table

Prompted by #19911.

Merge pull request #19930 from yuwata/update-comments

Update comments

fix typo

tree-wide: add missing whitespace at the end of comments

Merge pull request #19924 from yuwata/sd-event-fix-assertion

sd-event: fix assrtion in sleep_between()

test-seccomp: Check for __NR_ppoll before use (#19858)

some newer architectures like riscv32 do not have __NR_ppoll from get go

Merge pull request #19913 from yuwata/network-fix-counter

network: add missing increment of Link::set_flags_messages

repart: show partitions we don't grow/create as "unchanged"

The previous string was "unknown", but that's wrong, because we *do*
know what we are going to do with those partitions: we leave them
unmodified, hence say "unchanged" in the output, to be clearer.

repart: align all sizes in table to the right

test: correctly configure the NUMA node memory pool

The currently hardcoded value works with the default configuration, but
breaks when QEMU_MEM != 512M (in sanitizer runs, for example).

```
# QEMU_MEM=1G make -C test/TEST-36-NUMAPOLICY/ run
make: Entering directory '/home/fsumsal/repos/@systemd/systemd/test/TEST-36-NUMAPOLICY'
TEST-36-NUMAPOLICY RUN: test NUMAPolicy= and NUMAMask= options
+ /bin/qemu-kvm -smp 8 -net none -m 1G -nographic -kernel /boot/vmlinuz-5.12.5-300.fc34.x86_64 -drive format=raw'
qemu-kvm: total memory for NUMA nodes (0x20000000) should equal RAM size (0x40000000)
E: QEMU failed with exit code 1
```

Merge pull request #19905 from yuwata/network-set-mac-try-again

network: try to bring down interface before setting MAC address

network: update wlan information when IFF_LOWER_UP flag is gained

Fixes the issue mentioned at
https://github.com/systemd/systemd/issues/19832#issuecomment-860269320.

network: add brief comment about reconfiguring interfaces

This also renames link_reconfigure_internal() -> link_reconfigure_impl().

network: do not partially update wlan information on failure

network: add missing increment of Link::set_flags_messages

link_up_or_down() will decrement the counter when the subsequent
RTM_GETLINK netlink method is finished. So, we need to increment
the counter here.

Fixes the issue mentioned at
https://github.com/systemd/systemd/issues/19832#issuecomment-860255692.

test: add a test case for #19920

core/service: fix assertion when Type=dbus but BusName= is not specified

Fixes #19920.

sd-event: always reshuffle time prioq on changing online/offline state

Before 81107b8419c39f726fd2805517a5b9faab204e59, the compare functions
for the latest or earliest prioq did not handle ratelimited flag.
So, it was ok to not reshuffle the time prioq when changing the flag.

But now, those two compare functions also compare the source is
ratelimited or not. So, it is necessary to reshuffle the time prioq
after changing the ratelimited flag.

Hopefully fixes #19903.

sd-event: make event_source_time_prioq_reshuffle() accept all event source type

But it does nothing for an event source which is neither a timer nor
ratelimited.

sd-event: use usec_add()

sd-event: use CMP() macro

sd-event: drop unnecessary "else"

Revert "Mount all fs nosuid when NoNewPrivileges=yes"

This reverts commit d8e3c31bd8e307c8defc759424298175aa0f7001.

A poorly documented fact is that SELinux unfortunately uses nosuid mount flag
to specify that also a fundamental feature of SELinux, domain transitions, must
not be allowed either. While this could be mitigated case by case by changing
the SELinux policy to use `nosuid_transition`, such mitigations would probably
have to be added everywhere if systemd used automatic nosuid mount flags when
`NoNewPrivileges=yes` would be implied. This isn't very desirable from SELinux
policy point of view since also untrusted mounts in service's mount namespaces
could start triggering domain transitions.

Alternatively there could be directives to override this behavior globally or
for each service (for example, new directives `SUIDPaths=`/`NoSUIDPaths=` or
more generic mount flag applicators), but since there's little value of the
commit by itself (setting NNP already disables most setuid functionality), it's
simpler to revert the commit. Such new directives could be used to implement
the original goal.

Merge pull request #19898 from yuwata/network-multipath-route-without-interface-name

network: set link ifindex when multi-path routes specified without interface name

Merge pull request #19894 from yuwata/network-fix-configure-without-carrier

network: always check dynamic address assignments before entering con…

Merge pull request #19866 from yuwata/sd-device-new-from-ifindex

sd-device: introduce sd_device_new_from_ifindex()

network: drop misleading debugging logs about MTU

This fixes the following spurious logs on enumerating links:
```
wlan0: Saved original MTU 1500 (min: 256, max: 2304)
wlan0: MTU is changed: 0 → 1500 (min: 256, max: 2304)
```

network: try to bring down before setting MAC address

Most real network devices refuse to set MAC address when its operstate
is not down. So, setting MAC address once failed, then let's bring down
the interface and retry to set.

Closes #6696.

network: check the size of hardware address before setting MAC address

Also, skip to set MAC address when the current address equals to the
requrested one.

Add a network timeout option to journal-upload

doc: Add RISC-V to FAQ

Was added to the spec in 499f0f8209f33da0b6e3b0d6f3a13996edcc7555.

test/TEST-17-UDEV: use default image

No need to build a separate 700MB image to save 3080 bytes in the default image.

network: always check dynamic address assignments before entering configured state

Previously (v248 or earlier), even if no static address is configured,
the link did not enter configured state, as e.g. Link::static_addresses_configured
is false until the link gained its carrier.
But, after the commit 1187fc337577cecd685d331eeab656be186ba3b2, the
situation was changed. Static addresses, routes, and etc are requested even
if the link does not have its carrier, and thus the link enters configured
state when no static address and etc are specified.

This makes the link does not enter configured state before it gains its
carrier when at least one of dynamic address assignment protocols (e.g.
DHCP) except for NDISC is enabled.

Note that, unfortunately, netplan always enables ConfigureWithoutCarrier=
for all virtual devices, e.g. bridge. See,
https://github.com/canonical/netplan/commit/978e20f902f6b92a46dc6e0050e2172e834e4617
So, we need to support e.g. the following strange config:
```
[Netowkr]
ConfigureWithoutCarrier=yes
DHCP=yes
```

Fixes #19855.

test-network: disable dynamic addressing protocols when ConfigureWithoutCarrier= is enabled

test-network: refuse routable state when no-carrier is expected

If we do not set the maximum operstate in wait-online, then routable
state is allowed when no-carrier state is expected.

test-network: add a testcase for MultiPathRoute= without interface name

network: route: set link ifindex when multi-path routes specified without interface name

And do not set RTA_OIF attribute when multi-path routes are specified.
See kernel's fib_get_nhs() in net/ipv4/fib_semantics.c.

Fixes #19890.

sd-dhcp: refuse to set iaid if we cannot find the interface

dhcp: do not use ifindex when generating iaid in tests

sd-dhcp: do not use detect_container() to guess udev is running or not

nspawn: path_is_read_only_fs() may return negative errno

And we usually assume /sys is not read only on error.

tree-wide: use sd_device_new_from_ifindex/ifname()

sd-device: introduce sd_device_new_from_ifname/ifindex()

netlink: make rtnl_resolve_link_alternative_name() optionally return the main interface name

netlink: check input name is valid before calling netlink method

netlink: move resolve_ifname() or friends to netlink-util.[ch]

network: use link_get_by_name()

docs: update coding style a bit

Say that r should be declared at the top of the function.

Don't say that fixed buffers result in truncation, right after saying that they
must only be used if size is known.

Adjust order of examples to be consistent.

Merge pull request #19882 from keszybz/test-stat-util-more

Add a test for path_is_read_only_fs()

update TODO

core: Avoid spurious realization of unit cgroups

Cgroups may be unnecessarily realized when they are not needed. This
happens, e.g. for mount units parsed from /proc/$PID/mountinfo, check

        touch /run/ns_mount
        unshare -n sh -c "mount --bind /proc/self/ns/net /run/ns_mount"
        # no cgroup exists
        file /sys/fs/cgroup/system.slice/run-ns_mount.mount
        systemctl daemon-reload
        # the vain cgroup exists
        file /sys/fs/cgroup/system.slice/run-ns_mount.mount

. (Such cgroups can account to a large number with many similar mounts.)

The code already accounts for "lazy" realization (see various checks for
Unit.cgroup_realized) but the unit_deserialize() in the reload/reexec
path performs unconditional realization.

Invalidate (and queue) the units for realization only if we know that
they were already realized in the past. This is a safe thing to do even
in the case the reload brings some new cgroup setting (controllers, BPF)
because units that aren't realized will use the updated setting when the
time for their realization comes. (It's not even needed to add a code
comment because the current formulation suggests the changed behavior.)

man: clarify that global search domains apply to global servers, not all interfaces

Fixes #19257.

test-stat-util: don't fail under chroot

I wanted to see what is_path_read_only_fs() and is_path_temporary_fs() return
in a chroot, and various tests would fail. For most of our codebase, we can
assume that /proc and such are mounted, and it doesn't make sense to make the
tests work in a chroot. But let's do it here. (In general, it would be useful
for most stuff in src/basic/, since it's linked into libraries which might be
invoked in incorrectly set up environments and should not fail too badly.)

Merge pull request #19811 from anitazha/revert_mount_rl

sd-event: fix failure to exit rate limiting state

Merge pull request #19864 from keszybz/serialization-cleanup

Serialization cleanup

kbd-model-map: add Latvian keyboard layout mapping

Merge pull request #18851 from yuwata/dissect-try-to-find-partition-on-timeout

dissect: try to find partition more frequently

Merge pull request #19870 from keszybz/install-foo-again

Tweak the install logic again

Merge pull request #19878 from poettering/large-key-file-cryptsetup

add back support for large key files to systemd-cryptsetup

docs/CODING_STYLE: fix some typos

core/serialize: drop bogus deserialization of ipcns sockets

a70581ffb5c13c91c76ff73ba6f5f3ff59c5a915 added ExecRuntime.ipcns_storage_socket[], and
serialization in exec_runtime_serialize(), and deserialization in exec_runtime_deserialize_one(),
but also deserialization in exec_runtime_deserialize_compat(). exec_runtime_deserialize_compat()
is for deserializating ExecRuntime when it was serialized as part of the unit before
e8a565cb660a7a11f76180fe441ba8e4f9383771. There was never any code which would serialize
ExecRuntime.ipcns_storage_socket[] this way, so the deserialization attempts are pointless.

core/serialization: shorten code, treat all oom error the same

core/serialization: drop misleadingly-named unit_can_serialize()

All unit types can be serialized. This function was really checking whether the
unit type has custom serialization/deserialization code. But we don't need a
function for this.

Also, the check that both .serialize() and .deserialize_item() are defined is
better written as an assert. Not we have a function which would skip
serialization/deserializaton for the unit if we forgot to set either of the
fields.

test-stat-util: add a very basic test for test_path_is_read_only()

test-stat-util: add standard test logging

cryptsetup: improve error message when key files to load are too large

Let's make this easier to grok for users.

Prompted-by: #19193

fileio: bump limit for read_full_file() and friends to 64M

Apparently people use such large key files. Specifically, people used 4M
key files, and we lowered the limit from 4M to 4M-1 back in 248.

This raises the limit to 64M for read_full_file() to avoid these
specific issues and give some non-trivial room beyond the 4M files seen
IRL.

Note that that a 64M allocation in glibc is always immediately done via
mmap(), and is thus a lot slower than shorter allocations. This means
read_virtual_file() becomes ridiculously slow if we'd use the large
limit, since we use it all the time for reading /proc and /sys metadata,
and read_virtual_file() typically allocates the full size with malloc()
in advance.  In fact it becomes so slow, that test-process-util kept
timing out on me all the time, once I blindly raised the limit.

This patch hence introduces two distinct limits for read_full_file() and
read_virtual_file(): the former is much larger than the latter and the
latter remains where it is. This is safe since the former uses an
exponentially growing realloc() loop while the latter uses the
aforementioend ahead-of-time full limit allocation.

Fixes: #19193

shared/install: ignore enablement of template units w/o instance when presetting

When we have a unit which cannot be enabled:
 # foo@.service:
 ...
 [Install]
 WantedBy=foo.target  # there is no instance, so we don't know what to enable

we should throw an error when invoked directly with 'enable', but
not when doing 'preset' or 'preset-all'.

Fixes #19856.

shared/install: pass UnitFileFlags down into the call chain

This just propagates the parameter down into leaf functions,
without any functional change.

Merge pull request #19857 from yuwata/tmpfile-fix

tmpfiles: fix an issue found by Coverity

Merge pull request #19863 from keszybz/coverity-drop-unitialized-workarounds

Drop some -Wmaybe-unitialized workarounds to help coverity

Merge pull request #19867 from yuwata/ether-addr-util

ether-addr-util: introduce hw_addr_equal() and friends

core/dbus: rename internal variable for clarity

man: add note about operation without swap in systemd-oomd

Clarify help information for --global

sd-event: change ordering of pending/ratelimited events

Instead of ordering non-pending before pending we should order
"non-pending OR ratelimited" before "pending AND not-ratelimited".
This fixes a bug where ratelimited events were ordered at the end of the
priority queue and could be stuck there for an indeterminate amount of
time.

tmpfile: always get file descriptor of root or current directory

Fixes CID#1457467.

Merge pull request #19871 from yuwata/man-network-missing-settings

man: add missing settings

cryptsetup: remove unitialized workaround

Doesn't seem needed anymore.

test-capability: drop work-around initialization

Since those workarounds have been added, work has been done to tighten
up log_*() return values. Seems we get no warning with
gcc-11.1.1-1.fc34.x86_64 and -O0/-O2.

networkd: drop one workaround initialization

As for the other ones in src/network/, if they are removed, gcc warns when they
are removed.

Should fix Coverity CID#1457466.

Merge pull request #19861 from poettering/hwdb-249

hwdb + syscall database updates for 249

man: add missing settings

Fixes #19869.

network: sort settings about netdev

man: merge several settings about netdev

shared/install: improve message about template mismatch

$ systemctl enable --root=/ serial-getty@.service
Failed to enable unit, unit getty.target is a non-template unit.
↓
Failed to enable serial-getty@.service, destination unit getty.target is a non-template unit.

shared/install: remove custom error handling in unit_file_preset_all()

This had some purpose back in the day, but right now I cannot see what
difference this makes. It's hard to keep the list of all possible errors up to
date. So let's remove this, hopefully nothing breaks.

shared/install: ignore failures for auxiliary files

If Also= fails, warn, but otherwise ignore the failure.

Fixes #19407.

test: add extended test for triggering mount rate limit

It's hard to trigger the failure to exit the rate limit state in
isolation as it needs multiple event sources in order to show that it
gets stuck in the queue. Hence why this is an extended test.

man: fix missing markdown & minor errors

In #19771 there were a few missing markdown tags a few style issue.

Signed-off-by: Peter Morrow <pemorrow@linux.microsoft.com>

ether-addr-util, network: introduce ETHER_ADDR_TO_STR() macro and use it

network: use hw_addr_equal() or friends

ether-addr-util: introduce hw_addr_compare(), hw_addr_equal(), and hw_addr_is_null()

ether-addr-util: drop redundant "addr" from struct hw_addr_data

Also, this makes always specifiy "struct" for hw_addr_data.

Merge pull request #19854 from poettering/journal-enum-uniq-fix

fix journalctl -F

seccomp: add some recently added syscalls to filter groups

core/serialization: call exec_runtime_deserialize_compat() independently of whether .serialize is defined

There is no reason to tie the two together: in principle we may have
in the future a unit type which does not define .serialize/.deserialize_item,
but we would still want to call the compat deserialization code for it.