Preset user units on first boot as well

We need to make sure the presets from /usr/lib/systemd/user-preset
are applied as well. Currently only the ones from
/usr/lib/systemd/system-preset are applied.

Merge pull request #34511 from YHNdnzj/strv-extend-strv-consume

basic/strv: introduce strv_extend_strv_consume()

Merge pull request #34502 from yuwata/strv_find_closest

strv: introduce strv_find_closest()

Merge pull request #34515 from YHNdnzj/machined-machines-by

machined: modernize a bit

core/cgroup: drop `allow_cache` parameter in `unit_get_io_accounting()`

The name of the parameter is misleading and it does not save us much
work because it is not used during regular unit property queries.
It is only used during unit_log_resources(), and the cgroup is already
dead by that point so it won't be read anyway.

tree-wide: use strv_extend_strv_consume() where appropriate

basic/strv: introduce strv_extend_strv_consume()

machined-dbus: move manager_add_machine() and _get_machine_by_pid() to -core

machined: rename machine_{units,leaders} to machines_by_*

Also port machines_by_leader to store PidRef-s.

machined: fix bogus error check for machine_link()

basic/strv: make string_strv_hash_ops static, add missing assertions

shared/bus-map-properties: modernize map_basic() and bus_map_strv_sort()

basic/memory-util: make mempcpy_typesafe() take number of obj rather than raw size

Follow-up for eda6223942a172fa6777901cf5fbd47438f285ce

basic: replace size_multiply_overflow() with MUL_ASSIGN_SAFE where applicable

basic/macro: move DECIMAL_STR_FMT to shared/tests

This shall never be used outside of test functions.

Add an extra debug log to dissect_image()

machine: fix bus method argument name: who -> whom

Follow-up for cd2fb04960b8aa0b353d12005b8cfc3b9d2bcfee.

verbs: use strv_find_closest()

This also makes the list of verbs is always shown on failure.

strv: introduce strv_find_closest()

Follow-up for 1e1ac5d53b0f126b6c4419506c7c42b67c07537f.

Merge pull request #34510 from keszybz/mkosi-version-checks

Bump minimum required mkosi version

mkosi: bump mkosi MinimumVersion

dbff64ddf06f64ab94bd314df27d6c089b75de52 bumped the hash to
a commit after 24.3, so let's tell the users that 25~devel is
the minimum required.

mkosi: supress error messages from git

When updating, I get a message like:
fatal: Not a valid object name a67221c3f0d0b81b9b5b3230a71d09044342f1a4^{commit}

The failure here is expected, it just means that an update is
necessary, so suppress output.

sd-ipv4acd: fix assertion triggered when an ARP received in STARTED state

When a network is busy, an ARP may be received before the timer event
source triggered first time.

Fixes #34489.

NEWS: fix typos and remove backticks

Merge pull request #34499 from YHNdnzj/sd-path-trivial-cleanup

sd-path: trivial cleanups

Merge pull request #34496 from YHNdnzj/tmpfiles-neg-errno

tmpfiles: ERRNO_IS_NOINFO -> _IS_NEG_, correct negative errno checks

firstboot: add similar input suggestion

This uses the same logic as similar verb suggestion for command line
utilities. Try to be helpful when the user entered something invalid
instead of just showing the prompt again.

basic/memory-util: introduce mempcpy_typesafe

boot: use INC_SAFE where appropriate

Merge pull request #34480 from yuwata/test-seccomp-suppress-sync

seccomp-util: pass negative fds as is to fsync() and friends

kernel-install: add uki.conf example

sd-path: trivial cleanups for sd_path_lookup{,_strv}()

sd-path: modernize from_user_dir()

Deduplicate logic through sd_path_lookup() and from_home_dir().
Besides, rename to from_xdg_user_dir() to indicate
it's a XDG thing.

tmpfiles: ERRNO_IS_NOINFO -> _IS_NEG_, correct negative errno checks

tmpfiles: use RET_GATHER more

man: update PCR and Secure Boot key names and paths

firstboot: Prompt for keymap

It's rather crucial to have a good firstboot experience that you
can immediately set the right keymap so let's make sure we prompt
for it.

