Merge pull request #26179 from medhefgo/boot-no-gnu-efi

boot: Use size_t/unicode string literals

repart: Add roothash to output of all verity siblings

This can be used to match verity partitions together using the repart
JSON output.

Merge pull request #26195 from mrc0mmand/update-uapi

basic/linux: update l2tp.h

Merge pull request #26192 from mrc0mmand/fix-errno-check

sysupdate: fix errno check

test-execute: Skip when /sys is read-only

The test depends on /sys being writable, so let's skip it when /sys
is read-only.

test: skip firstboot --prompt-keymap check if keymaps are missing

Fixes: #26165

Merge pull request #26197 from poettering/journal-file-size-t-fix

journal: some trivial size_t array size fixes

boot: Use unicode literals

No changes in behavior.

boot: Use unsigned for beep counting

boot: Replace UINTN with size_t

No changes in behavior.

dlfcn: add new safe_dclose() helper

Let's allow destructing loaded module handles in our usual way that is
fine with NULL handles, and also returns the NULL handle again.

partition: fix build with newer linux/btrfs.h uapi header

linux/btrfs.h needs  to be included after sys/mount.h, as since [0]
linux/btrfs.h includes linux/fs.h causing build errors:

```
In file included from /usr/include/linux/fs.h:19,
                 from ../src/basic/linux/btrfs.h:29,
                 from ../src/partition/growfs.c:6:
/usr/include/sys/mount.h:35:3: error: expected identifier before numeric constant
   35 |   MS_RDONLY = 1,                /* Mount read-only.  */
      |   ^~~~~~~~~
[1222/2169] Compiling C object systemd-creds.p/src_creds_creds.c.o
ninja: build stopped: subcommand failed.
```

See: https://github.com/systemd/systemd/issues/8507

[0] https://github.com/torvalds/linux/commit/a28135303a669917002f569aecebd5758263e4aa

basic/linux: update linux uapi headers

IPPROTO_L2TP was moved from linux/l2tp.h to linux/in.h [0], so let's
reflect that change to fix build with newer kernels:

```
In file included from ../src/libsystemd/sd-netlink/netlink-types-genl.c:10:
../src/basic/linux/l2tp.h:16: error: "IPPROTO_L2TP" redefined [-Werror]
   16 | #define IPPROTO_L2TP            115
      |
In file included from ../src/libsystemd/sd-netlink/netlink-types-genl.c:3:
/usr/include/netinet/in.h:85: note: this is the location of the previous definition
   85 | #define IPPROTO_L2TP            IPPROTO_L2TP
      |
cc1: all warnings being treated as errors
```

When at it, update the rest of the headers we ship as well.

[0] https://github.com/torvalds/linux/commit/65b32f801bfbc54dc98144a6ec26082b59d131ee

journal-file: cast file size to to fixed size type

(We generally avoid using off_t for file sizes/offsets, and instead use
uint64_t to get the same behaviour everywhere. Do so here too.)

journal-file: fix type of array counter

Merge pull request #26193 from aafeijoo-suse/cryptenroll-unlock-fido2-device-man-and-bash-completion-fix

Add missing --unlock-fido2-device to systemd-cryptenroll man and bash-completion

man: add missing --unlock-fido2-device to systemd-cryptenroll

update TODO

bash-completion: add missing --unlock-fido2-device to systemd-cryptenroll

bootctl-status: several follow-ups for unlink command

Follow-ups for 8702496bfb0205764569782a9a2ebd11fd80e5e8.

- add missing error cause in logging,
- add several missing assertions,
- drop an unnecessary initialization,
- make boot_config_find_in() return negative errno if nothing found,
- and several coding style fixlets.

bootctl-uki: several follow-ups for inspect_osrel()

Follow-ups for #26124 and #26158.

- use os_release_pretty_name(),
- constify the buffer passed to inspect_osrel(),
- propagate errors in inspect_osrele(), and ignore them in the caller
  side,
- and several coding style fixlets.

coccinelle: skip the empty-to-null transformation on the macro itself

