tests: skip static asserts on old clang versions

We declare clang >= 10 is supported. The workaround is easy enough, so let's
just do that. Fixes #28598. With Debian clang version 11.0.1-2:

[267/384] Compiling C object test-bitfield.p/src_test_test-bitfield.c.o
FAILED: test-bitfield.p/src_test_test-bitfield.c.o
clang -Itest-bitfield.p -I. -I.. -Isrc/basic -I../src/basic -Isrc/fundamental -I../src/fundamental -Isrc/systemd -I../src/systemd -I../src/libsystemd/sd-bus -I../src/libsystemd/sd-device -I../src/libsystemd/sd-event -I../src/libsystemd/sd-hwdb -I../src/libsystemd/sd-id128 -I../src/libsystemd/sd-journal -I../src/libsystemd/sd-netlink -I../src/libsystemd/sd-network -I../src/libsystemd/sd-resolve -Isrc/shared -I../src/shared -fcolor-diagnostics -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=gnu11 -O0 -g -Wno-missing-field-initializers -Wno-unused-parameter -Warray-bounds -Wdate-time -Wendif-labels -Werror=format=2 -Werror=implicit-function-declaration -Werror=implicit-int -Werror=incompatible-pointer-types -Werror=int-conversion -Werror=missing-declarations -Werror=missing-prototypes -Werror=overflow -Werror=override-init -Werror=return-type -Werror=shift-count-overflow -Werror=undef -Wfloat-equal -Winit-self -Wmissing-include-dirs -Wmissing-noreturn -Wnested-externs -Wold-style-definition -Wpointer-arith -Wredundant-decls -Wshadow -Wstrict-aliasing=2 -Wstrict-prototypes -Wunused-function -Wwrite-strings '-Wno-error=#warnings' -Wno-string-plus-int -fdiagnostics-show-option -fno-common -fstack-protector -fstack-protector-strong --param=ssp-buffer-size=4 -Wno-typedef-redefinition -Wno-gnu-variable-sized-type-not-at-end -Werror=shadow -fno-strict-aliasing -fvisibility=hidden -fno-omit-frame-pointer -include config.h -DTEST_CODE=1 -MD -MQ test-bitfield.p/src_test_test-bitfield.c.o -MF test-bitfield.p/src_test_test-bitfield.c.o.d -o test-bitfield.p/src_test_test-bitfield.c.o -c ../src/test/test-bitfield.c
../src/test/test-bitfield.c:216:9: error: static_assert failed due to requirement '__builtin_constant_p(({
    static_assert(sizeof(unsigned char) <= sizeof(unsigned long long), "sizeof(uint8_t) <= sizeof(unsigned long long)");
    static_assert(__builtin_choose_expr(__builtin_constant_p(1), 1, 0) < (int)(sizeof(unsigned char) * 8), "__builtin_choose_expr(__builtin_constant_p(1), 1, 0) < (int)(sizeof(uint8_t) * 8)");
    __builtin_choose_expr(__builtin_constant_p(1), ((unsigned char)1) << (1), ({
        int __unique_prefix__i751 = (1);
        do {
            if ((__builtin_expect(!!(!(__unique_prefix__i751 < (int)sizeof(unsigned char) * 8)), 0)))
                log_assert_failed("UNIQ_T(_i, 751) < (int)sizeof(uint8_t) * 8", (&"../src/test/test-bitfield.c"[(sizeof ("..") - sizeof(const char)) + 1]), 216, __func__);
        } while (0);
        ((unsigned char)1) << __unique_prefix__i751;
    }));
}))' "__builtin_constant_p(INDEX_TO_MASK(uint8_t, 1))"
        assert_cc(__builtin_constant_p(INDEX_TO_MASK(uint8_t, 1)));
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../src/fundamental/macro-fundamental.h:109:25: note: expanded from macro 'assert_cc'
                        ^             ~~~~
/usr/include/assert.h:143:24: note: expanded from macro 'static_assert'
                       ^
...

gpt-auto-generator: fix warnings about unused func when !ENABLE_EFI

Merge pull request #28596 from keszybz/doc-updates

Two documentation updates

man/systemd.service: advise Type=exec instead of Type=simple

The descriptions of various options are reworked: first say what protocol
actually is, i.e. describe what type of notification the manager waits
for. Only after that describe various steps and things the service should
do. Also, apply some paragraph breaks.

