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.

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

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

Closes #18241.

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

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

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

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"

man: emphasize a bit more that PID files suck

manager: taint systemd if cgroupsv1 is used

let's mark cgroupsv1 systems as "tainted" in the taint string logic.
It's time.

Merge pull request #18695 from keszybz/xdg-generator-silence-warning

xdg-autostart-generator: silence warning

Merge pull request #18702 from poettering/news-v248-more

more v248 NEWS stuff

Merge pull request #18694 from keszybz/links-and-syntax-highlighting

Links and syntax highlighting

basic/alloc-util: also reset the cleaned-up variable in freep()

freep() has it's own definition, so I missed it in fd421c4adc7406de02e03e2f7ceede2cc6e1d9d4.

Again, there is a small growth, but the compiler should be able to optimize it away:
-Dbuildtype=debug:

-rwxrwxr-x 1 zbyszek zbyszek 4106816 Feb 19 12:52 build/libsystemd.so.0.30.0
-rwxrwxr-x 1 zbyszek zbyszek 7492952 Feb 19 12:52 build/src/shared/libsystemd-shared-247.so
-rwxrwxr-x 1 zbyszek zbyszek 4472624 Feb 19 12:53 build/systemd

-rwxrwxr-x 1 zbyszek zbyszek 4107056 Feb 19 13:03 build/libsystemd.so.0.30.0
-rwxrwxr-x 1 zbyszek zbyszek 7493480 Feb 19 13:03 build/src/shared/libsystemd-shared-247.so
-rwxrwxr-x 1 zbyszek zbyszek 4472760 Feb 19 13:03 build/systemd

Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=31055.

hwdb: whitespace fixes

update NEWS

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

network: add nexthop ID in debugging logs

network: add NextHop= setting in [Route] section

network: also manage nexthops by ID

It will be used in later commits.

Merge pull request #18646 from yuwata/network-ip-masquerade-follow-ups

network: several follow-ups for recent IPMasquerade= change

Fix grammar and a small typo on a comment

potentically -> potentially
(And a small grammar fix)

xdg-autostart-generator: ignore DBusActivatable=true

See https://wiki.gnome.org/HowDoI/DBusApplicationLaunching and
https://wiki.gnome.org/Initiatives/GnomeGoals/DBusActivatable for a description
of this key:
> Instead of the typical UNIX-style fork()/exec() approach to process creation,
> launching an application is done by sending a D-Bus message to the well-known
> name of that application, causing a D-Bus activation.
>
> Starting processes with D-Bus activation ensures that each application gets
> started in its own pristine environment, as a direct descendent of the
> session -- not in the environment of whatever its parent happened to be. This
> is important for ensuring the app ends up in the correct cgroup, for example.

So this motivation is not important for us: we launch stuff ourselves better.

This fixes warnings during boot:
systemd-xdg-autostart-generator[2274]: /etc/xdg/autostart/org.freedesktop.problems.applet.desktop:92: Unknown key name 'DBusActivatable' in section 'Desktop Entry', ignoring.

xdg-autostart-generator: reindent

man: fix links to various pages

nss-dns doesn't have any man page that I could find.

tree-wide: fix links to systemd.io pages

Having the extra slash at the end is not a problem, just inconsistent. But the links with
.html or .md return 404.

docs/ENVIRONMENT: syntax highlighting and some rewordings

Use backticks for commands and functions and variables, suffix directories with
a slash. Some sentences were reworded.

network: nexthop: update ID of nexthop created without specifiying ID

Otherwise, nexthop_configure() -> nexthop_add() creates NextHop object
without ID in link->nexthop, and then, manager_rtnl_process_nexthop() ->
nexthop_add_foreign() also creates another NextHop object with ID in
link->nexthop_foreign.

network: configure nexthop before routes that requires gateway

This is a preparation for later commits. When RTA_NH_ID is set to a
route, then the corresponding nexthop must be exist.

tree-wide: fix typo

GH Actions: Fix labeler bot

network: warn when any positive boolean string is specified for IPMasquerade=

Previously, any positive boolean string for IPMasquerade= enables only IPv4
masquerade. The commit 48ed276647c754bfb3ed5c6c5af9404e073ffe54 adds
IPv6 masquerade support. However, only "yes" is handled as "ipv4", and other
positive boolean strings are handled as "both".

This makes all positive boolean strings considered as "ipv4", warn that they
are deprecated, and suggest to use "ipv4" or "both".

Follow-up for 48ed276647c754bfb3ed5c6c5af9404e073ffe54.

network: address: reuse Address:ip_masquerade_done for IPv6 case

It is not necessary to use different flag for each address family.

Follow-up for 48ed276647c754bfb3ed5c6c5af9404e073ffe54.

Merge pull request #18659 from poettering/permyriadification

use scaling relative to UINT32_MAX instead of percentages/permyriads wherever we can

Recommend drop-ins over modifications to the main config file

As discussed in https://github.com/systemd/systemd/pull/18347.

table: drop trailing white spaces of the last cell in row

Fixes #18415.

update NEWS

