core: automatically add udev dependency for units using RootImage=

We use udev to wait for /dev/loopX devices to be fully proped hence we
need an implicit ordering dependency on it, for RootImage= to work
reliably in early boot, too.

Fixes: #14972

Merge pull request #15508 from keszybz/add-dbus-entities-to-index

Add dbus entities to index

Merge pull request #15505 from keszybz/man-sd-hwdb-sd-journal

Document remaining sd-journal and sd-hwdb functions

man: document how to get the boot menu with zero time-out

Fixes: #15125

man: add SD_HWDB_FOREACH_PROPERTY with an example

man: add description of all the sd-hwdb funtions

man: add markers to put all dbus entities in the directives index

Follow-up for f92c8d1c67bcdeba097e3203d8aafe3a31230ada.

directives.index:
- This index contains 3398 entries in 19 sections, referring to 333 individual
+ This index contains 4316 entries in 19 sections, referring to 333 individual

man: run systemd1(5) through the updater

For some reason I must've forgotten this page in 4fb222c4b2.

Merge pull request #15476 from boucman/directive_dbus

Generate systemd.directive entries from dbus documentation

Merge pull request #15495 from keszybz/resolve-debugging-and-stub-handling

Resolve debugging and stub handling

Merge pull request #15314 from keszybz/network-server-access-functions

Define network server access functions

Merge pull request #15502 from DaanDeMeyer/sd-bus-exit-on-disconnect-docs

sd-bus: Add sd_bus_get/set_exit_on_disconnect docs

Merge pull request #15503 from DaanDeMeyer/sd-bus-get-docs

sd-bus: Add sd_bus_get_scope/tid/unique_name docs

man: add sd_j_open_namespace as refname

This was forgotten in 241c8f67f65a1051068a1728daaca5bc49183c69.

check-api-docs: sd_journal_open_container is deprecated

man: document sd_journal_*_with_location

sd-bus: Rewrap sd_bus_set_close_on_exit + small fixes

sd-bus: Add sd_bus_get_scope/tid/unique_name docs

sd-bus: Rewrap sd_bus_set_description docs

sd-bus: sd_bus_get_fd docs typo fix

update-dbus-docs: automatically add variablelist for introspected items

Add a <variablelist/> tag after every programlisting we auto-generate that
will be read by make-directive-index to cross-reference all dbus elements.

sd-bus: Rewrap sd_bus_get_fd docs

sd-bus: Small sd_bus_set_server doc fix

sd-bus: Add sd_bus_get/set_exit_on_disconnect docs

sd-bus: Add sd_bus_is_server + sd_bus_set/is_bus_client docs

make-directive-index: allow variablelist to specify an element to index

This commit looks for a new "extra-ref" attribute in <variablelist>
If this attribute is specified, its content will be index as pointing to
the current man-page in systemd.directives

make-directive-index: allow pages to specify the path to search

So far, make-directive-index would look for
./valistentry/term/varname for elements to add to the directive man page.

This commit allows to specify xpath= in the varlist directive to tell
the generator what to look for.

sd-bus: sd_bus_get_current_* docs fixes

resolve: when the stub listener is disabled, symlink stub-resolv.conf to resolv.conf

When the stub listener is disabled, stub-resolv.conf is useless. Instead of
warning about this, let's just make stub-resolv.conf point to the private
resolv.conf file. (The original bug report asked for "mirroring", but I think
a symlink is nicer than a copy because it is easier to see that a redirection
was made.)

Fixes #14700.

resolve: when writing of private resolv.confs fails, do not remove old copies

All callers ignore the return value.

This is almost entirely theoretical, since writing to /run is unlikely to
fail..., but the user is almost certainly better off keeping the old copy
around and having working dns resolution with an out-of-date dns server list
than having having a dangling /etc/resolv.conf symlink.

resolved: include actual path in error message

An error with a full path is immediately clear. OTOH, a user might not be
familiar with concenpt like "private resolv.conf".

I opted to use %s-formatting for the path, because the code is much easier to
read this way. Any difference in t speed of execution is not important.

resolved: return org.freedesktop.resolve1.DnsError.NXDOMAIN on LLMNR resolution failure

Fixes #14922.

TODO: add a hypothetical

man: add forgotten <para> tags around a paragaph in resolve1(5)

