units: drop unnecessary 'StandardOutput=syslog'

DefaultStandardOutput is syslog anyway. There's no reason to assume that
the administrator would want these units to be excluded when he configures
a different DefaultStandardOutput.

man: fix a typo in signal number

utmp: for DEAD_PROCESS write the current time to wtmp

Zeroed .ut_tv values in wtmp confuse chkrootkit.

Reported and debugged by Norman Smith. This is based on his patch,
but modified to behave more like upstart did in F14 and cleaned up.

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

utmp: initialize store with the found entry, not with the lookup key

utmp: no need to zero a struct before overwriting it with memcpy

utmp: remove unneded parameters

With these functions no caller ever passes anything else than 0
for 't' (meaning the current time will be used).

update TODO

cgroup: immediately remove all cgroups which run empty

Some controllers have scaling problems when many empty cgroups exist.
Hence, as soon as we get a notification that a cgroup is empty, delete
it. This is also nice to keep the systemd-cgls output short.

Merge remote-tracking branch 'zbigniew/systemadm_changes'

bash-completion: rename file since it is no longer for systemctl only

bash-completion: add completions for systemd-loginctl

This script is straightforward and should give proper completions for
all of systemd-loginctl's verbs.

bash-completion: update with new verbs and arguments

Adds arguments --root= --runtime --no-legend.
Adds verbs link mask unmask reenable list-unit-files.
Also uses list-unit-files to make nicer enable and disable completions.

Rebased due to changes in systemctl.

Add Mageia support

This patch adds support for the Mageia Linux distribution:
 http://www.mageia.org/

Mageia is a fork of Mandriva although some divergence has already occured
and thus inclusion of these changes upstream allow us to (hopefully)
migrate more rapidly to the new standard approaches systemd offers.
Indeed, we already use the preferred mechanism of OS identification via
the /etc/os-release file rather than a distro specific variation.

This patch mostly mirrors the patch added previously for Mandriva
support. In addition to those original authors, this patch was mostly
written by Dexter Morgan with help from Colin Guthrie and Eugeni Dodonov.

update TODO

cryptsetup-generator: avoid ordering cycle on swap

Devices with random keys (swap), should not be ordered before local-fs.target,
as this creates a cycle with systemd-load-random-seed.service (and also it
does not make sense, a swap device is not a local-fs).

units: drop [Install] section from remote-fs-pre.target

remote-fs-pre.target is not a unit a user should ever explicitly enable.
Instead services which need to hook before network mounts should pull it
in.

mount: order remote mounts after both network.target and remote-fs-pre.target

Since remote-fs-pre.target is optional we cannot count on it to order
remote mounts after network.target, so let's add that order explicitly
in addition to remote-fs-pre.target.

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

cgroup: always recreate cgroup before we try to apply attributes

We might have trimmed the cgroup tree previously, hence don't trust our
own "realized" flag, always recreate cgroup tree before applying our
attributes to make sure this actually works out.

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

initctl: don't use dbus connection after PID 1 got respawned

After reexec PID 1 our bus connection is invalidated. Hence don't try to
reuse it, just terminate so that when we are spawned the next time we
just get a new one.

Spotted by Marti Raudsepp.

condition: Fix file descriptor leak in test_capability()

Detected by cppcheck.

random-seed: convert poolsize from bits to bytes

The problem was first noted in a bug report against Arch's initscripts.

Reported-by: Taylan Ulrich Bayırlı <taylanbayirli@gmail.com>
Reported-by: Gerardo Exequiel Pozzi <vmlinuz386@yahoo.com.ar>

plymouth: fix ply proto endianess issues

Plymouth enforces LE even for the local Ply proto, hence we should do
the conversion properly for BE arch compat.

Tracked down by Harald Hoyer.

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

README: explain dependency on cgroups

update TODO

systemctl: make list-unit-files output more economical

The first column is given the width of the widest entry,
if possible, otherwise all entries are ellipsized to fit
in ($COLUMNS - (width of second column)).

[ Added a few fixes, calculate state_cols too, respect '--no-legend',
  better handling of '--full' -- michich ]

systemctl-completion: always invoke with --no-legend

In the case of completion for the 'restart' verb, passing the invalid
unit name (the colums header) causes completion to cease functioning
entirely, with the error:

  Failed to issue method call: Unit name UNIT is not valid.

This adds a small wrapper function for systemctl which can have common
options added to it.

audit: do not complain if kernel lacks audit

When running on a kernel without audit support, systemd currently
writes a mysterious-sounding error to its log:

systemd[1]: Failed to connect to audit log: Protocol not supported