Instead of recommending Type=simple, recommend Type=exec. Say explicitly that
Type=simple, Type=forking are not recommended. Type=simple ignores failure in a
way that doesn't make any sense except as a historical accident. We introduced
'exec' instead of changing 'simple' to keep backwards-compatiblity, but
'simple' is not very useful. 'forking' works, but is inefficient: correctly
programming the interface requires a lot of work, and at runtime, the
additional one or two forks are just a waste of CPU resources. Furthermore, we
now understand that because of COW traps, they may also increase memory
requirements. There is really no reason to use 'forking', except if it's
already implemented and the code cannot be changed to use 'notify'.

Also, remove the recommendations to use Type=simple to avoid delaying boot. In
most cases, if the service can support notifications about startup, those
should be done.

Overall, for new services, "notify", "notify-reload", and "dbus" are the
types that make sense.

systemd-battery-check: provide more debug logs

The logs would give no hint about the answer to most interesting question: why
we decided to return true or false from the program. If we find batteries
that are low or uncertain, log at info level.

Merge pull request #28474 from yuwata/chase-fix

chase: handle ".." more carefully

NEWS: fix misstatement

We don't have type 'ready'.

Merge pull request #28595 from medhefgo/elf2efi-fixes

elf2efi: Fixes

Merge pull request #28269 from yuwata/udev-builtin-net_id-cleanups-part1

udev: cleanups and several fixes for net_id builtin

Merge pull request #28567 from yuwata/meson-versiondep

meson:  cleanups for version dependency

stub: Also reserve sections for EFI stub

The stub image may not have enough sections to cause enough PE header
space to be free for later expansion. Given that the stub is guaranteed
to be expanded we should always reserve enough sections for it.

This also bumps the reservation to 15. It doesn't add more space
compared to current builds but it more closely reflects the amount of
sections that a UKI could have in total if all optional sections are
used.

Fixes: #28593

elf2efi: Fix header size calculation

The PE header size calculation failed to take the PE magic and coff
header size into account, which will lead to header truncation if we are
writing only 5 sections.

Merge pull request #28573 from yuwata/network-address-next-part2

network: make Address object more consistent with assigned address

test-network: add testcase for overriding Address.RouteMetric=

For issue #25843.

network/address: also save/update priority of prefix route

Strictly speaking, this is not necessary, but let's make the managed
Address objects more consistent with the kernel's addresses.

network/address: introduce address_get_harder() and use it where appropriate

With the previous change, now Address objects under requesting are not
owned by Link object, hence we need to also search corresponding Address
object in the request queue.

network/address: do not add Address object to Link on requesting

Then, all addresses managed by a link really exist (unless the kernel
silently removes addresses).
By this change, now Address objects managed by Link and owned by Request
are decoupled, and always requested settings will be applied.

Fixes #25843.

network/address: make Address object more consistent with assigned address

This makes `manager_rtnl_process_address()`
- first read minimal information to get managed Address object,
- then update the managed Address object directly.

Previously, we updated the Address object managed by Link partially,
e.g. peer address did not updated.
This makes the managed Address object more consistent with the assigned
address.

network: fix typo

Follow-up for bbeceaf2956f6cc12d75d2dc571cfc9fb1d9d839.

resolved: added show-server-state verb and DumpStatistics  varlink method

Added show-server-state verb to resolvectl
Added DumpStatistics and ResetStatistics  methods to varlink

Merge pull request #28591 from yuwata/network-neighbor-next

network/neighbor: follow the kernel's way how to manage neighbors

Merge pull request #28575 from yuwata/network-address-next-part3

network: check specified address settings in more detail

Merge pull request #28572 from yuwata/network-ipv4acd

network: split-out sd_ipv4acd object management from Address object

tmpfiles.d: adjust /dev/vfio/vfio access mode

kmod-static-nodes.service sets the mode to 0600, but we want it
to be 0666 as specified in 50-udev-default.rules.

Resolves: #28588

test-network: add tests for several invalid neighbor settings, and overriding settings

This also makes the checks stricter.

network/neighbor: do not add Neighbor object to Link on requesting

Then, all neighbors managed by a link really exist (unless the kernel
silently removes neighbors).

Merge pull request #28428 from ldv-alt/ERRNO_IS

treewide: cleanup use of ERRNO_IS_*(r)

network/neighbor: follow the way how kernel distinguish neighbor settings

The kernel manages neighbors by the destination address, and the
LinkLayerAddress is mutable. Let's manage neighbors in the same way, and
dedup settings.

network/neighbor: skip to request neighbors with unmatching link layer address length

network/ipv4acd: split out sd_ipv4acd management from Address to Link

Then, we can start IPv4ACD without adding corresponding Address object
to Link.

Hopefully, no functional change, just refactoring and preparation for
later commits.

in-addr-util: introduce PTR_TO_IN4_ADDR() and IN4_ADDR_TO_PTR()

