network/l2tp: use hash_ops with destructor for managing sessions

mkosi: Two improvements (#37155)

mkosi: Reuse main image prepare scripts in subimages

In the subimages we also want to make sure all dependencies of the
systemd packages are cached so reuse the same prepare scripts from
the main image to do that.

We only want required dependencies in the subimages, not recommended
or suggested dependendencies, so add an environment variable
$SYSTEMD_REQUIRED_DEPS_ONLY which the prepare scripts can check for
and enable it for the subimages.

mkosi: Make sure coreutils is installed in initrd/exitrd

This is already installed but Fedora/CentOS systems are nudged towards
installing coreutils-single which then later causes issues when we try
to install coreutils as a dependency of systemd so let's make sure we
pick coreutils from the beginning.

bootctl: replace --no-variables by --variables=BOOL

I think the current behaviour of not doing EFI variables when we are run
in a container makes a ton of sense, but in some cases it's useful to
do EFI var setup even when a set of namespaces is set up for us, for
example to recover a hosed installation from a rescue disk.

While we are at it, let's remove some duplicate checks, and
systematically output information why we skip various operations.

Fixes: #36174 #35005

elf2efi: Add back python 3.7 compat

The walrus operator was introduced in python 3.9 so using it
accidentally bumped the minimum python version to 3.9 from 3.7 and
bluca still cares about keeping this working on 3.8 so let's remove
the usage of the walrus operator.

Follow up for d6a29eee2be6ec8ef1473b8097f6d0a2b9e09f4b

use hashmap_dump_sorted() and friends at more several places (#37150)

update TODO

docs: fix mkosi section for Environment= setting

conf-files: use hashmap_dump_sorted() at one more place

resolve: use set_dump_sorted() at one more place

resolve: fix memleak

Follow-up for 5c274555c3f6d5e188cb7e3e0adafa9ae55d5f49.

xattr: fix assert that breaks importctl (#37146)

getxattr_at_malloc() calls getxattr_pinned_internal() with size 0 to get
the size of the xattr, and then asserts because n > 0
```
Assertion '(size_t) n <= size' failed at src/basic/xattr-util.c:107, function getxattr_pinned_internal().
```

Follow-up for b8df25dcfe674e37ceb3d54e00a31e1d33e96057

core: remove cgroup v1 properties (#37026)

cgroup-setup: use path_join() to get path to cgroup attribute

We already have the path to the cgroup. Hence, it is not necessary to
call cg_get_path() again.

cgroup-setup: drop unnecessary controller argument

This also removes unnecessary cg_{create,attach,trim}_everywhere(),
and rename cg_enable_everywhere() -> cg_enable().

cgroup-setup: drop cgroup v1 support

core: drop several more cgroup version check

core/bpf-foreign: drop unnecessary check for cgroup v1

core/bpf-devices: drop cgroup v1 support

core/cgroup: drop cgroup v1 specific code in unit_update_cgroup()

core/cgroup: drop unnecessary cgroup version check in several getters

After 2b61489e5a53eb2611e1e573519c209ca07de98e, these checks are not
necessary anymore.

core/cgroup: drop unnecessary _unified_ modifier from function names

core/cgroup: remove logic of CGroup v1 attribute assignment

core/cgroup: remove unused legacy parameters in CGroupContext

core/cgroup: remove CGroup v1 settings in dump message

bus-unit-util: refuse setting CGroup v1 properties through DBus

core: deprecate CGroup v1 DBus properties

core: remove deprecated CGroup v1 settings

test: replace deprecated CGroup v1 settings with v2 ones

journal: hash_ops related cleanups (#37115)

resolve: hash_ops related cleanups (#37116)

network-tools: hash_ops related cleanups (#37118)

journal: drop old kernels support (#37137)

This drops several codes for supporting old kernels.
Also, this cleanups code a bit.

import/export: add support for zstd

journal/kmsg: drop old kernels (<3.6) support

The flags field was added by kernel v3.6:
https://github.com/torvalds/linux/commit/d39f3d77c9b1fe7cc33a14beb4a4849af0a4ac22

Now our baseline on the kernel is v5.4, so we can assume there exists
the flags field after the monotonic timestamp.

This also adds several short comments.

journal/kmsg: modernize server_open_dev_kmsg()

Let's assign fd and event source only on success.

journal/kmsg: drop Server.dev_kmsg_readable flag and use Server.read_kmsg

When /dev/kmsg is opened, the two flags have the same value.

This also adds an assertion in server_read_dev_kmsg().

journal/kmsg: drop support old kernels (<3.5) without reading /dev/kmsg support

/dev/kmsg is readable since kernel v3.5:
https://github.com/torvalds/linux/commit/e11fea92e13fb91c50bacca799a6131c81929986

Now our baseline on the kernel is v5.4. Let's assume /dev/kmsg is always
readable.

journal: remove old kernels (<3.2) support in monitoring hostname

Similar to cc983fc9dceb989f2cd7cb5ae393630b6c8c2f12.

test: drop error conditions for old kernels (<3.2)

Now our baseline on the kernel is 5.4.

tree-wide: IFA_FLAGS is supported since kernel v3.14

Now our baseline on the kernel is v5.4.
Let's unconditionally use the IFA_FLAGS attribute, and drop the getter
and setter for the deprecated header element.

Continuation of 29c2d4e1587e03a4d7589d045b81f7bea91c283e.

sd-netlink,sd-device: drop old kernels (<4.2) support

NETLINK_LIST_MEMBERSHIPS is supported since kernel v4.2, specifically
https://github.com/torvalds/linux/commit/b42be38b2778eda2237fc759e55e3b698b05b315

Our baseline on the kernel is v5.4. Let's drop unnecessary conditions.

udev-builtin-net_id: drop old kernels (<4.20) support

Since kernel v4.20, specifically since
https://github.com/torvalds/linux/commit/9b8b2a323008aedd39a8debb861b825707f01420
IP-over-Infiniband driver also uses dev_port.

As our baseline on the kernel is v5.4. Let's remove the fallback code.
For more details about the fallback, see
https://github.com/systemd/systemd/pull/10082#issuecomment-421416461

test-condition: drop cgroup version check

After f74ca66e6833c340a1532578176e55cb494590d0, we do not check the
cgroup version of the runtime environment, and unconditionally returns 0
for "v1", and 1 for "v2".

resolve: drop code for supporting old kernels (<3.2)

Now our baseline on the kernel is 5.4.

Note, the removed code triggered assert_return() in
sd_event_source_set_description().

loop-util: drop workaround for kernel older than v5.0

Now our baseline on the kernel is v5.4.

core: hash_ops related cleanups (#37114)

shared/condition: remove cgroup hierarchy check (#37101)

oomd: it's safe to assume cgv2 now

shared/condition: remove cgroup hierarchy check

core/taint: decrease stage array size for "cgroupsv1" removal

Follow-up for 399791eaf0f53887e52488b9f4e966d71abeea6a

socket-proxy: fix use-after-free

Fixes a bug in 10e41cdddc45548cc5938a307a16711b47d68062.
Fixes CID#1609061.

coredump: verify pidfd after parsing data in usermode helper

Ensure the pidfd is still valid before continuing

Follow-up for 313537da6ffdea4049873571202679734d49f0a1

network/wiphy: use hash_ops with destructor for managing Wiphy objects

network/link: use hash_ops with destructor for managing Link objects

network/network: use hash_ops with destructor for managing Network objects

hashmap: introduce ordered_hashmap_free_and_replace()

network/network: use hash_ops with destructor for managing stacked netdevs

network/bridge-fdb,mdb: use hash_ops with destructor

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.