resolve: allow setting the log level dynamically as in pid1

This is useful to raise the log level for a single transaction or a few,
without affecting other state of the resolved as a restart would.
The log level can only be set, I didn't bother with having the ability
to restore the original as in pid1.

Merge pull request #15439 from keszybz/man-resolve

Fix formatting in in resolved.conf(5) and nss-resolve(8)

Merge pull request #15491 from keszybz/resolvconf-compat

Two tweaks to the description of resolvconf compatibility

resolved: fix minor confusion in comment

Suffixing != prefixing.

man: use manpages.ubuntu.com for resolvconf(8) link

Add manpages.debian.org as a man citeref "project", so we can refer to man
pages hosted there. resolvconf(8) doesn't seem to appear on any of the ones we
currently have defined.

We are doing a trick, where our man page is installed as resolvconf(1), and
we refer the reader to resolvconf(8). This can be pretty confusing, so the
least we can do is to provide a non-broken symlink ;)

v2:
- link to manpages.d.o instead of manpages.ubuntu.com, because debian is the
  upstream here, and because with debian we can use /unstable/ as the version,
  and ubuntu doesn't seem to have any such shortcut, so the version would need
  to be periodically updated.

man: fix formatting in resolved.conf(5)

man: reindent nss-resolve(8)

Merge pull request #15480 from DaanDeMeyer/sd-bus-add-node-enumerator-docs

sd-bus: Add sd_bus_add_node_enumerator docs

Merge pull request #15478 from DaanDeMeyer/dbus-api-docs-fixes

D-Bus api docs fixes

sd-bus: Add sd_bus_get_current_* docs

sd-bus: Add sd_bus_emit_* docs

sd-bus: Small sd_bus_add_object_manager docs fix

sd-bus: Add sd_bus_add_node_enumerator docs

sd-bus: sd_bus_add_object docs fixes

man: add a note that resolvconf updates /etc/resolv.conf in specific circumstances

When someone knows how the whole ecosystem works, this is understandable.
But for someone coming from a system where resolvconf updates /etc/resolv.conf
directly, this can be rather surprising.

For https://bugzilla.redhat.com/show_bug.cgi?id=1815605.

sd-bus: Add sd_bus_set_server, sd_bus_get_bus_id docs

D-Bus docs: systemd1 fixes

sd-bus: Add sd_bus_add_object_manager docs

sd-bus: Add sd_bus_add_filter docs

D-Bus docs: Use method instead of call

Merge pull request #15463 from keszybz/resolvectl-query-formatting

Fix resolvectl query formatting

Merge pull request #15444 from poettering/audit-enable

journald: make whether we enable auditing at start-up optional

Merge pull request #15396 from keszybz/dbus-api-docs

D-bus API docs

systemctl: hide first column with --plain instead of --no-legend

Hiding the first column, which may contain bullet circles, with --no-legend
is undocumented and potentially unexpected. On the other hand, not printing
bullet circles with --plain is documented so hiding the column with that
switch is sensible.

The combination "--full --no-legend --no-pager --plain" is appropriate for
automated processing of systemctl output.

Merge pull request #15459 from keszybz/remove-seat_can_multi_session

Remove seat_can_multi_session

network: add TUN/TAP vt-* network rule for VMs

VM typically use a TAP device, and work similarly to a veth device from the
host side.

resolvectl: fix indentation of hexdump'ed packets

ebf963c551e1077abaa5e0fa0f330e6d1c23565e changed the 'sep' argument to always
be either " " or "\n", which broke the indentation logic for the first line
in base64_append_width(). Since it now always is one character, and never NULL,
let's change the type to char and simplify the logic a bit.

$ COLUMNS=30 build/test-dns-packet test/test-resolve/org~20200417.pkts
============== test/test-resolve/org~20200417.pkts ==============
org IN DNSKEY 256 3 RSASHA1-NSEC3-SHA1
        AwEAAcLPVEcg0hFBheXQf
        QOqqLiRgckk69o2KTAsq3
        lNRY0c9mnEjzZDGsGmXNy
        2EQ6yelkIYYus7KLor2Fz
        x59hEqcM82zqkdHV6hXvZ
        yjxxSHG3nl8xQS6gF8mdI
        YouDTWWhTInfjSKoIeDok
        Hq3S67EjSngV7/wVCMTbI
        amS0NF4H
        -- Flags: ZONE_KEY
        -- Key tag: 37022
