Merge pull request #18751 from anitazha/oomdconditions

units: add Conditions for systemd-oomd.service

man: various improvements to systemd.generator(7)

In particular, make clear the .d/*.conf unit file drop-ins are OK to
generate from generators.

Inspired by: https://lists.freedesktop.org/archives/systemd-devel/2021-February/046148.html

Revert "meson: remove one more instance of install_dir:bindir"

This reverts commit 5ffa2eaa541b718122a3422e911ef95f4c9981d7.

It seems that if install_dir is not specified, meson decides install path
based on file type, and non-executable binary files are installed under
/usr/share.

kernel-install is a script. So, we need to set install_dir argument
explicitly.

Fixes #18754.

README: add config_psi as requirement for systemd-oomd

units: add Conditions for systemd-oomd.service

Prompted by https://bugzilla.redhat.com/show_bug.cgi?id=1927148 and
https://bugzilla.redhat.com/show_bug.cgi?id=1931181.

Adds condition check for cgroups v2 and PSI, which systemd-oomd cannot
start without.

resolved: add another explanatory message to stub resolv.conf files

Apparently people do "cat /etc/resolv.conf" and not realize that they
are looking at a file in /run.

meson: bump version numbers

NEWS: add contributors

resolved: actually use the specified rrsig argument

The argument only exists to be used to override/fill in the RRSIG record
of the answer item. Hence actually use it instead of ignore it.

(Not sure how this got lost earlier.)

Fixes: #18714

Merge pull request #18741 from poettering/stub-no-cname

resolved: don't follow CNAMEs in the stub anymore

shared/condition: add ConditionControlGroupController=v1|v2

Before, we only allowed conditionalizing on controllers, not the hierarchy.
This commit extends this to allow a simple check for v1 (i.e. classic or hybrid),
and v2 (full unified).

An alternative approach would be to add a separate Condition for this, but I'm
not too keen on that, considering that v1 is already being deprecrecated
(c.f. 82f3063218).

MountImages: parse unit specifiers after permissive hyphen

The permissive bit it not something a specifier might synthetise

Merge pull request #18745 from keszybz/stop-using-fstrings

Stop using fstrings

test-dns-packet: fix leak of DnsResourceRecord object

NEWS: a few more entries for v248-rc1

I left the stuff related to [NextHop] out. There are still
patches outstanding, and we can add a comprehensive entry once
things reached the final form.

Merge pull request #18704 from keszybz/fallback-hostame-override

Allow overriding of fallback hostname through envvar and os-release field

resolved: stick CNAME targets into main answer section in stub replies

resolved: add dns_resource_record_get_cname_target() helper

This determines the redirection target from a CNAME or DNAME RR given it
matches some given RR key.

resolved: remove duplicate comment

Practically the same comment is a few lines up covering both parts
anyway, let's remove one.

resolved: update comment in DNS stub code a bit

There's no "answer_auxiliary" object anymore, it's all one "answer"
object, and we have per-item flags that tell us which section things are
from, i.e. from the main answer section, or the additional or
authoritative ones.

resolved: don't follow CNAMEs in the stub anymore

CNAME following was broken by 775ae35403f8f3c01b7ac13387fe8aac1759993f
where we'd not properly collect RRs along the CNAME path. Good thing
though is that we don't have to anymore: since we nowadays propagate all
sections of the upstream replies into the cache and back to stub clients
all the information should already be available anyway, and there's no
need for us to collect it.

Fixes: #18690

Merge pull request #18575 from bugaevc/aa

Set the AA bit in answers for synthetic records & mDNS

Revert "generate-dns_type-gperf: modernize python syntax"

This reverts commit b0a336a66929ed2a146888178157bf1af5c8598c.

Fixes #18708.

Stop using f-strings in generate-sym-test.py

This partially reverts 7857b6e8383f5debab9544ef3abb15a27830fafa.
Debian 9 has python3.5 which does not have f-strings yet.

Partially fixes #18708.

docs: align tables vertically to top

Fixes #18706.

Translated using Weblate (Slovak)

Currently translated at 24.8% (47 of 189 strings)

Co-authored-by: Frantisek Sumsal <frantisek@sumsal.cz>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/master/sk/
Translation: systemd/main

Merge pull request #18718 from yuwata/network-nexthop-blackhole

network: introduce Blackhole= setting in [NextHop] section

Merge pull request #18734 from poettering/cryptsetup-description-escape

cryptsetup: unescape ID_PART_ENTRY_NAME udev field

MountImages: fix exec_context_dump printf

Use strempty as options might not be set, and add the separator
for each option tuple

Merge pull request #18731 from yuwata/backlight-trivial-cleanups

backlight: trivial cleanups

hostnamed: rename FallbackHostname to DefaultHostname

This follows the addition of DEFAULT_HOSTNAME= in os-release.
The distinction between the value from os-release or the env var and
the compile-time setting is not made in the api: HostnameSource is
"default" is all cases. I think that this level of detail is not needed,
because the users of this mostly care whether the hostname was set by
user configuration or not.

Use the DEFAULT_HOSTNAME field from os-release

This provides a fairly comprehensible fix for
https://bugzilla.redhat.com/show_bug.cgi?id=1893417.

This adds yet-another level of configuration:
- /etc/hostname
- transient hostname
- $SYSTEMD_DEFAULT_HOSTNAME
- DEFAULT_HOSTNAME is os-release
- -Dfallback-hostname=
- "linux"

It's a lot of layers, but each has it's own justification.

Allow the fallback hostname to be overriden using an environment variable

See https://bugzilla.redhat.com/show_bug.cgi?id=1893417 for the back story:
the fallback hostname matters a lot in certain environments. Right now the only
way to configure the fallback hostname is by recompiling systemd, which is
obviously problematic in case when the fallback hostname shall differ between
different editions of the same distro that share a single compiled rpm.

By making this overridable through an envvar, we're providing an escape hatch
without making this a top-level api. Later on a way to set this through
os-release is added, but I think the approach with the variable is still
useful. It it very convenient for testing, or to override settings only in a
particular service, etc.

basic/os-util: make the sentinel implicit

Move os-util.[ch] to basic/

parse_os_release() will be used basic/hostname-util.c later on.

sysext: move extension_release_validate() out of os-util.c

os-release: add the DEFAULT_HOSTNAME= setting

The motivation is that variants of the same distro that share the same compiled
rpm want to customize various aspects of the system, in particular the
hostname. In some sense the default hostname is part of the identity of the
system, so setting it through os-release makes sense. In particular, instead of
setting a default value in /etc/hostname, the appropriate default can be baked
into the image, leaving /etc/hostname for local overrides only.

Why make this a separate field instead of e.g. using NAME from os-release?
NAME is already used for other purposes, and it seems likely that people want
to set those independently.

man: use ~/.config instead of XDG_CONFIG_HOME

e3820eeaf11f3b4614cbdfbc85675bc16a486e21 did that replacement XDG_CONFIG_HOME, in one
of two places. Let's use ~/.config everywhere.

Quoting https://github.com/systemd/systemd/pull/18704#discussion_r579465254:
> I'd really drop XDG_CONFIG_HOME from the docs. It's confusing enough as it
> is. Where we don't need the indirections we should not confuse people with
> it, in particular as people might then think it's actually a good idea to use
> that env var and redirect things. I'd just show the literal path everywhere,
> even if we internally use the env var.

manager: add ManagerEnvironment configuration setting

This is useful for various variables that modify process behaviour. This makes
it easy to set it for pid1 without touching the kernel command line. Even for
the *user manager* this also can be convenient for the unprivileged user, who
cannot modify user@.service definition.

Variables that could be set like this include $SD_EVENT_PROFILE_DELAYS,
$SYSTEMD_FALLBACK_HOSTNAME, $SYSTEMD_MEMPOOL, $SYSTMED_RDRAND, etc.

basic/env-util: add putenv_dup()

Merge pull request #18735 from poettering/some-doc-fixes

three documentation fixes

test-network: add tests for Blackhole= setting in [NextHop] section

network: route: shorten code a bit

network: nexthop: add Blackhole= setting in [NextHop] section

As similar to unreachable type routes, blackhole nexthops do not have
NHA_OID attribute, so they are managed by Manager.

man: try to improve documentation of conditions/asserts

Fixes: #18725

docs: , → .

man: correct documentaiton of StandardInput='s defaults in regards to "data"

Fixes: #18710

cryptsetup: unescape ID_PART_ENTRY_NAME udev property before using it

Fixes: #18729

tree-wide: use sd_device_new_from_stat_rdev() whereever appropriate

backlight: reindent comments

backlight: reduce indentation a bit

fuzz-systemctl-parse-argv: add "telinit" to the argv[0] list

backlight: same_device() may return negative errno

backlight: exit earlier when unknown verb is specified

test-network: test wireguard peer in drop-in config

Closes #18241.

resolved: set the AA bit for mDNS responses

This is required by RFC 6762.

Fixes https://github.com/systemd/systemd/issues/17972

resolved: set the AA bit for synthetic answers

The stub DNS server is authoritative for the RRs we synthesize, such as
localhost, _gateway, and entries from /etc/hosts, and also for trust anchors.

Partially fixes https://github.com/systemd/systemd/issues/17972

Merge pull request #18684 from yuwata/sd-device-cleanups-and-udevadm-fix

sd-device, udev: several cleanups and one udevadm fix

manager: read ~/.config/systemd/user.conf in user mode

This changes the paths we read user manager config from in two ways:

- split-usr-root paths are dropped. The user manager is a poster boy for
  non-early-boot, so reading dropins only from /usr is appropriate.

- we look at ~/.config/systemd/user.conf. Users should be allowed to override
  their own config.

As user managers become more and more used, it becomes more important for users
to customize their own daemon. By reading from ~/.config, this is possible
without privileges.

Add DPI info for Kensington Trackball Expert Mouse in hwdb (#18709)

The DPI value is from the manufacturer documentation, and freq measured
from mouse-dpi-tool

Update docs about fdisk/gdisk support for discoverable partitions

This updates the docs about using fdisk/gdisk to make partitions with the right
partition type UUID as defined in the discoverable partitions spec.

Improve wording/grammar/formatting, and reflect that gdisk supports this as
well nowadays; see https://www.rodsbooks.com/gdisk/revisions.html

man: Small fix of sd_bus_message_send() man page

man: correct the unit file directory for attached images

Commit 83f72cd65fb8 ("man,docs: document the new unit file directory for
attached images") updated the docs and man page with the new unit file
directory for attached images but included a system.attached ->
systemd.attached typo in the man page portion of the change. Fix the
typo to document the correct path.

man: fix typo

shared/conf-parser: allow more than one location of the main config file

No functional change as long as only one path is passed.

Merge pull request #18711 from yuwata/network-nexthop-onlink

network: nexthop: add OnLink= setting

test-network: add a test for OnLink= in [NextHop] section

network: nexthop: add OnLink= setting

mount-tool: sd_device_get_sysattr_value() returns 0 on success

sd-device-enumerator: shorten code a bit

unit: ignore exit code of "udevadm trigger"

Suggested at https://github.com/systemd/systemd/pull/18559#issuecomment-781626226.

Some drivers may be broken and return unexpected errors on triggering
events. That should be logged but should not boot-up to be affected.

udevadm-trigger: introduce --quiet option

This may be useful to invoke the command by non-privileged users.

udevadm-trigger: do not return immediately on EACCES

Prompted by https://github.com/systemd/systemd/pull/18559.

udevadm-trigger: use sd_device_trigger() for triggering uevent

udev: use sd_device_trigger() to trigger uevent

sd-device: introduce tiny wrapper sd_device_trigger()

sd-device: do not cache action string for uevent file written by sd_device_set_sysattr_value()

As sd_device_get_sysattr_value() may be already used to read uevent
file. The read value of uevent file may be usually quite different
from the action string.

test: add more tests for path_startswith()

sd-device: make devpath check stricter

See assertion in sd_device_get_devpath().

sd-device: make sd_device_get_xxx() accept NULL for storing result

Though, setting NULL for some getters, e.g. sd_device_get_syspath()
may be meaningless, but anyway also let's make them accept NULL for
consistency.

sd-device: use appropriate error code

sd-device: add a brief comment in device_cache_sysattr_value()

sd-device: ignore error in device_cache_sysattr_value() and propagate original error code

There are three calls of device_cache_sysattr_value(). Two of them are
just caching the value. Hence, let's ignore its failure, and propagate
original error code.

One exception is the last call in sd_device_get_sysattr_value().
Unfortunately, it returns `const char *` instead of `char *`. So,
sd_device object must have the reference of the returned value.
Hence, error in updating the cache by device_cache_sysattr_value()
is critical, and we need to propagate the error in that case.

sd-device: the last argument of sd_device_get_sysattr_value() may be NULL

sd-device: use delete_trailing_chars()

Also, this drops trailing '\r' from sysattr value set in
sd_device_set_sysattr_value().

sd-device: do not cache an empty string but clear cache on failre

And propagate the original error on write.
Note that some attributes are read-only.

sd-device: move comment about NULL value

It is not intuitive, and I do not know why the function behave so.
But let's keep that behavior for compatibility, as such behavior
may be already used explicitly by callers.

sd-device: rename device_{add,get,remove}_sysattr_value()

They just handle cache of sysattr values. Hence, let's use more
explanatory names.

Revert "ci: switch back to meson-0.56.2"

This reverts commit c39e36203ff79a831c93dfb9380c967cd7fdbb99.

Now that meson-0.57.1 (where https://github.com/mesonbuild/meson/issues/8347
is fixed) is out it should be safe to keep rolling forward.

man: split out sd_bus_set_fd() man page from sd_bus_get_fd()

sd_bus_get_fd() and related calls are useful for integrating a bus
connection into arbitrary event loops. But sd_bus_set_fd() is quite a
different beast, it's for using D-Bus over pre-initialized sockets or
pairs of fifos or stuff, i.e. very advanced stuff.

Let's split this man page in two, in order not to confuse things
needlessly.

And while we are at it, let's slightly extend the documentation.

systemctl: don't search in the full argv[0] for the invocation name

argv[0] might be prefixed by a path, and we shouldn't get confused by
that. Hence provide a simple helper call that abstracts the checking
away, which we can use everywhere, and expose the same behaviour, even
if argv[0] is not set.

(While we are at it, port all other multi-call binaries over to the new
helper, too)

Follow-up for: d41a9e4fc1e1bcdefc8d358da2744a97aac5820a

network: fix typo

Merge pull request #18689 from yuwata/network-address-broadcast-no

network: make Broadcast= accept "no"

test-network: add tests for Broadcast= with boolean settings

network: make Broadcast= accept boolean value

And disable it for wireguard interfaces by default.

Closes #18492.

Merge pull request #18699 from yuwata/network-route-add-nexthop-setting

network: add NextHop= setting in [Route] section

sd-bus: simplify sd_bus_reply()

there's no point in having two arguments, if one does as well.

sysctl.d: silence warning if net.core.default_qdisc cannot be set

Kernels can be compiled without the attribute. It's fine if this is not
set, so silence the warning.

fileio: extend comment about file sizes in virtual file systems

Merge pull request #18703 from poettering/cgroupsv1-bad

consider cgroupsv1 usage grounds for "tainting"