Merge pull request #1141 from poettering/logind-fixes

Various logind fixes

logind: when parsing a boolean via sd-bus the type must be "int"

And not bool.

logind: treat an empty wall message like a NULL one

hwdb: Update database of Bluetooth company identifiers

udev: ignore ENOEXEC from cgroup lookup

The recent cgroup-rework changed the error code for un-mounted cgroupfs to
ENOEXEC. Make sure udev ignores it just like ENOENT and does not spill
warnings on the screen.

Merge pull request #1127 from neheb/master

hwdb: Add Mionix Mouse

Merge pull request #1134 from reverendhomer/patch-1

cgroup-util: Removed unreachable statement in cg_get_path

cg_get_path: Removed unreachable statement

controller cannot be NULL because if-statement in L509 has return
Coverity #1322379

Merge pull request #1123 from phomes/scope-no-bool-vs-int

scope: do not compare a bool return with "<= 0"

Merge pull request #1126 from phomes/indentation2

tree-wide: fix indentation

Add Mionix Mouse

tree-wide: fix indentation

scope: do not compare a bool return with "<= 0"

Merge pull request #1119 from teg/virtio-names

udev: net_id - support predictable ifnames on virtio buses

udev: net_id - support predictable ifnames on virtio buses

Virtio buses are undeterministically enumerated, so we cannot use them as a basis
for deterministic naming (see bf81e792f3c0). However, we are guaranteed that there
is only ever one virtio bus for every parent device, so we can simply skip over
the virtio buses when naming the devices.

Merge pull request #1118 from jsynacek/man-dot-d-v2

man: *.d conf directories: add note about initrd regeneration

Merge pull request #1116 from poettering/unified-rebased

core: unified cgroup hierarchy support

Merge pull request #1112 from poettering/sd-bus-container-fixes

machined and sd-bus container fixes

man: *.d conf directories: add note about initrd regeneration

Merge pull request #1117 from evverx/detect-parallels-virt

virt: detect parallels virtualization

virt: detect parallels virtualization

inspired by http://people.redhat.com/~rjones/virt-what/

see:
* http://git.annexia.org/?p=virt-what.git;a=blob;f=virt-what.in;h=a5ed33ef3e4bfa3281c9589eccac4d92dff1babe;hb=HEAD#l200
* http://git.annexia.org/?p=virt-what.git;a=blob;f=virt-what.in;h=a5ed33ef3e4bfa3281c9589eccac4d92dff1babe;hb=HEAD#l253

core: unified cgroup hierarchy support

This patch set adds full support the new unified cgroup hierarchy logic
of modern kernels.

A new kernel command line option "systemd.unified_cgroup_hierarchy=1" is
added. If specified the unified hierarchy is mounted to /sys/fs/cgroup
instead of a tmpfs. No further hierarchies are mounted. The kernel
command line option defaults to off. We can turn it on by default as
soon as the kernel's APIs regarding this are stabilized (but even then
downstream distros might want to turn this off, as this will break any
tools that access cgroupfs directly).

It is possibly to choose for each boot individually whether the unified
or the legacy hierarchy is used. nspawn will by default provide the
legacy hierarchy to containers if the host is using it, and the unified
otherwise. However it is possible to run containers with the unified
hierarchy on a legacy host and vice versa, by setting the
$UNIFIED_CGROUP_HIERARCHY environment variable for nspawn to 1 or 0,
respectively.

The unified hierarchy provides reliable cgroup empty notifications for
the first time, via inotify. To make use of this we maintain one
manager-wide inotify fd, and each cgroup to it.

This patch also removes cg_delete() which is unused now.

On kernel 4.2 only the "memory" controller is compatible with the
unified hierarchy, hence that's the only controller systemd exposes when
booted in unified heirarchy mode.

This introduces a new enum for enumerating supported controllers, plus a
related enum for the mask bits mapping to it. The core is changed to
make use of this everywhere.

This moves PID 1 into a new "init.scope" implicit scope unit in the root
slice. This is necessary since on the unified hierarchy cgroups may
either contain subgroups or processes but not both. PID 1 hence has to
move out of the root cgroup (strictly speaking the root cgroup is the
only one where processes and subgroups are still allowed, but in order
to support containers nicey, we move PID 1 into the new scope in all
cases.) This new unit is also used on legacy hierarchy setups. It's
actually pretty useful on all systems, as it can then be used to filter
journal messages coming from PID 1, and so on.

The root slice ("-.slice") is now implicitly created and started (and
does not require a unit file on disk anymore), since
that's where "init.scope" is located and the slice needs to be started
before the scope can.

To check whether we are in unified or legacy hierarchy mode we use
statfs() on /sys/fs/cgroup. If the .f_type field reports tmpfs we are in
legacy mode, if it reports cgroupfs we are in unified mode.

This patch set carefuly makes sure that cgls and cgtop continue to work
as desired.

When invoking nspawn as a service it will implicitly create two
subcgroups in the cgroup it is using, one to move the nspawn process
into, the other to move the actual container processes into. This is
done because of the requirement that cgroups may either contain
processes or other subgroups.