Since the empty_to_null() function was "macrofied", we need to use a bit
of black magic to make Coccinelle avoid running the transformation on
the macro itself.

Follow-up to ef2409cbde3.

sysupdate: fix errno check

NEWS: update date and location

shared/efi-loader: fix compilation with !ENABLE_EFI, improve messages

When compiled without ENABLE_EFI, efi_stub_measured() was not defined, so
compilation would fail. But it's not enough to add a stub that returns
-EOPNOTSUPP. We call this function in various places and usually print the error
at warning or error level, so we'd print a confusing message. We also can't add
a stub that always returns 0, because then we'd print a message like "Kernel
stub did not measure", which would be confusing too. Adding special handling for
-EOPNOTSUPP in every caller is also unattractive. So instead efi_stub_measured()
is reworked to log the warning or error internally, and such logging is removed
from the callers, and a stub is added that logs a custom message.

Merge pull request #26184 from keszybz/cleanups

Various fixups to recent commits

locale: rename new XKB variables to match Debian/Ubuntu's

Debian/Ubuntu use almost the same variables, but without '_'. Given
our usage is new, rename them so that they match and downstream tech
debt can be removed.

Follow-up for https://github.com/systemd/systemd/pull/25805

See:
 https://github.com/systemd/systemd/issues/24228
 https://github.com/systemd/systemd/pull/25412

docs: Update HACKING.md to mention latest mkosi is needed

Let's require users to run mkosi from git so we can fix any issues
forward instead of trying to keep the configs working with older
versions.

meson: Do not include headers in source lists

Meson+ninja+compiler do this for us and are better at it.

https://mesonbuild.com/FAQ.html#do-i-need-to-add-my-headers-to-the-sources-list-like-in-autotools

Update NEWS

Consolidate various TAKE_* into TAKE_GENERIC(), add TAKE_STRUCT()

man: clarify that MESSAGE= should not appear more than once in the same journal entry

One would think this was clear already, but apparently it's not clear
enough, hence let's be more explicit.

Fixes: #26175

test-sleep: reduce timeout

The timeout was raised during review and I wrote that I lowered it, but forgot
to actually commit the diff. Follow-up for 31f62bdd79472c32d52408956d5c82e9991ca425.

NEWS: update for v253-rc1

udevadm: add todo to remind us to remove compat code

https://github.com/systemd/systemd/pull/25145#issuecomment-1397606521

man/bootctl: add missing markup

Fixup for 53c368d71ba43da7414ac86c58291a11da05ba84.

Merge pull request #26152 from systemd/revert-26143-issue-26142

Revert "Resolves #26142 - Fix list of supported personalities"

Merge pull request #26158 from poettering/hostnamed-end-of-support

hostnamed/hostnamectl: support os-release END_OF_SUPPORT= field

path-util: rework file_in_same_dir() on top of path_extract_directory()

Let's port one more over.

Note that this changes behaviour of file_in_same_dir() in some regards.
Specifically, a trailing slash of the input path will be treated
differently: previously we'd operate below that dir then, instead of the
parent. I think that makes little sense however, and I think the code
using this function doesn't expect that either.

Moroever, addresses some corner cases if the path is specified as "/" or
".", i.e. where e cannot extract a parent. These will now be treated as
error, which I think is much cleaner.

bootctl-uki: several coding style fixlets

Mostly follow-ups for #26082.

bootctl: kernel-inspect: print os info

man: extend Personality= docs a bit

Let's make clear personalities are supported on ARM (as per
https://github.com/systemd/systemd/pull/26152#issuecomment-1400321944).

Also, clarify that on many archs this functionality is pretty pointless,
i.e. where only a single native arch was ever known.

Revert "Resolves #26142 - Fix list of supported personalities"

This reverts commit fdbbb5ffc04c48e6e1b97ddfff366672c2f8a073.

update TODO

hostnamectl: show support and among output

hostnamed: expose support end timestamp as property on the bus

os-util: optionally, return EOL time in os_release_support_ended()

os-util: accept SUPPORT_END= with empty string as explicit way to disable concept

os-util: when determining support end, don't treat /etc/os-release file as error

