sd-daemon: increase sd_notify() socket buffer size

Let's make sure we don't start blocking on sd_notify() earlier than
necessary, let's bump the socket buffer sizes to 8M.

We already do something similar for our logging socket buffers, hence
apply a similar bump here.

sysctl.d: bump number of queueable AF_UNIX/SOCK_DGRAM datagrams

The default of 16 is pretty low, let's bump this to accomodate for more
queued datagrams. This is useful for AF_UNIX/SOCK_DGRAM logging and
sd_notify() sockets as this allows queuing more datagrams before things
start to block, thus improving parallelization and logging performance.

sysv-generator: modernize

- Make sure we log each error at least once, and at most once

- Replace FOREACH_WORD loops by extract_first_word() loops

- Use FOREACH_DIRENT() for directory loops

- Use free_and_strdup() where appropriate

- Do not operate on half-loaded SysV files

- Always properly free all memory

core: bail our earlier when doing audit

Let's make sure we don't even try to create the audit socket

core,nspawn: minor coding style fixes

nspawn: fix minor memory leak

When rebooting nspawn containers about 400 times we'd otherwise hit the
fd limit and refuse further reboots.

core: constify a few things

systemctl: add a way to explicitly request client-side unit installing

This adds support for a new environment variable
SYSTEMCTL_INSTALL_CLIENT_SIDE, that ensures that systemctl executes
install operations client-side instead of passing them to PID1. This is
useful in debugging situations, but even beyond that. However, we don't
want to make it official API, hence let's just make it an undocumented
environment variable.

Similar, add a second variable, SYSTEMCTL_SKIP_SYSV which allows
skipping the SysV chkconfig fall-back if set. This is useful for similar
reasons, and exposed as undocumented as environment variable for similar
reasons, too.

Merge pull request #1734 from evverx/overwrite-sysemd-unit-path

core: set_unit_path overwrites SYSTEMD_UNIT_PATH

core: set_unit_path overwrites SYSTEMD_UNIT_PATH

Merge pull request #1729 from endocode/dongsu/jr-mhd-memlimit-v3

journal-remote: increase memory limit per connection to avoid errors …

journal-remote: increase memory limit per connection to avoid errors with HTTPS

Explicitly set MHD_OPTION_CONNECTION_MEMORY_LIMIT to a larger value,
when setting up microhttpd, to give more memory per HTTP(S) connection.

This way systemd-journal-remote can now prevent microhttpd from failing
in creating response headers with messages like "Not enough memory for
write", especially when lots of HTTPS requests arrive. That's precisely
because MHD_OPTION_CONNECTION_MEMORY_LIMIT in libmicrohttpd defaults to
32768, which is in practice insufficient in this case.
See also https://gnunet.org/bugs/view.php?id=4007 for more details.

Fixes: https://github.com/coreos/bugs/issues/927

Merge pull request #1720 from evverx/shell-completion-global-mode

shell-completion: systemctl: show completions for `user` in `global` …

hwdb: Update database of Bluetooth company identifiers

shell-completion: systemctl: show completions for `user` in `global` mode

Merge pull request #1709 from evverx/set-location-completion

shell-completion: hostnamectl: add the command set-location

Merge pull request #1717 from ronnychevalier/rc/test_capabilityboundingset2

test: add tests to ensure that the capabilities are properly set

test: add tests to ensure that the capabilities are properly set

Merge pull request #1713 from evverx/test-cap-bounding-set-merging

Another fix of the CapabilityBoundingSet parsing

core: fix capability bounding set parsing

bug: CapabilityBoundingSet= doesn't reset all caps

test: add test for capability bounding set parsing

shell-completion: hostnamectl: add the command set-location

Merge pull request #1700 from ssahani/word

Port to extract_first_word

Merge pull request #1707 from dvdhrm/notify-prio

core: fix priority ordering in notify-handling

core: fix priority ordering in notify-handling

