core: break line after sd_json_buildo( in varlink-manager.c

test: simple tests for io.systemd.Unit varlink interface

test: quote entire path to socket in io.systemd.Manager tests

core: add basic lookup by name/pidref in io.systemd.Unit.List

json: add unit tests for json_dispatch_const_unit_name()

json: json_dispatch_const_unit_name()

core: Unit Runtime in io.systemd.Unit.List method

core: skeleton for io.system.Unit interface and io.systemd.Unit.List method

fs-util: prefer linkat(AT_EMPTY_PATH) over /proc/self/fd/ shenanigans

The permission check got relaxed in kernel v6.10, so let's
switch the fallback order around.

This also effectively reverts 94d94f0c0a7d28816c815dc9770cc659769fe980,
as I just realized that link_fd() involves multiple paths and
we can't tell which one tripped ENOENT...

cgroup-util: drop handcrafted cg_is_empty(), always check cgroup.events populated field (#37624)

homed: generate proper error if we cannot create mountpoint for homedir

Let's make this easier to debug. (When this failed for me due to disk
full it took me a bit to figure out what was going on.)

udevadm-info: update error messages (#37619)

cgroup-util: remove now unused cg_read_event()

cg_get_keyed_attribute() is a more generic version of this,
and cg_is_empty_recursive() was the only user of this function,
which got converted in the previous commit.

cgroup-util: drop handcrafted cg_is_empty(), always check cgroup.events populated field

This effectively renames cg_is_empty_recursive() to cg_is_empty().
Note that all existing code calls the former and not the latter,
hence with cgv1 support being dropped it's trivial to consult
cgroup.events directly for populated state everywhere.

Additionally, use more generic cg_get_keyed_attribute() helper
rather than cg_read_event().

src/core/manager.c: log preset activity on first boot

This gives us a little more information about what units were enabled
or disabled on that first boot and will be useful for OS developers
tracking down the source of unit state.

An example with this enabled looks like:

```
NET: Registered PF_VSOCK protocol family
systemd[1]: Applying preset policy.
systemd[1]: Unit /etc/systemd/system/dnsmasq.service is masked, ignoring.
systemd[1]: Unit /etc/systemd/system/systemd-repart.service is masked, ignoring.
systemd[1]: Removed '/etc/systemd/system/sockets.target.wants/systemd-resolved-monitor.socket'.
systemd[1]: Removed '/etc/systemd/system/sockets.target.wants/systemd-resolved-varlink.socket'.
systemd[1]: Created symlink '/etc/systemd/system/multi-user.target.wants/var-mnt-workdir.mount' → '/etc/systemd/system/var-mnt-workdir.mount'.
systemd[1]: Created symlink '/etc/systemd/system/multi-user.target.wants/var-mnt-workdir\x2dtmp.mount' → '/etc/systemd/system/var-mnt-workdir\x2dtmp.mount'.
systemd[1]: Created symlink '/etc/systemd/system/afterburn-sshkeys.target.requires/afterburn-sshkeys@core.service' → '/usr/lib/systemd/system/afterburn-sshkeys@.service'.
systemd[1]: Created symlink '/etc/systemd/system/sockets.target.wants/systemd-resolved-varlink.socket' → '/usr/lib/systemd/system/systemd-resolved-varlink.socket'.
systemd[1]: Created symlink '/etc/systemd/system/sockets.target.wants/systemd-resolved-monitor.socket' → '/usr/lib/systemd/system/systemd-resolved-monitor.socket'.
systemd[1]: Populated /etc with preset unit settings.
```

Considering it only happens on first boot and not on every boot I think
the extra information is worth the extra verbosity in the logs just for
that boot.

polkit: reword 'Interactive authentication required' error

I guess the current wording of "Interactive authentication required." is
hard to grok for many users. Let's try to reword this, and say
explicitly:

1. That this is a form of "access denied" error
2. That interactive auth could remedy this
3. But that the client disabled interactive auth

I think these are the three primary elements the error msg needs to
convey. I tried to distill this in a short error string with this.

Fixes: #2081

core/cgroup: CGRuntime.cgroup_path indicates whether the cg is still alive

so drop redundant checks in attr getters. Memory and IO accounting
functions already follow this pattern.

varlinkctl: if returned error is an errno, print it as one in log message

hwdb: make Saitek PLC Pro Flight Rudder Pedals a joystick (#37601)

Fuzzy logic on joystick vs. advanced keyboard detection thinks the
rudders are a keyboard. Adding this forces udev to use this device as
joystick.

```
# udevadm info /dev/input/by-id/usb-Saitek_Saitek_Pro_Flight_Rudder_Pedals-event-joystick
P: /devices/pci0000:00/0000:00:14.0/usb1/1-3/1-3:1.0/0003:06A3:0763.000D/input/input22/event3
M: event3
R: 3
U: input
D: c 13:67
N: input/event3
L: 0
S: input/by-id/usb-Saitek_Saitek_Pro_Flight_Rudder_Pedals-event-joystick
S: input/by-path/pci-0000:00:14.0-usbv2-0:3:1.0-event-joystick
S: input/by-path/pci-0000:00:14.0-usb-0:3:1.0-event-joystick
E: DEVPATH=/devices/pci0000:00/0000:00:14.0/usb1/1-3/1-3:1.0/0003:06A3:0763.000D/input/input22/event3
E: DEVNAME=/dev/input/event3
E: MAJOR=13
E: MINOR=67
E: SUBSYSTEM=input
E: USEC_INITIALIZED=10981053700
E: ID_INPUT=1
E: ID_INPUT_JOYSTICK=1
E: ID_BUS=usb
E: ID_MODEL=Saitek_Pro_Flight_Rudder_Pedals
E: ID_MODEL_ENC=Saitek\x20Pro\x20Flight\x20Rudder\x20Pedals
E: ID_MODEL_ID=0763
E: ID_SERIAL=Saitek_Saitek_Pro_Flight_Rudder_Pedals
E: ID_VENDOR=Saitek
E: ID_VENDOR_ENC=Saitek
E: ID_VENDOR_ID=06a3
E: ID_REVISION=0101
E: ID_TYPE=hid
E: ID_USB_MODEL=Saitek_Pro_Flight_Rudder_Pedals
E: ID_USB_MODEL_ENC=Saitek\x20Pro\x20Flight\x20Rudder\x20Pedals
E: ID_USB_MODEL_ID=0763
E: ID_USB_SERIAL=Saitek_Saitek_Pro_Flight_Rudder_Pedals
E: ID_USB_VENDOR=Saitek
E: ID_USB_VENDOR_ENC=Saitek
E: ID_USB_VENDOR_ID=06a3
E: ID_USB_REVISION=0101
E: ID_USB_TYPE=hid
E: ID_USB_INTERFACES=:030000:
E: ID_USB_INTERFACE_NUM=00
E: ID_USB_DRIVER=usbhid
E: ID_PATH_WITH_USB_REVISION=pci-0000:00:14.0-usbv2-0:3:1.0
E: ID_PATH=pci-0000:00:14.0-usb-0:3:1.0
E: ID_PATH_TAG=pci-0000_00_14_0-usb-0_3_1_0
E: ID_FOR_SEAT=input-pci-0000_00_14_0-usb-0_3_1_0
E: LIBINPUT_DEVICE_GROUP=3/6a3/763:usb-0000:00:14.0-3
E: DEVLINKS=/dev/input/by-id/usb-Saitek_Saitek_Pro_Flight_Rudder_Pedals-event-joystick /dev/input/by-path/pci-0000:00:14.0-usbv2-0:3:1.0-event-joystick /dev/input/by-path/pci-0000:00:14.>
E: TAGS=:seat:uaccess:
E: CURRENT_TAGS=:seat:uaccess:
```

```
# lsusb |fgrep Sait
Bus 001 Device 013: ID 06a3:0763 Saitek PLC Pro Flight Rudder Pedals
```

Closes #34642.

tree-wide: Handle EINVAL as not supported for chattr_xxx()

F2FS returns EINVAL from FS_IOC_SETFLAGS when trying to set
FS_NOCOW_FL. Let's handle this by treating EINVAL as not supported.
While we're at it, make sure we use ERRNO_IS_IOCTL_NOT_SUPPORTED()
across the tree instead of ERRNO_IS_NOT_SUPPORTED() when calling any
of the chattr_xxx() functions.

Fixes #37593

socket-util: Move getsockopt_int() to implementation file

And switch back to negative_errno() at the same time.

machinectl: fix indentation of status output field

One field is differently aligned then the others in the "status" output.
Fix that.

udevadm-info: make the error messages slightly more consistent

At least across the udevadm-info.c file. No functional changes.

udevadm-info: print the original input on error

Print the original input value on error instead of the consumed string
(which will be empty).

Previously:
$ udevadm info /dev/block/251:0 -e --attr-match='foo=\'
Failed to parse key/value pair : Invalid argument
$ udevadm info /dev/block/251:0 -e --attr-match='foo'
Missing '=' in key/value pair (null).

Now:
$ build/udevadm info /dev/block/251:0 -e --attr-match='foo=\'
Failed to parse key/value pair foo=\: Invalid argument

(The second scenario can't be hit anymore since
d89b3004da54228eb5ab2f3326a773a6e97924b9 due to an extra check before
calling parse_key_value_argument().)

update TODO

units: enable watchdog notifications for vmspawn

nspawn supports it and enables it. Let's do this for vmspawn too. It
already supports it in code. Let's make it also work in the unit file.

tree-wide: Clean up more includes

basic + fundamental: Clean up includes (#37595)

basic + fundamental: Clean up includes

Split out of #37344.

login-util: Mark session_id_valid() as pure

basic: Move userns_supported() to namespace-util.h

strv: Drop unnecessary macros

basic: Move LogRatelimit struct to log-ratelimit.h as well

docs: add man pages for sd_device_enumerator_[new,ref,unref,unrefp] (#37586)

For #20929.

Various cleanups (#37597)

Two follow ups for #37591 (#37594)

clangd: Enable UnusedIncludes feature again

Now that the entire free doesn't trigger any clang unused include
violations anymore, let's re-enable the clangd option as it's much
more useful now that the tree is clean.

clang-tidy: Skip public headers

We're very limited in our ability to change these due to backwards
compat, so let's skip them from analysis since we won't be able to
fix the errors anyway.

ci: Setup clang-tidy meson env with extra options

We want the relevant code to be compiled so that it can be analyzed
by clang-tidy.

core: Add missing bpf-dlopen.h includes to bpf skeleton headers

fundamental: Move declaration of free() to macro-fundamental.h

Let's put it together with the macro that needs it (mfree()). Also,
get rid of the unnecessary <stdlib.h> include in iovec-util-fundamental.h.

generate-sym-test: Only include required headers

If we don't use any symbols from a header, let's not include it.

netlink-internal: Make message_get_serial() static inline again

There was no need to move this function to the implementation file,
so let's make it static inline again.

Follow up for #37591.

forward: Move ChaseFlags forward declaration to forward.h

libsystemd: Clean up includes

For libsystemd's headers, the changes made depend on whether the
header is installed or not. For installed headers, the only change
made is that commonly included headers were moved to _sd-common.h.
For these headers, there should be no noticeable change in behavior
when including them. For non-installed headers, includes were replaced
with forward declarations where possible as usual.

Split out of #37344.

basic: Move ratelimit logging functions to log-ratelimit.h

shared: Followups for #37575

libudev: Clean up includes

Split out of #37344.

Bugprone argument comment 7 (#37556)

Follow up from https://github.com/systemd/systemd/pull/37507

treewide: correct argument name to user_icon / askpw_icon

killall: correct argument comment

validatefs: correct argument comments

These functions come from util-linux and can't be renamed.

sd-bus: rename 'add_callback' -> 'install_callback'

Throughout the whole codebase it is called install_callback.

tpm2-util: rename 'policy' -> 'pcrlock_policy'

Follow the used argument comment naming.

creds-util: correct argument comment

pcrlock: correct argument comment

cgroup-util: rename 'kill_log' -> 'log_kill'

Rename according to the implementation.

cgroup-util: rename 's' -> 'killed_pids'

machine-id-setup: correct argument comments

dissect-image: rename 'dest' -> 'where'

Consistently name dissect_image_mount*.

sd-bus: prefix return arguments with 'ret_' for sd_bus_message_peek_type

fuzz: correct argument comment

shared: Clean up includes

Split out of #37344.

linux: update kernel headers from v6.15-rc7

hwdb: merge entries for Ettus Research B200

Follow-up for 9dad00ef94e6207642f04496c56a4123a396530c.

hostname: two trivial cleanups (#37583)

core: renames in io.systemd.Manager (#37582)

1. rename sources to follow `core/varlink-*` patter per @YHNdnzj's
comment in
https://github.com/systemd/systemd/pull/35098#issuecomment-2876509919
2. camelCase Context/Runtime in varlink spec. Varlink fields are
camelCase unless well defined config names.

loginctl,machinectl: use LESS_BY() where appropriate

And while we are at it, add a lower bound of the display output to 10
for all three cases, because it's pointless outputting someting with
less space.

path-util: add flavour of path_startswith() that leaves a leading slash in place (#37572)

hwdb: add RTL-SDR devices

hostname: use FOREACH_ARGUMENT() at one more place

Follow-up for 29a352e8c14c51983ba661cb5f4ce51e2bc8664b.

hostname: use proper type

hostname: add hardware family, sku and version (#36956)

Add support for the hardware family, SKU (Stock-Keeping Unit) and
version. The values are read from the file /etc/machine-info or DMI as
fallback.

The entries are useful to provide an unified interface to collect detail
hardware information. The /etc/machine-info entries enable embedded
devices without UEFI support to provide the information via a common
interface.

Fixes: #31134

core: lowercase Context/Runtime

Varlink fields are camelCase unless well defined config names.

core: manager-varlink.{c,h} -> varlink-manager.{c,h}

hwdb: add Ettus Research SDR devices

boot: do not stop parsing profiles if .cmdline is missing

When profiles were added with a new interation this was left as
a return statement instead of a continue

Follow-up for 382e4da4a5b25ad0c225b4a8092ed24a44c84e18

path-util: invert PATH_STARTSWITH_ACCEPT_DOT_DOT flag

As requested: https://github.com/systemd/systemd/pull/37572#pullrequestreview-2861928094

cgroup: port some code over to path_startswith_full()

path-util: add flavour of path_startswith() that leaves a leading slash in place

tests: fix TEST-74-AUX-UTILS.varlinkctl.sh (#37562)

per Daan's explanation:
other subtests running as testuser apparently use systemd-run --user
--machine testuser@.host which turns user tracking in logind into "by
pin" mode. when the last pinning session exits it terminates the user.

More header cleanups (#37571)

Split out of #37344

tree-wide: Clean up includes

Preparation for cleaning up includes in src/shared.

meson: Make sure <sys/socket.h> is included in af-from-name.gperf

basic: Add missing process-util.h include to origin-id.h

logind: fix serialization of display user field

Follow-up for: 232f4e12fe526bfbcd5c7bd857bbb6a0db818cbc

forward: Add more forward declarations

CGroupMask was erronously added to core-forward.h even though it
comes from src/basic so we move it to the right location.

Restore compatibility for coredump --backtrace (#37559)

Clang tidy headers (#37566)

meson: Run clang-tidy on libsystemd headers as well

meson: Use .inc for generated code meant to be included in other files

These files use the .h extension but are not standalone so let's not
make it appear like they are proper headers and instead use the .inc
extension.

boot: Clean up includes

Instead of introducing efi-forward.h, we use efi.h for the same purpose
in the boot/ folder.

Revert "logind: always enclose list fields (that may contain spaces) in quotes"

This breaks GNOME on Wayland, login no longer works:

gnome-shell[2250]: Running GNOME Shell (using mutter 48.2) as a Wayland display server
gnome-shell[2250]: Failed to setup: Failed to find any matching session
systemd[2054]: org.gnome.Shell@wayland.service: Failed with result 'protocol'.
systemd[2054]: Failed to start org.gnome.Shell@wayland.service - GNOME Shell on Wayland.

This reverts commit b0555149de780f81ec6f741d36bdb41b4d1d8a94.

Various cleanups (#37564)

core: Clean up includes

Follow up for 836e4e7ea8fa24a0a08a3579f154ed4f4987c5df

test: Remove unused includes

machine: Fix comment

core: Clean up includes

Split out of #37344.

networkd: print a meaningful error on failure

test_keep_configuration_on_restart() works, but the error printed is
misleading because self.assertNotEmpty() doesn't exist.

Add a working assert statement so, when the unmanaged interface is
altered, the test fails with a meaningful error, like:

        ### ip monitor dev unmanaged0 BEGIN
        222:33::/64 proto kernel metric 256 pref medium
        FAIL
        [...]
        Traceback (most recent call last):
          File "/work/src/test/test-network/systemd-networkd-tests.py", line 5085, in test_keep_configuration_on_restart
            self.assertEqual(line, '')
        AssertionError: '222:33::/64 proto kernel metric 256 pref medium' != ''
        - 222:33::/64 proto kernel metric 256 pref medium

While at it, strip the trailing newline so we can print easily the
string (and in future build more a robust regexp which uses the $ token)

hostname: add hardware version

Add support for the hardware version. The version describes a distinct
version of compatibility hardware. The value is read from the file
/etc/machine-info or DMI as fallback.

The integration provides an unified interface to collect detail hardware
information. The /etc/machine-info entry enables embedded devices
without UEFI support to read the information from a custom store.