core: don't unset reload result, unless we begin a start or reload operation

Much like the result of the service itself we should not reset the
reload result unless we actually start from the beginning, so that
clients can query it at any time.

Specifically, let's reset the result states only when we begin with a
start operation (for both the main result, and the reload result), when
we begin with a reload operation (only for the load result), or when the
use explicitly asks for that via "systemctl reset-failed".

This is a more generic fix for #1447.

Fixes #1447.

man: include numeric prefixes in example file names for .link, .netdev, .network files

In order to avoid confusion with the default files we ship, let's use a
low prefix number for all examples.

Fixes #1409.

man: fix journalctl --syslog-identifier= documentation

We don't actually accept patterns, hence don't claim so. This is a
fix-up for 730836403aee5f5bb998e6e3622ea7068fce0699.

Fixes #1256.

man: document that the automatic journal limits are capped to 4G by default

Fixes #1441.

nspawn: fix --image= when nspawn is run as service

nspawn needs access to /dev/loop to implement --image=, hence grant that
in the service file.

Fixes #1446.

journal: rework vacuuming logic

Implement a maximum limit on number of journal files to keep around.
Enforcing a limit is useful on this since our performance when viewing
pays a heavy penalty for each journal file to interleve. This setting is
turned on now by default, and set to 100.

Also, actully implement what 348ced909724a1331b85d57aede80a102a00e428
promised: use whatever we find on disk at startup as lower bound on how
much disk space we can use. That commit introduced some provisions to
implement this, but actually never did.

This also adds "journalctl --vacuum-files=" to vacuum files on disk by
their number explicitly.

journal: use automatic clenup for ACL types

journal: improve some messages

Indicate that we are ignoring errors, when we ignore them, and log that
at LOG_WARNING level.

Use the right error code for the log message.

journal: simplify things by using the LESS_BY() macro

journal: don't affect atime of journal files when vacuuming

Let's try to use O_NOATIME if we can when vacuuming old journal files,
if we have the permissions for it, so that vacuuming doesn't count as
proper journal read access.

journal: prefer stack allocation

journal: make journal_file_close() return NULL

The way it is customary everywhere else in our sources.

journal: port over to fd_getcrtime_at()

Let's use fd_getcrtime_at(), since that *at() family of calls is how we
read the rest of the file metadata, too.

util: rework fgetxattrat_fake() to use O_PATH

That way, we don't ever open the file, thus leave the atime untouched,
and this works even when unprivileged.

Merge pull request #1438 from poettering/rfkill-rework

An rfkill rework among other things

Merge pull request #1430 from evverx/driver-syslog

journald: add syslog fields for driver messages

update NEWS

rfkill: rework and make it listen on /dev/rfkill

With this rework we introduce systemd-rfkill.service as singleton that
is activated via systemd-rfkill.socket that listens on /dev/rfkill. That
way, we get notified each time a new rfkill device shows up or changes
state, in which case we restore and save its current setting to disk.

This is nicer than the previous logic, as this means we save/restore
state even of rfkill devices that are around only intermittently, and
save/restore the state even if the system is shutdown abruptly instead
of cleanly.

This implements what I suggested in #1019 and obsoletes it.

core: add new setting Writable= to ListenSpecial= socket units

Writable= is a new boolean setting. If ture, then ListenSpecial= will
open the specified path in O_RDWR mode, rather than just O_RDONLY.

This is useful for implementing services like rfkill, where /dev/rfkill
is more useful when opened in write mode, if we want to not only save
but also restore its state.

systemctl: fix how we decode types and states

There was a bad memory access among other smaller issues.

man: minor wording improvements to USB FunctionFS documentation

fix typo in NEWS

Merge pull request #1434 from reverendhomer/patch-1

systemctl: fix memory leak in systemctl_parse_argv

Merge pull request #1433 from michich/log-errno

tree-wide: add missing errno arguments to log_*_errno()

udev: fix format string after conversion to log_*_errno()

Use %m where previously %s was used together with strerrno().

Fixes: e53fc357a9b "tree-wide: remove a number of invocations of
       strerror() and replace by %m"

sd-device: fix format strings after conversion to log_*_errno()

Use %m where previously %s was used together with strerrno().