Currently, we dispatch NOTIFY messages in a tight loop. Regardless how
much data is incoming, we always dispatch everything that is queued.
This, however, completely breaks priority event-handling of sd-event.
When dispatching one NOTIFY event, another completely different event
might fire, or might be queued by the NOTIFY handling. However, this
event will not get dispatched until all other further NOTIFY messages are
handled. Those might even arrive _after_ the other event fired, and as
such completely break priority ordering of sd-event (which several code
paths rely on).

Break this by never dispatching multiple messages. Just return after each
message that was read and let sd-event handle everything else.

(The patch looks scarier that it is. It basically just drops the for(;;)
 loop and re-indents the loop-content.)

nspwan: port to extract_first_word

install: port to extract_first_word

journal-remote: port to extract_first_word

Merge pull request #1705 from filbranden/testutil1

Move tests for extract-word and parse-util into separate test cases

test-parse-util: Move parse-util tests into their own test case

Tests for the functions defined in src/basic/parse-util.c. Reorder them
to match the order in which the functions are defined in the source
file. Adjusted the list of include files to remove the ones no longer
needed in test-util.c.

Tested that `make check` still passes as expected. Also checked the
number of lines removed from test-util.c matches the expected, as an
additional verification that no tests were dropped or duplicated in the
move.

test-extract-word: Move extract-word tests into their own test case

Tests for the functions defined in src/basic/extract-word.c.

Tested that `make check` still passes as expected.

build-sys: Keep .gitignore sorted

Let's try to keep it that way! :-)

Merge pull request #1703 from ronnychevalier/rc/test_execute_check_nobody2

test-execute: check if nobody exists before running some tests

Merge pull request #1699 from filbranden/cpuaffinity9

cpu-set-util: Support ranges in parse_cpu_set_and_warn (v4)

test-execute: check if nobody exists before running some tests

Merge pull request #1701 from hosiet/l10n-cn

l10n: add Simplified Chinese translation.

l10n: add zh_CN translation.

man: Update man page documentation for CPUAffinity

Document support for commas as a separator and possibility of specifying
ranges of CPU indices.

Tested by regenerating the manpages locally and reading them on man.

cpu-set-util: Support ranges in parse_cpu_set_and_warn

Tested CPUAffinity ranges on both a service unit and in system.conf and
confirmed they work as expected (by inspecting /proc/PID/status, for the
main pid of the service and for pid 1).  Also mixed ranges with both
spaces, commas, trailing commas and spaces.

Added new tests to increase coverage of ranges and prevent regressions.

parse-util: Introduce new parse_range function

This function will be useful for CPUAffinity settings that involve
ranges of CPUs.

Make it generic and include test coverage to prevent regressions.

cpu-set-util: Accept commas as separators in parse_cpu_set_and_warn

Tested CPUAffinity settings on both a service unit and in system.conf
and confirmed they work as expected.

Added a new test to confirm that trailing commas and spaces work and to
prevent any regressions in that area.

Merge pull request #1698 from poettering/pid-is-unwaited-fix

fixup for previous pr

service: exiting early is nicer than deeply-indented code blocks

core: all unit types that watch control PIDs should use the same logic

When coldplugging the unit state, make sure to follow the same basic
logic for all unit types: always verify whether the control PID is still
a waitable process before proceeding.

core: don't pass uninitialzed PIDs to pid_is_unwaited()

Since 5fd9b2c5467b0a42ccdabc7eb8e516d512609a8e passing a pid of 0 to
pid_is_unwaited() and pid_is_live() is considered as a request on the
current process, similar how the other calls in process-util.c handle a
PID of 0. This broke service.c, which passes a 0 PID and expects it to
be considered an unwaited process.

This fix make sure we can boot again.

Merge pull request #1695 from evverx/fix-cap-bounding-merging

core: fix CapabilityBoundingSet merging

Merge pull request #1691 from poettering/util-lib-3

split up util.h and macro.h into even more bits

util-lib: move inotify-related definitions to fs-util.[ch]

util-lib: move fdname_is_valid() to fd-util.[ch]

automount: move generically userful call repeat_mount() into mount-util.[ch]

process-util: move a couple of process-related calls over