These should be useful when an IPv4 address is used as a key of a
hashmap.

sd-ipv4acd: introduce sd_ipv4acd_is_bound()

network/ipv4acd: drop unused argument

network/ipv4acd: use IN4_ADDR_TO_STRING()

network/address: ignore address settings with unsupported flags

network/address: drop IPv6 settings when the kernel does not support IPv6

network: ndisc - Honour CurHopLimit

RFC4861 Neighbor Discovery – Sections 4.2 and 6.3.4

From section 4.2. Router Advertisement Message Format:

Cur Hop Limit        8-bit unsigned integer.  The default value that
                     should be placed in the Hop Count field of the IP
                     header for outgoing IP packets.  A value of zero
                     means unspecified (by this router).

network/address: merge address_needs_to_set_broadcast() with address_get_broadcast()

No functional change, preparation for later commits.

Merge pull request #28574 from yuwata/network-queue-next

network/queue: preparations

Merge pull request #28571 from yuwata/network-address-next-part1

network: several trivial cleanups

network/neighbor: drop IPv6 settings when the kernel does not support IPv6

network: rename sd_netlink_message* req -> m

This also merges neighbor_configure_message() with neighbor_configure().

Merge pull request #28564 from YHNdnzj/gpt-auto-kill-more-duplicate

gpt-auto: don't mount ESP if there's an fstab entry for it

meson: use kwargs to declare efi binaries

No functional change, just refactoring.

analyze: fix pcrs verb output without TPM support

If we don't have TPM support then `alg` is NULL and passing this to
table_new() means we'd get a table with only two columns instead of
three, leading up to a very confusing output:

$ build/systemd-analyze pcrs
System lacks full TPM2 support, not showing PCR state.
                 NR NAME
                  0 platform-code
                  - 1
    platform-config -
                  2 external-code
                  - 3
    external-config -
                  4 boot-loader-code
                  - 5
 boot-loader-config -
                  6 -
                  - 7
...

Let's name the header in this case with a simple dash, as it's going
to be hidden anyway, to make the table nice again:

$ build/systemd-analyze pcrs
System lacks full TPM2 support, not showing PCR state.
NR NAME
 0 platform-code
 1 platform-config
 2 external-code
 3 external-config
 4 boot-loader-code
 5 boot-loader-config
 6 -
 7 secure-boot-policy
...

meson: unconditionally add version dependency for all executables and libraries

Otherwise, we can easily forget the dependency.
No functional change, just refactoring.

tree-wide: drop unnecessary inclusion of version.h

boot: always set SBAT_DISTRO_VERSION

meson: drop short script to generate vcs tag

tree-wide: fix typo

network/queue: detach request from queue when netlink reply received

Then, we can find and use the Request object after sending netlink
message.

Preparation for later commits.

network/queue: free assigned userdata only when a new request is queued

network/address: split-out address_match_null()

No functional change, preparation for later commits.

network: drop unnecessary conditions

When link_get_by_index() succeeds, the result is always non-NULL.

network: use address_remove_and_drop()

network/address: free Address object by caller that passed to link_request_address()

Follow-up for 9684a8ded083dd427f843b0c40aa0292e6c7ae06.

Now, the input Address object is always copied, hence it is not
necessary to free it in link_request_address().

udev-builtin-net_id: make parse_hotplug_slot_from_function_id() always set return value on success

This also updates comment in the code.

udev-builtin-net_id: make names_xen() self-contained

It is not necessary to store its partial result into NetNames.

No functional changes, just refactoring.

udev-builtin-net_id: use sd_device_get_sysnum() to get index of netdevsim

udev-builtin-net_id: make names_netdevsim() self-contained

It is not necessary to store its partial result into NetNames.

No functional changes, just refactoring.

udev-builtin-net_id: do not assume the current ifname is ethX

Otherwise, `udevadm test-builtin` may not work.

udev-builtin-net_id: make names_platform() self-contained

It is not necessary to store its partial result into NetNames.

No functional changes, just refactoring.

udev-builtin-net_id: do not assume the current interface name is ethX

Otherwise, `udevadm test-builtin` does not work as expected.

udev-builtin-net_id: make names_vio() self-contained

It is not necessary to store its partial result in NetNames.

No functional changes, just refactoring.

udev-builtin-net_id: make names_ccw() self-contained

It is not necessary to store its partial result in NetNames.

No functional changes, just refactoring.

udev-builtin-net_id: make dev_devicetree_onboard() self-contained

and rename it to names_devicetree().

No functional changes, just refactoring.

udev-builtin-net_id: make names_mac() self-contained

