coredump: rework coredumping logic

This reworks the coredumping logic so that the coredump handler invoked from the kernel only collects runtime data
about the crashed process, and then submits it for processing to a socket-activate coredump service, which extracts a
stacktrace and writes the coredump to disk.

This has a number of benefits: the disk IO and stack trace generation may take a substantial amount of resources, and
hence should better be managed by PID 1, so that resource management applies. This patch uses RuntimeMaxSec=, Nice=, OOMScoreAdjust=
and various sandboxing settings to ensure that the coredump handler doesn't take away unbounded resources from normally
priorized processes.

This logic is also nice since this makes sure the coredump processing and storage is delayed correctly until
/var/systemd/coredump is mounted and writable.

Fixes: #2286

activate: add a new switch --inetd to enable inetd-style socket activation

Previously, using --accept would enable inetd-style socket activation in addition to per-connection operation. This is
now split into two switches: --accept only switches between per-connection or single-instance operation. --inetd
switches between inetd-style or new-style fd passing.

This breaks the interface of the tool, but given that it is a debugging tool shipped in /usr/lib/systemd/ it's not
really a public interface.

This change allows testing new-style per-connection daemons.

activate: minor fixes

activate: add new --seqpacket switch for testing SOCK_SEQPACKET sockets

resolved: include inttypes.h in resolved-def.h given that we use UINT64_C

build-sys: move coredump logic into subdir of its own

Merge pull request #2569 from zonque/removals

Remove some old cruft

Merge pull request #2571 from martinpitt/master

NEWS: Various clarification and grammar fixes

Merge pull request #2570 from zonque/build

configure.ac: Fix help text of --enable-* options

configure.ac: Fix help text of --enable-* options

Fixes #2567

NEWS: Various clarification and grammar fixes

tree-wide: remove Emacs lines from all files

This should be handled fine now by .dir-locals.el, so need to carry that
stuff in every file.

Merge pull request #2564 from poettering/fix-2467

Fix for #2467

Makefile.am: remove some relicts

util-lib: make sure usec_sub() doesn't degrade USEC_INFINITY

As suggested in: https://github.com/systemd/systemd/pull/2542#issuecomment-181877820

core: make the StartLimitXYZ= settings generic and apply to any kind of unit, not just services

This moves the StartLimitBurst=, StartLimitInterval=, StartLimitAction=, RebootArgument= from the [Service] section
into the [Unit] section of unit files, and thus support it in all unit types, not just in services.

This way we can enforce the start limit much earlier, in particular before testing the unit conditions, so that
repeated start-up failure due to failed conditions is also considered for the start limit logic.

For compatibility the four options may also be configured in the [Service] section still, but we only document them in
their new section [Unit].

This also renamed the socket unit failure code "service-failed-permanent" into "service-start-limit-hit" to express
more clearly what it is about, after all it's only triggered through the start limit being hit.

Finally, the code in busname_trigger_notify() and socket_trigger_notify() is altered to become more alike.

Fixes: #2467

Merge pull request #2568 from poettering/prepare-229

edit config updates, and a new NEWS file

Merge pull request #2563 from 0xAX/timespec_load_nsec-static

time-util: make timespec_load_nsec() static

time-util: make timespec_load_nsec() static

The timespec_load_nsec() function has no callers outside of the
time-util.c, so we can make it static.

NEWS: start putting together a NEWS file for 229

Totally incomplete, but let's get this started.

editors: only extend line width to 119 for C and XML files

For all other files leave the line width at 79 as before. This is a good idea
since we generally don't want text files such as catalog files, unit files or
README/NEWS files to be line-broken at 119 since they are regularly browsed on
text terminals.

While we are at it, also add a couple of comments to the various files.

