Revert "sd-bus: let sd_bus_call() use the synchronous kdbus method"

This reverts commit 021b89861d0b1defcbd6ba71d1aaf6271785a942.

Something is not quite right, "KillUnit" sent from systemctl is not
handled correctly and shutdown has problems.

logind: introduce session "positions"

logind has no concept of session ordering. Sessions have a unique name,
some attributes about the capabilities and that's already it. There is
currently no stable+total order on sessions. If we use the logind API to
switch between sessions, we are faced with an unordered list of sessions
we have no clue of.

This used to be no problem on seats with VTs or on seats with only a
single active session. However, with the introduction of multi-session
capability for seats without VTs, we need to find a way to order sessions
in a stable way.

This patch introduces session "positions". A position is a simple integer
assigned to a session which is never changed implicitly (currently, we
also don't change it explicitly, but that may be changed someday). For
seats with VTs, we force the position to be the same as the VTnr. Without
VTs, we simply find the lowest unassigned number and use it as position.
If position-assignment fails or if, for any reason, we decide to not
assign a position to a session, the position is set to 0 (which is treated
as invalid position).
During session_load() or if two sessions have the same VTnr, we may end up
with two sessions with the same position (this shouldn't happen, but lets
be fail-safe in case some other part of the stack fails). This case is
dealt with gracefully by ignoring any session but the first session
assigned to the position. Thus, session->pos is a hint, seat->positions[i]
is the definite position-assignment. Always verify both match in case you
need to modify them!

Additionally, we introduce SwitchTo(unsigned int) on the seat-dbus-API.
You can call it with any integer value != 0 and logind will try to switch
to the request position. If you implement a compositor or any other
session-controller, you simply watch for ctrl+alt+F1 to F12 and call
SwitchTo(Fx). logind will figure a way out deal with this number.
For convenience, we also introduce SwitchToNext/Previous(). It should be
called on ctrl+alt+Left/Right (like the kernel-console used to support).

Note that the public API (SwitchTo*()) is *not* bound to the underlying
logic that is implemented now. We don't export "session-positions" on the
dbus/C API! They are an implementation detail. Instead, the SwitchTo*()
API is supposed to be a hint to let logind choose the session-switching
logic. Any foreground session-controller is free to enumerate/order
existing sessions according to their needs and call Session.Activate()
manually. But the SwitchTo*() API provides a uniform behavior across
session-controllers.

Background: Session-switching keys depend on the active keymap. The XKB
specification provides the XKB_KEY_XF86Switch_VT_1-12 key-symbols which
have to be mapped by all keymaps to allow session-switching. It is usually
bound to ctrl+alt+Fx but may be set differently. A compositor passes any
keyboard input to XKB before passing it to clients. In case a key-press
invokes the XKB_KEY_XF86Switch_VT_x action, the keypress is *not*
forwarded to clients, but instead a session-switch is scheduled.

This actually prevents us from handling these keys outside of the session.
If an active compositor has a keymap with a different mapping of these
keys, and logind itself tries to catch these combinations, we end up with
the key-press sent to the compositor's clients *and* handled by logind.
This is *bad* and we must avoid this. The only situation where a
background process is allowed to handle key-presses is debugging and
emergency-keys. In these cases, we don't care for keymap mismatches and
accept the double-event. Another exception is unmapped keys like
PowerOff/Suspend (even though this one is controversial).

exec: introduce PrivateDevices= switch to provide services with a private /dev

Similar to PrivateNetwork=, PrivateTmp= introduce PrivateDevices= that
sets up a private /dev with only the API pseudo-devices like /dev/null,
/dev/zero, /dev/random, but not any physical devices in them.

update TODO

sd-dhcp-client: refactor state machine check

systemctl: skip native unit file handling if sysv file handling already handled everything

Issue pointed out by Colin Guthrie.

update TODO

sd-bus: let sd_bus_call() use the synchronous kdbus method

bus-kernel: move bus_kernel_make_message

This makes future commits more readable.

networkd: use 'up'/'down' rather than 'on'/'off'

networkd: dhcp - by default ignore the MTU

sd-dhcp-client: refactor client_{free,new}

Make them more simiar to sd_bus and friends. Also factor out the event attachment. In the future,
we will likely want to support external main-loops, so this is a first step. For the time being,
we are still requiring an sd_event to be attached though.

sd-resolv: declare functions from .h public

sd-resolve: rename get_next() and get_n_queries() + cleanup

sd-resolve: Allocate objects with new()

sd-resolve: get rid of sd_resolve_freeanswer()

sd-dhcp-client/networkd: add domainname support

networkd: don't hard depend on system bus

We may not have a dbus daemon in the initrd (until we can rely on kdbus). In
this case, simply ignore any attempts at using the bus. There is only one user
for now, but surely more to come.

In order to work reliably in the real root without kdbus, but at the same time
don't delay boot when kdbus is in use, order ourselves after dbus.service.

networkd: resolv.conf - reword comment

Take into account that users may want to use resolvconf(8), or similar. Also,
avoid repeated calls to fputs().

libsystemd: sync kdbus.h

units: drop [Install] section from multi-user.target and graphical.target

They were supposed to make it easy to make the default.target a symlink
to these targets, but this was never advertised and we have a better
command for this now in "systemctl set-default". Since the install
section makes the output of "systemctl list-unit-files" confusing (since
it makes the units appear as "disabled"), let's drop the sections.

Update TODO

sd-resolve: rename structs to fit coding-style e.g 'struct MixedCase'

lets get this right once, and if not for all, atleast for now :)