...

$ COLUMNS=120 build/test-dns-packet test/test-resolve/org~20200417.pkts
============== test/test-resolve/org~20200417.pkts ==============
org IN DNSKEY 256 3 RSASHA1-NSEC3-SHA1 AwEAAcLPVEcg0hFBheXQfQOqqLiRgckk69o2KTAsq3lNRY0c9mnEjzZDGsGmXNy2EQ6yelkIYYus7KLor
                                       2Fzx59hEqcM82zqkdHV6hXvZyjxxSHG3nl8xQS6gF8mdIYouDTWWhTInfjSKoIeDokHq3S67EjSngV7/w
                                       VCMTbIamS0NF4H
        -- Flags: ZONE_KEY
        -- Key tag: 37022
...

test-resolve: add a bunch more packets for testing

Let's append the date to the domain in the file name, to be able
to have multiple versions for the same domain.

There is no particular rhyme or reason to the domains being used:
I just pulled a few domains that happened to be present in issues reported
on github, even though the issues were not about pretty printing.

headers: get rid of pointless /* deprecated */ comment

The _sd_deprecated_ annotation gives it all away, and the comment adds
no additional value.

sd-login: get rid of seat_can_multi_session()

Follow-up for fa2cf64a917d31605d40d34e98ce9e2e066064fa.
Backwards-compat is retained. A short note is added in docs, in case
people see sd_seat_can_multi_session() mentioned somewhere and wonder what
happened to it.

Also see https://github.com/systemd/systemd/pull/15337#issuecomment-610369404.

errno-util: let's beef up ERRNO_IS_NOT_SUPPORTED() with socket not supported errors

journald: add configuration option for enabling/disabling audit during journald startup

Let's make it optional whether auditing is enabled at journald start-up
or not.

Note that this only controls whether audit is enabled/disabled in the
kernel. Either way we'll still collect the audit data if it is
generated, i.e. if some other tool enables it, we'll collect it.

Fixes: #959

man/systemd-service: clarify env variable expansion

This clarifies some more aspects of `${FOO}` expansions in service
units, mostly trying to answer my own doubts about what happens when
the matching variable is not defined.

Add .clang-format file

The result of automatic formatting is not perfect, but if someone uses an
editor that uses that, it's probably better than nothing.

Fixes #15415.

[zjs: I resurrected this patch from #11811 and edited the extended comment
heavily.]

Merge pull request #15454 from keszybz/codespell-fixes

Spelling fixes based on codespell suggestions

Merge pull request #15347 from hakman/journald-upload-public-cloud

Support journal-upload HTTPS without key and certificate

Merge pull request #15452 from keszybz/fix-ipproto-assert

Fix static assertion failure with recent glibc

network: fix typo

I was worried about backwards compat, but this was only added
in af94bb24b59a2abc8d9f3ba6b5ba168372c9237c, and can be changed
with impunity.

resolved: fix typo in an unused function and add comment

Follow-up for 2615691003b9d73a92590b8250a54ad135e3a33b.

man: fixes from online review

Also includes the issues pointed out by @boucman.

man: update description of polkit rules for systemd1

Fixes #2562.

v2: the erroneous part about CAP_SYS_ADMIN is removed

shared/reboot-util: indentation update

man: locale1(5) — adjust page structure and fill in the missing parts

man: login1(5) — fix markup and fill in the missing descriptions

Partially fixes #1042.

man: review of import1(5)

man: various tweaks to org.freedesktop.hostname1(5)

Co-Authored-By: Daan De Meyer <daan.j.demeyer@gmail.com>

man: run dbus api docs through the updater

This replaces the api export tables with updated versions, and inserts
comments for all "undocumented" items. The slow work of documented them
is left for later ;)

lxml does some formatting changes that are not significant for lxml processing,
but generate spurious difference in the diff (namely: ulinks become one-line,
and double quotes are used instead of single quotes for element attribute
values). This should be a one-time thing: subsequent renegeration should be
idempotent with regards to this.

update-dbus-docs: add support for settings printing just one selected interface

So far the units there were being documented had only one custom interface.
But for the pid1 case, something more flexibile is needed. So let's add
an annotation in the page what we want to print, and filter in the generator.