process-util: make some minor corrections to PID live detection

process-util: rename get_parent_of_pid() → get_process_ppid()

In order to match the other get_process_xyz() calls.

process-util: actually move rename_process() over

The prototype was moved long ago, actually move the definition over now,
too.

mount-util: move fstype_is_network() and name_to_handle_at() definitions over

update TODO

util-lib: split out allocation calls into alloc-util.[ch]

util-lib: split out globbing related calls into glob-util.[ch]

stat-util.h: move F_TYPE_EQUAL() macro definition to stat-util.h

util-lib: split out printf() helpers to stdio-util.h

macro.h: move definition of MODE_INVALID to parse-util.h

macro: move DECIMAL_STR_WIDTH() into macro.h

So that it sets next to DECIMAL_STR_MAX(), where it belongs.

socket-util: move CMSG_FOREACH() from macro.h to socket-util.h

io-util.h: move iovec stuff from macro.h to io-util.h

process-util: move PID related macros from macro.h to process-util.h

user-util: move UID/GID related macros from macro.h to user-util.h

util-lib: move web-related calls into web-util.[ch]

util-lib: split out syslog-related calls into syslog-util.[ch]

detect-virt: add new --chroot switch to detect chroot() environments

util-lib: move running_in_chroot() into virt.[ch]

It's a very weak kind of virtualization, after all.

util-lib: move /proc/cmdline parsing code to proc-cmdline.[ch]

util-lib: move is_main_thread() to process-util.[ch]

src/basic: rename audit.[ch] → audit-util.[ch] and capability.[ch] → capability-util.[ch]

The files are named too generically, so that they might conflict with
the upstream project headers. Hence, let's add a "-util" suffix, to
clarify that this are just our utility headers and not any official
upstream headers.

util-lib: split out umask-related code to umask-util.h

util-lib: move more locale-related calls to locale-util.[ch]

util-lib: move status printing stuff into src/core/

It's very specific to the core, and not used elsewhere, hence move it
out of terminal-util.[ch].

util-lib: move string table stuff into its own string-table.[ch]

util-lib: split stat()/statfs()/stavfs() related calls into stat-util.[ch]

util: move string_is_safe() to string-util.[ch]

util-lib: move a number of fs operations into fs-util.[ch]

util-lib: split out file attribute calls to chattr-util.[ch]

util-lib: split xattr-related calls into xattr-util.[ch]

util-lib: introduce dirent-util.[ch] for directory entry calls

Also, move a couple of more path-related functions to path-util.c.

util-lib: split out resource limits related calls into rlimit-util.[ch]

util-lib: move take_password_lock() to user-util.[ch]

Also, rename it take_etc_passwd_lock(), in order to make it more
expressive.

util: move filename_is_valid() and path_is_safe() to path-util.[ch]

util-lib: move mount related utility calls to mount-util.[ch]

util-lib: move fstab_node_to_udev_node() to fstab-util.[ch]

util-lib: move more file I/O related calls into fileio.[ch]

util: remove path_get_parent(), in favour of dirname_malloc()

We don't need two functions that do essentialy the same, hence drop
path_get_parent(), and stick to dirname_malloc(), but move it to
path-util.[ch].

util-lib: split out hex/dec/oct encoding/decoding into its own file

util-lib: split string parsing related calls from util.[ch] into parse-util.[ch]

core: fix CapabilityBoundingSet merging

Fixes: #1221

Merge pull request #1693 from ssahani/word

timesysnd: port to extract_first_word

resolved: port to extract_first_word

timesysnd: port to extract_first_word

Merge pull request #1688 from phomes/typo-fix

path-util: do not return NULL as int

path-util: do not return NULL as int

strv_split will only return NULL on oom so we should return -ENOMEM
instead.

Looks like an oversight from the changes in 0f474365

Merge pull request #1679 from evverx/refuse-manual-start-by-reload-or-restart

core: don't allow manual start with reload-or-restart too

Merge pull request #1686 from medhefgo/remount-ro-fix

unmount: Pass in mount options when remounting read-only