network/dhcp-server: use hash_ops with destructor for static lease

network/radv: use hash_ops with destructor for managing prefixes

network,udev: use hash_ops with destructor to manage SR-IOV configs

network/ndisc: replace set_free_free() with set_free()

They uses in_addr_prefix_hash_ops_free, hence set_free() is enough.

network: use in6_addr_hash_ops_free in Network.ipv6_proxy_ndp_addresses

This also one adjustment to network_adjust_ipv6_proxy_ndp().

network: use dns_name_hash_ops_free in dnssec negative trust anchors

test-networkd-conf: config_parse_ether_addrs() uses ether_addr_hash_ops_free

Hence, set_free() is enough to free the result.

network-generator: use hash_ops with destructor for Network, NetDev, and Link

wait-online: move manager_process_link() to link.c

Then, rename it to rtnl_process_link().

No functional change, just refactoring.

wait-online: use hash_ops with destructor for managing Link

sysv-generator: introduce hash_ops for SysvStub

This also renames free_sysvstub() -> sysvstub_free(), to follow our
usual coding style.

sysusers: use trivial_hash_ops_free for storing user/group name

sd-netlink: introduce custom hash_ops for GenericNetlinkFamily

sd-device-enumerator: use custom hash_ops with destructor

sd-device: replace set_free_free() with set_free()

These uses string_hash_ops_free, hence not set_free() is enough.

coredump: replace custom cleanup function with specific hash_ops with destructor

analyze: replace set_free_free() with set_free()

set_put_strdup() uses &string_hash_ops_free, hence set_free() also
frees stored contents.

resolved-dns-trust-anchor: use hash_ops with destructor for managing DnsAnswer

resolved-dns-trust-anchor: use hash_ops with destructor for managing RRs

resolved-dns-scope: use hash_ops with destructor for conflict_queue

resolve: use dns_name_hash_ops_free for dnssec negative trust anchor

journal-remote: introduce custom hash_ops with destructor for MHDDaemonWrapper

Then, we can move declaration of the type from journal-remote.c to
journal-remote-main.c, and drop several #if ... #endif.

sd-journal: use hash_ops with destructor that closes JournalFile

journal: use hash_ops with destructor that make JournalFile offline and close

journal: replace unnecessary use of set_free_free() with set_free()

These uses pcre2_code_hash_ops_free, hence set_free() is enough.

unit-file: use set_free() to free paths

As it uses path_hash_ops_free.

core/unit: use string_hash_ops_free for Unit.aliases

core,systemctl: unit_file_find_fragment() provides names with string_hash_ops_free

names are assigned with set_put_strdup(), hence string_hash_ops_free is
used and we can use set_free() rather than set_free_free().

core: replace unnecessary use of set_free_free() with set_free()

These variables are assigned with set_put_strv(), hence
string_hash_ops_free is used.

This also drops unnecessary call of set_ensure_allocated() with wrong
hash ops.

install: use hash_ops with destructor for InstallInfo

install: use path_hash_ops_free