So comments and input about nameing is very welcome.

Cheers

sd-resolve: get rid of _t post-fixes for sd_ objects

We dont want _t prefixes, right?

build: Skip .busname generator when kdbus is off

bus: Fix read_word_le() function

update TODO

libsystemd: rename LIBSYSTEMD_BUS to LIBSYSTEMD

bus: update kdbus.h (ABI break)

networkd: resolv.conf - extend the comment a bit

sd-dhcp-client: minor style fixes

Also introduce a cleanup macro for DHCPLease

sd-dhcp-client: refactor DNS support

Rather than keeping an array of pointers to addresses, just keep an array of addresses.

sd-dhcp-client/networkd: add transient hostname support

networkd: dhcp - only set the MTU option once

Only set MTU request when creating the dhcp client, not every time it is restarted.

networkd: link - only save original MTU when necessary

networkd: improve logging

bus: fix issue with reference counting

b7fc42e03 introduced a regression.

build-sys: increase valgrind stack limit so all tests pass again

bus: break reference cycle between bus and messages

Previously (6ee4f99 bus: break reference cycle between bus and
messages) I committed the test code, but not the actual fix :)

build-sys: fix --enable-dbus

The incorrect shell code of AS_IF always get false value, so it always
build without dbus though build with "--enable-dbus" explicitely.

sd-dhcp-client/networkd: add interface MTU support

sd-rtnl: only alter link flags when reqeusted to

sd-dhcp-client: fix memory corruption

event: extend priority raneg to 64bit

The kdbus prioq logic will use 64bit prios too, so let's keep this
similar in style. Using 64bit here has the advantage, that pointers and
pretty much anything else can be mapped naturally to priorities if so
desired.

systemctl: introduce new "import-environment" command

This may be used in graphical session start-up scripts to upload
environment variables such as $DISPLAY into the systemd manager easily.

logind: get rid of X11 display socket symlink

X11 never made use of it anyway and it's probably better to just push
$DISPLAY into the systemd daemon from gnome-session (or equivalent
program) than to change libX11 to look for this socket. In particular
since we won't need this for Wayland anyway and we shouldn't add
infrastructure for stuff that's on its way out anyway.

shell-completion: remove load from systemctl

Revert "Revert "coredumpctl: in case of error free pattern after print""