Better to suppress the audit_open() failure message when (and only
when) it is due to running on a kernel without audit support, since in
this case the admin probably does not mind systemd not writing to the
audit log.  This way, more serious errors like ENOMEM and EACCES will
stand out more.

service: Drop rcN.d runlevels from SysV services that also exist in rcS.d

Services which claim to start in both rcN.d and rcS.d generate
loops which for some reason seems to usually end up with dbus not
starting and the whole machine being quite unhappy. We now rather
assume that if a service can be started in rcS, it should not also
start in rcN.d.

Fixes Debian bug #637037

manager: fix a crash in isolating

HASHMAP_FOREACH is safe against the removal of the current entry, but
not against the removal of other entries. job_finish_and_invalidate()
can recursively remove other entries.

It triggered an assertion failure:
  Assertion 'j->installed' failed at src/manager.c:1218, function
  transaction_apply(). Aborting.

Fix the crash by iterating from the beginning when there is a
possibility that the iterator could be invalid.

It is O(n^2) in the worst case, but that's better than a crash.

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

util: properly detect what the last capability is

build-sys: bump release for v37

service: don't try to guess PID for SysV services anymore

As it turns out there are quite a number of SysV services too broken to
make the guessing work: instead of returning in the parent only after
the child is fully initialized they return immediately. The effect is
that the guessing in systemd might happen too early, at a time where the
final main process doesn't exist yet.

By turning this off we won't try to detect the main pid anymore, with
the effect that all processes of the service in question are considered
equally likely to be the main process.

localed: make sure s-s-k doesn't create any X11 config files anymore

localed: shorten generate-kbd-model-map

Output is identical.

pager: add _noreturn_ to pager_fallback()

src/pager.c: In function ‘pager_fallback’:
src/pager.c:35:13: warning: function might be possible candidate for attribute ‘noreturn’ [-Wmissing-noreturn]

unit: introduce ConditionCapability

units: forgot target units

units: increase LimitNOFILE a bit

since we need one fd per session (for logind) and one fd per service
(for stdout-syslog-bridge) increase the default rlimit a bit.

logind: fail gracefully if too many sessions are created

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

timedate: fall back to /etc/sysconfig/clock on Fedora, for compatibility with legacy

units: remount root and API FS before all mount units are applied

In order to ensure that bind mounts copy the final mount settings to the
new bind mount make the root and API FS mount options are applied before
the other file systems are mounted.

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

units: introduce local-fs-pre.target and remote-fs-pre.target

This hook target enables services to order themselves between
network.target and remote mounts, which is needed for GFS2 and similar
systems.

l10n: Reword the polish translation a bit

s/siedzenie/stanowisko/: they both translate to seat in English, but 'stanowisko'
is a place where you work, while 'siedzenie' is like a chair.

s/podłączanie/podłączenie/: we are doing it now, now sometime in the future.

man: document list-unit-files

It's documented in the --help, but not in the manpage.

sd-login.h: correct spelling mistakes in comments

hostname-setup: Frugalware switched to /etc/hostname

readahead: lower max file size for readahead

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

man: .include directive does not include as textual include it includes by parsing the include file.

This means that section headers must be used inside of
a .include file otherwise all the lines are ignored.

man: for ExecStart= provide more details on env var substitution and how that turns into arguments.

For EnvironmentFile= explain that double quotes can be used
to protect whitespace.

update TODO

util: fix close() call on wrong variable

Detected by "cppcheck" (actually it detected a file descriptor leak)

tmpfiles: fix file descriptor leak

Detected by "cppcheck"

update TODO

test_virtualization: do not try to compare id in !virt context

update TODO

update TODO

hashmap: use different version of DJB's hash algorithm that uses shifting instead of multiplication

update TODO

logind: properly generate session creation response on dbus

This fixes the various reported issues with dbus message parsing when
running su.

systemctl: fix corrupted output of units

On some systems the list of units in systemctl output came out wrong,
all on one line and with missing descriptions.

It turns out printf() really attempts to mmap 2G memory when INT_MAX is
passed as the field width. On machines with small virtual memory
without overcommit the mmap inside printf() failed.

Do not use INT_MAX for unlimited width.

localed: handle multiple X11 kbd layouts

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

systemctl: fix error message when no connection to systemd is available

systemctl: fix detection whether the dbus connection is available

build-sys: fix build

vconsole: don't parse Fedora's KEYMAP= kernel parameters anymore

KEYMAP are use to pass keymap configuration to initrd, but not to the
system itself. Since the initrd might get out of date we need to make
sure that changes made in userspace override the settings from the
cmdline, hence drpo any use of it all for these variables.

