units: typo fix /proc/<pid>/exec → /proc/<pid>/exe

Fix a pretty relevant typo introduced in
c7faa23235694a1e803ba093cba6d6e0193a093e.

cryptsetup: use strjoin() for concatenating strings

units: document why CAP_SYS_PTRACE is needed by journald

tree-wide: fix typo

specifiers: introduce common macros for generating specifier tables

In many cases the tables are largely the same, hence define a common set
of macros to generate the common parts.

This adds in a couple of missing specifiers here and there, so is more
thant just refactoring: it actually fixes accidental omissions.

Note that some entries that look like they could be unified under these
macros can't really be unified, since they are slightly different. For
example in the DNSSD service logic we want to use the DNSSD hostname for
%H rather than the unmodified kernel one.

Merge pull request #17706 from ddstreet/test-move-pam-systemd-user

test/test-functions: copy /usr/lib/pam.d into $initdir

NEWS: mention that we intend to retrigger udev devices on package upgrade

Also, mention RISCV GPT partition types have been defined.

pam_systemd_home: export password as PAM_AUTHTOK

Merge pull request #17680 from yuwata/udev-link-mac-address

udev: to make MACAddress= take effect, MACAddressPolicy= must be "none"

seccomp: also move munmap into @default syscall filter set

Follow-up for 5abede3247591248718026cb8be6cd231de7728b.

sd-device-enumerator: do not return error when a device is removed

If /sys/class/OOO node is created and destroyed during booting (kernle driver initialization fails),
systemd-udev-trigger.service fails due to race condition.

***** race condition ***********************************************************************************
 1. kernel driver create /sys/class/OOO
 2. systemd-udev-trigger.service execues "/usr/bin/udevadm trigger --type=devices --action=add"

 3. device_enumerator_scan_devices() => enumerator_scan_devices_all() => enumerator_scan_dir("class") =>
    opendir("/sys/class") and iterate all subdirs ==> enumerator_scan_dir_and_add_devices("/sys/class/OOO")

 4. kernel driver fails and destroy /sys/class/OOO
 5. enumerator_scan_dir_and_add_devices("/sys/class/OOO") fails in opendir("/sys/class/OOO")

 6. "systemd-udev-trigger.service" fails
 7. udev coldplug fails and some device units not ready
 8. mount units asociated with device units fail
 9. local-fs.target fails
 10. enters emergency mode
********************************************************************************************************

***** status of systemd-udev-trigger.service unit ******************************************************
$ systemctl status systemd-udev-trigger.service
 systemd-udev-trigger.service - udev Coldplug all Devices
   Loaded: loaded (/usr/lib/systemd/system/systemd-udev-trigger.service; static; vendor preset: enabled)
   Active: failed (Result: exit-code) since Thu 2020-01-02 13:16:54 KST; 22min ago
     Docs: man:udev(7)
           man:systemd-udevd.service(8)
  Process: 2162 ExecStart=/usr/bin/udevadm trigger --type=subsystems --action=add (code=exited, status=0/SUCCESS)
  Process: 2554 ExecStart=/usr/bin/udevadm trigger --type=devices --action=add (code=exited, status=1/FAILURE)
  Main PID: 2554 (code=exited, status=1/FAILURE)

  Jan 02 13:16:54 localhost udevadm[2554]: Failed to scan devices: No such file or directory
  Jan 02 13:16:54 localhost systemd[1]: systemd-udev-trigger.service: Main process exited, code=exited, status=1/FAILURE
  Jan 02 13:16:54 localhost systemd[1]: systemd-udev-trigger.service: Failed with result 'exit-code'.
  Jan 02 13:16:54 localhost systemd[1]: Failed to start udev Coldplug all Devices.
*******************************************************************************************************

