core: remove one {}

Merge pull request #13225 from mrc0mmand/networkd-test-replace-adduser-with-useradd

test: use `useradd` instead of `adduser`

test: print an error when networkctl returns an unexpected EC

If networkctl crashes, like recently with SIGABRT, it returns absolutely
no output, which may be confusing during debugging. Help it a little
with a short informative message.

analyze-security: move assert above first use of the variable (#13238)

Merge pull request #13239 from poettering/coverity-fixes

four coverity fixes

man: reword DeviceAllow= documentation

Don't claim we'd use cgroup.deny much. It's just a way to remove stuff
from device lists, which is nothing we allow users to explicitly
configure.

Also, extend documentation when wildcards may be used, and when not.

job: fix coverity issue

Fixes coverity issue 1403550

test-chase-symlinks: fix coverity issue

Fixes coverity issue 1403708

unit-file: fix coverity issue

Fixes coverity issue 1403820

cryptsetup-generator: fix coverity issue

Fixes coverity issue 1403772

NEWS: Remove DisableControllers from v243

We already released this in v240 and had a NEWS entry then.

test-execute: skip test_exec_systemcallfilter_system under ASan

This particular test case keeps intermittently failing due to crashing
LSan  when running under clang+ASan. Generally, sanitizers don't
like seccomp filters, so the best option here is to just switch this
test off for this scenario.

analyze: declare dump_exit_status outside of HAVE_SECCOMP block

Fixes: 76ed04d936f757763c32db5dbaaebd8b13785d7b
Closes: https://github.com/systemd/systemd/issues/13230

po: update Polish translation

test: use `useradd` instead of `adduser`

`adduser` is in certain cases a standalone package which provides a
better user experience. In other cases it's just a symlink to `useradd`.
And some distributions don't have `adduser` at all, like Arch Linux.
Let's use the `useradd` binary instead, which should provide the same
functionality everywhere.

boot: fix build with gnu-efi older than 3.0.5

Merge pull request #13119 from keszybz/unit-loading-2

Rework unit loading to take into account all aliases

NEWS: more entries and some rewordings

bash-completion: support "systemd-analyze exit-status"

Merge pull request #13189 from yuwata/network-initrd-generator-unit

unit: update initrd-network-generator.service

networkctl: TABLE_BPS requires uint64_t

test-unit-file: allow printing of information about specific units

Useful for manual debugging.

pid1: drop unit caches only based on mtime

v2:
- do not watch mtime of transient and generated dirs

  We'd reload the map after every transient unit we created, which we don't
  need to do, since we create those units ourselves and know their fragment
  path.

analyze: add "unit-files" to dump the unit fragment map

I'm not convinced that this is useful enough to be included... But it is
certainly nice when debugging.

pid1: use a cache for all unit aliases

This reworks how we load units from disk. Instead of chasing symlinks every
time we are asked to load a unit by name, we slurp all symlinks from disk
and build two hashmaps:
1. from unit name to either alias target, or fragment on disk
   (if an alias, we put just the target name in the hashmap, if a fragment
    we put an absolute path, so we can distinguish both).
2. from a unit name to all aliases

Reading all this data can be pretty costly (40 ms) on my machine, so we keep it
around for reuse.

The advantage is that we can reliably know what all the aliases of a given unit
are. This means we can reliably load dropins under all names. This fixes #11972.

shared/unit-file: add a function to validate unit alias symlinks

It turns out most possible symlinks are invalid, because the type has to match,
and template units can only be linked to template units.

I'm not sure if the existing code made the same checks consistently. At least
I don't see the same rules expressed in a single place.

TEST-15-DROPIN: add test for details of unit aliasing

I adjusted the tests to pass. I don't think the behaviour makes much sense,
even if we ignore the issue with "lazy loading" of aliases. E.g. in the
last section, the fact that dropins for yup@.service and yup@3.service are
not loaded seems to be a plain old bug.

test: use "ln -fs"

Without this, repeated runs of "make -C TEST/... setup" fail when trying
to create the symlink.

docs: fix typo

Merge pull request #13219 from poettering/named-exit-codes-tweaks

quick follow-up for the symbolic exit status PR #13207

update TODO

Merge pull request #13216 from poettering/busctl-format-table

port "busctl list" to format-table.h

Merge pull request #13217 from poettering/TODO-updates

udev: warn if rules files are executable

shared/install: typo

Merge pull request #13004 from shinygold/master

cryptsetup: Improve support for keyfiles on removable storage

analyze: clarify that 'exit-status' optionally takes arguments

Merge pull request #13201 from yuwata/networkctl-merge-table_add_cell

networkctl cleanups and improvements

unit: rename initrd-network-generator.service -> systemd-network-generator.service

network-generator: rename generated unit files

The tool may be used in the initrd-less systems.

logind: notify dbus of changes to session State

This adds the option to monitor transitions between the state Opening ->
Active.

man: remove trailing space in link in HTML output

analyze: right-align exit status column

analyze: rename "exit-codes" to "exit-status"

waitid(2) and the libc function signature calls this "exit status", and
uses "exit code" for something different. Let's stick to the same
nomenclature hence.

analyze: always use "int" to display exit status values

That's what it actually is, hence let's use that and cast when the type
is bigger.

exit-status: rename EXIT_STATUS_GLIBC → EXIT_STATUS_LIBC

After all these two exit codes are defined by ISO C as part of the C
library, and it's not the GNU implementation defines them.

Merge pull request #13207 from keszybz/symbolic-exit-code-names

Symbolic exit code names

update TODO

TODO: drop udev access mode item we just implemented

udev: warn on rules files with weird access modes

TODO: remove item implemented in f05e1ae6660

pid1: use LOG_DEBUG/INFO/NOTICE for unit resource consumption message

We now log at LOG_INFO for any unit. Let's vary the log level
a bit, so that for normal short lived-units (less than 1 sec CPU),
we only log if debugging is enabled.

busctl: port "busctl list" to format_table.h

Among generally being prettier this gives us JSON output for basically
free.

format-table: add TABLE_PID cell type

format-table: automatically show empty cells in grey

format-table: add table_fill_empty() to fill in empty cells until the specified column is reached

format-table: if NULL is spcified as data, let's patch to an empty cell

This should make various calls easier that currently generate either an
empty cell or a regular cell depending on whether they have data to
show.

format-table: optionally show a specific string in empty cells

For some cases it might make sense to show "-" instead of just spaces
for empty cells.

test-network: add tests for new entries in "networkctl status"

networkctl : Add support to display vxlan properties

networkctl: Add support to display bridge properties

time-util: introduce jiffies_to_usec()

table: add missing NULL initialization

table: add TABLE_UINT8 or friends

table: add TABLE_IN_ADDR and TABLE_IN6_ADDR

networkctl: merge multiple table_add_cell() by using table_add_many()

analyze: dot(1) man page link should be urlified

I mean, let's not miss out on this excellent opportunity to use
hyperlinks on terminals.

(Unfortunately not see unless you invoke 'systemd-analyze --no-pager
--help', because 'less' is so much stuck in the past :-(.)

Merge pull request #13212 from poettering/sleep-conf-default

sleep: document default HibernateDelaySec=

NEWS: add entry about exit status changes

analyze: add exit-codes verb

units: use symbolic exit code names

(nspawn uses 133 which doesn't have a name. That's reasonable, because
there's less chance of conflict with a return value from the payload.)

systemctl: do print all statuses/signals received from pid1

If for some reason we do not know some signal, instead of silently
skipping it, let's print it numerically. Likewise, 'show' is not the
right place to do value filtering for exit codes. If pid1 accepted it,
let's just print it with no fuss.

pid1: fix message about triggers missing services

systemd[1]: systemd-tmpfiles-clean.timer: Refusing to start, unit systemd-tmpfiles-cle
an.timer to trigger not loaded.

pid1,systemctl: allow symbolic exit code names

shared/bus-util: fix dbus serialization of {RestartPrevent,RestartForce,Success}ExitStatus

We were passing 1/4th of the size in bytes as argument. So depending
on the size of the array, either we'd only transfer a subset of values,
or we'd get an alignment error.

bus-util: convert bus_log_{parse,create}_error into defines

With SYSTEMD_LOG_LOCATION=1, it is much more useful to see the location
where the call to bus_log_{parse,create}_error() was made, rather then
the one-line body of the helper function. Also, it's our internal code,
so having a one-line non-inline function doesn't make much sense anyway.

shared/exit-status: use Bitmap instead of Sets

I opted to embed the Bitmap structure directly in the ExitStatusSet.
This means that memory usage is a bit higher for units which don't define
this setting:

Service changes:
        /* size: 2720, cachelines: 43, members: 73 */
        /* sum members: 2680, holes: 9, sum holes: 39 */
        /* sum bitfield members: 7 bits, bit holes: 1, sum bit holes: 1 bits */
        /* last cacheline: 32 bytes */

        /* size: 2816, cachelines: 44, members: 73 */
        /* sum members: 2776, holes: 9, sum holes: 39 */
        /* sum bitfield members: 7 bits, bit holes: 1, sum bit holes: 1 bits */

But this way the code is simpler and we do less pointer chasing.

shared/bitmap: constify various operators which don't modify bitmap

shared/exit-status: add exit_status_from_string()

shared/exit-status: turn status level into a bitmask, add "test"

The "test" doesn't really test much automatically, but it is still useful
to look at the mappings.

Merge pull request #13209 from poettering/nspawn-volatile-merged-usr

make incompatibility of non-/usr-merged distros with --volatile=yes more discoverable

Merge pull request #13184 from poettering/nspawn-usr-root

Allow "systemd-nspawn -D / --volatile=yes" to work

systemd-sleep: use swaps in priority order

In situations where hibernation is requested but resume= and
resume_offset= kernel parameters are not configured, systemd
will attempt to locate a suitable swap location by inspecting
/proc/swaps. This change will use the first suitable swap with
the highest configured priority.

man: document default HibernateDelaySec= setting

update TODO

man: document that --volatile=yes is not supported for split /usr systems

nspawn: print an explanatory error when people try to use --volatile=yes on distros that are not /usr-merged

more 243 news

man: document the kernel command line options to use when logging to kmsg

Prompted by:

https://lists.freedesktop.org/archives/systemd-devel/2019-July/043149.html

nspawn: when operating on the host image, let's move the root to a different directory first, via a bind mount

shared: allow LOCK_SH locks on the host root in OS images

See the add comments for the justification.

nspawn: always take exclusive locks of ephemeral OS tree copies

nspawn: don't look for .nspawn file above the top-level directory, it makes no sense

nspawn: allow --volatile=yes instances of -D /

news corrections and improvements (#13200)

* missing whitespace.

* NEWS: some small fixes (?) and improvements (???).

* a number of small corrections and (hopefully) improvements

Merge pull request #13203 from yuwata/update-ja-po

po: include network1.policy and update ja.po

catalog: update Polish translation

Update Touchpad for g6

po: update ja.po

po: include network1.policy in the list for generating .pot file

Merge pull request #13120 from keszybz/journal-min-use

Increase journald min_use value