udev: tag "leds" and "backlight" devices for association to a seat

These devices do not become user-accessible this way, but they are
logically assigned to a seat, which makes a lot of sense, since they are
human-facing output devices, and such should belong to one.

machine-image: openat() doesn't operate on the cwd if the first argument is specified

A fix-up for bcb846f30f9ca8f42e79d109706aee9f2032261b.

Merge pull request #12430 from poettering/seccomp-kill-process

use SCMP_ACT_KILL_PROCESS for SystemCallFilters=

test-execute: turn off coredump generation in test services

These services are likely to coredump, and we expect that but aren't
interested in the coredump. Hence let's turn off processing by setting
RLIMIT_CORE to 0/0.

test-execute: let's ignore the difference between CLD_KILLED and CLD_DUMPED

Depending on system configuration and whether SCMP_ACT_KILL_PROCESS or
SCMP_ACT_KILL_THREAD is available/used processes might coredump on
specific coredumps or are just plain killed. For our test case the
difference doesn't really matter, hence let's hide it away.

test-execute: check exit code before exit status

The meaning of the status changes depending on the code, hence let's
always compare the code first, status second.

service: tweak capitalization of unit description

NEWS: document the new SystemCallFilter= behaviour

nspawn: expose the new seccomp actions in the OCI logic

test: use the new action in our tests

This way, we know that it works as intended.

core: prefer SCMP_ACT_KILL_PROCESS for SystemCallFilter= behaviour

If we have it, use it. It makes a ton more sense.

Fixes: #11967

seccomp: add scmp_act_kill_process() helper that returns SCMP_ACT_KILL_PROCESS if supported

Merge pull request #12601 from keszybz/two-log-color-adjustments

Two log color adjustments

Merge pull request #12590 from keszybz/unicode-cmdlines

Use unicode for cmdline printing

Drop support for /usr/sbin/halt.local

/usr/sbin/halt.local is a Fedora/Red Hat anachronism from pre-systemd
times.