(Note that .editorconfig doesn't carry line-width information, simply because
the specification doesn't know the concept.)

Merge pull request #2565 from poettering/fix-2315

core: clarify which unit file is masked in error message

After all, the masked unit file error might be returned when enqueuing a unit that is not masked but requires a masked
unit. In this case it should really be clear which unit is meant here.

man: link resolved bus API documentation from its man page

This is similar how we already do it for logind or machined.

units: downgrade dependency on /tmp in basic.target to Wants=

Now that requiring of a masked unit results in failure again, downgrade the dependency on /tmp to Wants= again, so that
our suggested way to disable /tmp-on-tmpfs by masking doesn't result in a failing boot.

References: #2315

core: change internal error code for masked units from EBADR to ESHUTDOWN

This commit changes the mapping of the BUS_ERROR_UNIT_MASKED error to ESHUTDOWN. This error is used whenever the
transaction engine is asked to operate on a masked unit. ESHUTDOWN is what is used for the similar case when the unit
file enable/disable logic hits a masked unit file, hence is a natural candidate to be used here too.

Background: before this patch both "job type not applicable" and "unit masked" where mapped to EBADR, which
transaction_add_job_and_dependencies() then checked for. It actually wanted to check exclusively for the former error
condition, not the latter but due to the same mapping this failed to work.

This patch semi-undoes an accidental change made in caffa4ef700fdd0eadd6c0b2ef9925611672a1bc, however restores the
error number to ESHUTDOWN instead of the original ENOSYS (for the reasons indicated above).

To make this easier to grok for the future, I added comments to explaining which error conditions are checked for.

Fixes: #2315

Merge pull request #2561 from msekletar/virtio-blk-by-path

path_id: reintroduce by-path links for virtio block devices

Merge pull request #2559 from chaloulo/drop_monotonicity_check_from_journal_file

journal: Drop monotonicity check when appending to journal file

Merge pull request #2557 from whot/hwdb-updates

hwdb: add Dell Lattitude E6320 pointing stick speed fix

path_id: reintroduce by-path links for virtio block devices

Enumeration of virtio buses is global and hence
non-deterministic. However, we are guaranteed there is never going to be
more than one virtio bus per parent PCI device. While populating
ID_PATH we simply skip virtio part of the syspath and we extend the path
using the sysname of the parent PCI device.

With this patch udev creates following by-path links for virtio-blk
device /dev/vda which contains two partitions.

ls -l /dev/disk/by-path/
total 0
lrwxrwxrwx 1 root root  9 Feb  9 10:47 virtio-pci-0000:00:05.0 -> ../../vda
lrwxrwxrwx 1 root root 10 Feb  9 10:47 virtio-pci-0000:00:05.0-part1 -> ../../vda1
lrwxrwxrwx 1 root root 10 Feb  9 10:47 virtio-pci-0000:00:05.0-part2 -> ../../vda2

See:
http://lists.linuxfoundation.org/pipermail/virtualization/2015-August/030328.html

Fixes #2501

journal: Drop monotonicity check when appending to journal file

Remove the check that triggers rotation of the journal file when the arriving log entry had a monotonic timestamp smaller that the previous log entry. This check causes unnecessary rotations when journal-remote was receiving from multiple senders, therefore monotonicity can not be guaranteed. Also, it does not offer any useful functionality for systemd-journald.

hwdb: add Dell Lattitude E6320 pointing stick speed fix

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

Merge pull request #2542 from 0xAX/get_ts_delta

time-util: cleanups

Merge pull request #2547 from 0xAX/generalize-format-timestamp-internal

time-util: merge format_timestamp_internal() and format_timestamp_int…

time-util: introduce usec_sub()

The dual_timestamp_from_realtime(), dual_timestamp_from_monotonic()
and dual_timestamp_from_boottime_or_monotonic() shares the same
code for comparison given ts with delta. Let's move it to the
separate inline function to prevent code duplication.

time-util: merge format_timestamp_internal() and format_timestamp_internal_us()

The time_util.c provides format_timestamp_internal() and
format_timestamp_internal_us() functions for a timestamp formating. Both
functions are very similar and differ only in formats handling.

We can add additional boolean parameter to the format_timestamp_internal()
function which will represent is a format for us timestamp or not.
This allows us to get rid of format_timestamp_internal_us() that is prevent
code duplication.

We can remove format_timestamp_internal_us() safely, because it is static and
has no users outside of the time_util.c. New fourth parameter will be passed
inside of the format_timestamp(), format_timestamp_us() and etc, functions,
but the public API is not changed.

Merge pull request #2535 from PhillipSz/remove-exit-0

test: remove exit 0 at the end

Merge pull request #2533 from keszybz/read-only-seed

random-seed: provide nicer error message when unable to open file

Merge pull request #2549 from rhatdan/journalctl

Journalctl command completion should show machines

Merge pull request #2550 from evverx/fix-457

tests: fix #457

Journalctl command completion should show machines

Merge pull request #2543 from keszybz/build-sys-and-man-fixes

Build sys and man fixes

tests: fix #457

There are some limits on XFS:
major_max=511
minor_max=262143
see https://github.com/torvalds/linux/blob/master/fs/xfs/xfs_iops.c

if (S_ISCHR(mode) || S_ISBLK(mode)) {
    if (unlikely(!sysv_valid_dev(rdev) || MAJOR(rdev) & ~0x1ff))
        return -EINVAL;

Merge pull request #2546 from torstehu/fix-typo3

treewide: fix typos and spacing

Merge pull request #2536 from vcaputo/journal-misc-cleanups

journal: miscellaneous cleanups

Merge pull request #2541 from mgorny/append-flags-fix

build-sys: Perform flag tests in context of existing flags

treewide: fix typos and spacing

random-seed: provide nicer error message when unable to open file

If /var is read-only, and the seed file does not exist, we would print
a misleading error message for ENOENT. Print both messages instead, to
make it easy to diagonose.

Also, treat the cases of missing seed file the same as empty seed file
and exit successfully. Initialize the return code properly.

Fixes https://github.com/systemd/systemd/issues/2530,
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=813599.

shell-completion: fix header

man: clarify that generators are run before /var

Just a clarification. At least systemd-openqa-generator from
openqa gets this wrong.

man/hostnamectl: refer to transient hostname as "fallback"

Saying it is the "default" is misleading, because it is almost
never used.

man/hostname: remove obsolete text

All distribution-specific code was removed ages ago.

build-sys: simplify conditionals around shell completion files

Repeating those conditionals for every program is
annoying. Use a helper variable to avoid conditionals.

Also always add generated completion files to CLEANFILES.

build-sys: move CLEANFILES additions out from under HAVE_GNUEFI

It's better to always include them in 'make clean'.
It is also easier to read Makefile.am when less stuff is conditional.

build-sys: Perform flag tests in context to existing flags

Fix the CC_CHECK_FLAG_APPEND macro to test appended flags in context to
current flag values. Otherwise, it is possible to append flags colliding
with user's *FLAGS or even previously appended flags that will cause
the build to fail.

Merge pull request #2534 from indrajitr/patch-1

Fix typo in rescue shell

Merge pull request #2525 from chaloulo/journal-remote-microhttp-max-memory-usage

journal-remote: decrease microhttpd memory limit

Merge pull request #2540 from 0xAX/use-dual-timestamp-get-in-timeutil

time-util: use dual_timestamp_get()

time-util: use dual_timestamp_get()

The time-util.c provides dual_timestamp_get() function for getting
realtime and monotonic timestamps. Let's use it instead of direct
realtime/monotonic calculation.

journal: add missing space to switch statement

Merge pull request #2138 from stefwalter/journal-combine

Combine journal catalog entries with the same id

journal: remove template from open_journal args

None of the callers take advantage of this parameter, it's always NULL,
this is just a private helper function to simplify the call sites so
drop the template parameter altogether.  If a caller emerges later who
needs it, it can be restored.

journal: Add test for merging journal entries

journal: Refactor test-catalog importing tests

One function per test. Remove shared state between tests.

journal: Combine journal catalog entries with the same id

Instead of discarding duplicate catalog entries, we now combine
them. This allows software or admins to add or override catalog
headers, or add additional text to the catalog message.

journal: move mmap() ENOMEM loop to function

Introduces mmap_try_harder()

journal: add asserts on f->(data|field)_hash_table

Functions dereferencing these members should assert their non-NULL state.

journal: add asserts for f->header

Just some additional asserts in functions dereferencing f->header.

journal: move window initialization to window_add

test: remove exit 0 at the end

We don't need that at the end, as it will always exit with 0 if
everything is okey.

Fix typo in rescue shell

Merge pull request #2531 from whot/hwdb-updates

hwdb: add HP Pavilion dm4 axis corrections

hwdb: add HP Pavilion dm4 axis corrections

https://bugs.freedesktop.org/show_bug.cgi?id=91615

Merge pull request #2524 from poettering/bag-of-stuff

Bag of stuff

journal-remote: decrease microhttpd memory limit

Set the MHD_OPTION_CONNECTION_MEMORY_LIMIT to 128KB. The precious value was DATA_SIZE_MAX, which was defined as 1024*1024*768. This caused journal-remote to allocate 756MB for each journal-upload connection, thus exhausting the available memory.

resolved: correctly store interface index of RRs in cache

Fixes: #2361

resolved: properly turn off DNSSEC for LLMNR/mDNS scopes

core: rework job_get_timeout() to use usec_t and handle USEC_INFINITY time events correctly

resolved: don't follow CNAMEs originating from DNS on LLMNR

Fixes: #2514

core: move service_unwatch_control_pid() call into service_enter_running()

When we enter the running state we should forget about any control processes, in all cases, and not just when hit a
reload timeout...

journal: fix boolean handling in MMapCache

Let's use bitfields for our booleans, and don't try to apply binary OR or addition on them, because that's weird and we
should instead use logical OR only.

core: log about path_is_mount_point() errors

We really shouldn't fail silently, but print a log message about these errors. Also make sure to attach error codes to
all log messages where that makes sense.

(While we are at it, add a couple of (void) casts to functions where we knowingly ignore return values.)

core: when a service's ExecStartPre= times out, skip ExecStop=

This makes sure we never run two control processes at the same time, we cannot keep track off.

This introduces a slight change of behaviour but cleans up the definition of ExecStop= and ExecStopPost=. The former is
now invoked only if the service managed to start-up correctly. The latter is called even if start-up failed half-way.
Thus, ExecStopPost= may be used as clean-up step for both successful and failed start-up attempts, but ExecStop='s
purpose is clearly defined as being responsible for shutting down the service and nothing else.

The precise behaviour of this was not documented yet. This commit adds the necessary docs.

Fixes: #1254

util: add check that makes sure time_t and TIME_T_MAX work the way we assume they do

gpt-auto: handle errors from blkid more correctly

Let's make sure we don't choke if blkid_probe_lookup_value() returns a NULL string.

Also, make sur we propagate the correct error when blkid_probe_lookup_value() fails.

nspawn: make sure --help fits it 79ch

nspawn: optionally run a stub init process as PID 1

This adds a new switch --as-pid2, which allows running commands as PID 2, while a stub init process is run as PID 1.
This is useful in order to run arbitrary commands in a container, as PID1's semantics are different from all other
processes regarding reaping of unknown children or signal handling.

resolved: apply epoch to system time from PID 1

For use in timesyncd we already defined a compile-time "epoch" value, which is based on the mtime of the NEWS file, and
specifies a point in time we know lies in the past at runtime. timesyncd uses this to filter out nonsensical timestamp
file data, and bump the system clock to a time that is after the build time of systemd. This patch adds similar bumping
code to earliest PID 1 initialization, so that the system never continues operation with a clock that is in the 1970ies
or even 1930s.

nspawn: add new --chdir= switch

Fixes: #2192

Merge pull request #2509 from bengal/dhcp-nak-delay-v4

dhcp: delay restarts after NAKs (v4)

Merge pull request #2508 from fishilico/selinux-logind

Load SELinux labelling systemd in systemd-logind

Merge pull request #2512 from 0xAX/mount-setup

mount-setup: introduce mount_points_setup

Merge pull request #2522 from 0xAX/check-early-mount

manager: print fatal error if early mount failed

Merge pull request #2519 from msekletar/journalctl-device-log-current-boot-v2

journalctl: add match for the current boot when called with devpath (v2)

Merge pull request #2453 from poettering/journalctl-f

journalctl --fields logic

man: clarify what happens when journalctl is called with devpath

journalctl: add match for the current boot when called with devpath

Merge pull request #2507 from evverx/fix-q-on-tmpfs

tmpfiles: don't skip path_set_perms on error

Merge pull request #2518 from msekletar/udev-manpage-fix

man: document that udev_device_new_* functions set errno on error

man: document that udev_device_new_* functions set errno on error