localed: add SetX11Keyboard() and SetVConsoleKeyboard() bus calls

analyze: always draw top kernel, initramfs, userspace bars

Based on Koen Kooi's patch. Renamed initrd to initramfs.

systemctl: better width and alignment of fields

Provide nicer output by taking advantage of the available columns.

Expand UNIT and DESCRIPTION if possible. Economize the space:
 - never give UNIT more width than it can use.
 - if there's a shortage of space, give 25 columns to UNIT, even
   if it would result in overflowing the total columns. The field would not
   be useful below width 25.
 - then if possible, assign up to 25 columns to DESCRIPTION.
 - then split any remaining space evenly between UNIT and DESCRIPTION.

Keep the columns aligned even with '--full'.

systemctl: print the legend regardless of stdout type, add --no-legend

Print the legend (the column headers and the footer with hints) by
default even to non-tty output. People seem to get confused by the
difference when they redirect the output.

Add a parameter to suppress the printing of the legend.

Related-to: https://bugzilla.redhat.com/show_bug.cgi?id=713567

pager: add a trivial internal pager

In the very unlikely scenario where none of the external pagers is
available, use an internal implementation to pass stdin to stdout.

Don't bother with trying 'cat', because it's no more useful than the
internal pager.

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

build-sys: bump release

readahead: opt out of bridge logging for readahead in order not to delay things unnecessarily

service: fix up std output/error before we add dependencies to the bridge socket

virt: add missing header inclusion

condition: optionally test against type of virtualization (vm vs. container)

util: move virtualization detection into its own files, and extend return codes

analyze: report startup time in plot mode as well

It now prints something like "Startup finished in 1507ms (kernel) + 850ms (userspace) = 2357ms" below the legend.

analyze: always draw 1s marker for scale

In situations like this:

root@omap4430-panda:~# systemd-analyze
Startup finished in 1499ms (kernel) + 916ms (userspace) = 2416ms

The svg plot will only have the 0s marker and no subsequent markers for scale. This patch forces the 1s marker to always be drawn.

unit: fix complementing of requirement deps with After deps for targets

'man systemd.target' says:
  Unless DefaultDependencies= is set to false, target units will
  implicitly complement all configured dependencies of type
  Wants=, Requires=, RequiresOverridable= with dependencies of type
  After=  if the units in question also have DefaultDependencies=true.

It did not work because of a forgotten negation.

util: don't fail if no id was passed to detect_container()

util: detect systemd-nspawn without relying on ns cgroup tree

nspawn: set env var container=systemd-nspawn, following the scheme lxc introduced

loginctl: fix typo

llvm-analyze: change a few things to make llvm-analyze show fewer false positives

llvm-analyze: fix some bugs found by llvm-analyze

util: improve signal_to_string, signal_from_string

signal_to_string:
 Produce names for SIGRTMIN+n.
 Never give an "n/a" result. In the worst case give the number itself as
 a string.

signal_from_string:
 Parse "RTMIN+n".
 Parse any valid signal number.

manager: fix job mode for SIGRTMIN+1, +2

The test did not work as intended and always resulted in JOB_REPLACE.

condition: fix reversed tests if path does not exist at all

CONDITION_PATH_IS_DIRECTORY, CONDITION_PATH_IS_SYMBOLIC_LINK and
CONDITION_FILE_IS_EXECUTABLE gave reversed results when the path
did not exist at all.

coverity: change a few things so that coverity doesn't show so many false positives

coverity: fix a couple of bugs found by coverity

units: fix rescue.service race with plymouth

It was possible for the "ExecStartPre=-/bin/plymouth quit" to race
with plymouth-start.service which is pulled in indirectly by
basic.target -> sysinit.target.

The race left plymouth running on the terminal, making it unusable for
rescue purposes.

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

man: document that PathChanged= does not care for simple writes but only for close()-after-write()

unit: don't recheck conditions when a unit is already starting but unit_start() is invoked

man: document pkg-config line for sd-daemon

man: document that sd-daemon.[ch] is now available as shared library

service: warn if a service fails to write its PID file

Warn if a service promises to write a PID file (using 'PIDFile=' in the
unit file or '# pidfile:' in SysV header), but fails to keep the
promise.

This warning will likely trigger also for the forking services with a
racy daemonization, which exit the original process before the PID file
is written.

update TODO

selinux: reopen log when loading the SELinux policy fails

service: if StandardInput=socket and StandardOutput=inherit imply socket for output, don't imply default output

This is useful for inetd-style per-connection services, so that they
again can simply specify StandardOutput=socket to connect all three fds
to the socket.

udev: asynchronously mount fuse/confifgs as soon as the kmods are loaded