In the case of the error set_consume will free the pointer 'pattern'
which is later used in log_error. Either we should stop priniting that
or use simple set_put and free pattern manually.

This reverts commit ece6b8fd5bbc1fee16f652e680e3033f2f3efc4a.

sd-resolv: rename to sd-resolve

Lennart pointed out that we were misspelling 'resolve'. Let's not repeat the mistakes of 'umount'
and 'resolv.conf'.

README: list CONFIG_FHANDLE in main section

readme: CONFIG_FHANDLE is a requirement

Revert "coredumpctl: in case of error free pattern after print"

This reverts commit cf50a55277ee432e1217ad894afcbd9a664ff96c.

set_consume frees on error.

coredumpctl: in case of error free pattern after print

sd-resolv: remove left-over comment

We are anyway breaking the API by renaming everything, so that is not a concern.

sd-resolv: require SOCK_CLOEXEC

sd-resolv: rename old asyncns variables

sd_resolv: Rename structs- and function names to sd_resolv

bus: update kdbus.h (ABI break)

bus: update kdbus.h (ABI break)

networkd: DHCPv4 - allow opting out of using DNS servers

Setting UseDNS=no will ignore any received DNS servers.

libsystemd-dns: merge into libsystemd

Also rename sd-dns -> sd-resolv.

libsystemd-rtnl: merge into libsystemd

libsystemd-dhcp: merge into libsystemd

libsystemd-bus: rename to libsystemd

Documentation was updated to refer to either 'libsystemd' or 'sd-bus' in place
of libsystemd-bus.

TODO: update

sd-dns: coding style cleanup

networkd: fix wording

TODO: update rtnl/network sections

networkd: generate resolv.conf

This adds support to generate a basic resolv.conf in /run/systemd/network.
This file will not take any effect unless a symlink is created from
/etc/resolv.conf.

Nameservers received over DHCP takes precedence over statically configured ones.

Note: /etc/resolv.conf is severely limited, so in the future we will likely
rather provide a much more powerfull nss plugin (or something to that effect),
but this should allow current users to function without any loss of
functionality.

networkd: bridge - remove redundant state

We will not insist on getting the reply from rtnl that the bridge
was created before considering the bridge ready, as we will be
notified about that via udev. We will listen for the rtnl response
however, in case the creation of the bridge failed.

networkd: don't automatically reload config at runtime

We don't know if the config will be consistent, so do as systemd itself and only
load config when the daemon starts (and possibly, in the future, when explicitly requested).

networkd: improve logging a bit

bus-proxyd: add some more debugging

Make sure we print a message when exiting with an error.

core: don't allow setting NOTIFY_SOCKET and similar env vars we need ourselves via SetEnvironment bus calls

We just quietly eat them up, so that simple environment importing still
works without error.

update TODO

core: clean up environment block for --user instances a bit

bus: add test case for signature type

man: add busctl(1)

bus: properly handle EOF error conditions in proxyd

EOF is not an error so we should not print an error message about it.

journald: do not free space when disk space runs low

Before, journald would remove journal files until both MaxUse= and
KeepFree= settings would be satisfied. The first one depends (if set
automatically) on the size of the file system and is constant.  But
the second one depends on current use of the file system, and a spike
in disk usage would cause journald to delete journal files, trying to
reach usage which would leave 15% of the disk free. This behaviour is
surprising for the user who doesn't expect his logs to be purged when
disk usage goes above 85%, which on a large disk could be some
gigabytes from being full. In addition attempting to keep 15% free
provides an attack vector where filling the disk sufficiently disposes
of almost all logs.

Instead, obey KeepFree= only as a limit on adding additional files.
When replacing old files with new, ignore KeepFree=. This means that
if journal disk usage reached some high point that at some later point
start to violate the KeepFree= constraint, journald will not add files
to go above this point, but it will stay (slightly) below it. When
journald is restarted, it forgets the previous maximum usage value,
and sets the limit based on the current usage, so if disk remains to
be filled, journald might use one journal-file-size less on each
restart, if restarts happen just after rotation. This seems like a
reasonable compromise between implementation complexity and robustness.