Fixes: e53fc357a9b "tree-wide: remove a number of invocations of
       strerror() and replace by %m"

systemctl: fix memory leak in systemctl_parse_argv

This commit fixes Coverity #1325228

tree-wide: add missing errno arguments to log_*_errno()

A few of the recent conversions to log_*_errno() were missing the errno
value arguments.

Fixes: e53fc357a9b "tree-wide: remove a number of invocations of
       strerror() and replace by %m"

Merge pull request #1429 from poettering/dhcp-hostname

DHCP and journal remoting fix, as well as NEWS update

Merge pull request #1427 from dbuch/master

sd-pppoe: use strna() instead of strempty()

Merge pull request #1426 from poettering/log-syntax

logging fixes and more

journald: add syslog fields for driver messages

journal: make sure to set MHD_USE_PIPE_FOR_SHUTDOWN for libmicrohttpd servers

This makes sure libmicrohttpd won't call shutdown() on our listening
sockets, which make sure socket activation and re-activation will work
cleanly.

See:

https://github.com/systemd/systemd/pull/1286
https://lists.gnu.org/archive/html/libmicrohttpd/2015-09/msg00014.html

Fixes #1286

dhcp: make sure we can deal with a single trailing NUL byte in the hostname

Fixes #1337

update NEWS

sd-pppoe: use strna() instead of strempty()

In cases where we use for instance. log_debug() with a statement + string
it is better to output (Service-name: N/A) instead of (Service-name: )

Merge pull request #1425 from kaysievers/wip

gpt-auto-generator: check fstab for /boot entries

Merge pull request #1422 from keszybz/sd-daemon-mq-badf

sd-daemon: fix return value for sd_is_mq

Merge pull request #1419 from keszybz/shell-completion

Shell completion tweaks

core: make sure we expose the empty string when we detect no virtualization

On the bus, stick to exposing the empty string as "no virtualization",
instead of none.

Fixes #1423

tree-wide: remove a number of invocations of strerror() and replace by %m

Let's clean up our tree a bit, and reduce invocations of the
thread-unsafe strerror() by replacing it with printf()'s %m specifier.

basic: split out cpu set specific APIs into cpu-set-util.[ch]

conf-parser: make use of DEFINE_PARSER macro to parse mode_t

Let's unify more code.

Also, rework the macro to accept a trailing semicolon, to make the code
prettier.

util: there cannot be trailing garbage when parsing cpu sets

extract_first() already skips trailing whitespace, hence no reason to
explicitly check for it.

util: rename parse_cpu_set() to parse_cpu_set_and_warn()

It's pretty untypical for our parsing functions to log on their own.
Clarify in the name that this one does.

log: move log_invalid_utf8() to log.h

Also, make sure it follows the same scheme as log_syntax() does in its
behaviour.

tree-wide: clean up log_syntax() usage

- Rely everywhere that we use abs() on the error code passed in anyway,
  thus don't need to explicitly negate what we pass in

- Never attach synthetic error number information to log messages. Only
  log about errors we *receive* with the error number we got there,
  don't log any synthetic error, that don#t even propagate, but just eat
  up.

- Be more careful with attaching exactly the error we get, instead of
  errno or unrelated errors randomly.

- Fix one occasion where the error number and line number got swapped.

- Make sure we never tape over OOM issues, or inability to resolve
  specifiers

Merge pull request #1424 from evverx/journalctl-rotate

journalctl: add --rotate option

gpt-auto-generator: check fstab for /boot entries

We need to prevent the creation of the gpt automount unit, which will not
get overridden by the fstab mount unit.

https://github.com/systemd/systemd/issues/1378

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`

sd-daemon: fix sd_is_mq for non-mq fds

mq_getattr returns -1/EBADF for file descriptors which are not mq.
But we should return 0 in this case.

We first check that fd is a valid fd, so we can assume that if
mq_getattr returns EBADF, it is simply a non-mq fd. There is a slight
race, but there doesn't seem to be a nice way to fix it.

Merge pull request #1421 from keszybz/formatting-fixes

logind: avoid a dot before parenthesis

Merge pull request #1420 from keszybz/doc-updates

Doc updates

systemctl: do not set variable to be immediately overwritten

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.