mkosi: update to Fedora30 (#12642)

Merge pull request #12645 from poettering/journal-mmap-einval

journald: output a proper error message when the journal is used on f…

shared/machine-image: avoid passing NULL to log_debug_errno

Fixes: https://github.com/systemd/systemd/issues/12534

Merge pull request #12618 from yuwata/test-network-improvements

network: several cleanups

login: fixup button_open() fd

Merge pull request #12647 from cdown/dup

test: Remove superfluous error check

test: Remove superfluous error check

This is already checked above before we set any manager attributes,
immediately after manager_new().

Merge pull request #12631 from keszybz/doc-and-error-message-tweaks

Doc and error message tweaks

journald: output a proper error message when the journal is used on fs that doesn't do mmap() properly

Prompted by:

https://lists.freedesktop.org/archives/systemd-devel/2019-May/042708.html

Merge pull request #12431 from poettering/tmpfiles-chmod-chown-order

tmpfiles: run chown() before chmod()

man: reword systemd.socket(5) description to avoid "file" when talking about services

This text was written a long time ago, when we mostly talked about "service
files".  Strictly speaking, we don't need a "file", because the service might
be just an instance of a template, or an alias to another service. So let's use
the more modern parlance of "service units".

https://bugzilla.redhat.com/show_bug.cgi?id=1623781#c21

pid1: improve message when setting up namespace fails

I covered the most obvious paths: those where there's a clear problem
with a path specified by the user.

Prints something like this (at error level):
May 21 20:00:01.040418 systemd[125871]: bad-workdir.service: Failed to set up mount namespacing: /run/systemd/unit-root/etc/tomcat9/Catalina: No such file or directory
May 21 20:00:01.040456 systemd[125871]: bad-workdir.service: Failed at step NAMESPACE spawning /bin/true: No such file or directory

Fixes #10972.

man: tweak description of #12247 a bit

It wasn't really wrong, but maybe we can make it a bit clearer.

Fixes #12247.

man: add note that %h/%u/%U are mostly useless

Fixes #12389.

core: unset HOME=/ that the kernel gives us

Partially fixes #12389.

%h would return "/" in a machine, but "/root" in a container. Let's fix
this by resetting $HOME to the expected value.

sd-bus: improve error message for invalid org.freedesktop.DBus.Properties.Set signature

Fixes #12551.

man: add note which tmpfiles services read which files

Fixes #12596.

login: use _cleanup_ to simplify error handling

When logging about a device, try to include the path to it everywhere in
messages. Also fixes an invalid assert(fd).

Inspired by b2774a3ae692113e1f47a336a6c09bac9cfb49ad.

loop-util: invoke LOOP_CTL_GET_FREE in a loop

if we don't call it in a loop the device it tells us to open might
already be gone, taken by somebody else racing against us. Hence try a
few times.

test-network: drop wait_online() from test_ip6gre_tunnel()

network: unify link_ipv4ll_enabled() and link_ipv4ll_fallback_enabled()

test-network: wait for bridge slave to be enslaved in test_bridge_ignore_carrier_loss_frequent_loss_and_gain()

network: drop unused variable

network: update master's ifindex in link_update()

And use it in link_is_enslaved().

test-network: drop unused variable

network: drop duplicated logs

link_set_state() already logs about state change.

test-network: add test that vcan device can be configured state

network: make CAN devices go through LINK_STATE_CONFIGURING

test-network: add basic tests for veth interfaces

network: deprecate OneQueue= for tun or tap devices

IFF_ONE_QUEUE has no effect since kernel-3.8. See kernel's commit
5d097109257c03a71845729f8db6b5770c4bbedc.

test-network: add tests for flags of tun or tap devices

network: extend warning messages

network: Allow IFF_VNET_HDR to also be set for tun devices

f5f07dbf06f6df9d12e092c05ad8cfbe244d203f adds VnetHeader= for tap
devices, but the flag is also used for tun devices.
This adds VnetHeader= setting in [Tun] section.

test-network: add tests for LinkLocalAddressing= on ipvlan or friends

network: disable IPv4LL for ipvlan with L3 or L3S mode

As L3 or L3S mode do not support ARP.

test-network: add tests for Flags= in [IPVTAP]

test-network: add tests for Flags= in [IPVLAN]

test-network: add tests for Mode= setting in [MACVTAP] section

test-network: add tests for Mode= setting in [MACVLAN] section

test-network: convert jiffies to sec

test-network: show interface status when wait_online() fails

test-network: use wait_online() more

test-network: introduce check_link_exists()

test-network: use wait_online() more

test-network: use splitlines() at one more place

networkctl: add support to display Transmit/Recieve queue length (#12633)

```
(networkctl)⚡ % build/networkctl status veth-test
● 13: veth-test
            Link File: /usr/lib/systemd/network/99-default.link
         Network File: /usr/lib/systemd/network/veth0.network
                 Type: ether
                State: routable (configured)
               Driver: veth
           HW Address: 8a:a6:1c:3f:a6:1a
                  MTU: 1500
          Minimum MTU: 68
          Maximum MTU: 65535
Transmit Queue Length: 1
 Receive Queue Length: 1
              Address: 192.168.5.31
                       fe80::88a6:1cff:fe3f:a61a
              Gateway: 192.168.5.1
                  DNS: 192.168.5.1
                  NTP: 192.168.5.1
            Time Zone: Asia/Kolkata
         Connected To: Zeus on port peer-test
```

bus_open leak sd_event_source when udevadm trigger。

On my host, when executing the udevadm trigger, I only receive the change event, which causes memleak

basic/utf8: reduce memory usage

Add 8bit-version of get_process_cmdline() and use in cgroup-show.c

This restores show_pid_array() output in legacy locales on the console.
Only one call to get_process_cmdline() is changed, all others retain
utf8-only mode. This affects systemd-cgls, systemctl status, etc, when
working locally.

Calls to get_process_cmdline() that cross a process boundary always use
utf8. It's the callers responsibility to convert this to some encoding that
they use. This means that we always pass utf8 over the bus.

basic/process-util: convert bool arg to flags

In preparation for the next commit…

basic/escape: add truncation to xescape too

This does for ASCII and non-unicode encodings what utf8_escape_non_printable_full()
does for utf8-based encodings.

TODO: update

util-lib: do not truncate kernel comm names

It turns out that the kernel allows comm names higher than our expected limit
of 16.
$ wc -c /proc/*/comm|sort -g|tail -n3
35 /proc/1292317/comm
35 /proc/1293610/comm
36 /proc/1287112/comm
$ cat /proc/1287112/comm
kworker/u9:3-kcryptd/253:0

Rework cmdline printing to use unicode

The functions to retrieve and print process cmdlines were based on the
assumption that they contain printable ASCII, and everything else
should be filtered out. That assumption doesn't hold in today's world,
where people are free to use unicode everywhere.

This replaces the custom cmdline reading code with a more generic approach
using utf8_escape_non_printable_full().
For kernel threads, truncation is done on the parenthesized name, so we'll
get "[worker]", "[worker…]", …, "[w…]", "[…", "…" as we reduce the number of
available columns.

This implementation is most likely slower for very long cmdlines, but I don't
think this is very important. The common case is to have short commandlines,
and should print those properly. Absurdly long cmdlines are the exception,
which needs to be handled correctly and safely, but speed is not too important.

Fixes #12532.

v2:
- use size_t for the number of columns. This change propagates into various
  other functions that call get_process_cmdline(), increasing the size of the
  patch, but the changes are rather trivial.

util-lib: add truncation based on printable width to utf8_escape_non_printable()

udev/scsi_id: fix incorrect page length when get device identification VPD page

The length of device identification VPD page is filled with two bytes,
but scsi_id only gets the low byte. Fix it.

Signed-off-by: Zhang Xianwei <zhang.xianwei8@zte.com.cn>

Merge pull request #12629 from ssahani/networkctl

networkctl: MTU enhancements

Merge pull request #12626 from keszybz/oompolicy-check

Make the check if oom-killer fired more robust

networkctl: Add support to display min and max MTU

(networkctl)⚡ % ./networkctl status enp0s31f6                                                                                                                          ~/tt/networkctl/build
● 4: enp0s31f6
       Link File: /usr/lib/systemd/network/99-default.link
    Network File: n/a
            Type: ether
           State: n/a (unmanaged)
            Path: pci-0000:00:1f.6
          Driver: e1000e
          Vendor: Intel Corporation
           Model: Ethernet Connection (2) I219-LM
      HW Address: 8c:16:45:6c:83:b9 (LCFC(HeFei) Electronics Technology co., ltd)
             MTU: 1500
     Minimum MTU: 68
     Maximum MTU: 9000

sd-netlink: Add netlink property IFLA_MIN_MTU and IFLA_MAX_MTU,

networkctl: Display MTU

Now the MTU is not displayed. with this patch it's fixed.

(networkctl)⚡ % ./networkctl status enp0s31f6                                                                                                                          ~/tt/networkctl/build
WARNING: systemd-networkd is not running, output will be incomplete.

● 4: enp0s31f6
       Link File: /usr/lib/systemd/network/99-default.link
    Network File: n/a
            Type: ether
           State: n/a (unmanaged)
            Path: pci-0000:00:1f.6
          Driver: e1000e
          Vendor: Intel Corporation
           Model: Ethernet Connection (2) I219-LM
      HW Address: 8c:16:45:6c:83:b9 (LCFC(HeFei) Electronics Technology co., ltd)
             MTU: 1500

shared/cgroup-show: fix off-by-one in column counting

We'd get one column too many, and the trailing ellipsis would end up in
the next row.

test-utf8: add function headers and rename tests after functions they test

Introduce sc_arg_max() helper

Just a cast and an assert.

cgtop,test: use consistent capitalization for CGroup

Merge pull request #12510 from keszybz/test-directives

test: run check-directives.sh as part of the test suite

Merge pull request #12519 from keszybz/man-on-demand

Build man pages on demand only

meson-vcs-tag: enhance version info generation

When build from release tarball and where there is parent .git dir,
this situtaion will get wrong version info. (build with buildroot)

The systemd running show wrong version in dmesg log:
systemd[1]: systemd 2019.02-1086-gf5f17c4 running in system mode.
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Shoule be:
systemd[1]: systemd 241 running in system mode. (-PAM -AUDIT -SEL
            ^^^^^^^^^^^

man: systemd-nspawn: Update syntax to launch an image

To access a shell on a disk image, the man page on Fedora-29 says to
run: `systemd-nspawn -M Fedora-Cloud-Base-28-1.1.x86_64.raw`.  Let's
try.

List existing images:

    $> machinectl list-images | awk '{print $1,$2}';
    NAME TYPE
    Fedora-Cloud-Base-30… raw

    1 images

Now invoke `systemd-nspawn` as noted in the man page:

    $> systemd-nspawn -M Fedora-Cloud-Base-30-1.2.x86_64.raw
    No image for machine 'Fedora-Cloud-Base-30-1.2.x86_64.raw'.

Removing the ".raw" extension launches the image and gives a shell.
Update the man page to reflect that.

Frantisek Sumsal on #systemd (Freenode) noted the reason: "In older
versions systemd -M accepted both image-name.raw and image-name as a
valid image names, however, on Fedora 29 (systemd-239) with all the
BTRFS stuff around it accepts only -M image-name (without the
extension)"

    - - -

While at it, update the fedora_{latest_version, cloud_release}
variables.

Signed-off-by: Kashyap Chamarthy <kchamart@redhat.com>

Merge pull request #12612 from keszybz/bootctl-column

Rename TEST-31-OOMPOLICY to avoid conflict and look in $BOOT for kernels

sysctl: bump pid range only on 64-bit systems

Closes #12604.

json: fix minor memory leak on error path

core: do an extra check if oom was triggered when handling sigchild

Should fix #12425.

core/service: drop {}

small fixes: make get_process_state() static and fix typo

Merge pull request #12602 from keszybz/two-man-page-tweaks

Two man page tweaks

travis: use variable to avoid one call

tests: install perl for CI

It is useful for various tests (test-directives, test-udev, …).

test: run check-directives.sh as part of the test suite

Merge pull request #12603 from ssahani/ndisc-blacklist

networkd: Ability to selectively ignore IPv6 prefixes supplied via ro…

efivars: allow plus in the entry name

Closes #12572.

networkd: Ability to selectively ignore IPv6 prefixes supplied via router advertisement

Closes https://github.com/systemd/systemd/issues/10647

TEST-32-OOMPOLICY: drop unneeded headers

test: rename TEST-31-OOMPOLICY to TEST-32-OOMPOLICY

TEST-31-DEVICE-ENUMERATION got merged first.

test: search $BOOT for kernel+initrd

$BOOT is the preferred location for kernel images. If $BOOT does not
exist, -x will print the path to ESP, so well fall back to previous
behaviour.