locale-util,kbd-util: several cleanups (#37090)

core/execute: set_get_strv() does not transfer ownership of strings

Prompted by #37090

fstab/gpt-auto: hook up validatefs to /sysroot/usr/ rather than /sysusr/usr/ (#37109)

socket-proxy: several cleanups (#37107)

stdio-bridge: minor coding style cleanup (#37110)

kbd-util: use string_hash_ops_free and set_to_strv()

locale-util: use string_hash_ops_free and set_to_strv()

Then we can drop hackish combination of use of set_free_free() and
set_free().

strv: constify arguments for strv_equal_ignore_order()

Follow-up for 5072f4268b89a71e47e59c434da0222f722c7f0e.

set: introduce set_to_strv()

It is similar to set_get_strv(), but invalidates the set on success.

locale-util: coding style cleanups

locale-util: filter out non-UTF-8 locales even when failed to parse $SYSTEMD_LIST_NON_UTF8_LOCALES

locale-util: fix argument for munmap()

socket-proxy: reset connection only in callback functions

This makes connection_complete(), connection_start(), and friends not
reset connection on failure.

socket-proxy: use event_source_hash_ops for managing event sources

event-util: use DEFINE_HASH_OPS_WITH_VALUE_DESTRUCTOR() to define event_source_hash_ops

Currently, the hash_ops is only used by set, so this does not change
anything. But, there are several benefits,
- we can drop cast in the definition,
- the hash_ops can be also used by hashmap.

socket-proxy: use hash_ops with destructor for managing Connection

This also renames context_clear() -> context_done(), to follow our
recent coding style.

socket-proxy: make connection_free() return NULL

To make it follow our coding style.

socket-proxy: use event_reset_time_relative()

Also, split out context_reset_timer() from connection_release().
No functional change. Preparation for later change.

stdio-bridge: use DEFAULT_SYSTEM_BUS_ADDRESS

stdio-bridge: minor modernization for parse_argv()

fstab-generator: honor x-systemd.validatefs from kernel cmdline too

fstab/gpt-auto: hook up validatefs to /sysroot/usr/ rather than /sysusr/usr/

We carry a special logic for /usr/ in initrd that it initially gets
mounted to /sysusr/usr/ and later bind mounted to /sysroot/usr/.
When hooking up validatefs it is hence necessary to wait for
the bind mount and validate that, rather than the /sysusr/usr/
established as intermediate transport.

Alternative to #37102

gpt-auto: do not add /sysusr/usr/ -> /sysroot/usr/ after switch-root

Follow-up for 7852e301e0eb839adf4bf45aa41e39c0dfc03403

libudev: use 'type* func()' style rather than 'type *func()'

libudev-list: use strdup_to()

No functional change, just refactoring.

libudev-list: use hashmap_dump_sorted()

libudev-list: use custom hash_ops with destructor for udev_list_entry

TODO: drop getenv item, done where needed

Use secure_getenv() instead of getenv() where appropriate

busctl: several cleanups for introspect command (#37103)

busctl: use table to format result of introspect command

busctl: use set_dump_sorted() and FOREACH_ARRAY()

busctl: use strdup_to() and set_consume()

busctl: set destructor to member_hash_ops

Then, we can drop the custom destructor of 'Set *mmebers'.

network/netdev: remove support of netdevsim

netdevsim device creation by networkd was deprecated by
59a9ef102240adad039f58524cfa5774f9e78a7d (v246), and it does not work
since kernel v5.2:
https://github.com/torvalds/linux/commit/e05b2d141fef22cfac1928cf0eb6890e5dae4216

Now our baseline for the kernel is v5.4. Let's completely kill the
feature.

test: also wait for all queued events being processed

Otherwise, if already 'remove' uevent for the device is queued, then
the device node may not exist any more when 'udevadm wait' command succeeds.

Follow-up for 90941d458a2f6b03cb782ab55c16f605b44c4fd8.

core: drop cgroup v1 synthetic empty event logic (#37099)

fido2: hook up with plymouth for notifications (#37089)

Show notifications for fido2 messages in plymouth, so that they
show up in the initrd like the passphrase prompt already does.

mkosi: update debian commit reference to 91a4d114e0b5a01385477f8862caedc1056fa68b

* 91a4d114e0 Install new file for upstream profile

mkosi: enable autologin also with --console=gui

fido2: hook up with plymouth for notifications

Show notifications for fido2 messages in plymouth, so that they
show up in the initrd like the passphrase prompt already does.

core: unit_watch_pidref() gracefully handles EEXIST

No need to check that at callsite.

core: drop cgroup v1 synthetic empty event logic

cgroup v2's empty events are reliable, hence we'd not bother
with validating it again in unit_add_to_cgroup_empty_queue()
either.

udev-rules: add more verbose messages for ATTR{} and ATTRS{}

This also makes udevd/udevadm mention that the parent device rules
also tested with the device currently processing.

With the following rules file:
```
ATTRS{idVendor}=="0001", ATTRS{idProduct}=="44ff", MODE="0666", GROUP="input",
```

Before:
```
event0: /tmp/hoge.rules:1 Checking conditions for parent devices: ATTRS{idVendor}=="0001", ATTRS{idProduct}=="44ff"
event0: /tmp/hoge.rules:1 No parent device passed parent conditions.
```
After:
```
event0: /tmp/hoge.rules:1 Checking conditions for parent devices (including self): ATTRS{idVendor}=="0001", ATTRS{idProduct}=="44ff"
event0: /tmp/hoge.rules:1 ATTRS{idVendor}=="0001": Cannot read sysfs attribute: No such file or directory
input0: /tmp/hoge.rules:1 ATTRS{idVendor}=="0001": Cannot read sysfs attribute: No such file or directory
PNP0C0E:00: /tmp/hoge.rules:1 ATTRS{idVendor}=="0001": Cannot read sysfs attribute: No such file or directory
LNXSYBUS:00: /tmp/hoge.rules:1 ATTRS{idVendor}=="0001": Cannot read sysfs attribute: No such file or directory
LNXSYSTM:00: /tmp/hoge.rules:1 ATTRS{idVendor}=="0001": Cannot read sysfs attribute: No such file or directory
event0: /tmp/hoge.rules:1 No parent device passed parent conditions.
```

Prompted by #37063.

udev: support reverting/serializing/deserializing configurations set by 'udevadm control' (#37067)

Previously, log level, properties, maximum number of worker processes,
and so on set by 'udevadm control' are discarded on restart. This makes
the configuration serialized on stop and deserialized in next
invocation. Also, this introduces 'udevadm control --revert' to clear
previous configurations.

core/cgroup: drop v1 handling in unit_attach_pids_to_cgroup()

mount-setup: swap MountMode and condition_fn in struct

To avoid unnecessary 'NULL's in structured initializer.

TEST-17-UDEV: add test cases for reverting/serializing/deserializing config

udev-config: serialize/deserialize dynamical configurations

Previously, configurations set by 'udevadm control' such as log level,
maximum number of childrens, global properties, and so on were discarded
on restart. This makes udevd serialize those configurations on stop, and
deserialize them in the next invocation.

daemon-util: expose notify_remove_fd_warn()

udevadm: introduce --revert option to call io.systemd.service.Revert

udev-varlink: introduce io.systemd.Udev.Revert method

It reverts dynamically set configurations.

mkosi: update fedora commit reference to 6646d13acae64665f63354cd60ecf963ee563b96 (#37065)

* 6646d13aca fmf: Run tests from systemd-tests rpm if possible

mkosi: update fedora commit reference to 6646d13acae64665f63354cd60ecf963ee563b96

* 6646d13aca fmf: Run tests from systemd-tests rpm if possible

test: Work around bug in meson when installing directory symlinks

Installing symlinks pointing to directories with install_subdir() is
broken (see https://github.com/mesonbuild/meson/pull/14471). Let's work
around the issue for now by manually installing the standalone directory
until the issue is fixed upstream and available in meson in all supported
distributions.

meson: Fix installation directory for integration-tests when using rsync

JSON User/Group records: Add properties for UUIDs (#37024)

It is useful to have stable and unique identifiers for a security
principal. The majority of identitiy management systems in use with Unix
systems today (e.g. Active Directory objectGUID, FreeIPA ipaUniqueID,
Kanidm UUIDs) assign each account and group a unique UUID and exposing
that to applications allows them to refer to accounts in a stable
manner.

At this time we are merely adding the properties to the user/group
records. Adding ways to perform lookups by these IDs is left for a
future PR.

See [discussion](https://mastodon.social/@pid_eins/114283987142625086) and
[this comment](https://github.com/systemd/systemd/issues/24032#issuecomment-2745246757).

I'm sure there are wording aspects which could be improved, but I
believe this is a reasonable initial stab at the problem.

integration-tests: Don't enable tpm for every single test

test: Fix typing error in integration-test-wrapper.py

mkosi: Make sure the mkosi image can be built without the source tree available (#37068)

Let's make sure the mkosi can be built (with `NO_BUILD` enabled) without
the source tree available. This allows running the integration tests
when only distribution packages are available but the source tree is
not.

Some cleanups and a CI fix for test-sd-device (#37083)

mkosi: Move TEST-24-CRYPTSETUP files to mkosi/ directory

If the integration tests have been installed in the systemd-tests
package, the path to these in mkosi.postinst.chroot will be wrong.
Let's fix the issue by moving these files into the mkosi/ directory
as they're only used by mkosi regardless so they make more sense to
be there anyway.

mkosi: Rely on tmpfiles to put nsswitch.conf in place

Let's rely on tmpfiles to put our nsswitch.conf in place instead of
doing it in the post-install script. This moves us one step closer
to being able to build the mkosi image without having the source
tree available when NO_BUILD is used.

mkosi: update debian commit reference to 11efce9445f987f1e588719c182a93794eba2565

* 11efce9445 Install /usr/share/factory for upstream profile
* 4c3d753649 d/t/upstream: copy mkosi key from mkosi/ subdir if it exists
* 00f2ab1bce Install etc.conf tmpfiles.d in upstream builds
* dcf5869729 Refresh patch for upstream review changes
* f94714d8cc d/copyright: use GPL URL instead of old FSF postal address
* bf005e69f5 Update changelog for 257.5-2 release
* 709e474e5b Backport new patch to workaround /lib64 symlink incompatibility
* fa6c61db40 Update changelog for 257.5-1 release
* 9c9ca29ceb Remove conflicts with dracut:arm64 and build nspawn:arm64 again
* 5899bcc55d Update changelog for 257.5-1 release
* dd5cb92d08 Drop backports, included in 257.5
* c1373fb99e d/t/upstream: run mkosi genkey before summary
* 223d7a412a Install new files for upstream
* b9d337abd9 Use Conflicts instead of Breaks/Replaces for file move
* 9379847813 d/t/upstream: write mkosi.local.conf in subdir if the rest of the configs are in subdir
* 86fc24b565 d/t/upstream: do not fail if 10-root.conf is not present

test-sd-device: limit the number of iterations when testing device parent/child functions

The test "hangs" and times out on some arm64 machines. It actually works as
expected, but the machine has 2016 children under /sys/devices/system/memory/,
and the tests do a double loop over this, which is slow enough to hit the 120 s
limit. Add a limit on the number of iterations.

Another option would be to exclude "memory" subsystem. But we may have other
subsystems which have the same problem in the future, so I think it'll be more
robust to not try to limit the fix to a specific subsystem.

test-sd-device: dedeplicate common setup fragment

repart: mkfs.btrfs --subvol option format change

Adapt repart mkfs.btrfs invocation to match the latest format:
(btrfs-progs@8ba72e45c8a6da1275b448ad4912d96e0a2c54b5)

Fixes #37074

man: mention special functionality for reload-or-restart with --marked (#37076)

We had a downstream discussion on what `systemctl reload-or-restart
--marked` does, until upstream chimed in and pointed on very special
behavior for that combination. 😜

The second references the first, but not vice versa. Let's fix this.

nspawn: replace prefix_roota() with chase()