gpt-auto-generator: use EBADSLT code when unable to detect partition type

ENODEV suggests that something is missing, which is be misleading
here.

gpt-auto-generator: skip nonexistent devices

The devices we work with have eMMC chips for storage. The chips
provide four "hardware" partitions.  The first is /dev/mmcblk0, it
takes almost whole space and holds a GPT with several real partitions
(/dev/mmcblk0p?). Then there are three block devices (mmcblk0boot0,
mmcblk0boot1, rpmb) that are part of the same hardware as mmcblk0 that
are presented by the kernel as children of the latter. That relationship
makes gpt-auto-generator try to peek them but since they are not GPT
partitions blkid_do_safeprobe() returns -2 making verify_gpt_parition()
function return -ENODEV.

ycm: autocompose flags from the Makefile

This should hopefully be self-maintaining.

pam: skip DBUS_SESSION_BUS_ADDRESS export when kdbus is not active

make socket_instantiate_service use cleanup gcc attribute

bus: fix length calculation when serializing SD_BUS_TYPE_SIGNATURE

core: fix unused variable

network: use GNU-ism to simplify macros

Thanks David!

journalctl: allow globbing in --unit and --user-unit

This is a continuation of e3e0314b systemctl: allow globbing in commands
which take multiple unit names.

Multiple patterns can be specified, as separate arguments, or as one argument
with patterns seperated by commas.

If patterns are given, at least one unit must be matched (by any of the patterns).
This is different behaviour than systemctl, but here it is necessary because
otherwise anything would be matched, which is unlikely to be the intended
behaviour.

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

journal: fix access to munmapped memory in sd_journal_enumerate_unique

sd_j_e_u needs to keep a reference to an object while comparing it
with possibly duplicate objects in other files. Because the size of
mmap cache is limited, with enough files and object to compare to,
at some point the object being compared would be munmapped, resulting
in a segmentation fault.

Fix this issue by turning keep_always into a reference count that can
be increased and decreased. Other callers which set keep_always=true
are unmodified: their references are never released but are ignored
when the whole file is closed, which happens at some point. keep_always
is increased in sd_j_e_u and later on released.

sd-dns: simplify headers

As noted by Lennart: HAVE_CONFIG_H is done implicitly and sys/prctl.h is
mandatory.

sd-dns: update/add license texts

sd-dns: cleanup public header

Don't include internal headers
Update license text
Use _SD_BEGIN/END_DECLARATIONS

sd-dns: initial commit

Origin: <http://0pointer.de/lennart/projects/libasyncns/>

[tomegun: renamed some more files asyncns -> sd-dns and moved to libsystemd-bus as
requested by Lennart]

udev/net_id: Introduce predictable network names for Linux on System z

Use the bus-ID to create predicatable devices names for network interfaces
on Linux on System z instances.  The bus-ID identifies a device in the s390
channel subsystem.

Network interfaces of device type Ethernet are named as:
    enccw0.0.1234    (13 characters)
up to
    enccwff.7.ffff   (14 characters)

CTC network devices of device type SLIP, use a different prefix as follows:
    slccw0.0.1234    (13 characters)

See also Red Hat Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=870859
[tomegun: typical problem of netdevs switching names between reboots.]

hwdb: Add support for Toshiba Satellite P75-A7200 keyboard

Revert "bus: export bus address also when compiled without kdbus"

This reverts commit 9130f2128b64de19a3b7d6db7e0d371adfd296c2.

It's too early to do that. For now we should only "break" the
--enable-kdbus case, not the normal one.

man: clarify sd_bus_e_get_errno with NULL arg

bus: convert a bus error with NULL name into an errno of 0 in sd_bus_error_get_errno()

man: mention which variables will be expanded in ExecStart