hwdb: add keymaps for Acer Nitro 5 AN515-47 (#34493)

This fixes the microphone mute key and NitroSense key for Acer Nitro 5 AN515-47,
so they can be recognized by desktop environments.

test: add tests for seccomp_suppress_sync()

seccomp-util: pass negative fds as is to fsync() and friends

Closes #34478.

Co-authored-by: Mike Yuan <me@yhndnzj.com>

sysupdated: Rearrange error logging a little bit

First, this fixes a case where an error is logged twice at the LOG_ERR
level. Second, this goes through and raises the log level where
appropriate so that sysupdated is easier to debug. Finally, it replaces
EINVAL with EPROTO where appropriate, since EINVAL implies that the
caller passed some incorrect arguments (which is incorrect; sysupdate
passed some invalid JSON back to us)

Merge pull request #34486 from DaanDeMeyer/test-process-util

test-process-util: Migrate to new assertion macros

Merge pull request #34481 from yuwata/has-tpm2

tpm2-util: several cleanups for tpm2_support()

updatectl: check: Don't print an empty table

If there are no updates available, `updatectl check` would previously
just print an empty table. We should instead just tell the user that
there are no updates available.

Merge pull request #34483 from yuwata/network-conf-parser-neighbor-nexthop

network: several cleanups for conf parsers

test-process-util: Ignore EINVAL from setresuid() and setresgid()

If we're running in a user namespace with a single user and without
the nobody user, we'll get EINVAL from these system calls so make
sure we handle those gracefully.

test-process-util: Use FORK_REOPEN_LOG everywhere we close all fds

To make sure logging works in the child processes.

test-process-util: Migrate to new assertion macros

man/systemd-analyze: mention required libraries for TPM2 support

Closes #34477.

tpm2-util: colorize output of 'systemd-analyze has-tpm2'

shell-completion/analyze: add has-tpm2

tpm2-util: update comment

has-tpm2 command is moved to systemd-analyze.

Follow-up for 58e359604ffdca12bb4d2c5807b96e070611c0f6.

tpm2-util: do not load tpm2 libraries when not interested in the existence of the libraries

For example, 'bootctl status' only interested in if the efi has TPM2
support and a TPM2 driver is loaded. Hence, not necessary to load
libtss2.

tpm2-util: introduce tpm2_is_fully_supported()

Merge pull request #34479 from yuwata/sd-json-dispatch-field-table-static

tree-wide: make sd_json_dispatch_field table static

network/nexthop: introduce generic conf parser for [NextHop] section

network/nexthop: make conf parsers for Family= and Gateway= independent of each other

network/nexthop: use log_section_warning() and friend

Add ASSERT_OK_ZERO_ERRNO() and ASSERT_OK_EQ_ERRNO()

Merge pull request #34474 from DaanDeMeyer/user-group

Two integration test fixes

test: Run TEST-74-AUX-UTILS in virtual machine

Various tests skip themselves when running in a container so make
sure the test runs in a virtual machine so we get full coverage.

test: create .netdev file at last

Previously, when the test ran on mkosi, then networkd was not masked, and
might be already started. In that case, the interface test2 would be created
soon after the .netdev file is created, and the .link file would not be
applied to the interface. Hence, the later test case for
'networkctl cat @test2:link' would fail.

This make networkd always started at the beginning of the test, and
.netdev file created after .link file is created. So, .link file is
always applied to the interface created by the .netdev file.

man: fix typo

Follow-up for 8aee931e7ae1adb01eeac0e1e4c0aef6ed3969ec.

efi: fix typo

Follow-up for f4e081051d950a09ce9331ba55eaf604dac72652.

ci: Don't add testuser to wheel and systemd-journal groups

This breaks TEST-74-AUX-UTILS when run in a VM as the user gets access
to journal files that the test expects it can't access.

test: drop removed SCSI passthrough feature

This feature has been deprecated since QEMU 5.0 and finally removed in
QEMU 9.1 [0] which now causes issues when running the storage tests on
latest Arch:

------ testcase_long_sysfs_path: BEGIN ------
...
qemu-system-x86_64: -device virtio-blk-pci,drive=drive0,scsi=off,bus=pci_bridge25: Property 'virtio-blk-pci.scsi' not found
E: qemu failed with exit code 1

[0] https://github.com/qemu/qemu/commit/a271b8d7b2f39275a05e49deb7c8edc20b7a8279

TEST-64-UDEV-STORAGE: remove obsolete NVMe symlink for nvme_subsystem

In the nvme_subsystem test, there are only namespace IDs 16 and 17,
so there would no longer be an "obsolete" symlink created, since this
test scenaro does not create a namespace with ID 1.

Signed-off-by: Bryan Gurney <bgurney@redhat.com>

Pin obsolete NVMe symlinks to namespace 1

In order to preserve backwards compatibility with legacy NVMe devices,
create the obsolete symlinks, but only if they are associated with
namespace ID 1.

Signed-off-by: Bryan Gurney <bgurney@redhat.com>

network/neighbor: use log_section_warning_errno()

network/neighbor: introduce generic Neighbor section parser

network/neighbor: use struct in_addr_data

conf-parser: drop unnecessary temporary variable

conf-parser: introduce config_parse_in_addr_data()

tree-wide: drop unnecessary 'struct'

nsresource: make sd_json_dispatch_field table static

This also adds missing error check of sd_json_dispatch().

Follow-up for 54452c7b2aa03536331fc1ec04439c89d0320b57.

creds-util: make sd_json_dispatch_field table static

resolve: make sd_json_dispatch_field table static

resolvectl: make sd_json_dispatch_field table static

updatectl: make sd_json_dispatch_field table static

This also fixes memory leak of Version object on failure.

Follow-up for ec15bb71c2ccd57702325a6e6626053a343f3511.

varlinkctl: make sd_json_dispatch_field table static

ssh-generator: make sd_json_dispatch_field table static

machine: make sd_json_dispatch_field table static

nsresourced: fix build without libbpf

```
In file included from ../src/nsresourced/nsresourced-manager.c:9:
../src/shared/bpf-link.h:5:10: fatal error: bpf/libbpf.h: No such file or directory
    5 | #include <bpf/libbpf.h>
      |          ^~~~~~~~~~~~~~
```

Follow-up for 46718d344fdaaaf523d854a0c728197e7406a55a

repart: Drop unprivileged subvolumes logic for btrfs

The functionality was explicitly not included in 6.11 for some
unknown reason so drop the logic from systemd-repart as well so
we don't release v257 with it included.

man/repart: use <varname> instead of <variable>

Otherwise, `<variable>$BOOT</variable>` is rendered:

```
[2548/2992] Generating man/repart.d.5 with a custom command
Element variable in namespace '' encountered in para, but no template matches.
Element variable in namespace '' encountered in para, but no template matches.
```

hwdb: Add accel orientation quirk for the IdeaPad Duet 3 10IGL5-LTE

Signed-off-by: Marius Hoch <mail@mariushoch.de>

repart: Support specifying multiple directories to ExcludeFiles=

systemd: rewatch pids under cgroup v1 when sigchld of processes more than main pid and control pid is captured

If `Delegate` is configured in service, cgroup agent will never send out
any datagram as .control subcgroup is generated. Thus systemd will watch
all processes on the cgroup hierarchy for SIGCHLD to deal with unreliable
cgroup notifications.

In this way, systemd should rewatch all processes when any SIGCHLD is
captured, more than the control pid or main pid.

man: document ShowStatus and SetShowStatus()

SetShowStatus() was added in order to fix #11447. Recently, I ran into
the exact same problem that OP was experiencing in #11447. I wasn’t able
to figure out how to deal with the problem until I found #11447, and it
took me a while to find #11447.

This commit takes what I learned from reading #11447 and adds it to the
documentation. Hopefully, this will make it easier for other people who
run into the same problem in the future.

Merge pull request #34464 from yuwata/test-space-in-path

test: allow to run tests under directory that contains spaces

test: quote paths to executables

Fixes #34459.

kernel-install: unquote plugin paths in KERNEL_INSTALL_PLUGINS

To support the case that paths to plugins contain spaces.

Prompted by #34459

Merge pull request #34460 from yuwata/test-86-follow-ups

test: follow-ups for TEST-86

tmpfiles.d: Remove purge flag from lines that don't support it

Fixes db15657dfb8100c3c6cd8cf60fa68521e019d47a

ukify: Remove debug log

This prints a python data structure which we shouldn't do during
normal operation.

Merge pull request #34040 from AdrianVovk/repart-dollar-boot

repart: Implement $BOOT support

Merge pull request #34440 from yuwata/network-log-no-matching-network

network: log when no matching .network file found

network/dhcp4: use device_get_property_bool() at link_needs_dhcp_broadcast()

No functional change, just refactoring.

test: drop unused test.sh for TEST-86-MULTI-PROFILE-UKI

The test cannot run with the bash test runner, as it requires python.
Hence, test.sh is not necessary.

Follow-up for a37640653cab108bb2cfdcba7e9422cced925e95.

test: minor coding style fixlets

Follow-up for a37640653cab108bb2cfdcba7e9422cced925e95.