Merge pull request #32324 from mrc0mmand/more-website-fixes

docs: use absolute links for our pages

doc: fix .ssh credential examples

Let's create the .ssh dir with the right perms first.

Suggested by @gcb.

Fixes: #28172

docs: Add note on packages produced by mkosi builds

udev: fix assignment of ret_truncated

Follow-ups for 089bef66316e5bdc91b9984148e5a6455449c1da.

blockdev-util: fix typo

Follow-up for 33ff155957327f51dde740a7a75f19122bff1ebc.

Merge pull request #32192 from yuwata/part-scan

blockdev-util: fix detection of partscan

Merge pull request #32249 from CodethinkLabs/vmspawn/predicatable_tap_names

vmspawn: generate predicatable TAP device names and MAC addresses

blockdev-util: also read 'ext_range' sysattr to check if the partscan is enabled

The 'capability' sysattr was deprecated by
https://github.com/torvalds/linux/commit/e81cd5a983bb35dabd38ee472cf3fea1c63e0f23 (v6.3).

sd-device: introduce device_get_sysattr_unsigned_full()

man: fix wrong version info (#31949)

Fixes #31920.

systemctl: allow user to suppress output when no action scheduled (#32278)

journalctl: update help to say "priority range" (#32323)

Clarify that `-p, --priority=` always treats its option as a priority range, even when given
a single log level per the full man page description.

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

Merge pull request #32308 from yuwata/sd-ndisc-disable-timer-on-recv

sd-ndisc: disable one more timer event source

Merge pull request #32318 from YHNdnzj/skip-leading-char

journal-gatewayd: use skip_leading_chars

Merge pull request #32320 from bluca/softreboot_serialize

Soft reboot timestamp follow-ups

man: mention that sd_journal_test_cursor() needs a positioning call

Fixes #30331.

Merge pull request #32317 from DaanDeMeyer/mkosi

mkosi: Set up -ffile-prefix-map= correctly when building debuginfo packages

docs: use absolute links for our pages

Since 56b2970 has proven to be a no-go for us, as it breaks existing
links, let's embrace the trailing slash and use absolute links
everywhere for our pages. This way we'll get around browser cleverly
appending the relative link to the current location (since it ends with
a slash), and given our docs/ layout is flat it's not much of a hassle
either.

Converted using this beauty:
  $ sed -ri 's/(\[.+\]\()([A-Z_]+\))/\1\/\2/g' *.md

Resolves: #32088 (again) and #32310

Revert "docs: don't suffix page permalink with a slash"

This breaks existing links with a trailing slash, so it's a no-go.

This reverts commit d1a7e030c5daa4a09ee93b8af6b28ecaac5d34d0.

core: rename SoftRebootStartTimestamp -> ShutdownStartTimestamp and generalize

Follow-up for 54f86b86ba8168faccbfc2ad16ceeccefde98a51

manager: serialize current objective

So that we can tell what happened before the exec. It is overwritten
shortly after deserialization. Use it to bump the soft reboots counter.

Merge pull request #32315 from poettering/hwdb-256

hwdb/autosuspend/NEWS update for 256-rc1

Merge pull request #31648 from neighbourhoodie/review-content

I did the merge manually to resolve a trivial conflict.

mkosi: Drop workarounds

With the latest Github Actions image release none of these should
be required anymore (20240414.1.1).

mkosi: Switch to O0 and disable FORTIFY_SOURCE

-Og still causes a lot of "<optimized out>" in GDB so let's use -O0
instead and disable FORTIFY_SOURCE as it doesn't work without
optimizations enabled.

Merge pull request #32143 from yuwata/magic

basic/linux: copy magic.h from kernel to our code

NEWS: prepare -rc1

autosuspend: update for v256

hwdb: update for v256

Merge pull request #32311 from YHNdnzj/write-ellipsis

Some minor cleanup for string-util (ellipsis)

mkosi: Disable DWARF debug info optimization

We care more about fast builds in this case so disable the DWARF
debug optimization that's done by default.

NEWS: udev and networkd pick up more things from creds

journal-gatewayd: use skip_leading_chars where appropriate

Prompted by c5d675472588815dedf67af8c36da0fac398fb3e

journal-gatewayd: add missing assertions

mkosi: Override build_cflags instead of _distro_extra_cflags

The latter is not supported on CentOS Stream 9.

journal-gatewayd: remove duplicate call to `startswith()`

mkosi: Set up -ffile-prefix-map= correctly when building debuginfo packages

This makes sure that the debuginfo files contain source files pointing
to the source files shipped by the debugsource package.

Normally this should be done automatically by rpm invoking debugedit
but for some unknown reason debugedit refuses to rewrite the source
files in our binaries.

Given that debugedit is completely undebugable (does not generate any
logs at all, and its source code is ridiculously obtuse), let's set
-ffile-prefix-map= when building instead which achieves the same
effect.

build(deps): bump pkg/debian from `e477254` to `30c77a7`

Bumps pkg/debian from `e477254` to `30c77a7`.

---
updated-dependencies:
- dependency-name: pkg/debian
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

core/show-status: use strlen where appropriate

We don't use sizeof() outside of array definitions
these days. The compiler should be capable of
optimizing this out.

string-util: use mempcpy where appropriate

string-util: use special_glyph() at one more place

No functional change, just refactoring.

fuzz-ndisc-rs: also test packets with sd-radv

Addresses https://github.com/systemd/systemd/pull/32267#discussion_r1566715787.

Suggested-by: Evgeny Vereshchagin <evvers@ya.ru>

test: fix typo

Follow-up for 236634c82c56e280348f9452129f3cdfd1fb7a45.

sd-ndisc: improve debugging logs

sd-ndisc: disable one more timer event source

Otherwise, even if we have already received RA, timeout callback will be
called. Currently, networkd mostly does nothing on timeout, hence should
not change any effective behavior.

Follow-up for e7cb80474d4e580f7cc7c8645e1412e4a59f7588.

Merge pull request #32300 from mrc0mmand/assorted-tweaks

test: split TEST-50-DISSECT into smaller parts

Merge pull request #32303 from CodethinkLabs/push_down_disk_partitioning

test-64-udev-storage: Push disk partitioning into the testsuite itself

cryptenroll: Use CTAP2.1 credProtect extension

When enrolling a new FIDO2 token with a client PIN, this tells the authenticator to require the PIN on all uses.

It also collects a PIN before attempting to create a credential.

Works around #31443 in most (not all) scenarios.

test-64-udev-storage: partition disk inside testcase - testcase_multipath_basic_failover

test-64-udev-storage: partition disk inside testcase - testcase_virtio_scsi_identically_named_partitions

test-64-udev-storage: partition disk inside testcase - testcase_long_sysfs_path

Merge pull request #32302 from DaanDeMeyer/mkosi

Various mkosi improvements

Merge pull request #32236 from poettering/journalctl-list-namespace-followup

journalctl: --list-namespaces followup

mkosi: Add support for building debug packages

This allows building debug packages by setting WITH_DEBUG=1. This
slows down the build a lot so it's opt in. We don't yet install
these but can do so in a future commit.

mkosi: Add --noclean when running rpmbuild

The entire build environment is ephemeral anyway so everything is
cleaned regardless. By specifying --noclean, we make debugging
easier as the rpm build root can be inspected when using --debug-shell.

mkosi: Install which in image

gdb-headless in Fedora is missing a dependency on which:
https://bugzilla.redhat.com/show_bug.cgi?id=2275274

This happens all the time so let's make sure which is always installed
in the image.

Update debugging with vscode section

- We have ssh-generator now, so need for mkosi's Ssh= option anymore.
- By enabling RuntimeBuildSources= by default, we don't need the gdb
  config file in the image anymore, since the build and source
  directories will be mounted at the expected locations.

test: rework systemd-sysext test

Let's rework the test a bit, namely:
  - condense the code a bit
  - drop unnecessary braces around variables
  - drop unnecessary explanations around `touch` calls
  - drop/rename functions to make the code more self-explanatory
  - simplify cleanup a bit
  - create R/O bind mounts directly (supported since util-linux 2.27)

test: simplify unit check

test: split some overly long lines

Let's get the line count back up again by splitting some overly long
lines in hopes to make the code slightly more readable.

test: split TEST-50-DISSECT into smaller parts

As the current version has almost 2k lines.

journalctl: tighten rules on parsing namespace journal dir suffixes

The dot must follow the machine ID immediately, let's check for that.
Also, I think it's generally better to parse the machine ID and then
comparing it, instead of comparing the string representation. That's
more in line how we usually do it, as we parse 128bit IDs generally
case-insensitively.

journalctl: show human readable text if no namespace is found

The common case will be that no namespaces are defined, hence by default
we'd just output "NAMESPACE" and exit, which is very confusing.

Generate a nice message instead, how many of our tools do it. Suppress
it on --quiet however.

journalctl: add missing --help line for --list-namespaces

Follow-up for: #32236 68f66a171398e27280a95e58ae7464219cccaaec

shared/vpick: also align function params in header

Follow-up for 6d8690ecd34a6c9450e78e242e07b1a4479d6d27

Merge pull request #32289 from bluca/counter

soft-reboot counter follow-ups

Merge pull request #32294 from yuwata/network-generator-creds

network-generator: also load drop-ins for networkd.conf from credentials

Merge pull request #32297 from yuwata/networkctl-cat-without-args

networkctl: allow to call 'networkctl cat' without arguments

NEWS: a few fixes, add systemd-cat --namespace=

network: fix use of wrong flag

Fixes a bug introduced by 86a66e9b95048b1a3a4e297ba2884afcedd1585e (v255).

test: add test case for 'networkctl cat' without arguments

networkctl: allow to call 'networkctl cat' without arguments

Then, show networkd.conf and its drop-ins.

man: add missing drop-in directory

test: add test case for network.conf.* credential

network-generator: also copy drop-ins for networkd.conf from credential

Follow-up for 1a30285590c2f40f256d0628950ef9243b2c1938.

NEWS: fix typo

Merge pull request #32103 from yuwata/udev-load-credentials

udev: load udev rules from credentials

Merge pull request #32267 from yuwata/sd-ndisc-router-solicit

sd-radv: introduce sd_ndisc_router_solicit and use it

test: add test for systemd-udev-load-credentials.service

sd-radv: use ICMP6Packet and sd_ndisc_router_solicit

Then, sd-radv can reject invalid Router Solicit messages, e.g. that have
invalid options.

No effective functional change, just refactoring.

sd-radv: introduce sd_ndisc_router_solicit

It is not used in this stage, but will be used later in sd-radv.

units: introduce systemd-udev-load-credentials.service

udevadm-control: add --load-credentials option

When specified, credentials udev.conf.* and udev.rules.* are copied to
the corresponding directories.

udevadm-control: split out send_control_commands()

No functional change, just refactoring and preparation for later
commits.

udevadm-control: check if control command really specified

Previously, 'udevadm control' only checked the number of the arguments.
So, if only `--timeout` is specified, it spuriously did nothing and succeeded.
This makes the command request at least one control command.

creds-util: move pick_up_credentials() from network-generator

The function is quite general, and can be used at other places.
Let's move to our shared library.

network-generator: rename NETWORKD_UNIT_... -> NETWORK_UNIT_...

As .link files are used by systemd-udevd, rather than systemd-networkd.

Also, this makes it ends with '/', to follow our recent coding style.

analyze: show soft-reboot counter next to time deltas

man: mention initial value of SoftRebootsCount

Follow-up for 66f35161f6568386526bdfeab18e6b74b8b13608

analyze: reset more timestamps when analyzing soft reboots

Follow-up for 7addfba9c45af97305a628e98acf684eedb7d510

core: use usec_sub_unsigned() to subtract deltas

Follow-up for 54f86b86ba8168faccbfc2ad16ceeccefde98a51

test: initialize _cleanup_ variables

Otherwise static analysis will complain

Merge pull request #32286 from YHNdnzj/vpick-null-result

shared/vpick: add missing condition on ret_result

Merge pull request #32258 from yuwata/network-tc-fix-stack-overflow

network/tc: fix stack overflow

units: order repart after systemd-tpm2-setup-early.service

This mimics what we do for systemd-cryptsetup@.service (see
src/shared/generator.c), and makes sense since repart might lock up the
root volume against a TPM, which ideally has its SRK already set up by
then.

More importantly though, this ensures that we ordered correctly after
tpm2.target (which systemd-tpm2-setup-early.service has a dependency
on), for systems where the TPM drivers are not compiled into the kernel.

See: https://lists.freedesktop.org/archives/systemd-devel/2024-April/050201.html

shared/vpick: add missing condition on ret_result

Fixes #32285

shared/vpick: drop unneeded strempty()

prefix_roota() handles NULL root param gracefully.

shared/vpick: align function args correctly, add missing assertion

man: slightly reword LogFilterPatterns= description

As there was something missing in the existing sentence.

dissect: refuse to use diskseq if ID_IGNORE_DISKSEQ=1 is specified

Follow-up for 1a81ddef00a0a25f6bcdd1e6633430e8b240b87f.