cgroup: small cleanups and coding style fixes

A number of simplications and adjustments to brings things closer to our
coding style.

cgroup: don't allow hidden cgroups

We really should care for all cgroups, and not allow hidden ones.

cgroup: never migrate kernel threads out of the root cgroup

It won't work anyway.

units: enable waiting for unit termination in certain cases

The legacy cgroup hierarchy does not support reliable empty
notifications in containers and if there are left-over subgroups in a
cgroup. This makes it hard to correctly wait for them running empty, and
thus we previously disabled this logic entirely.

With this change we explicitly check for the container case, and whether
the unit is a "delegation" unit (i.e. one where programs may create
their own subgroups). If we are neither in a container, nor operating on
a delegation unit cgroup empty notifications become reliable and thus we
start waiting for the empty notifications again.

This doesn't really fix the general problem around cgroup notifications
but reduces the effect around it.

(This also reorders #include lines by their focus, as suggsted in
CODING_STYLE. We have to add "virt.h", so let's do that at the right
place.)

Also see #317.

unit: suppress unnecessary cgroup empty check

Rework the "service is good" check, to only check the cgroup state if we
really need to instead of always.

This allows us to suppress going to the cgroupfs for an empty check for
the majority of services.

No functional change.

manager: don't write first-boot flag file all the time

Instead, remember that we have already written it.

sd-login: improve error handling

let's return ENXIO whenever we don't know something rather than ENOENT.

ENOENT suggests this was really about a file or directory, while ENXIO
is a more generic "not found" indicator.

cgtop: properly show "/" instead of empty string in cgroup list

set: return NULL on destructors

Like we do it pretty much everywhere else.

Merge pull request #1066 from ssahani/tunnel

networkd: add support for tunnel encap limit

Merge pull request #1100 from martinpitt/master

logind: Listen to WMI hotkeys to catch SW_DOCK state/events

logind: Listen to WMI hotkeys to catch SW_DOCK state/events

On Dell and HP laptops the dock state/events (SW_DOCK) come from the "{Dell,HP}
WMI hotkeys" input devices. Tag them as power-switch so that login actually
considers them. Use a general match in case this affects other vendors, too.

Thanks to Andreas Schultz for debugging this!

https://launchpad.net/bugs/1450009

Merge pull request #1096 from evverx/systemd-path-bash-completion

shell-completion: bash: add systemd-path completion

Merge pull request #1097 from teg/dhcp-server-2

dhcp-server: make pool configurable

networkd: dhcp-server - allow configuration of the pool

The constraints we place on the pool is that it is a contiguous
sequence of addresses in the same subnet as the server address, not
including the subnet nor broadcast addresses, but possibly including
the server address itself. If the server address is included in the
pool it is (obviously) reserved and not handed out to clients.

networkd: dhcp-server - default to manage the whole subnet

Don't restrict yourselves to 32 leases, simply manage the whole subnet by default.

sd-dhcp-server: simplify pool creation

Merge sd_dhcp_server_set_address() and sd_dhcp_server_set_lease_pool() into
sd_dhcp_server_configure_pool() as the behavior of the two former depends
on the order they are called in. The flexibility is not needed, so let's
just do this in one call.

shell-completion: bash: add systemd-path completion

login: support user-bus on dbus1

dbus-1.10 was just released, including systemd units to run
`dbus-daemon --session` as systemd user unit. This allows using a
user-bus with dbus1, just like we do per default with kdbus.

All the dbus libraries have already been fixed long ago to use the
user-bus as default. Hence, there's no need to set
DBUS_SESSION_BUS_ADDRESS= if we use the user-bus. However, gdm and
friends continue to spawn a session bus if this variable is not set
(instead of checking for the existence of the user-bus). Hence, we force
the user-bus, if it is available, in pam_systemd. Once gdm and friends
are fixed, we can continue to drop this again. However, that might take
a while.

With this in place, all that is needed to make the user-bus work is:
    `systemctl --global enable dbus.socket`

If dbus.socket is not enabled, the legacy session-bus is still used.

Based on a patch by: Jan Alexander Steffens <jan.steffens@gmail.com>

Merge pull request #1093 from poettering/unified-prepare

A variety of clean-ups

cgtop: rework error handling

Never report errors twice.

sd-event: improve debug message when we fail to remove and fd from an epoll

Let's help users to debug issues with epoll fd removal by printing the
name of the event source.

cgls: pretty print root cgroup path

Make sure show it as "/" rather than empty string.

manager: remove ask-password fd from sd_event before closing it

Otherwise we might attempt to remove a non-existing fd from epoll.

man: document relationship between keys and switches of cgtop

cgtop: allow toggling of --recursive= and -k at runtime

cgtop: recursively count cgroup member tasks