oomd: increase accuracy of SwapUsedLimit= to permyriads too

oomd.conf has two parameters with fractionals: SwapUsedLimit= and
DefaultMemoryPressureLimit=, but one accepts permyriads, the other only
percentages, for no apparent reason. One carries the "Percent" in the
name, the other doesn't.

Let's clean this up: always accept permyriads, and drop the suffix,
given that it is misleading.

I figure we should internally try to focus on scaling everything
relative to UINT32_MAX, and if that isn't in the cards at least 10000,
but never permille nor percent unless there's a really really good
reason for it (e.g. interface defined by someone else).

core: use our usual UINT32_MAX scaling for OOMD limits

So far OOMD limits used permyriads, as an upgrade from the original
percent.

The rest of our codebase typically scales stuff relative to UINT32_MAX.
Let's clean this up, an make sure this happens here too. This is
particularly relevant, as this is exposed in unit files and API, and
before we mark this stable we should get the APIs right.

parse-util: add format string macro for outputting permyriad

Let's define a set of macros for making output of permyriad values easy.
They are printed in pure ASCII, i.e. without the permille/permyriad
suffix, using just percent and two places after the dot.

tree-wide: port various pieces of code over to UINT32_SCALE_FROM_PERMYRIAD()

util: add some helpers for converting percent/permille/permyriad to parts of 2^32-1

At various places we accept values scaled to the range 0…2^32-1 which
are exposed to the user as percentages/permille/permyriad. Let's add
some helper macros (actually: typesafe macro-like functions) that help
with converting our internal encoding to the external encodings.

benefits: some of the previous code rounded up, some down. let's always
round to nearest, to ensure that our conversions are reversible. Also,
check for overflows correctly.

This also adds a test that makes sure that for the full
percent/permille/permyriad ranges we can convert forth and back without
loss of accuracy.

percent-util: when parsing permyriads, permit percents too with 1 place after the dot

Previously, when parsing myriads, we'd support:

 x%          → percent, no places after the dot
 x.yz%       → percent, two places after the dot
 x‰          → permille, no places after the dot
 x.y‰        → permille, one place after the dot
 x‱          → permyriad, no places after the dot

What's missing is:

 x.y%        → percent, one place after the dot

Let's add it in.

util: move percent/permille/permyriad parser into percent-util.[ch]

A good chunk of parse-util.[ch] has been about parsing parts per
hundred/thousand/ten-thousand. Let's split that out into its own file.

No code changes, just some shuffling around.

main: let's use physical_memory_scale() where appropriate

This way we can take benefit of the fact that physical_memory_scale()
aligns on page sizes.

tree-wide: parse permyriads wherever we can

Given that we now have a parser for permyriads, let's use it everywhere
for greater accuracy. This means wherever we previously supported % and
‰, we now also support ‱.

limits-util: tweak overflow checks for (physical_memory|system_tasks)_max_scale()

Also, shortcut two special cases for passing through values as-is, so
that we are not needlessly subjected to overflow issues for them.

update TODO

network: rename UseFQDN= -> UseHostname=

Prompted by
https://github.com/systemd/systemd/pull/18543#issuecomment-780798451.

Merge pull request #18685 from poettering/network-type-tweaks

sd-network.h type tweaks

random-util: fix type of random_u64_range()

As the name of the function suggests this is supposed to return
uint64_t, of course. Fix it. Not sure how this mistake happened in the
first place...

Merge pull request #18546 from poettering/sd-device-action

export sd_device_get_action() API + more

https://github.com/systemd/systemd/issues/15360

The additional rules fix a firmware bug in the en keyboard of Librem notebooks by Purism.

By doing so the break the keyboard with other layouts which do not have the same bug in their firmware.

resolved: improve debug logging on incoming messages a bit

Merge pull request #18679 from keszybz/rfkill-size-leniency

struct rfkill_event read/write size leniency

journald: when we fail to add a new entry to a journal, return the seqno

Fixes: #18005

udevadm: after validating action, use our internal string instead of optarg

This doesn't really change anything, but feels nicer, since it abstracts
away what device_action_from_string()/device_action_to_string() do
internally, and always uses a normalized action string (yes, there's no
ambiguity, but it's nice to stay abstract, maybe one day there is
ambiguity around this)

sd-device: add sd_device_get_action() + sd_device_get_seqnum() + sd_device_new_from_stat_rdev()

To make sd-device properly usable for all programs we need to provide an
API for the "action" field of an event, it's one of the most relevant
ones, and it was so far missing.

This also adds sd_device_get_seqnum(), which isn't that interesting,
except for generating pretty debug output, which we use it ourselves
for.

This also makes device_new_from_stat_rdev() public, as it is truly
useful, as we can see in our own uses of it, and I think is fairly
generic to show up in the public APIs.

Merge pull request #18664 from poettering/resolved-defrag

resolved: many UDP fragmentation tweaks

sd-device: don't compare pointers with numeric zero

Our coding style says no to this.

Merge pull request #18667 from poettering/resolved-change-notification

resolved: optimize dbus change notification signals a bit

network: tighten table alignment a bit