scope: on unified, make sure to unwatch all PIDs once they've been moved to the cgroup scope

Commit 428a9f6f1d0396b9eacde2b38d667cbe3f15eb55 freed u->pids which is
problematic since the references to this unit in m->watch_pids were no more
removed when the unit was freed.

This patch makes sure to clean all this refs up before freeing u->pids by
calling unit_unwatch_all_pids().

sd-event: fix delays assert brain-o (#17790)

s/sizeof/ELEMENTSOF/

Bug introduced in 34b87517749caa4142b19eb3c63bdf349fafbc49.

mkosi: make ubuntu image bootable by default

man: Speicfy exact return values of sd_bus_message_enter_container

po: add units/systemd-journald.service.in to POTFILES.skip

hwdb: add axis range corrections for Lenovo ThinkPad T14 Gen1

Merge pull request #17761 from keszybz/documentation-followups

Two tiny fixlets

Update logind-button.c

test-networkd-conf: add missing assert_se

Coverity CID#1437351. Fixup for f706340979a.

semaphore: temporarily disable the timedated test suite

All otherwise passing Semaphore CI builds are now getting stuck on the
timedated test suite. Let's temporarily skip it until the issue is
resolved.

See: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=975010

hwdb: 60-keyboard: Add Dell Inspiron 11 3168 Fn+Home/End/PageUp/PageDown key mappings

On the Dell Inspiron 11 3168 the Fn and Arrow Keys control the Home, End, Page Up and Page Down keys. This commit allows those keys to work correctly.

Merge pull request #17734 from yuwata/parse-hwdb

hwdb: add missing Group()

Merge pull request #17732 from yuwata/core-use-synthetic_errno

core,network: use SYNTHETIC_ERRNO() macro

NEWS: really fix kernel version number reference

Fixes #17736.

coccinelle: ignore specific cases to use SYNTHETIC_ERRNO() macro

cryptsetup: drop unnecessary bracket

test: use for(;;) instead of while(true)

core/scope: use set_ensure_put()

mount-util: use mfree()

oom: use CMP() macro

core: fix typo

coccinelle: add rules for log_unit_error_errno() or friends

network: fix return values

network: use SYNTHETIC_ERRNO() macro

core: use SYNTHETIC_ERRNO() macro

core/timer: drop unnecessary brackets

Merge pull request #17474 from yuwata/network-drop-link-deserialization-logic

network: drop link deserialization logic

Merge pull request #17478 from yuwata/split-network-internal

libsystemd-network: split network-internal.c

resolved: allow cache responses from local DNS servers

oom: fix oomd.conf install location

The file is read in pkgsysconfdir, so install it there.

home: fix homed.conf install location

The install location changed in d7aa78c32f076c305ceeb183fc06eb1de0960deb,
I think this was not intentional. Keep the condition but revert the path.

hwdb: add missing Group()

This fixes the following warning:
```
parse_hwdb.py:120: UserWarning: warn_ungrouped_named_tokens_in_collection: setting results name 'SETTINGS*' on And expression collides with 'HZ' on contained expression
  dpi_setting = (Optional('*')('DEFAULT') + INTEGER('DPI') + Suppress('@') + INTEGER('HZ'))('SETTINGS*')
```

Not sure about for the mount_matrix, but LGTM.com warns in that line,
and, adding Group() does not change the parse result.

hwdb: enable diagnostic switches

Merge pull request #17738 from keszybz/hwdb-news-update-v247

Update hwdb and news for v247

shared/seccomp-util: address family filtering is broken on ppc

This reverts the gist of da1921a5c396547261c8c7fcd94173346eb3b718 and
0d9fca76bb69e162265b2d25cb79f1890c0da31b (for ppc).

Quoting #17559:
> libseccomp 2.5 added socket syscall multiplexing on ppc64(el):
> https://github.com/seccomp/libseccomp/pull/229
>
> Like with i386, s390 and s390x this breaks socket argument filtering, so
> RestrictAddressFamilies doesn't work.
>
> This causes the unit test to fail:
> /* test_restrict_address_families */
> Operating on architecture: ppc
> Failed to install socket family rules for architecture ppc, skipping: Operation canceled
> Operating on architecture: ppc64
> Failed to add socket() rule for architecture ppc64, skipping: Invalid argument
> Operating on architecture: ppc64-le
> Failed to add socket() rule for architecture ppc64-le, skipping: Invalid argument
> Assertion 'fd < 0' failed at src/test/test-seccomp.c:424, function test_restrict_address_families(). Aborting.
>
> The socket filters can't be added so `socket(AF_UNIX, SOCK_DGRAM, 0);` still
> works, triggering the assertion.

Fixes #17559.

NEWS: fix kernel version number reference

Fixes #17736.

NEWS: version 247

hwdb: update chromiumos autosuspend rules

hwdb: update for v247

Only some small changes, because we updated recently. As usual, it seems that there are mostly
additions with a smaller amount of corrections, no big removals.

docs/RELEASE: clarify which steps are done when

test: use cap_last_cap() for max supported cap number, not capability_list_length()

This test assumes capability_list_length() is an invalid cap number,
but that isn't true if the running kernel supports more caps than we were
compiled with, which results in the test failing.

Instead use cap_last_cap() + 1.

If cap_last_cap() is 63, there are no more 'invalid' cap numbers to test with,
so the invalid cap number test part is skipped.

Translated using Weblate (Chinese (Simplified))

Currently translated at 83.9% (157 of 187 strings)

Co-authored-by: Whired Planck <fungdaat31@outlook.com>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/master/zh_CN/
Translation: systemd/master

Merge pull request #17622 from yuwata/udev-database-set-udev-version

sd-device: make sd_device_has_current_tag() compatible with udev database generated by older udevd

add touchpad-edge-detect output for the Lenovo Thinkpad Extreme to evdev.hwdb

Merge pull request #17709 from yuwata/test-seccomp-skip

test: skip several tests in test-seccomp when running under valgrind or ASAN

sd-device: make sd_device_has_current_tag() and friends compatible with database version 0

sd-device: introduce database version and save it in udev database V field

sd-device: make device_add_property_internal() inline

man: slightly update the man page of sd_bus_message_read_basic()

Follow-up for 73a1d7d2433edd1872ec53db3e804009298ebb1d.

man: Fixed an incomplete sentence

man: Specify that sd_bus_message_read_basic returns 0 if end of array had been reached.

docs: document what VPNs should do to systemd-resolved.service

Fixes: #17588 #17512

Prompted-by: #17529
(Also relevant: #6076)

core/mount: mount command may fail after adding the corresponding proc mountinfo entry

Hopefully fixes #17617.

Merge pull request #17715 from yuwata/specifier-follow-ups

Sort specifiers

cryptsetup: use log_warning_errno() where we can, instead of log_warning()

Merge pull request #17721 from poettering/more-strjoin-cryptsetup

two minor fixes/clean-ups

mkosi: make sure our mkosi files work with f33

test: skip several tests in test-seccomp when running on asan

test: skip several tests in test-seccomp when running on valgrind

man: clarify DefaultTasksMax doc

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

tree-wide: sort specifiers and move common comments to specifier.h

man: sort specifiers alphabetically

man: add missing specifiers supported in [INSTALL] section

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