***** journal log with Environment=SYSTEMD_LOG_LEVEL=debug in systemd-udev-trigger.service  ***********
  Jan 01 21:57:20 localhost udevadm[2039]: sd-device-enumerator: Scanning /sys/bus
  Jan 01 21:57:20 localhost udevadm[2522]: sd-device-enumerator: Scan all dirs
  Jan 01 21:57:20 localhost udevadm[2522]: sd-device-enumerator: Scanning /sys/bus
  Jan 01 21:57:21 localhost udevadm[2522]: sd-device-enumerator: Scanning /sys/class
  Jan 01 21:57:21 localhost udevadm[2522]: sd-device-enumerator: Failed to scan /sys/class: No such file or directory
  Jan 01 21:57:21 localhost udevadm[2522]: Failed to scan devices: No such file or directory
*******************************************************************************************************

kernel-install: drop redundant "/"

Follow-up for 1cdbff1c844ce46f1d84d8feeed426ebfd550988.

After the commit 1cdbff1c844ce46f1d84d8feeed426ebfd550988, each entry .conf contains
redundant slash like the following:
```
$ cat xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx-5.9.8-200.fc33.x86_64.conf
title      Fedora 33 (Thirty Three)
version    5.9.8-200.fc33.x86_64
machine-id xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
options    root=/dev/nvme0n1p2 ro rootflags=subvol=system/fedora selinux=0 audit=0
linux      //xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/5.9.8-200.fc33.x86_64/linux
initrd     //xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/5.9.8-200.fc33.x86_64/initrd
```

man: bootctl set-default/oneshot accept an empty string

Revert "semaphore: temporarily explicitly use the US image mirror"

This reverts commit 9ac47d926146ed177aef805bba75f07bd3ed8643.

test/test-functions: copy /usr/lib/pam.d into $initdir

The systemd-user file has been moved from /etc/pam.d into /usr/lib/pam.d,
so test-functions needs to copy it from /usr/lib/pam.d instead.

This will copy it from either location.

unit: update comment about OOM score

Follow-up for 6b2229c6c60d0486f5eb9ed3088f9c780d7c0233.

Merge pull request #17669 from yuwata/coccinelle-update

tree-wide: shorten code a bit suggested by coccinelle

Merge pull request #17676 from yuwata/lgtm-fixes

tree-wide: several fixes prompted by LGTM

gpt: add RISC-V GPT partition typecode uuid

curl-util: fix callback prototype

CURLMOPT_SOCKETFUNCTION callback is an easy handle, not a multi.

Merge pull request #17685 from yuwata/curl-no-oldies

meson: set CURL_NO_OLDIES in developer mode

Merge pull request #17687 from yuwata/use-localtime_or_gmtime_r

use localtime_or_gmtime_r()

backlight: add several debug logs and adjust log level of non critical error

This may help to investigate issues.

networkd/dhcp6: allow layer3 devices without MAC

Devices with multicast but without mac addresses i.e. tun devices
are not getting setuped correctly:

$ ip tuntap add mode tun dev tun0
$ ip addr show tun0
16: tun0: <NO-CARRIER,POINTOPOINT,MULTICAST,NOARP,UP> mtu 1500 qdisc fq_codel state DOWN group default qlen 500
    link/none
$ cat /etc/systemd/network/tun0.network
[Match]
Name = tun0

[Network]
Address=192.168.1.1/32
$ ./systemd-networkd
tun0: DHCP6 CLIENT: Failed to set identifier: Invalid argument
tun0: Failed

man: Fix misspelling of directive and typos in systemd-coredump man page

This commit fixes
 * misspelling of LimitCORE directive of systemd.exec
 * typos
in systemd-coredump man page.