Instead, let's just treat it as "field unset".

format-table: add new cell type TABLE_TIMESPAN_DAY

This is just like TABLE_TIMESPAN_MSEC, but shows time spans in accuracy
of days.

os-util: determine distro EOL in UTC, not local timezone

Presumably vendors establish a global EOL time, hence its better to
interpret the date specification everywhere the same, instead of
timezone dependent.

tree-wide: unify how we pick OS pretty name to display

Merge pull request #26176 from bluca/rc

preparations for rc1

docs: update instructions for translation strings

docs: drop manual rc PR warning step

it is not automated, yay

meson: bump numbers for v253-rc1

Update hwdb

NEWS: add list of contributors

repart: Mention broken XFS protofile format

machinectl: add --now to start/stop containers when enabling/disabling

Closes #26154

man: fix explanation for Credential= match in .link files

.link files are handled by systemd-udevd, not by systemd-networkd,
so fix the man page to say that the Credential= match uses credentials set
on the systemd-udevd.service

Merge pull request #26167 from ldv-alt/docs-fixes

docs: fix a few relatively trivial issues

Merge pull request #26164 from mrc0mmand/test-bus-timeout

test: bump the client-side timeout in sd-bus as well

Merge pull request #26159 from keszybz/capacity-to-crash

Resolve crash in systemd-sleep and other minor fixes

Merge pull request #26110 from medhefgo/boot-no-gnu-efi

boot: Stop linking against libefi.a

docs, man: consistently use comma after "For example"

Merge pull request #26162 from YHNdnzj/machinectl-check-target-enabled

machinectl: also enable machines.target when enabling machines

docs: fix formatting a bit

docs: fix grammar a bit

docs: fix a few typos

boot: Use objcopy with arm64

Binutils 2.38 added support for efi-app-aarch64
Still use binary mode if we have an older objcopy
Add check for incompatible gnu-efi crt0 containing the header section
which gets added by objcopy and if used results in duplicate header
and subsequently a broken binary

Signed-off-by: Callum Farmer <gmbr3@opensuse.org>

sleep: fix memleak

Those hashmaps are created anew in each iteration of the loop. The
leak wasn't really a problem, because the loop is bounded and the
hashmaps were not huge, but it's nicer to be correct.

sleep: reduce double logging and improve messages and comments a bit

read_battery_capacity_percentage() was already logging, but with a slightly
different wording.

More could be done, I just touched the most noticable places. Especially
in debug messages, it is much more useful to be direct about what couldn't
be accessed or parsed, instead of providing "descriptive names" which are
not useful to the user at all, who then needs to read the code to figure out
what was the actual property name.

test-sleep: add a very simple test that prints battery suspend estimates

sleep: do not abort if we try to query capacity of missing battery

Fixes #25584.

From the issue:
Assertion 'capacity >= 0' failed at src/shared/sleep-config.c:58, function PTR_TO_CAPACITY(). Aborting.

(gdb) bt

The problem is that PTR_TO_CAPACITY(hashmap_get(last_capacity, battery_name))
will abort if it's called with a name not present in the hashmap. We want to
skip the device silently in this case instead.

Merge pull request #26153 from DaanDeMeyer/repart-remove-userns

mkfs-util: Remove user namespace owner => root mapping

core: ensure init.scope is realized after drop-ins have been loaded

If we add a drop-in for init.scope (e.g.: to set some memory limit),
it will be loaded long after the cgroup has already been realized.
Do it again when creating the special unit.

build: add some coloring to --version output

Make it easier to discern enabled and disabled build options.

Merge pull request #26157 from medhefgo/meson

meson: Small improvements

Merge pull request #25805 from yuwata/locale-xkb-save-vconsole

locale: also save X11 keyboard settings to /etc/vconsole.conf

test: bump the container spawn timeout to 60s

As 30s might be not enough on busy systems (and we already bumped the
reboot timeout from 30s to 60s for this reason).

test: bump the client-side timeout in sd-bus as well

Since c78d18215b D-Bus services now have 60s to start, but the client
side (sd-bus) still waits only for 25s before giving up:

