Merge pull request #1419 from keszybz/shell-completion

Shell completion tweaks

Merge pull request #1424 from evverx/journalctl-rotate

journalctl: add --rotate option

man: journalctl: add info about --rotate

shell-completion: journalctl: add --rotate

journalctl: add --rotate option

shortcut for `systemctl kill --kill-who main --signal SIGUSR2 systemd-journald`

Merge pull request #1421 from keszybz/formatting-fixes

logind: avoid a dot before parenthesis

Merge pull request #1420 from keszybz/doc-updates

Doc updates

NEWS: typos

man: update description of exit.target

exit.target is now used for both system and user sessions,
so remove "on user service manager exit". Also reword that
paragraph: services will be killed before the manager exits,
even if they do not conflict with shutdown target, but we
recommend that they conflict with shutdown target so that
systemd schedules them to be stopped immediately when starting
to exit.

In the first paragraph, containers should be mentioned last,
and the more general systems first.

logind: avoid a dot before parenthesis

systemd-logind[27]: System is rebooting. (Applied kernel updates.)

is changed to

systemd-logind[27]: System is rebooting (Applied kernel updates).

Users should not add a dot in the sentence in --message, i.e. the correct usage is now:
$ systemctl reboot --message "Applied kernel updates"

man: add link to kernel docs for net_cls

man: use "=" when referring to configuration file settings

This convention is almost universal in systemd man pages, and makes
it easier to visually parse the docs.

Also fix some markup along the way.

man: add link to functionfs docs

Also reword some ungainly grammar constructs.

Merge pull request #1415 from aakoskin/bootchart-v2

[v2] bootchart: don't fail if "model name" is missing from cpuinfo

Merge pull request #1418 from poettering/base-fs

base-fs: consider inability to create /root non-fatal

Merge pull request #1417 from poettering/nspawn-and-more

Various fixes, primarily for nspawn, but other stuff too

base-fs: consider inability to create /root non-fatal

Fixes #1414

Merge pull request #1394 from steelman/fsck-force-reboot

fsck: start reboot.target irreversibly

analyze: add new set-log-target subcommand

We already have the property writable, hence let's add a command to set
it.

log: properly return -EINVAL from log_set_max_level_from_string()

If we just return the value we got from log_level_from_string() on
failure we'll return -1, which is not a proper error code.

log_set_target_from_string() did get this right already, hence let's fix
this here too.

nspawn: mount /sys as tmpfs, and then mount only select subdirs of the real sysfs below it

This way we can hide things like /sys/firmware or /sys/hypervisor from
the container, while keeping the device tree around.

While this is a security benefit in itself it also allows us to fix
issue #1277.