When showing the number of tasks in a cgroup, recursively count tasks in
child cgroups and include them in the number. This ensures that the
number of tasks is cummulative the same way as memory, cpu and IO
resources are.

Old behaviour can be restored by passing the new --recursive=no switch.

cgtop: ignore kernel threads when counting tasks

However, allow them to be counted in by specifying -k

cgls: print the expressive error message we have

process-util: trivial optimization

cgtop: show resource usage relative to cgroup root only

This way the output is restricted to cgroups from a container when run
in one.

unit: minor simplification

util: treat 'C' and 'POSIX' locale identical

pager: set $LESSCHARSET when we output UTF8 chars

This way we can be sure that less has the same idea of the terminal as
we do.

This solves issues in systems that have locale uninitalized, where
systemd would output UTF-8 but less wouldn't allow it and show them as
control characters.

unit: unify how we assing slices to units

This adds a new call unit_set_slice(), and simplifies
unit_add_default_slice(). THis should make our code a bit more robust
and simpler.

unit: add new macros to test for unit contexts

core: use DUAL_TIMESTAMP_NULL where we can

core: don't generate stub unit file for transient units

We store the properties for transient units in drop-ins anyway, and
units don't have to have fragment files, hence don't bother with them,
and don't create them.

build-sys: add missing Makefile symlink

update TODO

Merge pull request #1091 from marmolak/master

socket: fix setsockopt call. SOL_SOCKET changed to SOL_TCP.

socket: fix setsockopt call. SOL_SOCKET changed to SOL_TCP.

Merge pull request #1083 from filbranden/configparse1

A few more tweaks to extract_first_word

man: add man for tunnel encap limit

networkd: add support for tunnel encap limit

The Tunnel Encapsulation Limit option specifies how many additional
levels of encapsulation are permitted to be prepended to the packet

Merge pull request #1088 from filbranden/perltest2

test: udev - Signal that test was skipped when running as non-root

Merge pull request #1084 from crrodriguez/master

misc bugfixes

Merge pull request #1087 from whot/hwdb-updates

hwdb: Add trackpoint sensitivity setting for Thinkpad X230

test: udev - Signal that test was skipped when running as non-root

It is more accurate to report that the test was skipped rather than
succeeded if we did not get to run it because it requires root.

Tested by running `make check` and inspecting the log file:

  $ cat test/udev-test.pl.log
  Must have root permissions to run properly.
  SKIP test/udev-test.pl (exit status: 77)

The `make check` output also reported:

  SKIP: test/udev-test.pl

hwdb: move the Performance MX to 1000dpi

Original entry may have been skewed by an earlier setting on the device, we
have a confirmed 1000dpi for a new device.

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

hwdb: Add trackpoint sensitivity setting for Thinkpad X230

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

extract_first_word: Refactor EXTRACT_DONT_COALESCE_SEPARATORS handling

Refactor allocation of the result string to the top, since it is
currently done in both branches of the condition.

Remove unreachable code checking for EXTRACT_DONT_COALESCE_SEPARATORS
when state == SEPARATOR (the only place where SEPARATOR is assigned to
state follows a check for EXTRACT_DONT_COALESCE_SEPARATORS that jumps to
the end of the function.)

Tested by running test-util successfully.

Follow up to: 206644aedeb8859801051ac170ec562c6a113a79

extract_first_word: Refactor allocation in empty argument case

This covers the case where an argument is an empty string, such as ''.

Instead of allocating the empty string in the individual conditions when
state == VALUE, just always allocate it at the end of state == START, at
which point we know we will have an argument.

Tested that test-util keeps passing after the refactor.

Follow up to: 14e685c29d5b317b815e3e9f056648027852b07e

sd-bus: Fix incorrect cleanup attribute in bus_match_to_string()

Must be _cleanup_fclose_ not _cleanup_free_

core: add attribute printf to null_log()

locale: add printf attribute to log_xkb()

Merge pull request #1072 from piotrdrag/master

po: update Polish translation

Merge pull request #1073 from phomes/master

util: make malloc0 ask calloc for one block of size n

Merge pull request #1074 from phomes/networkd-initialize-var

networkd: initialize tz to null

networkd: initialize tz to null

We may not set it before checking it for null.

util: make malloc0 ask calloc for one block of size n

... instead of an array of n individual bytes.

Silences a lot of warnings in smatch.

po: update Polish translation

nspawn: don't try to extract quotes from option string, glibc doesn't do that either

Follow-up regarding #649.

Merge pull request #649 from eyakubovich/nspawn-rbind

nspawn: add --rbind and --rbind-ro options

Merge pull request #1069 from evverx/networkctl-bash-completion

shell-completion: bash: add networkctl completion

nspawn: add (no)rbind option to --bind and --bind-ro

