Merge pull request #14208 from poettering/json-homed-prepare

json bits from homed PR

hwdb: Add Bluetooth-attached Logitech MX Master

Except for the product IDs, the original MX Master appears identical to
the MX Master 2S.

Merge pull request #14360 from yuwata/udev-alternative-names-policy

udev: introduce AlternativeNamesPolicy= setting

systemctl: show what verbs support --dry-run in the help page

Signed-off-by: Bart Willems <bwillems@protonmail.com>

Merge pull request #14241 from keszybz/resume-timeout

Bump resume timeout to infinity

network: set AlternativeNamesPolicy= in 99-default.link

udev: introduce AlternativeNamesPolicy= setting

udev: do not fail if kernel does not support alternative names

Merge pull request #14339 from keszybz/invalid-enablement-logs

Small enhancements to logs for enabling of invalid units

random-util: call initialize_srand() after fork()

Merge pull request #14351 from yuwata/util-constify-strv-xxx

util: constify arguments of strv_xxx()

udev: extend the length of ID_NET_NAME_XXX= to ALTIFNAMSIZ

efivars: properly NUL terminate EFI variables when reading

A follow-up for 35b9eb0a72b6254568a294f0ebd011da20958a64.

basic/tmpfile: avoid maybe-uninitialized warning in mkostemp_safe()

The variable is always initialized, but the compiler might not notice
that. With gcc-9.2.1-1.fc31:

    $ CFLAGS='-Werror=maybe-uninitialized -Og' meson build
    $ ninja -C build
    [...]
    ../src/basic/tmpfile-util.c: In function ‘mkostemp_safe’:
    ../src/basic/tmpfile-util.c:76:12: error: ‘fd’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
       76 |         if (fd < 0)
          |            ^

TODO: drop entry

Implemented in 7d1e91d1a9504ab1bc03894038f90a8e87a4e982.

shared/install: log syntax error for invalid DefaultInstance=

Ideally, we would want to report this over back over dbus. But that is pretty hard,
because the unitfile parsing logic doesn't provide any feedback.
systemd-analyze verify also doesn't notice the issue, because it doesn't look
at the [Install] section at all. Let's print a message in the logs at least.

Added Trekstor Primetab S11B

Merge pull request #14354 from poettering/link-fix

two trivial doc fixes

docs: CSS files should not be executable

github: use systemd.io links in issue template

Merge pull request #14261 from keszybz/loop-utils-and-efivars

Fixes for networkd, shared/loop-util, basic/efivars

Merge pull request #14196 from keszybz/gpt-auto-generator-debugging

gpt-auto-generator debugging

Merge pull request #14350 from yuwata/network-udev-altnames-support

network, udev: add altname support

util: constify arguments of strv_xxx()

test-network: pass environment variables to networkctl

test-network: add a test case for netdev altname

networkctl: show alternative names

network: make Name= in [Match] support alternative names of interfaces

udev: support AlternativeName= setting in .link file

util: introduce ifname_valid_full()

test: add a test for sd_netlink_message_{append,read}_strv()

sd-netlink: introduce sd_netlink_message_append_strv()

sd-netlink: introduce sd_netlink_message_read_strv()

The combination of sd_netlink_message_enter_container() and
sd_netlink_message_read_string() only reads the last element if the attribute is
duplicated, such a situation easily happens for IFLA_ALT_IFNAME.
The function introduced here reads all matched attributes.

shared/loop-util: spin on open() returning ENOENT too

https://github.com/systemd/systemd/pull/14261#discussion_r355001559

basic/efivars: do not return EIO if an efivar read is shorten than fstat size

On my machine stat returns size 22, but only 20 bytes are read:

openat(AT_FDCWD, "/sys/firmware/efi/efivars/LoaderTimeInitUSec-4a67b082-0a4c-41cf-b6c7-440b29bb8c4f", O_RDONLY|O_NOCTTY|O_CLOEXEC) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=22, ...}) = 0
read(3, "\6\0\0\0", 4)                  = 4
read(3, "7\0001\0001\0003\0005\0002\0007\0\0\0", 18) = 16
Failed to read LoaderTimeInitUSec: Input/output error

Let's just accept that the kernel is returning inconsistent results.
It seems to happen two only two variables on my machine:
/sys/firmware/efi/efivars/LoaderTimeInitUSec-4a67b082-0a4c-41cf-b6c7-440b29bb8c4f
/sys/firmware/efi/efivars/LoaderTimeMenuUSec-4a67b082-0a4c-41cf-b6c7-440b29bb8c4f
so it might be related to the way we write them.

shared/efi-loader: add some debugging statements

Should make it easier to figure out why some operations fail...

shared/loop-util: spin on LOOP_CTL_REMOVE

If we call LOOP_CLR_FD and LOOP_CTL_REMOVE too rapidly, the kernel cannot deal
with that (5.3.13-300.fc31.x86_64 running on dual core
Intel(R) Core(TM) i7-6500U CPU @ 2.50GHz).