```
[  226.196380] testsuite-71.sh[556]: + assert_in 'Static hostname: H' ''
[  226.332965] testsuite-71.sh[576]: + set +ex
[  226.332965] testsuite-71.sh[576]: FAIL: 'Static hostname: H' not found in:
[  228.910782] sh[577]: + systemctl poweroff --no-block
[  232.255584] hostnamectl[565]: Failed to query system properties: Connection timed out
[  236.827514] systemd[1]: end.service: Consumed 2.131s CPU time.
[  237.476969] dbus-daemon[566]: [system] Successfully activated service 'org.freedesktop.hostname1'
[  237.516308] systemd[1]: system-modprobe.slice: Consumed 1.533s CPU time.
[  237.794635] systemd[1]: testsuite-71.service: Main process exited, code=exited, status=1/FAILURE
[  237.818469] systemd[1]: testsuite-71.service: Failed with result 'exit-code'.
[  237.931415] systemd[1]: Failed to start testsuite-71.service.
[  238.000833] systemd[1]: testsuite-71.service: Consumed 5.651s CPU time.
[  238.181030] systemd[1]: Reached target testsuite.target.
```

Let's override the timeout in sd-bus as well to mitigate this.

Follow-up to c78d18215b3e5b0f0896ddb1d0d72c666b5e830b.

machinectl: remove unnecessary initialization

machinectl: also enable machines.target when enabling machines

It's mostly desired to enable machines.target
along with the machine since only when both are
enabled it would really work.

repart: Add note about UIDs/GIDs of copied files and directories

Revert "repart: Ensure files end up owned by root in generated filesystems"

This reverts commit e59678b2cf42e4206ddabc959d3cf9a5a865ecdc.

We also modify the repart integration tests to make them pass with the
changes in this commit. In short, we have to make sure every file is
owned by the user executing repart. We use tee instead of cat since it
makes that easier. This also has the benefit of improving debugability
as seeing the config file contents on stdout makes it easier to know
which test is failing.

Merge pull request #25168 from valentindavid/valentindavid/umount-move-recursive-list

shutdown: move busy mounts to not block parent mounts

meson: Use python module for detection

Revert "repart: Make sure all files in the image are owned by root"

This reverts commit d2ac7698cb43807a2dd0af727599db486180ebf1.

meson: Move bootctl sources defintion to its own file

The root meson.build file is already large enough.

Merge pull request #25145 from yuwata/udevadm-trigger

udev: make 'udevadm trigger --settle' work even if device is renamed

test: fix TEST-73-LOCALE on Debian

Fix incorrection assumption about the Debian patch being buggy and
actually making TEST-73-LOCALE fail on Debian.

```
 # localectl set-locale LANG=C.UTF-8
 # cat /etc/default/locale
 LANG=C.UTF-8
```

locale: also save XKB settings to vconsole.conf

Closes #24228.
Replaces #25412.

locale: make errors in writing files not critical

Suggested at https://github.com/systemd/systemd/pull/25805#discussion_r1054871210

> we now write multiple files and we cannot possibly guarantee that
> either both updates worked or neither. unix fs apis simply have
> no concept that would allow us to implement that. hence I think
> we should update what we can, log about what we cannot, but return
> success as long as our in-memory state was updated.

locale: merge vconsole_convert_to_x11_and_emit() with method_set_vc_keyboard()

Similary, this also merges x11_convert_to_vconsole_and_emit() with
method_set_x11_keyboard().

No functional changes, preparation for later commits.

locale: split out XKB settings to X11Context

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

locale: rename context_free_vconsole() -> context_clear_vconsole()

Similary, this also renames context_free_x11() -> context_clear_x11(),
to make it clear that these functions do not free Context.

Also, this makes context_clear() support to be called more than once.

locale: do not use alloca() for strings which can be potentially arbitrary

locale: several coding style fixlets

- add missing assertions,
- rename arguments for storing results,
- always initialize results on success,
- use _cleanup_ attribute at more places, etc.

Merge pull request #26147 from keszybz/silence-gcc-13-warning

Silence gcc-13 warning