The MAC address is only used by names_mac() and ieee_oui(). Let's parse
it in names_mac() and drop it from LinkInfo.

No functional changes, just refactoring.

udev-builtin-net_id: split out get_ifname_prefix()

No functional changes, just refactoring.

udev-builtin-net_id: swap arguments for streq() and friends

udev-builtin-net_id: drop unused value from NetNameType

chase: propagate error in dir_fd_is_root()

Otherwise, if it fails, chaseat() may return unexpected result and
triggers an assertion in chase().

chase: check root path in more detail

In chaseat() we call dir_fd_is_root() several places, and the final
result depends on it. If the root path specified to `chase()` is not
normalized but points to "/", e.g. "/../", assertions in `chaseat()` or
`chase()` may be triggered.

chase: carefully handle result of extracting parent directory

Should not change any behavior.

chase: add two more assertions

When 'need_absolute' is true, 'done' should always contain "/" at the
beginning, and thus should not be NULL.

chase: drop CHASE_AT_RESOLVE_IN_ROOT earlier

The flag will be anyway dropped in chaseat(), but let's shortcut.

chase: fix CHASE_STEP with ".."

chase: reuse "done" to open fd of starting point

For readability that 'done' and 'fd' are always consistent with each other.

- dir_fd == AT_FDCWD:
  - path is absolute:
    - previous: fd = open("/")
    - current:  fd = openat(AT_FDCWD, "/")
  - path is relative:
    - previous: fd = openat(AT_FDCWD, ".")
    - current:  fd = openat(AT_FDCWD, ".")
- dir_fd >= 0:
  - dir_fd points to "/":
    - previous: fd = openat(dir_fd, ".")
    - current:  fd = openat(dir_fd, "/")
  - dir_fd does not point to "/":
    - previous: fd = openat(dir_fd, ".")
    - current:  fd = openat(dir_fd, ".")

Hence, this should not change any behavior. Just refactoring.

Merge pull request #28562 from yuwata/ukify-option-length

ukify: check option length

Merge pull request #28563 from keszybz/configure-meson

configure: update meson invocation

labeller: add build-system label

configure: update meson invocation

New meson says:
WARNING: Running the setup command as `meson [options]` instead of `meson setup [options]` is ambiguous and deprecated.

Inspired by https://github.com/systemd/systemd/issues/28482.

gpt-auto: don't mount ESP if there's an fstab entry for it

Follow-up for #28511

Fixes #28550

fstab-util: add fstab_has_node

Merge pull request #28557 from bluca/utils

utils: add serialize_bool_elide() and send/receive helpers for FD array

Merge pull request #28508 from yuwata/network-next-dhcp4

network: several cleanups and fixes for DHCPv4 client

test-ukify: add tests for an empty argument

Merge pull request #28409 from yuwata/network-dhcp4-drop-duid-only

network,dhcp4: drop "duid-only" option support in ClientIdentifier=

serialize: add serialize_bool_elide() helper

In many case we serialize into objects that have been zero-initialized.
To save some time and resources when there are a lot of booleans, add a
helper that serializes only when the boolean is true.

socket-util: add send/receive helpers for FD array

mkosi: set CONFIG_AUTOFS_FS rather than CONFIG_AUTOFS4_FS

Since Linux 4.18 CONFIG_AUTOFS4_FS just enables CONFIG_AUTOFS_FS, its
description in fs/autofs/Kconfig reads:

   This name exists for people to just automatically pick up the
   new name of the autofs Kconfig option. All it does is select
   the new option name.

   It will go away in a release or two as people have
   transitioned to just plain AUTOFS_FS.

NEWS: mention about the removal of duid-only option

network,dhcp: drop support of ClientIdentifier=duid-only

The setting has not been never worked, not tested, and should not been
used. As the option is RFC incompliant. Let's drop it.

Closes #25562.

ukify: check option length

Follow-up for df4a46733a609f1673de0bebb38e89fffd70c16c.

Merge pull request #28558 from bluca/docs

Update docs/RELEASE.md and NEWS

NEWS: open for v255 business

docs: note that Github Pages configuration has to be updated after a release

Merge pull request #28391 from ssahani/rp-filter

networkd: allow setting rp_filter for an interface

Merge pull request #28552 from bluca/test_execute

test-execute: measure, count and log test runs

test-execute: count and log the number of individual tests actually executed

Drop split-usr and unmerged-usr support

As previously announced, execute order 66:

https://lists.freedesktop.org/archives/systemd-devel/2022-September/048352.html

The meson options split-usr, rootlibdir and rootprefix become no-ops
that print a warning if they are set to anything other than the
default values. We can remove them in a future release.