$ sudo strace -eioctl build/test-dissect-image /tmp/foobar3.img
ioctl(3, TCGETS, 0x7ffcee47de20)        = -1 ENOTTY (Inappropriate ioctl for device)
ioctl(4, LOOP_CTL_GET_FREE)             = 9
ioctl(5, LOOP_SET_FD, 3)                = 0
ioctl(5, LOOP_SET_STATUS64, {lo_offset=0, lo_number=0, lo_flags=LO_FLAGS_READ_ONLY|LO_FLAGS_AUTOCLEAR|LO_FLAGS_PARTSCAN, lo_file_name="", ...}) = 0
ioctl(5, BLKGETSIZE64, [299999744])     = 0
ioctl(5, CDROM_GET_CAPABILITY, 0)       = -1 EINVAL (Invalid argument)
ioctl(5, BLKSSZGET, [512])              = 0
Waiting for device (parent + 0 partitions) to appear...
Found root partition, writable of type btrfs at #-1 (/dev/block/7:9)
ioctl(5, LOOP_CLR_FD)                   = 0
ioctl(3, LOOP_CTL_REMOVE, 9)            = -1 EBUSY (Device or resource busy)
Failed to remove loop device: Device or resource busy

This seems to be clear race condition, and attaching strace is generally enough
to "win" the race. But even with strace attached, we will fail occasionally.
Let's wait a bit and retry. With the wait, on my machine, the second attempt
always succeeds:

...
Found root partition, writable of type btrfs at #-1 (/dev/block/7:9)
ioctl(5, LOOP_CLR_FD)                   = 0
ioctl(3, LOOP_CTL_REMOVE, 9)            = -1 EBUSY (Device or resource busy)
ioctl(3, LOOP_CTL_REMOVE, 9)            = 9
+++ exited with 0 +++

Without the wait, all 64 attempts will occasionally fail.

shared/loop-util: fix error handling in loop_device_make_full()

The function no longer returns the fd. This complicated semantics, because it
wasn't clear what holds the ownership: the return value or the output
parameter.  There were no users of the fd in the return value, so let's
simplify things conceptually and only return the fd once.

Reduce the scope of variables.

LOOP_CLR_FD was called on the wrong fd. Let's use a cleanup function to make
this automatic and reduce chances of a mixup in the future.

CID 1408498.

sd-netlink: support IFLA_PROP_LIST and IFLA_ALT_IFNAME attributes

linux: update headers

Add Acer Spin 1 SP111-33 to sensor hwdb

hwdb: Add accel orientation quirk for Thundersoft TST168 tablet

Add a quirk to fix the accelerometer orientation on the Thundersoft
TST168 tablet.

shared/install: provide a nicer error message for invalid WantedBy=/Required= values

$ build/systemctl --user cat badinstall
 # /home/zbyszek/.config/systemd/user/badinstall.service
[Service]
ExecStart=true

[Install]
WantedBy=asdf

$ build/systemctl --user enable badinstall
Failed to enable unit: "asdf" is not a valid unit name.

Fixes #4209.

shared/install: remove duplicated check

install_info_add() does the exact same check.

nspawn: fix overlay with automatic temporary tree

This makes --overlay=+/foobar::/foobar work again, i.e. where the middle
parameter is left out. According to the documentation this is supposed
to generate a temporary writable work place in the midle. But it
apparently never did. Weird.

Merge pull request #14333 from poettering/markdown-header-fixes

docs: make sure there's only one # markdown header in each file

docs: make sure there's only one # markdown header in each file

@bertob wants us to be strict here, and only have one "#" header per
markdown file, and use "##" (or "###", …) for all others. Interestingly,
we mostly got this right already, but this fixes a few cases where this
wasn't correct.

blockdev-util: rework get_block_device()

Let's open the specified path once, and use the same fd for all lookups.
Also, don't check for btrfs twice.

The behaviour remains unmodified.

Merge pull request #13915 from ddstreet/ipv6_mtu

Set ipv6 mtu after link-up or device mtu change

Merge pull request #14320 from yuwata/network-tc-fq_codel-more

network: tc: introduce more FQ-CoDel settings

Merge pull request #14269 from DaanDeMeyer/enable-mounts-on-root

nspawn: Enable specifying root as the mount target directory.

nspawn-mount: Use FLAGS_SET to check flags.

nspawn: Only bind-mount directory when necessary.

nspawn-mount: Remove unused parameters

nspawn: Enable specifying root as the mount target directory.

Fixes #3847.

test: increase qemu timeout for TEST-08 and TEST-09

These tests two have shown to be flaky in Ubuntu test infrastructure,
but got more reliable with increased timeouts of 300 seconds each.

Signed-off-by: Christian Ehrhardt <christian.ehrhardt@canonical.com>