Previously we'd mount /sys before creating the user namespace, in order
to be able to mount /sys/fs/cgroup/* beneath it (which resides in it),
which we can only mount outside of the user namespace. To ensure that
the user namespace owns the network namespace we'd set up the network
namespace at the same time as the user namespace. Thus, we'd still see
the /sys/class/net/ from the originating network namespace, even though
we are in our own network namespace now. With this patch, /sys is
mounted before transitioning into the user namespace as tmpfs, so that
we can also mount /sys/fs/cgroup/* into it this early. The directories
such as /sys/class/ are then later added in from the real sysfs from
inside the network and user namespace so that they actually show whatis
available in it.

Fixes #1277

bootchart: use get_proc_field and PROC_CPUINFO_MODEL

Use get_proc_field and PROC_CPUINFO_MODEL to get the CPU type. Also don't
fail if it missing, instead restore the old behaviour and print "Unknown".

Fixes the following error seen with v225 on powerpc:

Unable to read module name from cpuinfo.
Error generating svg file: No such file or directory

architecture.h: define PROC_CPUINFO_MODEL

Define PROC_CPUINFO_MODEL to indicate the CPU model field in /proc/cpuinfo.

fileio: make get_status_field() more generic

All users of get_status_field() expect the field pattern to occur in
the beginning of a line, and the delimiter is ':'.

Hardcode this into the function, and also skip any whitespace before ':'
to support fields in files like /proc/cpuinfo. Add support for returning
the full field value (currently stops on first whitespace).

Rename the function so it's easier to ensure all callers switch to new
semantics.

fix typos in NEWS

nspawn: fix user namespace support

We didn#t actually pass ownership of /run to the UID in the container
since some releases, let's fix that.

update NEWS

update NEWS

Add Coccinelle patch for strempty() usage

nspawn: order includes

nspawn: make sure mount_legacy_cgroup_hierarchy() can deal with NULL root directories

update TODO

systemctl: when reading legacy -t argument for shutdown, don't drop following parameter

We currently completely ignore the following parameter, but we really
should not, as that is actually the time to shut down on.

systemctl: various modernizations

Merge pull request #1412 from zonque/strempty

tree-wide: use strempty() where possible

tree-wide: use strempty() where possible

Also add a Coccinell patch to detect such locations in the future.

Merge pull request #1408 from poettering/systemctl-and-more

Systemctl and more

systemctl: when a shutdown is scheduled, always go via logind

systemctl: minor modernizations

systemctl: port verb dispatching to generic verbs.[ch] code

Let's parse the command line the same way in our various tools.

systemctl: prefer shutting down through logind even as root

Otherwise, wall messages will not be generated for root.

systemctl: split out code invoking SetExitCode() into function of its own

pager: minor modernizations

systemctl: don't assert on the arguments array unnecessarily

In most verbs it's fine to treat the arguments array being NULL is empty
array, hence do so.

systemctl: make use of strv_skip() where appropriate

systemctl: make use of log_error_errno() where we can

Also adds a couple of "return" statements, where they are missing.

systemctl: rework how we connect to busses

Sometimes we have to connect to the system manager directly (early boot,
initrd, late boot, ...), sometimes through the system bus (unprivileged,
remote, logind, ...). Instead of guessing in advance, which kind of
connection we require (and sometimes guessing incorrectly), let's make
sure each time we need  bus connection we request the right bus
explicitly.

This way, we set up exactly the bus connections require, never guess
incorrectly, and do so only immediately when necessary.

As effect this reworks avoid_bus() into install_client_side(), since
that's all it determines now: whether to install unit files client-side
or server-side (i.e. in PID 1).

core: add a "Requires=" dependency between units and the slices they are located in

We place the processes we fork off in the cgroup anyway, and we probably
shouldn't be able to get that far if we couldn't set up the slice due to
resource problems or unmet conditions. Hence upgrade the dependency
between units and the slices they are located in from Wants= to
Requires=.

update-utmp: flush and close the bus used for connections after use

update-utmp: minor modernizations

bus-util: rename bus_open_transport() to bus_connect_transport()

In sd-bus, the sd_bus_open_xyz() family of calls allocates a new bus,
while sd_bus_default_xyz() family tries to reuse the thread's default
bus. bus_open_transport() sometimes internally uses the former,
sometimes the latter family, but suggests it only calls the former via
its name. Hence, let's avoid this confusion, and generically rename the
call to bus_connect_transport().

Similar for all related calls.

And while we are at it, also change cgls + cgtop to do direct systemd
connections where possible, since all they do is talk to systemd itself.

pager: minor simplification

systemctl: rename parse_time_spec() to parse_shutdown_time_spec()

Let's clarify that this function is specific to shutdown time
specifications, nothing else.

systemctl: conditionalize /dev/initctl fallback on HAVE_SYSV_COMPAT

systemctl: allocate arg_wall only on the heap

Previously, we'd allocate it sometimes from the heap, but otherwise let
it point directly int argv[]. Let's clean this up, so that we know
exactly how to release its resources, and do so at the end.

systemctl: add 'const' where appropriate

systemctl: remove client-side wall message support

logind sends out wall messages now, let's remove this from the systemctl
client side hence. If people build systemd without logind support they
won't get wall messages now, but that's OK.

systemctl: move strv_skip_first() out of systemctl.c

Make it generic, call it strv_skip() and move it to strv.[ch]

systemctl: split out code that schedules shutdowsn into its own function

systemctl: rename all logind-specific functions to logind_xyz()

systemctl: move shutdown cancelling code into its own function

Let's make sure the main switch statement only invokes functions, but
doesn't do anything real on its own.

systemctl: don't special case ACTION_RUNLEVEL anymore

Let's move its dispatching to the main switch statement.

systemctl: introduce a single function to set the wall message

Let's not have the same code three times, but reduce it to one function.

core: rework crash handling

This introduces a new systemd.crash_reboot=1 kernel command line option
that triggers a reboot after crashing.

This also cleans up crash VT handling. Specifically, it cleans up the
configuration setting, to be between 1..63 or a boolean. This is to
replace the previous logic where "-1" meant disabled. We continue to
accept that setting, but only document the boolean syntax instead.

This also brings the documentation of the default settings in sync with
what actually happens.

The CrashChVT= configuration file setting is renamed to CrashChangeVT=,
following our usual logic of not abbreviating unnecessarily. The old
setting stays support for compat reasons.

Fixes #1300

socket: rename ffs_xyz to usbffs_xyz

The relation to USB should really be clearer.

core: turn a large if statement into a switch

selinux: add _cleanup_ concepts to SELinux label allocation

core: simplify how we create socket fds

Let's always return the allocated fds as return values where possible,
and make more use of _cleanup_close_

core: small error message wording improvements

core: properly use the ExitCode bus property when exiting is --user instance

core: allow setting WorkingDirectory= to the special value ~

If set to ~ the working directory is set to the home directory of the
user configured in User=.

This change also exposes the existing switch for the working directory
that allowed making missing working directories non-fatal.

This also changes "machinectl shell" to make use of this to ensure that
the invoked shell is by default in the user's home directory.

Fixes #1268.

man: also properly indent our examples to 8ch

detect-virt: various modernizations

journal-cat: various modernizations

journal-cat: return a correct error, not -1

machine-id-commit: merge machine-id-commit functionality into machine-id-setup

And remove machine-id-commit as separate binary.

There's really no point in keeping this separate, as the sources are
pretty much identical, and have pretty identical interfaces. Let's unify
this in one binary.

Given that machine-id-commit was a private binary of systemd (shipped in
/usr/lib/) removing the tool is not an API break.

While we are at it, improve the documentation of the command substantially.

units: increase watchdog timeout to 3min for all our services

Apparently, disk IO issues are more frequent than we hope, and 1min
waiting for disk IO happens, so let's increase the watchdog timeout a
bit, for all our services.

See #1353 for an example where this triggers.

sd-daemon: Simplify sd_booted() check

We can just use access() to check whether /run/system/system/ is a
directory, no need to involve stat(). The trick is to suffix the path
name with a dash.

systemctl: make "systemctl is-system-running" return "offline" if we are not booted with systemd

This sounds like the better place to expose this than in "systemd-notify
--booted".

Also document the so far undocumented "unknown" state the command might
return. And rearrange the table of states documented to be more like the
one for "is-running".

Also, don't document the precise exit code of this function, just say
errors are reported != 0 or > 0...

Merge pull request #1398 from evverx/man-systemd-journald

man: journald: add commands for /var/log/journal

Merge pull request #1407 from dvdhrm/prioq

prioq: drop stability guarantee

tty-ask-password: minor modernizations

util: introduce common version() implementation and use it everywhere

This also allows us to drop build.h from a ton of files, hence do so.
Since we touched the #includes of those files, let's order them properly
according to CODING_STYLE.

util: unify implementation of NOP signal handler

This is highly complex code after all, we really should make sure to
only keep one implementation of this extremely difficult function
around.

core: remove set_default_unit()

The new free_and_strdup() call does pretty much the same thing these
days, no need to keep a private limited purpose version around.

core: modernize prepare_reexecute()

core: exit early if we hit OOM

core: use %m rather than strerror() where we can

main: minor clean-ups

Add (void) casting for a couple of functions where we knowingly ignore
the returning error code.

Use EXIT_FAILURE where appropriate.

Try to initialize structures at declaration time, or at once.

tree-wide: take benefit of the fact that fdset_free() returns NULL

core: order #includes in main.c

strv: add strv_free_free() to strv.c and make use of it

Let's teach it a new trick, and make it return NULL.

tree-wide: port more code to use send_one_fd() and receive_one_fd()

Also, make it slightly more powerful, by accepting a flags argument, and
make it safe for handling if more than one cmsg attribute happens to be
attached.

core: sort includes of manager.[ch] according to CODING_STYLE

sd-event: don't provide priority stability

Currently, we guarantee that if two event-sources with the same priority
fire at the same time, they're always dispatched in the same order. While
this might sound nice in theory, there's is little benefit in providing
stability on that level. We have no control over the order the events are
reported, hence, we cannot guarantee that we get notified about both at
the same time.

By dropping the stability guarantee, we loose roughly 10% Heap swaps in
the prioq on a desktop cold-boot. Krzysztof Kotlenga even reported up to
20% on his tests. This sounds worth optimizing, so drop the stability
guarantee.

prioq: never shuffle identical entries

Skip shuffling identical entries in shuffle_up(), just like we already do
in shuffle_down().

Merge pull request #1403 from dvdhrm/prioq-comment

prioq: add introduction comment

prioq: add introduction comment

Add comment to prioq.c explaining what it does. And more importantly,
mention that we implement a Heap. It's more than annoying having to
figure out what the code actually does, without ever mentioning the word
'heap'.

bash-completion: remove one more sort and sub-shell

{} is preferred to (), because the first is just grouping, while the
second invokes a separate shell.

bash-completion: use builtins when generating list