Add updater for dbus introspection in man pages

Compares to gdbus output, the values of properties are replaced by ellipses.
For arrays and strings, the outer markers are kept. This is obviously also told
by the type string, but it seems a bit easier to read this way.

For any elements which are undocumented, a comment is inserted in sources.
"Undocumented" means that the expected element was not found. This might
require some adjustments if I missed some markup types.

Invocation is manual:
$ tools/update-dbus-docs.py tools/update-dbus-docs.py man/org.freedesktop.login1.xml
$ tools/update-dbus-docs.py tools/update-dbus-docs.py man/org.freedesktop.resolve1.xml
$ tools/update-dbus-docs.py tools/update-dbus-docs.py man/org.freedesktop.systemd1.xml
...

If some object is not found on the bus, the existing output is retained. So the
user needs to make sure that the appropriate objects have been instantiated
before calling this. We don't change the dbus interface very often, so I think
this manual mode is OK as a starting point. Making this fully automatic later
would be nice of course.

manager: add dbus parameter names

man: import org.freedesktop.systemd1(3) from the wiki

As usual, the formatting was fixed and various obvious updates
were done, but nothing major.

I removed documentation of snapshots and related methods though.

resolved: add dbus parameter names

man: import org.freedesktop.resolve1(3) from the wiki

resolved: tone down comment in /run/systemd/{stub-,}resolve.conf

There are legitimate reasons to access the file directly, as currently
discussed on fedora-devel. Hence tone things down from "must" to "should
typically not".

Also, let's use fputs() instead of fputs_unlocked() here,
fopen_temporary_label() turns off stdio locking anyway for the whole
FILE*, hence no need to do this manually each time.

Merge pull request #15448 from keszybz/logind-avoid-polkit-query

logind: avoid polkit query for SetWallMessage()

tree-wide: spellcheck using codespell

Fixes #15436.

network: use "FooOverUDP" as one word

The whole thing is one name, and I think it's confusing to break it
up into separate words.

network: fix static assertion on IPPROTO_MAX range

Builds with recent glibc would fail with:
../src/network/netdev/fou-tunnel.c: In function ‘config_parse_ip_protocol’:
../src/basic/macro.h:380:9: error: static assertion failed: "IPPROTO_MAX-1 <= UINT8_MAX"
  380 |         static_assert(expr, #expr)
      |         ^~~~~~~~~~~~~
../src/network/netdev/fou-tunnel.c:161:9: note: in expansion of macro ‘assert_cc’
  161 |         assert_cc(IPPROTO_MAX-1 <= UINT8_MAX);
      |         ^~~~~~~~~

This is because f9ac84f92f151e07586c55e14ed628d493a5929d (present in
glibc-2.31.9000-9.fc33.x86_64) added IPPROTO_MPTCP=262, following
v5.5-rc5-1002-gfaf391c382 in the kernel.

units: use correct path to refer to plymouth

This doesn't really matter, since in non-/usr-merged systems plymouth
needs to be in /bin and on merged ones it doesn't matter, but it is
still prettier to insert the right path, and avoid /bin on merged
systems, since it's just a compat symlink.

Replaces: #15351

watchdog: reduce watchdog pings in timeout interval

The watchdog ping is performed for every iteration of manager event
loop. This results in a lot of ioctls on watchdog device driver
especially during boot or if services are aggressively using sd_notify.
Depending on the watchdog device driver this may have performance
impact on embedded systems.
The patch skips sending the watchdog to device driver if the ping is
requested before half of the watchdog timeout.

Merge pull request #15437 from keszybz/man-nss-resolve

Move "files" and "dns" later in the recommended hosts: lines

Merge pull request #15441 from sartura/src/analyze

systemd-analyze: Add new meson configure option for controlling systemd-analyze support

logind: skip polkit query with --no-wall

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1800875.

loginctl: adjust indentation

Support journal-upload HTTPS without key and certificate

man: move "myhostname" right after "resolve"

The text in the man page provides the justification why I think this is
generally the right thing. An additional reason is that with the previous
commit (to move resolved earlier), since resolved internally implements the
same rules that nss-myhostname does, we'd have this strange inversion where
the priority of external configuration would be different in the "resolve"
path and in the fallback path.