nspawn: allow combination of private-network and network-namespace-path

Fixes: #14289

Merge pull request #14321 from yuwata/network-generator-14319

network-generator: allow empty hostname

test-network: add a test case for the new settings of FQ-CoDel

network: tc: support more attributes for FQ-CoDel

Merge pull request #14305 from ssahani/tc-fq

network tc: inroduce FQ - Fair Queue traffic policing

test: add a test case for network-generator

network-generator: allow empty hostname

Fixes #14319.

test-network: add a test case for FQ

network tc: inroduce FQ - Fair Queue traffic policing

Please see http://man7.org/linux/man-pages/man8/tc-fq.8.html

Merge pull request #14308 from poettering/man-naming-scheme-typo

superficial fixes to network naming docs

sd-netlink: add attributes for FQ

Merge pull request #14309 from yuwata/network-tc-vtable

network: tc: introduce QDiscVTable

Merge pull request #14307 from poettering/unicode-fuckup

locale-util: drop weird invisible unicode codepoints accidentally ins…

Merge pull request #14317 from poettering/hwdb-indent-fix

hwdb: trivial indentation fix

locale-util: drop weird invisible unicode codepoints accidentally inserted in comment

hwdb: trivial indentation fix

minor: avoid double title

The logo and H1 were clashing a bit

docs: make it pretty

Add custom Jekyll theme, logo, webfont and .gitignore

FIXME: the markdown files have some H1 headers which need to be replaced
with H2

network: tc: introduce QDiscVTable for future extendability

network: tc: drop unused element

network: tc: drop unused functions

docs: direct to systemd.io version of naming scheme docs

The document was imported back in
f9e6d49904e06d2cba3871f6873d0519fe086d0f. Let's make sure we actually
point to that version rather than the old one on freedesktop.

docs: fix markdown links

docs: use `` quotes for marking identifiers of some form

man: fix typo in net-naming-scheme man page

Merge pull request #14303 from yuwata/tc-use-typesafe-functions

network: tc: use typesafe functions

Merge pull request #14306 from poettering/markdown-category

beef up systemd.io pages with simple categorization

docs: beef up entrypoint documentation page

Let's use the rough categorization of the markdown pages to add basic
sections, via Jeykll templating. Also, add in a couple of additional
links via a JSON array that lists them.

So much web development, so much wow!

docs: place all our markdown docs in rough categories

man: fix typos (#14304)

network: tc: use typesafe functions to append netlink attributes

sd-netlink: make TCA_OPTIONS take NETLINK_TYPE_UNION

Merge pull request #14278 from ssahani/tc

network tc: qdisc parent add support to set ingress

Merge pull request #14297 from poettering/id128-size

cleaning up id128 formatting buffer sizing

ipv4ll: do not reset conflict counter on restart

Don't reset the conflict counter when trying a new pseudo random
address, so that after trying 10 addresses the londer timeout is used in
accordance with the RFC

Fixes #14299.

test-network: add a test case for fq-codel

network tc: Add support to conkfigure CoDel - Controlled-Delay Active Queue Management algorithm

```
$ tc qdisc show dev dummy99
qdisc fq_codel 8005: dev dummy99 root refcnt 2 limit 1000p flows 1024 quantum 1514 target 5.0ms interval 100.0ms memory_limit 32Mb ecn
$ network cat dumm99.network
[Match]
Name=dummy99

[TrafficControlQueueingDiscipline]
Parent=root
FairQueuingControlledDelayPacketLimit=1000

```

network tc: qdisc parent add support to set ingress

```
$ network tc qdisc
qdisc noqueue 0: dev dummy99 root refcnt 2
qdisc ingress ffff: dev dummy99 parent ffff:fff1 ----------------

$ network cat dumm99.network
[Match]
Name=dummy99

[TrafficControlQueueingDiscipline]
Parent=ingress
```

Merge pull request #14280 from yuwata/network-tbf-more

network: tc: add more options for TBF

Merge pull request #14295 from poettering/greedy-alloc-round-up

add new GREEDY_ALLOC_ROUND_UP() to make strv_extend() a bit less slow

tree-wide: use SD_ID128_STRING_MAX where appropriate

id128: introduce ID128_UUID_STRING_MAX for sizing UUID buffers

Merge pull request #14284 from yuwata/network-fix-segfault-14283

network: fix segfault in parsing SendOption=

strv: when growing strv arrays piecemeal actually allocate memory in exponential steps

Let's improve memory allocation for call such as strv_extend() that just
one item to an strv: these are often called in a loop, where they used
to be very ineffecient, since we'd allocate byte-exact space. With this
change let's improve on that, by allocating exponentially by rounding up
to the next exponent of 2. This way we get GREEDY_REALLOC()-like
behaviour without passing around state.

In fact this should be good enough so that we could replace existing
loops around GREEDY_REALLOC() for strv build-up with plain strv_extend()
and get similar behaviour.