hwdb: set fuzz value for lenovo x240 touchpad to improve cursor precision (#17659)

logs-show: use localtime_or_gmtime_r()

timedate: use localtime_or_gmtime_r() and mktime_or_timegm()

meson: set CURL_NO_OLDIES in developer mode

curl-util: fix type CURL -> CURLM

units: restore sysfs conditions in sys-fs-fuse-connections.mount and sys-kernel-config.mount

Commit 42cc2855ba2fe4c6f5d incorrectly removed the condition on sysfs in both
sys-fs-fuse-connections.mount and sys-kernel-config.mount. However there are
still needed in case modprobe of one of these modules is intentionally skipped
(due to lack of privs for example).

This patch restores the 2 conditions which should be safe for the common case,
since all conditions are only checked after all deps ordered before are
complete.

Follow-up for 42cc2855ba2fe4c6f5dc863507a1c843611350a0.

core/mount: mount_start() may be called during the state is MOUNT_MOUNTING_DONE

As, both MOUNT_MOUNTING and MOUNT_MOUNTING_DONE are mapped to
UNIT_ACTIVATING.

Fixes #17570.

core: serialize u->pids until the processes have been moved to the scope cgroup

Otherwise if a daemon-reload happens somewhere between the enqueue of the job
start for the scope unit and scope_start() then u->pids might be lost and none
of the processes specified by "PIDs=" will be moved into the scope cgroup.

man: to make MACAddress= take effect, MACAddressPolicy= must be "none"

link-config: warn when MACAddress= is set with MACAddressPolicy=persistent or random

link-config: make MACAddressPolicy= accept an empty string

Merge pull request #17649 from yuwata/resolve-dnssd-template-name

resolve: ignore invalid service template name

Merge pull request #17658 from jwrdegoede/hwdb-accel-work

hwdb accel work

systemctl: fix potential pointer overflow

tools: drop unnecessary "else" after for loop

ask-passwd: drop a condition which is always false

resolvectl: drop a condition which is always true

pstore: use log_oom()

coccinelle: always use SYNTHETIC_ERRNO() macro

tree-wide: use return value of log_xxx_errno()

coccinelle: add one more rule to use return value of log_xxx_errno()

seccomp: move brk+mmap+mmap2 into @default syscall filter set

These three syscalls are internally used by libc's memory allocation
logic, i.e. ultimately back malloc(). Allocating a bit of memory is so
basic, it should just be in the default set.

This fixes a couple of issues with asan/msan and the seccomp tests: when
asan/msan is used some additional, large memory allocations take place
in the background, and unless mmap/mmap2/brk are allowlisted these will
fail, aborting the test prematurely.

Merge pull request #17667 from fbuihuu/fix-module-loading-from-udev-rule

Fix module loading from udev rule

man: suffix settings name with = and enclose in <varname>

units: wait until some fs modules are entirely loaded before mounting their corresponding filesystem

udev requests to start the fs mount units when their respective module is
loaded. For that it monitors uevents of type "ADD" for the relevant fs modules.

However the uevent is sent by the kernel too early, ie before the init() of the
module is called hence before directories in /sys/fs/ are created.

This patch workarounds adds "Requires/After=modprobe@<fs-module>.service" to
the mount unit, which means that modprobe(8) will be called once the fs module
is announced to be loaded. This sounds pointless, but given that modprobe only
returns after the initialization of the module is complete, it should
workaround the issue.

As a side effect, the module will be automatically loaded if the mount unit is
started manually.

Fixes #17586.

Revert "units: skip modprobe@.service if the unit appears to be already loaded"

This reverts commit 9cbf1e58f9629af5c6b56777ee73dc6320306d6d.

The presence of /sys/module/%I directory can't be used to assert that the load
of a given module is complete and therefore the call to modprobe(8) can be
skipped. Indeed this directory is created before the init() function of the
module is called.

Users of modprobe@.service needs to be sure that once this service returns the
module is fully operational.

hwdb: Add accel orientation quirk for Acer Aspire Switch 10 SW3-016 2-in-1

Add a quirk to fix the accelerometer orientation on the
Acer Aspire Switch 10 SW3-016 2-in-1.

hwdb: Add accel orientation quirk for Voyo Winpad A15 tablet

Add a quirk to fix the accelerometer orientation on the
Voyo Winpad A15 tablet.

hwdb: Add accel orientation quirk for Lenovo ThinkPad Yoga 11e 4th gen

Add a quirk to fix the accelerometer orientation on the Lenovo
ThinkPad Yoga 11e 4th gen 360 degree hinges 2-in-1.

hwdb: Fix accel orientation quirk Z-axis for Lenovo ThinkPad Yoga 11e 3th gen

The Lenovo ThinkPad Yoga 11e 360 degree hinges style 2-in-1s use 2
accelerometers, 1 in the display and 1 in the base.

Kernel work is under way to also export the second accelerometer in
the base as an iio-device; and userspace work is underway to use
both accelerometers on 360 degree hinges style 2-in-1s (with 2 accels)
to figure out the angle between the 2 halves.

So far most orientation-matrix quirks have not cared much about the
Z-axis being correct, but in these 2 accelerometer setups getting
the Z-axis correct is important too.

hwdb: Add base accelerometer orientation quirk for base sensor of Medion Akoya E* series

The KIOX010A and KIOX02A ACPI hw-ids (HIDs) are used in 360 degree hinges
style 2-in-1s which have 2 accelerometers, 1 in the display (as usual) and
a second accelerometer in the base.

So far 60-sensor.hwdb has only defined a mount-matrix for the
sensor with the KIOX010A HID, which is the sensor in the display
half of the device. The reason for this is that sofar userspace has
only cared actually used the sensor in the display (for automatic
display rotation. Work is underway to make userspace use both sensors:
https://gitlab.freedesktop.org/hadess/iio-sensor-proxy/-/issues/216

Recently an entry was added for the Medion Akoya's E2221T base-sensor,
but that was added to mark it with ACCEL_LOCATION=base and the entry
simply used the identity-matrix for ACCEL_MOUNT_MATRIX since nothing
is using the mount-matrix info for the second accelerometer.
I believe that this entry was added because on some devices the second
accelerometer gets enumerated first and then iio-sensor-proxy will
wrongly use the second sensor for display-rotation, unless it is marked
with ACCEL_LOCATION=base.

Instead of adding info for the second accelerometer on a per device
basis use the same generic dmi matches as used for the first (KIOX010A)
sensor, replacing the special case added for the E2221T and also
update the ACCEL_MOUNT_MATRIX with the actual mount-matrix for the
KIOX020A sensor in the base of these devices.

This was tested on a Medion Akoya E2228T.

hwdb: Document how to properly set the mount-matrix for the base-accelerometer in 360 degree hinges style 2-in-1s with 2 accelerometers

Document how the mount-matrix for the base-accelerometer must be set on
360 degree hinges style 2-in-1s with 2 sensors (one in the display and
1 in the base).

Note the choice to define the lid being fully closed as an angle of
0 degrees is based on the ACPI tables of devices with a BOSC0200
ACPI device-node describing both sensors. In this case the ACPI
tables contain mount-matrix info (and the kernel will soon support
reading this and exporting it to userspace) and the mount-matrices
defined in these ACPI tables are such that the angle of the G-force
vector measured by the sensors is identical for both sensors when
the laptop's lid is fully closed.

This also feels more natural then defining the laptop being fully
open (180 degrees open) as the home / 0 degree angle position.

khash: fix structured initializer

Fixes #17646.

man: Fix sd_bus_message_append_array_space function signature

Merge pull request #17651 from yuwata/the-the

tree-wide: fix "the the" and "that that"

JOURNAL_FILE_FORMAT: fixup typos and punctuation

No significant changes

tree-wide: update "that that"

man,NEWS: fix "the the"

sd-device: drop unwanted newline in netlink message

test/udev-test: gracefully exit when imports fail

In Fedora rawhide various perl modules are now available as separate
packages that are not pulled in by dependencies. If we don't have some
package, skip the tests.

This ugly code is apparently the way to do conditional imports:
https://www.cs.ait.ac.th/~on/O/oreilly/perl/cookbook/ch12_03.htm.

resolve: ignore invalid service template name

Let's fiest test the template name, and then assign it.

resolve: make config_parse_dnssd_service_name() accepts an empty string

resolve: add a short comment about difference between dnssd_render_instance_name() and config_parse_dnssd_service_name()

resolve: wrap long line

Revert "resolve: check DNSSD service name template before assigning it"

This reverts commit 34136e1503cf60852051adbd8b9a002d6282b750.

Having the "%H" host name specifier in a DNSSD service name template
triggers a failed assertion during name template instantiation as
specifier_dnssd_host_name expects DnssdService in its userdata
pointer but finds NULL instead.

hwdb/60-keyboard: untabify and move comments to the same column

Follow-up for b0a3a3ff5d80d1605926fbba1d25a0f2448e383c.

Merge pull request #17640 from keszybz/meson-test-c++20

Also test headers against c++20

Merge pull request #17643 from keszybz/man-news-independent

NEWS and man page tweaks

NEWS,man: improve descriptions of Independent=

In NEWS, the new option was described twice, most likely because the first
description was tucked away in a paragraph about some other subject.

While at it, improve the descriptions in the man page to make it easier to grok
what that option really does.

man: drop misplaced phrase

I think it was added in 6c2b9c8da1eb1bac7e8f170b913f63dbd13a16d7 as c&p.

man: Add a paragraph to sd_bus_call explaning callback message lifetime

Merge pull request #17584 from yuwata/news-networkd

NEWS: several entries for networkd

network: use the common "cannot" form

Just for consistency.

man/systemd.netdev: remove bogus markup

There is no "Multicast" constant, and NULL doesn't make sense in the
context of addresses.

man/systemd.netdev: don't say "ranges"

NEWS: slightly fix explanation about oomd

man: DHCPv6PrefixDelegation= is disabled by dafault

NEWS: add several entries for networkd

Merge pull request #17599 from yuwata/meson-fuzz-tests

meson: do not build fuzzers when -Dfuzz-tests=false

test-oomd-util: remove memory_pressure == 0 checks

test_oomd_cgroup_context_acquire_and_insert reads the live cgroup data used
by the unit test. Under certain conditions, the memory pressure for the cgroup
can be non-zero (although most of the time it is 0 since these tests don't
generate much pressure).

Since these values are too dependent on the state of the system, remove the
checks. The type used is always >= 0 and test-psi-util already unit tests that
PSI values are parsed correctly from files so this test is redundant anyways.

man: mention that FirewallMark= optionally takes firewall mask

Closes #17587.

Merge pull request #17597 from yuwata/fix-typo

tree-wide: Fix typo

test: let's start getting ready for 2020

It'll be a wonderful year, I'm sure.

meson: use proper variable for libudev.h path

meson: fix build tests for c++14 and c++17

Merge pull request #17603 from yuwata/systemctl-fix-underline

systemctl: always show underline even if unit has no job ID

udev: use FOREACH_DEVICE_TAG() macro at one more place

Merge pull request #17612 from yuwata/fix-man-pages

man: fix two issues

Merge pull request #17614 from jwrdegoede/hwdb-logitech-kbd-fixes

hwdb Logitech keyboard fixes

man: Separated paragraph about making sd_bus object reply async

update TODO

man: add missing full stop

Merge pull request #17619 from yuwata/network-ndisc-dhcp6-client-always

network: honor M or O flag in RA even if IPv6AcceptRA.DHCPv6Cleint=always

meson: use "_" as separator in test names

Follow-up for ca121e20c42219e3bc4e5cb63dcc96cc5eae2879.

Fixes #17568.

sd-dhcp6-client: insert spaces around ternary operator