Merge pull request #1115 from phomes/hwdb-mice

hwdb: more mice

hwdb: more mice

Merge pull request #1098 from filbranden/cpuaffinity2

Getting rid of FOREACH_WORD_QUOTED and some more cleanup in config_parse_cpu_affinity2

Merge pull request #1107 from msekletar/selinux-get-raw-context

selinux: always use *_raw API from libselinux

sd-bus: when connecting to a container, don't fall back to host bus

We should never connect to the host bus as fallback if connecting to a
container failed via one method. Otherwise connecting to a dbus1
container will always result in a connection to the host.

sd-bus: when connecting to a kdbus container bus pass error up

We rely on the correct error used when opening the kdbus device node,
hence let's make sure we pass it up from the namespaced child process to
the process which actually wants to connect.

machined: introduce a ptsname_namespace() call and make use of it

The call is like ptsname() but does not assume the pty path was
accessible in the local namespace. It uses the same internal ioctl
though.

machined: call unlockpt() in container, not host

It makes assumptions about the pty path, hence better call it in the
container namespace rather than the host.

core: Log parse errors in config_parse_cpu_affinity2

Merge pull request #1111 from poettering/more-cgroup-fixes

More cgroup fixes

Merge pull request #1099 from filbranden/joincontrollers2

Getting rid of FOREACH_WORD_QUOTED in config_parse_join_controllers

Merge pull request #1102 from heftig/master

build-sys: Look for gcc-* binutils wrappers only if we're using GCC

Merge pull request #1109 from phomes/man-typos

man: fix typos in systemd-path.xml

Merge pull request #1110 from evverx/run-interactive-auth

run: enable interactive authorization

core: rework when we kill with which signal

When the user wants to explicitly send our own PID a signal, then do so.

Don't follow up SIGABRT with a SIGHUP if send_sighup is enabled. At that
point the process should have segfaulted, hence there's no point in
following up with a SIGHUP.

Send only termination signals to ourselves, never KILL or ABRT signals.

core: don't allow changing the slice of a unit while it is active

unit: small clean-ups

Always say when we ignore errors. Cast calls whose return value we
knowingly ingore to (void). Use "bool" where we actually mean a boolean,
even if we return it as an int later on.

core: when looking for the unit for a process, look at the PID hashmaps first

It's cheaper that going to cgroupfs, and also usually the better choice
since it's not racy and can map PIDs even if they were moved to a
different unit.

run: enable interactive authorization

cgroup: the root cgroup is always populated

cgroup: drop "ignore_self" argument from cg_is_empty()

In all cases where the function (or cg_is_empty_recursive()) ignoring
the calling process is actually wrong, as a process keeps a cgroup busy
regardless if its the current one or another. Hence, let's simplify
things and drop the "ignore_self" parameter.

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.

Merge pull request #1108 from phomes/dont-shadow-globals

tree-wide: do not shadow the global var timezone

man: fix typos in systemd-path.xml

tree-wide: do not shadow the global var timezone

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.

core: add OOM check in config_parse_join_controllers

core: Log parse errors in config_parse_join_controllers

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.

selinux: always use *_raw API from libselinux

When mcstransd* is running non-raw functions will return translated SELinux
context. Problem is that libselinux will cache this information and in the
future it will return same context even though mcstransd maybe not running at
that time. If you then check with such context against SELinux policy then
selinux_check_access may fail depending on whether mcstransd is running or not.

To workaround this problem/bug in libselinux, we should always get raw context
instead. Most users will not notice because result of access check is logged
only in debug mode.

* SELinux context translation service, which will translates labels to human
  readable form

build-sys: Look for gcc-* binutils wrappers only if we're using GCC

If we don't look for them, LT_INIT will and default to the unprefixed
tools.

Apparently clang doesn't like the wrappers being used. Should fix #1077.

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

core: Use extract_first_word in config_parse_join_controllers

Related to the TODO item to replace FOREACH_WORD_QUOTED with it.

Tested by setting `JoinControllers=cpu,cpuacct,memory net_cls,blkio' in
/etc/systemd/system.conf, rebooting the system with the patched binaries
and checking that the desired setup was created by inspecting the
entries under /sys/fs/cgroup.

No regressions observed in test cases.

util: Declare a cleanup routine for a cpu_set_t

Make use of it in config_parse_cpu_affinity2.

Tested by tweaking the `CPUAffinity' setting in /etc/systemd/system.conf
and reloading the daemon to confirm it is working as expected.

No regressions observed in test cases.

core: Use extract_first_word in config_parse_cpu_affinity2

Related to the TODO item to replace FOREACH_WORD_QUOTED with it.

Tested by setting `CPUAfinity=0 1' (and other similar settings) in
/etc/systemd/system.conf, booting the system with the patched binaries
(and also using `systemctl daemon-reload` to reconfigure) and checking
that /proc/1/status indicates only CPUs 0 and 1 are allowed for PID 1.

No regressions observed in test cases.

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