--bind and --bind-ro perform the bind mount
non-recursively. It is sometimes (often?) desirable
to do a recursive mount. This patch adds an optional
set of bind mount options in the form of:
--bind=src-path:dst-path:options
options are comma separated and currently only
"rbind" and "norbind" are allowed.
Default value is "rbind".

Merge pull request #1058 from spasche/hwdb-logitech-mice

hwdb: add DPI data for a few Logitech mice

Merge pull request #1071 from phomes/master

hwdb: update the mouse db

hwdb: update entry for logitech RX 250

I tested a RX 250 and measured 1000 DPI. The spec for the mouse
agrees:
http://www.logitech.com/en-gb/product/rx250-optical-mouse-business

This is an optical mouse so it is extra sensitive to what kind of
surface it is tested on.

hwdb: add a few more mice

shell-completion: bash: add networkctl completion

Merge pull request #1068 from evverx/networkctl-man

man: networkctl understands --no-legend too

man: networkctl understands --no-legend too

Merge pull request #1067 from teg/dhcp-server

sd-dhcp-server: improve predictability of leases

Merge pull request #1063 from poettering/dbus-interface-from-type

cgls/cgtop: a variety of modernizations

sd-dhcp-server: improve predictability of leases

Rather than having all clients attempt to get the same leases (starting at the
beginning of the pool), make each client star at a random offset into the pool
determined by their client id. This greatly increases the chances of a given
client receiving the same IP address even though both the client and server
have lost any lease information (and distinct server instances handing out
the same leases).

Merge pull request #1062 from poettering/cgroups-show

cgroup-show: unescape cgroups on presentation

Merge pull request #1061 from poettering/pager

A few auto-pager improvements

Merge pull request #1060 from poettering/resolved-man-2

man: Document that resolved requires nss-resolve to work with libc

Merge pull request #1064 from evverx/systemctl-completion

shell-completion: add `failed` state to --state compwords

shell-completion: add `failed` state to --state compwords

cgls: modernize cgls in preparation for unified cgroup hierarchy work

Fix --machine= handling, don't hardcode long-obsolete container
cgroup paths

Many other clean-ups.

cgtop: major modernizations

In preparation of the unified cgroup support, let's clean up cgtop:

a) rework time code to be based on "nsec_t" rather than "struct timespec"

b) Introduce long option --order= for selecting ordering

c) count number of processes only in the main hierarchy, don't bother
   with the controller hierarchies. We don't allow orthogonal
   hierarchies in systemd anymore, hence there's no point to check the
   other hierarchies.

d) Deal with non-monotonic cpuacct values (see #749)

e) When sorting groups, don't do prefix compare when ordering by number
   of tasks, since this is not accumulative for all children.

f) Actually make --cpu without parameter work

g) Don't output control characters when we get them as input.

Fixes #749.

core: add unit_dbus_interface_from_type() to unit-name.h

Let's add a way to get the type-specific D-Bus interface of a unit from
either its type or name to src/basic/unit-name.[ch]. That way we can
share it with the client side, where it is useful in tools like cgls or
machinectl.

Also ports over machinectl to make use of this.

cgroup-show: unescape cgroups on presentation

Let's unescape cgroups on presentation, so that we show literal unit
names.

pager: also redirect stderr

It's really confusing if stdout goes to the pager, but stderr is written
directly to the screen. Hence, make sure both stdout and stderr are
passed to the pager when doing autopaging.

pager: port fallback pager to use copy_bytes()

copy: add splice() based fallback

Apparently, sendfile() does not work between fifos and ttys, but
splice() does, hence let's optionally fall back to that. This is useful
to implement the fallback pager this way.

man: Document that resolved requires nss-resolve to work with libc

Closes #884.

Merge pull request #1059 from evverx/systemd-run-bash-completion

shell-completion: update systemd-run bash completion

shell-completion: update systemd-run bash completion

Many new options have been added since the bash completion was last
updated.

Merge pull request #1057 from poettering/resolve-drop-res-query

sd-resolve: drop res_query() API

Merge pull request #1055 from poettering/dhcp-updates

Various networkd and dhcp updates

hwdb: add DPI data for a few Logitech mice

update TODO

sd-resolve: drop res_query() API

Querying low-level DNS RRs should be done via resolved now, not via
glibc's awful res_query() API anymore. Let's not introduce an async
wrapper for it hence.

networkd: s/now/know

dhcp: say domain name, when we mean domain name

network: get rid of more RefCnt usage

A follow-up to 3733eec3e292e4ddb4cba5eb8d3bd8cbee7102d8

dhcp: fix n_ref type

Fixes fallout from 3733eec3e292e4ddb4cba5eb8d3bd8cbee7102d8.